MD History: Replace last usage of cr-shared-menu with cr-action-menu.

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: {
     /** @type {HistoryRange} */
     groupedRange: {
       type: Number,
       observer: 'groupedRangeChanged_',
     },
 
     // The path of the currently selected page.
     selectedPage_: String,
 
     // Whether domain-grouped history is enabled.
     grouped: Boolean,
 
     /** @type {!QueryState} */
     queryState: Object,
 
     /** @type {!QueryResult} */
     queryResult: Object,
+
+    /**
+     * @private {?{
+     *    index: number, item: !HistoryEntry,
+     *    path: string, target: !HTMLElement

[Dan Beam, 2016/12/21 22:06:38]

indent off, why are you wrapping like this?

[dpapad, 2016/12/21 22:41:09]

Done. If only there was a linter to tell me how to do this (and that also
operated on JSDoc)

+     * }}
+     */
+    actionMenuModel_: Object,
   },
 
   observers: [
     'searchTermChanged_(queryState.searchTerm)',
     'groupedOffsetChanged_(queryState.groupedOffset)',
   ],
 
   listeners: {
-    'history-list-scrolled': 'closeMenu_',
     'load-more-history': 'loadMoreHistory_',
-    'toggle-menu': 'toggleMenu_',
+    'open-menu': 'openMenu_',
   },
 
   /**
    * @param {HistoryQuery} info An object containing information about the
    *    query.
    * @param {!Array<!HistoryEntry>} results A list of results.
    */
   historyResult: function(info, results) {
     this.initializeResults_(info, results);
     this.closeMenu_();
@@ skipping 168 matching lines @@
     var dialog = assert(this.$.dialog.getIfExists());
     dialog.close();
   },
 
   /**
    * Closes the overflow menu.
    * @private
    */
   closeMenu_: function() {
     var menu = this.$.sharedMenu.getIfExists();
-    if (menu)
-      menu.closeMenu();
+    if (menu && menu.open) {
+      this.actionMenuModel_ = null;
+      menu.close();
+    }
   },
 
   /**
-   * Opens the overflow menu unless the menu is already open and the same button
-   * is pressed.
-   * @param {{detail: {item: !HistoryEntry, target: !HTMLElement}}} e
+   * Opens the overflow menu.
+   * @param {{detail: {
+   *    index: number, item: !HistoryEntry,
+   *    path: string, target: !HTMLElement
+   * }}} e
    * @private
    */
-  toggleMenu_: function(e) {
+  openMenu_: function(e) {
     var target = e.detail.target;
+    this.actionMenuModel_ = e.detail;
     var menu = /** @type {CrSharedMenuElement} */ this.$.sharedMenu.get();
-    menu.toggleMenu(target, e.detail);
+    menu.showAt(target);
   },
 
   /** @private */
   onMoreFromSiteTap_: function() {
     md_history.BrowserService.getInstance().recordAction(
         'EntryMenuShowMoreFromSite');
 
     var menu = assert(this.$.sharedMenu.getIfExists());
-    this.set('queryState.searchTerm', menu.itemData.item.domain);
-    menu.closeMenu();
+    this.set('queryState.searchTerm', this.actionMenuModel_.item.domain);
+    this.actionMenuModel_ = null;
+    this.closeMenu_();
   },
 
   /** @private */
   onRemoveFromHistoryTap_: function() {
     var browserService = md_history.BrowserService.getInstance();
     browserService.recordAction('EntryMenuRemoveFromHistory');
     var menu = assert(this.$.sharedMenu.getIfExists());
-    var itemData = menu.itemData;
+    var itemData = this.actionMenuModel_;
     browserService.deleteItems([itemData.item]).then(function(items) {
       // This unselect-all resets the toolbar when deleting a selected item
       // and clears selection state which can be invalid if items move
       // around during deletion.
       // TODO(tsergeant): Make this automatic based on observing list
       // modifications.
       this.fire('unselect-all');
       this.getSelectedList_().removeItemsByPath([itemData.path]);
 
       var index = itemData.index;
       if (index == undefined)
         return;
 
       var browserService = md_history.BrowserService.getInstance();
       browserService.recordHistogram(
           'HistoryPage.RemoveEntryPosition',
           Math.min(index, UMA_MAX_BUCKET_VALUE), UMA_MAX_BUCKET_VALUE);
       if (index <= UMA_MAX_SUBSET_BUCKET_VALUE) {
         browserService.recordHistogram(
             'HistoryPage.RemoveEntryPositionSubset', index,
             UMA_MAX_SUBSET_BUCKET_VALUE);
       }
     }.bind(this));
-    menu.closeMenu();
+    this.closeMenu_();
   },
 
   /**
    * @return {Element}
    * @private
    */
   getSelectedList_: function() { return this.$$('#' + this.selectedPage_); },
 
   /** @private */
   canDeleteHistory_: function() {
     return loadTimeData.getBoolean('allowDeletingHistory');
   }
 });