Fix all remaining print preview closure compiler errors

chrome/browser/resources/print_preview/search/destination_list_item.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';
 
   /**
    * Component that renders a destination item in a destination list.
    * @param {!cr.EventTarget} eventTarget Event target to dispatch selection
@@ skipping 20 matching lines @@
      * @private
      */
     this.destination_ = destination;
 
     /**
      * Active filter query text.
      * @type {RegExp}
      * @private
      */
     this.query_ = query;
-  };
+  }
 
   /**
    * Event types dispatched by the destination list item.
    * @enum {string}
    */
   DestinationListItem.EventType = {
     // Dispatched to check the printer needs to be configured before activation.
     CONFIGURE_REQUEST: 'print_preview.DestinationListItem.CONFIGURE_REQUEST',
     // Dispatched when the list item is activated.
     SELECT: 'print_preview.DestinationListItem.SELECT',
@@ skipping 16 matching lines @@
       print_preview.Component.prototype.enterDocument.call(this);
       this.tracker.add(this.getElement(), 'click', this.onActivate_.bind(this));
       this.tracker.add(
           this.getElement(), 'keydown', this.onKeyDown_.bind(this));
       this.tracker.add(
           this.getChildElement('.register-promo-button'),
           'click',
           this.onRegisterPromoClicked_.bind(this));
     },
 
-    /** @return {!print_preiew.Destination} */
+    /** @return {!print_preview.Destination} */
     get destination() {
       return this.destination_;
     },
 
     /**
      * Updates the list item UI state.
      * @param {!print_preview.Destination} destination Destination data object
      *     to render.
      * @param {RegExp} query Active filter query.
      */
@@ skipping 145 matching lines @@
           span.className = 'destination-list-item-query-highlight';
           span.textContent = sections[i];
           parent.appendChild(span);
         }
       }
     },
 
     /**
      * Shows/Hides the configuring in progress message and starts/stops its
      * animation accordingly.
-     * @param {bool} show If the message and animation should be shown.
+     * @param {boolean} show If the message and animation should be shown.
      * @private
      */
     updateConfiguringMessage_: function(show) {
       setIsVisible(
           this.getChildElement('.configuring-in-progress-text'), show);
       this.getChildElement('.configuring-text-jumping-dots')
           .classList.toggle('jumping-dots', show);
     },
 
     /**
@@ skipping 29 matching lines @@
       }
     },
 
     /**
      * Called when the key is pressed on the destination item. Dispatches a
      * SELECT event when Enter is pressed.
      * @param {KeyboardEvent} e Keyboard event to process.
      * @private
      */
     onKeyDown_: function(e) {
-      if (!hasKeyModifiers(e)) {
+      if (e && !hasKeyModifiers(e)) {
         if (e.keyCode == 13) {
           var activeElementTag = document.activeElement ?
               document.activeElement.tagName.toUpperCase() : '';
           if (activeElementTag == 'LI') {
             e.stopPropagation();
             e.preventDefault();
             this.onActivate_();
           }
         }
       }
     },
 
     /**
      * Called when the registration promo is clicked.
      * @private
      */
     onRegisterPromoClicked_: function() {
       var promoClickedEvent = new Event(
           DestinationListItem.EventType.REGISTER_PROMO_CLICKED);
       promoClickedEvent.destination = this.destination_;
       this.eventTarget_.dispatchEvent(promoClickedEvent);
     },
 
     /**
      * Handles click and 'Enter' key down events for the extension icon element.
      * It opens extensions page with the extension associated with the
      * destination highlighted.
-     * @param {MouseEvent|KeyboardEvent} e The event to handle.
+     * @param {Event} e The event to handle.

[dpapad, 2017/05/08 20:40:10]

!Event

[rbpotter, 2017/05/08 23:28:20]

This function has to have @param {Event} as the input, since it is assigned to
the onclick event of extensionIconEl, which requires  (function ((Event|null)):
?|null). See
https://cs.chromium.org/chromium/src/chrome/browser/resources/print_preview/s...

      * @private
      */
     onExtensionIconClicked_: function(e) {
       e.stopPropagation();
       window.open('chrome://extensions?id=' + this.destination_.extensionId);
     },
 
     /**
      * Handles key down event for the extensin icon element. Keys different than
      * 'Enter' are ignored.
-     * @param {KeyboardEvent} e The event to handle.
+     * @param {Event} e The event to handle.
      * @private
      */
     onExtensionIconKeyDown_: function(e) {
-      if (hasKeyModifiers(e))
+      if (e && hasKeyModifiers(e))

[dpapad, 2017/05/08 20:40:10]

Can |e| be ever null? If not let's change the annotation to !Event, and remove
the extra condition.

[rbpotter, 2017/05/08 23:28:20]

Same as above for this function - assigned to onkeydown here:
https://cs.chromium.org/chromium/src/chrome/browser/resources/print_preview/s...
Given that the assignment requires it accept a null argument I assume e can be
null, though not sure what would cause that.

         return;
       if (e.keyCode != 13 /* Enter */)
         return;
-      this.onExtensionIconClicked_(event);
+      this.onExtensionIconClicked_(e);
     }
   };
 
   // Export
   return {
     DestinationListItem: DestinationListItem
   };
 });