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

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

Issue 2656443004: MD History: Add routing for grouped history mode. (Closed)
Patch Set: Review comments 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/query_manager.js
diff --git a/chrome/browser/resources/md_history/query_manager.js b/chrome/browser/resources/md_history/query_manager.js
index a2a93bd7b4b9842810804fac5e1e404657c9de51..9fbe1d405f9ea862733e32bcbfab786b6e6f2b72 100644
--- a/chrome/browser/resources/md_history/query_manager.js
+++ b/chrome/browser/resources/md_history/query_manager.js
@@ -33,6 +33,9 @@ Polymer({
/** @type {QueryResult} */
queryResult: Object,
+
+ /** @type {?HistoryRouterElement} */
+ router: Object,
},
observers: [
@@ -63,14 +66,9 @@ Polymer({
*/
queryHistory_: function(incremental) {
var queryState = this.queryState;
- // Disable querying until the first set of results have been returned. If
- // there is a search, query immediately to support search query params from
- // the URL.
- var noResults = !this.queryResult || this.queryResult.results == null;
- if (queryState.queryingDisabled ||
- (!this.queryState.searchTerm && noResults)) {
+
+ if (queryState.queryingDisabled)
return;
- }
this.set('queryState.querying', true);
this.set('queryState.incremental', incremental);
@@ -127,8 +125,11 @@ Polymer({
needsUpdate = true;
}
- if (needsUpdate)
+ if (needsUpdate) {
this.queryHistory_(false);
+ if (this.router)
+ this.router.serializeUrl();
+ }
},
/**
« no previous file with comments | « chrome/browser/resources/md_history/list_container.js ('k') | chrome/browser/resources/md_history/router.js » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698