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

Unified Diff: chrome/browser/resources/md_history/history_list_behavior.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_list_behavior.js
diff --git a/chrome/browser/resources/md_history/history_list_behavior.js b/chrome/browser/resources/md_history/history_list_behavior.js
index 0a6d4393873bd6cf21308d8f844b9b6015d9d4a6..cc3111eaf0e21e374399061334fdaef5740dabcf 100644
--- a/chrome/browser/resources/md_history/history_list_behavior.js
+++ b/chrome/browser/resources/md_history/history_list_behavior.js
@@ -213,10 +213,12 @@ var HistoryListBehavior = {
if (paths.length == 0)
paths.push(item.path);
+ var selected = !this.selectedPaths.has(item.path);
+
paths.forEach(function(path) {
- this.set(path + '.selected', item.selected);
+ this.set(path + '.selected', selected);
- if (item.selected) {
+ if (selected) {
this.selectedPaths.add(path);
return;
}
« no previous file with comments | « chrome/browser/resources/md_history/history_item.js ('k') | chrome/test/data/webui/md_history/history_grouped_list_test.js » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698