MD History: Remove bookmark when clicking bookmark star

chrome/browser/resources/md_history/history_item.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.
 
 cr.define('md_history', function() {
   var HistoryItem = Polymer({
     is: 'history-item',
 
     properties: {
       // Underlying HistoryEntry data for this item. Contains read-only fields
@@ skipping 24 matching lines @@
      * or decreases its count of selected items accordingly.
      * @private
      */
     onCheckboxSelected_: function() {
       this.fire('history-checkbox-select', {
         countAddition: this.$.checkbox.checked ? 1 : -1
       });
     },
 
     /**
+     * Remove bookmark of current item when bookmark-star is clicked.
+     * @private
+     */
+    onRemoveBookmarkTap_: function() {
+      chrome.send('removeBookmark', [this.item.url]);

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

Consider moving this into a method in browser_service.js. We want to consolidate
all chrome.send calls there.

Opinions tsergeant?

[tsergeant, 2016/06/30 01:41:10]

Sure, sgtm.

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

Done.

+      this.fire('remove-bookmark-stars', this.item.url);
+    },
+
+    /**
      * Fires a custom event when the menu button is clicked. Sends the details
      * of the history item and where the menu should appear.
      */
     onMenuButtonTap_: function(e) {
       this.fire('toggle-menu', {
         target: Polymer.dom(e).localTarget,
         item: this.item,
       });
 
       // Stops the 'tap' event from closing the menu when it opens.
@@ skipping 47 matching lines @@
 
     if (searchedTerm)
       return currentItem.dateShort != nextItem.dateShort;
 
     return currentItem.time - nextItem.time > BROWSING_GAP_TIME &&
         currentItem.dateRelativeDay == nextItem.dateRelativeDay;
   };
 
   return { HistoryItem: HistoryItem };
 });