Print Preview: Fix compile errors in settings/ directory

chrome/browser/resources/print_preview/settings/advanced_settings/advanced_settings_item.js

 // Copyright 2014 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.
+/**
+ * Specifies a custom vendor capability.
+ * @typedef {{
+ *   id: (string),
+ *   display_name: (string),
+ *   localized_display_name: (string | undefined),
+ *   type: (string),
+ *   select_cap: ({
+ *     option: (Array<{
+ *       display_name: (string),
+ *       type: (string | undefined),
+ *       value: (number | string | boolean),
+ *       is_default: (boolean | undefined)
+ *     }>|undefined)
+ *   }|undefined)
+ * }}
+ */
+print_preview.VendorCapability;
 
 cr.define('print_preview', function() {
   'use strict';
 
   /**
    * Component that renders a destination item in a destination list.
-   * @param {!cr.EventTarget} eventTarget Event target to dispatch selection
-   *     events to.
    * @param {!print_preview.PrintTicketStore} printTicketStore Contains the
    *     print ticket to print.
-   * @param {!Object} capability Capability to render.
+   * @param {!print_preview.VendorCapability} capability Capability to render.
    * @constructor
    * @extends {print_preview.Component}
    */
-  function AdvancedSettingsItem(eventTarget, printTicketStore, capability) {
+  function AdvancedSettingsItem(printTicketStore, capability) {
     print_preview.Component.call(this);
 
     /**
-     * Event target to dispatch selection events to.
-     * @private {!cr.EventTarget}
-     */
-    this.eventTarget_ = eventTarget;
-
-    /**
      * Contains the print ticket to print.
      * @private {!print_preview.PrintTicketStore}
      */
     this.printTicketStore_ = printTicketStore;
 
     /**
      * Capability this component renders.
-     * @private {!Object}
+     * @private {!print_preview.VendorCapability}
      */
     this.capability_ = capability;
 
     /**
      * Value selected by user. {@code null}, if user has not changed the default
      * value yet (still, the value can be the default one, if it is what user
      * selected).
      * @private {?string}
      */
     this.selectedValue_ = null;
 
     /**
      * Active filter query.
      * @private {RegExp}
      */
     this.query_ = null;
 
     /**
      * Search hint for the control.
      * @private {print_preview.SearchBubble}
      */
     this.searchBubble_ = null;
 
     /** @private {!EventTracker} */
     this.tracker_ = new EventTracker();
-  };
+  }
 
   AdvancedSettingsItem.prototype = {
     __proto__: print_preview.Component.prototype,
 
     /** @override */
     createDom: function() {
       this.setElementInternal(this.cloneTemplateInternal(
           'advanced-settings-item-template'));
 
       this.tracker_.add(
-          this.select_, 'change', this.onSelectChange_.bind(this));
-      this.tracker_.add(this.text_, 'input', this.onTextInput_.bind(this));
+          assert(this.select_), 'change', this.onSelectChange_.bind(this));
+      this.tracker_.add(
+          assert(this.text_), 'input', this.onTextInput_.bind(this));
 
       this.initializeValue_();
 
       this.renderCapability_();
     },
 
     /**
      * ID of the corresponding vendor capability.
      * @return {string}
      */
@@ skipping 25 matching lines @@
 
     get searchBubbleShown() {
       return getIsVisible(this.getElement()) && !!this.searchBubble_;
     },
 
     /**
      * @return {HTMLSelectElement} Select element.
      * @private
      */
     get select_() {
-      return this.getChildElement(
-          '.advanced-settings-item-value-select-control');
+      return /** @type {HTMLSelectElement} */(
+          this.getChildElement('.advanced-settings-item-value-select-control'));
     },
 
     /**
      * @return {HTMLSelectElement} Text element.
      * @private
      */
     get text_() {
-      return this.getChildElement('.advanced-settings-item-value-text-control');
+      return /** @type {HTMLSelectElement} */(

[dpapad, 2017/05/04 01:20:04]

How about casting to !HTMLSelectElement here, and change the @return too, and
avoid having to assert() when using text_? Similar for select_ above.

[rbpotter, 2017/05/04 01:43:07]

Done.

+          this.getChildElement('.advanced-settings-item-value-text-control'));
     },
 
     /**
      * Called when the select element value is changed.
      * @private
      */
     onSelectChange_: function() {
       this.selectedValue_ = this.select_.value;
     },
 
@@ skipping 110 matching lines @@
         this.initializeSelectValue_();
       else
         this.initializeTextValue_();
     },
 
     /**
      * Initializes the select element.
      * @private
      */
     initializeSelectValue_: function() {
-      setIsVisible(
-          this.getChildElement('.advanced-settings-item-value-select'), true);
+      setIsVisible(assert(this.getChildElement(
+          '.advanced-settings-item-value-select')), true);
       var selectEl = this.select_;
       var indexToSelect = 0;
       this.capability_.select_cap.option.forEach(function(option, index) {
         var item = document.createElement('option');
         item.text = this.getEntityDisplayName_(option);
         item.value = option.value;
         if (option.is_default)
           indexToSelect = index;
         selectEl.appendChild(item);
       }, this);
       for (var i = 0, option; option = selectEl.options[i]; i++) {
         if (option.value == this.selectedValue_) {
           indexToSelect = i;
           break;
         }
       }
       selectEl.selectedIndex = indexToSelect;
     },
 
     /**
      * Initializes the text element.
      * @private
      */
     initializeTextValue_: function() {
-      setIsVisible(
-          this.getChildElement('.advanced-settings-item-value-text'), true);
+      setIsVisible(assert(this.getChildElement(
+          '.advanced-settings-item-value-text')), true);
 
       var defaultValue = null;
       if (this.capability_.type == 'TYPED_VALUE' &&
           this.capability_.typed_value_cap) {
         defaultValue = this.capability_.typed_value_cap.default || null;
       } else if (this.capability_.type == 'RANGE' &&
                  this.capability_.range_cap) {
         defaultValue = this.capability_.range_cap.default || null;
       }
 
@@ skipping 21 matching lines @@
         }
       });
     }
   };
 
   // Export
   return {
     AdvancedSettingsItem: AdvancedSettingsItem
   };
 });