MD History: Refresh the list when clearing browsing data

chrome/browser/resources/md_history/list_container.js

Index: chrome/browser/resources/md_history/list_container.js
diff --git a/chrome/browser/resources/md_history/list_container.js b/chrome/browser/resources/md_history/list_container.js
index 0c79ae11e590ad084ac77391bf1278a88820d4a7..4ab5e77df98e0c7836ee511153a41892f212ffdf 100644
--- a/chrome/browser/resources/md_history/list_container.js
+++ b/chrome/browser/resources/md_history/list_container.js
@@ -44,7 +44,8 @@ Polymer({
     }
 
     var list = /** @type {HistoryListElement} */(this.$['infinite-list']);
-    list.addNewResults(results);
+    list.addNewResults(results, this.queryState.incremental,
+        this.getSelectedItemCount());

[tsergeant, 2016/08/17 06:58:28]

For simplicity, I would suggest only checking the selected item count before you
reload the list (in clearHistory below). If there are items selected, exit
without trying to reload.

That way, you don't need to store clearingHistory_ and it simplifies the logic a
lot. It's possible for the reload to go through when it shouldn't, but that case
would be very difficult to trigger. 

[lshang, 2016/08/18 08:41:54]

Done. Great idea!

     if (info.finished)
       list.disableResultLoading();
   },
@@ -87,6 +88,11 @@ Polymer({
     ]);
   },
 
+  clearingHistory: function() {

[tsergeant, 2016/08/17 06:58:28]

For method names, use the plain verb form:

clearHistory()

For properties, you can still use the participle form:

this.clearingHistory_

to indicate that it's a continuous state.

[lshang, 2016/08/18 08:41:54]

Done.

+    /** @type {HistoryListElement} */ (this.$['infinite-list'])
+        .clearingHistory();
+  },
+
   /** @return {number} */
   getSelectedItemCount: function() {
     return this.getSelectedList_().selectedPaths.size;