[MD History] Factor out a common HistoryListBehavior.

chrome/browser/resources/md_history/list_container.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: 'history-list-container',
 
   properties: {
     // The path of the currently selected page.
     selectedPage_: String,
@@ skipping 67 matching lines @@
 
     var maxResults =
       queryState.range == HistoryRange.ALL_TIME ? RESULTS_PER_PAGE : 0;
     chrome.send('queryHistory', [
       queryState.searchTerm, queryState.groupedOffset, queryState.range,
       lastVisitTime, maxResults
     ]);
   },
 
   unselectAllItems: function(count) {
-    /** @type {HistoryListElement} */ (this.$['infinite-list'])
-        .unselectAllItems(count);
+    this.$.content.selectedItem.unselectAllItems(count);

[tsergeant, 2016/08/05 01:39:09]

Nit: Maybe extract this.$.content.selectedItem into a getter which returns a
HistoryListBehavior.

The idea being that then we are forced by closure in this file to not know which
type of list we are dealing with.

[calamity, 2016/08/09 02:56:00]

So this is exactly why I was asking how behaviors worked with Closure yesterday.
I can't seem to get a Type for HistoryListBehavior =(

[tsergeant, 2016/08/09 04:37:54]

That's unfortunate. I think it would be worthwhile to write a getter if you can
anyway, though.

[calamity, 2016/08/09 05:28:30]

Done.

   },
 
   /**
    * Delete all the currently selected history items. Will prompt the user with
    * a dialog to confirm that the deletion should be performed.
    */
   deleteSelectedWithPrompt: function() {
     if (!loadTimeData.getBoolean('allowDeletingHistory'))
       return;
 
@@ skipping 33 matching lines @@
           info.term == '' ? results[i].dateTimeOfDay : results[i].dateShort;
 
       if (results[i].dateRelativeDay != currentDate) {
         currentDate = results[i].dateRelativeDay;
       }
     }
   },
 
   /** @private */
   onDialogConfirmTap_: function() {
-    this.$['infinite-list'].deleteSelected();
+    this.$.content.selectedItem.deleteSelected();
     this.$.dialog.close();
   },
 
   /** @private */
   onDialogCancelTap_: function() {
     this.$.dialog.close();
   },
 
   /**
    * Closes the overflow menu.
@@ skipping 21 matching lines @@
     this.fire('search-domain', {domain: menu.itemData.domain});
     menu.closeMenu();
   },
 
   /** @private */
   onRemoveFromHistoryTap_: function() {
     var menu = /** @type {CrSharedMenuElement} */(this.$.sharedMenu);
     md_history.BrowserService.getInstance()
         .deleteItems([menu.itemData])
         .then(function(items) {
-          this.$['infinite-list'].removeDeletedHistory_(items);
+          this.$.content.selectedItem.removeItemsByPath(items[0].path);
           // This unselect-all is to reset the toolbar when deleting a selected
           // item. TODO(tsergeant): Make this automatic based on observing list
           // modifications.
           this.fire('unselect-all');
         }.bind(this));
     menu.closeMenu();
   },
 });