MD History: Remove bookmark when clicking bookmark star

chrome/browser/resources/md_history/history_list.js

 // Copyright 2015 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',
 
   properties: {
     // The search term for the current query. Set when the query returns.
     searchedTerm: {
@@ skipping 11 matching lines @@
     resultLoadingDisabled_: {
       type: Boolean,
       value: false,
     },
   },
 
   listeners: {
     'infinite-list.scroll': 'closeMenu_',
     'tap': 'closeMenu_',
     'toggle-menu': 'toggleMenu_',
+    'remove-bookmark-stars': 'removeBookmarkStars_',
   },
 
   /**
    * Closes the overflow menu.
    * @private
    */
   closeMenu_: function() {
     /** @type {CrSharedMenuElement} */(this.$.sharedMenu).closeMenu();
   },
 
   /**
    * Opens the overflow menu unless the menu is already open and the same button
    * is pressed.
    * @param {{detail: {item: !HistoryEntry, target: !HTMLElement}}} e
    * @private
    */
   toggleMenu_: function(e) {
     var target = e.detail.target;
     /** @type {CrSharedMenuElement} */(this.$.sharedMenu).toggleMenu(
         target, e.detail.item);
   },
 
+  /**
+   * Remove bookmark star of an url's all history items.

[calamity, 2016/06/29 07:36:05]

nit: Remove bookmark star for history items with matching URLs.

[lshang, 2016/07/01 01:15:16]

Done.

+   * @param {{detail: !string}} e
+   * @private
+   */
+  removeBookmarkStars_: function(e) {
+    var url = e.detail;
+
+    if (this.historyData_ === undefined)
+      return;
+
+    for (var i = 0; i < this.historyData_.length; i++) {
+      if (this.historyData_[i].url == url) {

[calamity, 2016/06/29 07:36:05]

nit: No braces needed for single line if.

[lshang, 2016/07/01 01:15:16]

Done.

+        this.set('historyData_.' + i + '.starred', false);
+      }
+    }
+  },
+
   /** @private */
   onMoreFromSiteTap_: function() {
     var menu = /** @type {CrSharedMenuElement} */(this.$.sharedMenu);
     this.fire('search-domain', {domain: menu.itemData.domain});
     menu.closeMenu();
   },
 
   /** @private */
   onRemoveFromHistoryTap_: function() {
     var menu = /** @type {CrSharedMenuElement} */(this.$.sharedMenu);
@@ skipping 183 matching lines @@
 
   /**
    * @param {number} index
    * @return {boolean}
    * @private
    */
   isFirstItem_: function(index) {
     return index == 0;
   }
 });