Add comments

Author: DonJayamanne

src/client/datascience/plotting/plotViewer.ts

@@ -17,7 +17,6 @@ import { traceError } from '../../common/logger';
 import { IFileSystem } from '../../common/platform/types';
 import { IConfigurationService, IDisposable, IExperimentsManager, Resource } from '../../common/types';
 import * as localize from '../../common/utils/localize';
-import { noop } from '../../common/utils/misc';
 import { ICodeCssGenerator, IPlotViewer, IThemeFinder } from '../types';
 import { WebViewHost } from '../webViewHost';
 import { PlotViewerMessageListener } from './plotViewerMessageListener';
@@ -166,7 +165,9 @@ export class PlotViewer extends WebViewHost<IPlotViewerMapping> implements IPlot
                         // tslint:disable-next-line: no-require-imports
                         const pdfkit = require('pdfkit') as typeof import('pdfkit');
 
-                        const dimensions = await getSVGDimensions(payload.svg).catch(noop);
+                        const dimensions = await getSVGDimensions(payload.svg).catch(ex =>
+                            traceError('Failed to get dimensions of SVG', ex)
+                        );
                         const options: Record<string, {}> = {};
                         if (dimensions && dimensions.width && dimensions.height) {
                             const LetterSize = { width: 612, height: 792.0 }; //https://github.com/foliojs/pdfkit/blob/master/lib/page.js#L62