MD History: Delete button in the toolbar allows deletion of multiple history-items.

chrome/browser/resources/md_history/history_item.js

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 2497db1bdbde186501d92266724742850b638673..519ce625ffaa190f4731c2a15ad1785e50878381 100644
--- a/chrome/browser/resources/md_history/history_item.js
+++ b/chrome/browser/resources/md_history/history_item.js
@@ -89,5 +89,9 @@ Polymer({
     });
     // Stops the 'tap' event from closing the menu when it opens.
     e.stopPropagation();
+  },
+
+  selectionAllowed_: function() {

[tsergeant, 2016/01/25 01:43:15]

The truthiness of your variable/function names flip around a lot:

In the html:
disabled = selectionAllowed

In the js:
selectionAllowed = not allowDeletingHistory

If you think through it, you can figure out that it's correct. It's really
confusing to read, though, since something 'truthy' is being set to something
'falsey'.

Can you rename things to make this clearer?

[hsampson, 2016/01/25 04:15:42]

Done.

+    return !loadTimeData.getBoolean('allowDeletingHistory');
   }
 });