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

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

Issue 2336503002: Replace paper-checkbox with a paper-icon-button-light (Closed)
Patch Set: fix closure + slightly more descriptive method Created 4 years, 3 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 4d369a3b423f0a66d1aa760427f3918cde85cb13..3a88c4488ff2e82ea7b04babc7367b64de8cb18e 100644
--- a/chrome/browser/resources/md_history/history_item.js
+++ b/chrome/browser/resources/md_history/history_item.js
@@ -14,7 +14,7 @@ cr.define('md_history', function() {
// Search term used to obtain this history-item.
searchTerm: {type: String},
- selected: {type: Boolean, notify: true},
+ selected: {type: Boolean, reflectToAttribute: true},
isFirstItem: {type: Boolean, reflectToAttribute: true},
@@ -74,6 +74,15 @@ cr.define('md_history', function() {
},
/**
+ * @param {boolean} selected
+ * @return {string}
+ * @private
+ */
+ getAriaChecked_: function(selected) {
+ return selected ? 'true' : 'false';
+ },
+
+ /**
* Remove bookmark of current item when bookmark-star is clicked.
* @private
*/
« no previous file with comments | « chrome/browser/resources/md_history/history_item.html ('k') | chrome/browser/resources/md_history/history_list_behavior.js » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698