Print Preview: Fix top level directory compile errors

chrome/browser/resources/print_preview/native_layer.js

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 cr.exportPath('print_preview');
 
 /**
  * @typedef {{selectSaveAsPdfDestination: boolean,
  *            layoutSettings.portrait: boolean,
  *            pageRange: string,
@@ skipping 61 matching lines @@
     global.onExtensionCapabilitiesSet =
         this.onExtensionCapabilitiesSet_.bind(this);
     global.onEnableManipulateSettingsForTest =
         this.onEnableManipulateSettingsForTest_.bind(this);
     global.printPresetOptionsFromDocument =
         this.onPrintPresetOptionsFromDocument_.bind(this);
     global.onProvisionalPrinterResolved =
         this.onProvisionalDestinationResolved_.bind(this);
     global.failedToResolveProvisionalPrinter =
         this.failedToResolveProvisionalDestination_.bind(this);
-  };
+  }
 
   /**
    * Event types dispatched from the Chromium native layer.
    * @enum {string}
    * @const
    */
   NativeLayer.EventType = {
     ACCESS_TOKEN_READY: 'print_preview.NativeLayer.ACCESS_TOKEN_READY',
     CAPABILITIES_SET: 'print_preview.NativeLayer.CAPABILITIES_SET',
     CLOUD_PRINT_ENABLE: 'print_preview.NativeLayer.CLOUD_PRINT_ENABLE',
@@ skipping 203 matching lines @@
         'previewModifiable': documentInfo.isModifiable,
         'generateDraftData': documentInfo.isModifiable,
         'fitToPageEnabled': printTicketStore.fitToPage.getValue(),
         'scaleFactor': printTicketStore.scaling.getValueAsNumber(),
         // NOTE: Even though the following fields don't directly relate to the
         // preview, they still need to be included.
         // e.g. printing::PrintSettingsFromJobSettings() still checks for them.
         'collate': true,
         'copies': 1,
         'deviceName': destination.id,
-        'dpiHorizontal': "horizontal_dpi" in printTicketStore.dpi.getValue() ?
+        'dpiHorizontal': 'horizontal_dpi' in printTicketStore.dpi.getValue() ?
            printTicketStore.dpi.getValue().horizontal_dpi : 0,
-        'dpiVertical': "vertical_dpi" in printTicketStore.dpi.getValue() ?
+        'dpiVertical': 'vertical_dpi' in printTicketStore.dpi.getValue() ?
            printTicketStore.dpi.getValue().vertical_dpi : 0,
         'duplex': printTicketStore.duplex.getValue() ?
             NativeLayer.DuplexMode.LONG_EDGE : NativeLayer.DuplexMode.SIMPLEX,
         'printToPDF': destination.id ==
                 print_preview.Destination.GooglePromotedId.SAVE_AS_PDF,
         'printWithCloudPrint': !destination.isLocal,
         'printWithPrivet': destination.isPrivet,
         'printWithExtension': destination.isExtension,
         'rasterizePDF': false,
         'shouldPrintBackgrounds': printTicketStore.cssBackground.getValue(),
@@ skipping 62 matching lines @@
         'shouldPrintBackgrounds': printTicketStore.cssBackground.getValue(),
         'shouldPrintSelectionOnly': false,  // Only used in print preview
         'previewModifiable': documentInfo.isModifiable,
         'printToPDF': destination.id ==
             print_preview.Destination.GooglePromotedId.SAVE_AS_PDF,
         'printWithCloudPrint': !destination.isLocal,
         'printWithPrivet': destination.isPrivet,
         'printWithExtension': destination.isExtension,
         'rasterizePDF': printTicketStore.rasterize.getValue(),
         'scaleFactor': printTicketStore.scaling.getValueAsNumber(),
-        'dpiHorizontal': "horizontal_dpi" in printTicketStore.dpi.getValue() ?
+        'dpiHorizontal': 'horizontal_dpi' in printTicketStore.dpi.getValue() ?
            printTicketStore.dpi.getValue().horizontal_dpi : 0,
-        'dpiVertical': "vertical_dpi" in printTicketStore.dpi.getValue() ?
+        'dpiVertical': 'vertical_dpi' in printTicketStore.dpi.getValue() ?
            printTicketStore.dpi.getValue().vertical_dpi : 0,
         'deviceName': destination.id,
         'fitToPageEnabled': printTicketStore.fitToPage.getValue(),
         'pageWidth': documentInfo.pageSize.width,
         'pageHeight': documentInfo.pageSize.height,
         'showSystemDialog': opt_showSystemDialog
       };
 
       if (!destination.isLocal) {
         // We can't set cloudPrintID if the destination is "Print with Cloud
@@ skipping 591 matching lines @@
 
     /**
      * Title of the document.
      * @type {string}
      * @private
      */
     this.documentTitle_ = documentTitle;
 
     /**
      * Whether the document has selection.
-     * @type {string}
+     * @type {boolean}

[dpapad, 2017/05/05 23:30:29]

Nit (optional): Since you are already doing small lint fixes in this CL, perhaps
consider using the shorthand syntax "@private {Type}" in this file.

[rbpotter, 2017/05/06 00:02:19]

Done.

      * @private
      */
     this.documentHasSelection_ = documentHasSelection;
 
     /**
      * Whether selection only should be printed.
-     * @type {string}
+     * @type {boolean}
      * @private
      */
     this.selectionOnly_ = selectionOnly;
 
     /**
      * ID of the system default destination.
      * @type {?string}
      * @private
      */
     this.systemDefaultDestinationId_ = systemDefaultDestinationId;
 
     /**
      * Serialized app state.
      * @type {?string}
      * @private
      */
     this.serializedAppStateStr_ = serializedAppStateStr;
 
     /**
      * Serialized default destination selection rules.
      * @type {?string}
      * @private
      */
     this.serializedDefaultDestinationSelectionRulesStr_ =
         serializedDefaultDestinationSelectionRulesStr;
-  };
+  }
 
   NativeInitialSettings.prototype = {
     /**
      * @return {boolean} Whether the print preview should be in auto-print mode.
      */
     get isInKioskAutoPrintMode() {
       return this.isInKioskAutoPrintMode_;
     },
 
     /**
@@ skipping 57 matching lines @@
       return this.serializedDefaultDestinationSelectionRulesStr_;
     }
   };
 
   // Export
   return {
     NativeInitialSettings: NativeInitialSettings,
     NativeLayer: NativeLayer
   };
 });