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

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

Issue 2684493002: MD History: Delete Grouped History (Closed)
Patch Set: Rebase Created 3 years, 10 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 9fbe1d405f9ea862733e32bcbfab786b6e6f2b72..3ad12d92542118e335e875a38353919024b652e7 100644
--- a/chrome/browser/resources/md_history/query_manager.js
+++ b/chrome/browser/resources/md_history/query_manager.js
@@ -17,16 +17,7 @@ Polymer({
// A query is initiated by page load.
querying: true,
queryingDisabled: false,
- _range: HistoryRange.ALL_TIME,
searchTerm: '',
- groupedOffset: 0,
-
- set range(val) {
- this._range = Number(val);
- },
- get range() {
- return this._range;
- },
};
},
},
@@ -79,15 +70,12 @@ Polymer({
lastVisitTime = lastVisit ? Math.floor(lastVisit.time) : 0;
}
- var maxResults =
- this.queryState.range == HistoryRange.ALL_TIME ? RESULTS_PER_PAGE : 0;
-
chrome.send('queryHistory', [
queryState.searchTerm,
- queryState.groupedOffset,
- queryState.range,
+ 0, // No grouped offset.
+ 0, // Disable grouping.
lastVisitTime,
- maxResults,
+ RESULTS_PER_PAGE,
]);
},
@@ -96,29 +84,9 @@ Polymer({
* @private
*/
onChangeQuery_: function(e) {
- var changes =
- /** @type {{range: ?HistoryRange, offset: ?number, search: ?string}} */
- (e.detail);
+ var changes = /** @type {{search: ?string}} */ (e.detail);
var needsUpdate = false;
- if (changes.range != null && changes.range != this.queryState.range) {
- this.set('queryState.range', changes.range);
- needsUpdate = true;
-
- // Reset back to page 0 of the results, unless changing to a specific
- // page.
- if (!changes.offset)
- this.set('queryState.groupedOffset', 0);
-
- this.fire('history-view-changed');
- }
-
- if (changes.offset != null &&
- changes.offset != this.queryState.groupedOffset) {
- this.set('queryState.groupedOffset', changes.offset);
- needsUpdate = true;
- }
-
if (changes.search != null &&
changes.search != this.queryState.searchTerm) {
this.set('queryState.searchTerm', changes.search);
« 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