[MD Bookmarks] Convert bookmark list to iron-list.

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,
   ],
 
   properties: {
-    /** @private {Array<string>} */
+    /** @private {Array<{id: string}>} */
     displayedList_: {
       type: Array,
+      // Use an empty list during initialization so that the databinding to
+      // hide #bookmarksCard takes effect.
       value: function() {
-        // Use an empty list during initialization so that the databinding to
-        // hide #bookmarksCard takes effect.
         return [];
       },
     },
 
+    /** @private {Array<string>} */
+    displayedIds_: {
+      type: Array,
+      observer: 'onDisplayedIdsChanged_',
+    },
+
     /** @private */
     searchTerm_: String,
   },
 
   listeners: {
     'click': 'deselectItems_',
   },
 
   attached: function() {
-    this.watch('displayedList_', function(state) {
+    var list = /** @type {IronListElement} */ (this.$.bookmarksCard);
+    list.scrollTarget = this;
+
+    this.watch('displayedIds_', function(state) {
       return bookmarks.util.getDisplayedList(state);
     });
     this.watch('searchTerm_', function(state) {
       return state.search.term;
     });
     this.updateFromStore();
   },
 
   getDropTarget: function() {
     return this.$.message;
   },
 
+  onDisplayedIdsChanged_: function(newValue, oldValue) {

[tsergeant, 2017/05/16 03:41:51]

Comment here with why all this is necessary/useful

(+ closure annotations)

[calamity, 2017/05/17 03:28:27]

Done.

+    if (!oldValue) {
+      this.displayedList_ = this.displayedIds_.map(function(id) {
+        return {id: id};
+      });
+    } else {
+      var splices = Polymer.ArraySplice.calculateSplices(newValue, 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(
+            'displayedList_', splice.index, splice.removed.length,
+            ...additions);

[tsergeant, 2017/05/16 03:41:51]

Check if uglify errors when parsing this file (I suspect it probably will). If
it does, you'll need to use call() to avoid it.

[calamity, 2017/05/17 03:28:27]

Done.

+      }.bind(this));
+    }
+  },
+
   /** @private */
   emptyListMessage_: function() {
     var emptyListMessage = this.searchTerm_ ? 'noSearchResults' : 'emptyList';
     return loadTimeData.getString(emptyListMessage);
   },
 
   /** @private */
   isEmptyList_: function() {
     return this.displayedList_.length == 0;
   },
 
   /** @private */
   deselectItems_: function() {
     this.dispatch(bookmarks.actions.deselectItems());
   },
 });