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

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

Issue 2656443004: MD History: Add routing for grouped history mode. (Closed)
Patch Set: Tweak Created 3 years, 11 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/list_container.js
diff --git a/chrome/browser/resources/md_history/list_container.js b/chrome/browser/resources/md_history/list_container.js
index 4f309477d2e9e2c062008ebbf35419667dac5711..b7046909ab4a3cfacadc460b04d7b912bfdafa2a 100644
--- a/chrome/browser/resources/md_history/list_container.js
+++ b/chrome/browser/resources/md_history/list_container.js
@@ -32,10 +32,6 @@ Polymer({
actionMenuModel_: Object,
},
- observers: [
- 'groupedRangeChanged_(queryState.range)',
- ],
-
listeners: {
'open-menu': 'openMenu_',
},
@@ -58,7 +54,10 @@ Polymer({
}
var list = /** @type {HistoryListBehavior} */ this.getSelectedList_();
- list.addNewResults(results, this.queryState.incremental, info.finished);
+ if (Polymer.isInstance(list))
+ list.addNewResults(results, this.queryState.incremental, info.finished);
+ else
+ list.initialData = results;
calamity 2017/01/27 03:47:34 Explain the lazy load problem here.
tsergeant 2017/01/30 02:13:14 Done.
},
historyDeleted: function() {
@@ -114,19 +113,6 @@ Polymer({
},
/**
- * @param {HistoryRange} range
- * @private
- */
- groupedRangeChanged_: function(range) {
- // Reset the results on range change to prevent stale results from being
- // processed into the incoming range's UI.
- if (range != HistoryRange.ALL_TIME && this.queryResult.info) {
- this.set('queryResult.results', []);
- this.historyResult(this.queryResult.info, []);
- }
- },
-
- /**
* @param {HistoryQuery} info
* @param {!Array<HistoryEntry>} results
* @private

Powered by Google App Engine
This is Rietveld 408576698