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

Unified Diff: chrome/browser/resources/md_history/history_item.js

Issue 2672723002: [MD History] Disable Clear Browsing Data button for supervised users. (Closed)
Patch Set: fix_test Created 3 years, 10 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_item.js
diff --git a/chrome/browser/resources/md_history/history_item.js b/chrome/browser/resources/md_history/history_item.js
index 4dca56c9957ce8939a64ff63a13b3f89a65c3358..8f53904e3b0abbe4baf956745309294188f6214e 100644
--- a/chrome/browser/resources/md_history/history_item.js
+++ b/chrome/browser/resources/md_history/history_item.js
@@ -111,6 +111,11 @@ cr.define('md_history', function() {
observer: 'ironListTabIndexChanged_',
},
+ selectionNotAllowed_: {
+ type: Boolean,
+ value: !loadTimeData.getBoolean('allowDeletingHistory'),
+ },
+
hasTimeGap: Boolean,
index: Number,
@@ -189,7 +194,7 @@ cr.define('md_history', function() {
}
}
- if (this.selectionNotAllowed_())
+ if (this.selectionNotAllowed_)
return;
this.$.checkbox.focus();
@@ -302,10 +307,6 @@ cr.define('md_history', function() {
this.$.icon.style.backgroundImage = cr.icon.getFavicon(this.item.url);
},
- selectionNotAllowed_: function() {
- return !loadTimeData.getBoolean('allowDeletingHistory');
- },
-
/**
* @param {number} numberOfItems The number of items in the card.
* @param {string} historyDate Date of the current result.
« no previous file with comments | « chrome/browser/resources/md_history/history_item.html ('k') | chrome/browser/resources/md_history/history_toolbar.js » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698