Compile print_preview, part 3: reduce down to 185 errors

chrome/browser/resources/print_preview/data/print_ticket_store.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.define('print_preview', function() {
   'use strict';
 
   // TODO(rltoscano): Maybe clear print ticket when destination changes. Or
   // better yet, carry over any print ticket state that is possible. I.e. if
   // destination changes, the new destination might not support duplex anymore,
@@ skipping 287 matching lines @@
      */
     init: function(
         thousandsDelimeter, decimalDelimeter, unitType, selectionOnly) {
       this.measurementSystem_.setSystem(thousandsDelimeter, decimalDelimeter,
                                         unitType);
       this.selectionOnly_.updateValue(selectionOnly);
 
       // Initialize ticket with user's previous values.
       if (this.appState_.hasField(
           print_preview.AppState.Field.IS_COLOR_ENABLED)) {
-        this.color_.updateValue(this.appState_.getField(
-            print_preview.AppState.Field.IS_COLOR_ENABLED));
+        this.color_.updateValue(
+            /** @type {!Object} */(this.appState_.getField(
+            print_preview.AppState.Field.IS_COLOR_ENABLED)));
       }
       if (this.appState_.hasField(
           print_preview.AppState.Field.IS_DUPLEX_ENABLED)) {
-        this.duplex_.updateValue(this.appState_.getField(
-            print_preview.AppState.Field.IS_DUPLEX_ENABLED));
+        this.duplex_.updateValue(
+            /** @type {!Object} */(this.appState_.getField(
+            print_preview.AppState.Field.IS_DUPLEX_ENABLED)));
       }
       if (this.appState_.hasField(print_preview.AppState.Field.MEDIA_SIZE)) {
-        this.mediaSize_.updateValue(this.appState_.getField(
-            print_preview.AppState.Field.MEDIA_SIZE));
+        this.mediaSize_.updateValue(
+            /** @type {!Object} */(this.appState_.getField(
+            print_preview.AppState.Field.MEDIA_SIZE)));
       }
       if (this.appState_.hasField(
           print_preview.AppState.Field.IS_LANDSCAPE_ENABLED)) {
-        this.landscape_.updateValue(this.appState_.getField(
-            print_preview.AppState.Field.IS_LANDSCAPE_ENABLED));
+        this.landscape_.updateValue(
+            /** @type {!Object} */(this.appState_.getField(
+            print_preview.AppState.Field.IS_LANDSCAPE_ENABLED)));
       }
       // Initialize margins after landscape because landscape may reset margins.
       if (this.appState_.hasField(print_preview.AppState.Field.MARGINS_TYPE)) {
         this.marginsType_.updateValue(
-            this.appState_.getField(print_preview.AppState.Field.MARGINS_TYPE));
+            /** @type {!Object} */(this.appState_.getField(
+            print_preview.AppState.Field.MARGINS_TYPE)));
       }
       if (this.appState_.hasField(
           print_preview.AppState.Field.CUSTOM_MARGINS)) {
-        this.customMargins_.updateValue(this.appState_.getField(
-            print_preview.AppState.Field.CUSTOM_MARGINS));
+        this.customMargins_.updateValue(
+            /** @type {!Object} */(this.appState_.getField(
+            print_preview.AppState.Field.CUSTOM_MARGINS)));
       }
       if (this.appState_.hasField(
           print_preview.AppState.Field.IS_HEADER_FOOTER_ENABLED)) {
-        this.headerFooter_.updateValue(this.appState_.getField(
-            print_preview.AppState.Field.IS_HEADER_FOOTER_ENABLED));
+        this.headerFooter_.updateValue(
+            /** @type {!Object} */(this.appState_.getField(
+            print_preview.AppState.Field.IS_HEADER_FOOTER_ENABLED)));
       }
       if (this.appState_.hasField(
           print_preview.AppState.Field.IS_COLLATE_ENABLED)) {
-        this.collate_.updateValue(this.appState_.getField(
-            print_preview.AppState.Field.IS_COLLATE_ENABLED));
+        this.collate_.updateValue(
+            /** @type {!Object} */(this.appState_.getField(
+            print_preview.AppState.Field.IS_COLLATE_ENABLED)));
       }
       if (this.appState_.hasField(
           print_preview.AppState.Field.IS_CSS_BACKGROUND_ENABLED)) {
-        this.cssBackground_.updateValue(this.appState_.getField(
-            print_preview.AppState.Field.IS_CSS_BACKGROUND_ENABLED));
+        this.cssBackground_.updateValue(
+            /** @type {!Object} */(this.appState_.getField(
+            print_preview.AppState.Field.IS_CSS_BACKGROUND_ENABLED)));
       }
       if (this.appState_.hasField(
           print_preview.AppState.Field.VENDOR_OPTIONS)) {
-        this.vendorItems_.updateValue(this.appState_.getField(
-            print_preview.AppState.Field.VENDOR_OPTIONS));
+        this.vendorItems_.updateValue(
+            /** @type {!Object.<string,string>} */(this.appState_.getField(

[Aleksey Shlyapnikov, 2014/09/22 22:00:40]

Space after comma.

[Vitaly Pavlenko, 2014/09/22 22:33:11]

Done.

+            print_preview.AppState.Field.VENDOR_OPTIONS)));
     }
     },
 
     /**
      * @return {boolean} {@code true} if the stored print ticket is valid,
      *     {@code false} otherwise.
      */
     isTicketValid: function() {
       return this.isTicketValidForPreview() &&
           (!this.copies_.isCapabilityAvailable() || this.copies_.isValid()) &&
@@ skipping 154 matching lines @@
     onDocumentInfoChange_: function() {
       cr.dispatchSimpleEvent(this, PrintTicketStore.EventType.DOCUMENT_CHANGE);
     },
   };
 
   // Export
   return {
     PrintTicketStore: PrintTicketStore
   };
 });