Roll closure compiler

chrome/browser/resources/md_bookmarks/list.js

 // Copyright 2016 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.
 
 Polymer({
   is: 'bookmarks-list',
 
   behaviors: [
     bookmarks.StoreClient,
   ],
@@ skipping 52 matching lines @@
    * focus on incremental update.
    * @param {Array<string>} newValue
    * @param {Array<string>} oldValue
    */
   onDisplayedIdsChanged_: function(newValue, oldValue) {
     if (!oldValue) {
       this.displayedList_ = this.displayedIds_.map(function(id) {
         return {id: id};
       });
     } else {
-      var splices = Polymer.ArraySplice.calculateSplices(newValue, oldValue);
+      var splices = Polymer.ArraySplice.calculateSplices(
+          /** @type {!Array<string>}*/ (newValue),

[dpapad, 2017/05/24 21:22:13]

Nit: Space after "}"
/** @type {!Array<string>} */

[scottchen, 2017/05/25 19:31:26]

There's issues with patching a CL with a jar involved, but I don't have access
to the workstation that uploaded this CL for 2 days, but this is blocking 2
other things, so I will land this now and address the nit in a separate CL.

[dpapad, 2017/05/25 20:34:40]

OK.

+          /** @type {!Array<string>}*/ (oldValue));
       splices.forEach(function(splice) {
         // TODO(calamity): Could use notifySplices to improve performance here.
         var additions =
             newValue.slice(splice.index, splice.index + splice.addedCount)
                 .map(function(id) {
                   return {id: id};
                 });
         this.splice.apply(this, [
           'displayedList_', splice.index, splice.removed.length
         ].concat(additions));
@@ skipping 10 matching lines @@
   /** @private */
   isEmptyList_: function() {
     return this.displayedList_.length == 0;
   },
 
   /** @private */
   deselectItems_: function() {
     this.dispatch(bookmarks.actions.deselectItems());
   },
 });