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

Unified 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: Address reviewer comments. 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 side-by-side diff with in-line comments
Download patch
Index: chrome/browser/resources/md_history/history_toolbar.js
diff --git a/chrome/browser/resources/md_history/history_toolbar.js b/chrome/browser/resources/md_history/history_toolbar.js
index 38b1c12e8c71d2c9fe514f087409b95dce27112b..1bcd11a7456a15b9b6184cbb1687d880e2647498 100644
--- a/chrome/browser/resources/md_history/history_toolbar.js
+++ b/chrome/browser/resources/md_history/history_toolbar.js
@@ -31,5 +31,17 @@ Polymer({
onClearSelectionTap_: function() {
this.fire('unselect-all');
+ },
+
+ onDeleteTap_: function() {
+ this.fire('delete-selected');
+ },
+
+ /**
+ * If the user is a supervised user the delete button is not shown.
+ * @private
+ */
+ deletingAllowed_: function() {
+ return loadTimeData.getBoolean('allowDeletingHistory');
}
});

Powered by Google App Engine
This is Rietveld 408576698