Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(54)

Issues Search
    My Issues | Starred     Open | Closed | All

Side by Side Diff: chrome/browser/resources/md_history/history-toolbar.js

Issue 1586373002: MD History: Delete button in the toolbar allows deletion of multiple history-items. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@patch_to_be_uploaded
Patch Set: Add TODO and remove console.logs. Created 4 years, 11 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
« chrome/browser/resources/md_history/history-card-manager.js ('K') | « chrome/browser/resources/md_history/history-toolbar.html ('k') | chrome/browser/ui/webui/md_history_ui.cc » ('j') | chrome/browser/ui/webui/md_history_ui.cc » ('J')
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright 2015 The Chromium Authors. All rights reserved. 1 // Copyright 2015 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 Polymer({ 5 Polymer({
6 is: 'history-toolbar', 6 is: 'history-toolbar',
7 properties: { 7 properties: {
8 // Number of history items currently selected. 8 // Number of history items currently selected.
9 count: { 9 count: {
10 type: Number, 10 type: Number,
(...skipping 13 matching lines...) Expand all
24 * This function changes the toolbar background color depending on whether or 24 * This function changes the toolbar background color depending on whether or
25 * not any history items are currently selected. 25 * not any history items are currently selected.
26 * @private 26 * @private
27 */ 27 */
28 changeToolbarView_: function() { 28 changeToolbarView_: function() {
29 this.itemsSelected_ = this.count != 0; 29 this.itemsSelected_ = this.count != 0;
30 }, 30 },
31 31
32 clearSelection: function() { 32 clearSelection: function() {
33 this.fire('unselect-all'); 33 this.fire('unselect-all');
34 },
35
36 deleteSelection: function() {
37 this.fire('delete-selected');
34 } 38 }
35 }); 39 });
OLDNEW
« chrome/browser/resources/md_history/history-card-manager.js ('K') | « chrome/browser/resources/md_history/history-toolbar.html ('k') | chrome/browser/ui/webui/md_history_ui.cc » ('j') | chrome/browser/ui/webui/md_history_ui.cc » ('J')

Powered by Google App Engine
This is Rietveld 408576698