Print Preview: Fix compile errors in settings/ directory

chrome/browser/resources/print_preview/settings/advanced_options_settings.js

Index: chrome/browser/resources/print_preview/settings/advanced_options_settings.js
diff --git a/chrome/browser/resources/print_preview/settings/advanced_options_settings.js b/chrome/browser/resources/print_preview/settings/advanced_options_settings.js
index e1284a749008675c20693907fa04844d0a27577e..54c1657c057edeac378dbab9292a44867e7d479b 100644
--- a/chrome/browser/resources/print_preview/settings/advanced_options_settings.js
+++ b/chrome/browser/resources/print_preview/settings/advanced_options_settings.js
@@ -7,7 +7,7 @@ cr.define('print_preview', function() {
 
   /**
    * Print options section to control printer advanced options.
-   * @param {!print_preview.ticket_item.VendorItems} ticketItem Ticket item to
+   * @param {!print_preview.ticket_items.VendorItems} ticketItem Ticket item to
    *     check settings availability.
    * @param {!print_preview.DestinationStore} destinationStore Used to determine
    *     the selected destination.
@@ -28,7 +28,7 @@ cr.define('print_preview', function() {
      * @private {!print_preview.DestinationStore}
      */
     this.destinationStore_ = destinationStore;
-  };
+  }
 
   /**
    * Event types dispatched by the component.
@@ -61,7 +61,7 @@ cr.define('print_preview', function() {
       print_preview.SettingsSection.prototype.enterDocument.call(this);
 
       this.tracker.add(
-          this.getButton_(), 'click', function() {
+          assert(this.getButton_()), 'click', function() {

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

Instead of asserting here, can we change getButton_() to return !HTMLElement and
assert there?

/**
 * @return {!HTMLElement}
 * @private
 */
getButton_: function() {
  return assert(this.getChildElement('.advanced-options-settings-button'));
},

Having said that, there is already wider TODO at
https://cs.chromium.org/chromium/src/chrome/browser/resources/print_preview/c...
to make getCHildElement() return !HTMLElement, but probably better to do this
separately (if at all).

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

Done.

             cr.dispatchSimpleEvent(
                 this, AdvancedOptionsSettings.EventType.BUTTON_ACTIVATED);
           }.bind(this));