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

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

Issue 2122823002: [MD History] Fix issues caused by refactor. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@funky_fresh
Patch Set: fix nit Created 4 years, 5 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/app.js
diff --git a/chrome/browser/resources/md_history/app.js b/chrome/browser/resources/md_history/app.js
index 887fa1cd76ce001e5fe653e783f3b211eaeff057..721e621ac3142c69ce9fc27d7e5ced01f381345a 100644
--- a/chrome/browser/resources/md_history/app.js
+++ b/chrome/browser/resources/md_history/app.js
@@ -21,7 +21,7 @@ Polymer({
incremental: false,
// A query is initiated by page load.
querying: true,
- queryingDisabled: Boolean,
+ queryingDisabled: false,
_range: HistoryRange.ALL_TIME,
searchTerm: '',
// TODO(calamity): Make history toolbar buttons change the offset
@@ -33,7 +33,17 @@ Polymer({
}
},
- sessionList: Array,
+ /** @type {!QueryResult} */
+ queryResult_: {
+ type: Object,
+ value: function() {
+ return {
+ info: null,
+ results: null,
+ sessionList: null,
+ };
+ }
+ },
},
listeners: {
@@ -96,6 +106,8 @@ Polymer({
*/
historyResult: function(info, results) {
this.set('queryState_.querying', false);
+ this.set('queryResult_.info', info);
+ this.set('queryResult_.results', results);
var listContainer =
/** @type {HistoryListContainerElement} */ (this.$['history']);
listContainer.historyResult(info, results);
@@ -116,7 +128,7 @@ Polymer({
if (!isTabSyncEnabled)
return;
- this.sessionList = sessionList;
+ this.set('queryResult_.sessionList', sessionList);
},
/**
« no previous file with comments | « chrome/browser/resources/md_history/app.html ('k') | chrome/browser/resources/md_history/compiled_resources2.gyp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698