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

Unified Diff: chrome/browser/resources/md_history/history_toolbar.html

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: add tests for deletion and logic for supervised users. 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.html
diff --git a/chrome/browser/resources/md_history/history_toolbar.html b/chrome/browser/resources/md_history/history_toolbar.html
index 2aa251e4e193a91679f1602d04b837deff6de494..eabf0aa470f6e4747dfdd71b34dca8f9f26ce196 100644
--- a/chrome/browser/resources/md_history/history_toolbar.html
+++ b/chrome/browser/resources/md_history/history_toolbar.html
@@ -66,7 +66,11 @@
<paper-button id="cancel-button" on-tap="clearSelection">
CANCEL
</paper-button>
- <paper-button id="delete-button">DELETE</paper-button>
+ <template is="dom-if" if="{{deletingAllowed_()}}">
+ <paper-button id="delete-button" on-tap="deleteSelection">
+ DELETE
+ </paper-button>
+ </template>
</div>
</template>

Powered by Google App Engine
This is Rietveld 408576698