MD History: Refresh the list when clearing browsing data

chrome/browser/resources/md_history/history_list.js

Index: chrome/browser/resources/md_history/history_list.js
diff --git a/chrome/browser/resources/md_history/history_list.js b/chrome/browser/resources/md_history/history_list.js
index aaba6536622c5929aba6360e9ff38e4a8f659dfc..9da58e4b1a52f8ec55c9b1d6b32ede550e4c0ab1 100644
--- a/chrome/browser/resources/md_history/history_list.js
+++ b/chrome/browser/resources/md_history/history_list.js
@@ -81,10 +81,15 @@ Polymer({
     }
 
     if (this.historyData_) {
-      // If we have previously received data, push the new items onto the
-      // existing array.
-      results.unshift('historyData_');
-      this.push.apply(this, results);

[lshang, 2016/08/04 00:17:04]

These two lines seem not right.

When there are new items coming and the list is refreshed automatically, the
fetched 'results' will include all updated history items including new items and
old items, and 'unshift' method will append 'results' to historyData_, which
will result in historyData_ having 2 copies of old items?

It hasn't caused any problems, which I think is because we now don't support
refreshing history list automatically when user navigates to new websites, so
that these code only get executed when search term changes, which will clear
historyData_ ahead on line 78.

Please correct me if I understand it wrong:-)

[tsergeant, 2016/08/04 01:25:55]

These lines support infinite scrolling. When we do an incremental query, we
receive new data which gets appended directly after the old data.

You'll need to keep this logic in place while adding in separate logic for
refreshing the list.

[lshang, 2016/08/05 03:39:03]

Ah, it's for infinite scrolling! Then it makes sense now. I've added them back.

+      // If we have previously received data, refresh the list unless the user
+      // has any items checked.
+      var selectedItem = this.historyData_.filter(function(item) {
+        return item.selected;
+      });
+      if (selectedItem.length == 0) {
+        this.splice('historyData_', 0, this.historyData_.length);
+        this.set('historyData_', results);
+      }
     } else {
       // The first time we receive data, use set() to ensure the iron-list is
       // initialized correctly.