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

Unified Diff: chrome/browser/resources/md_history/list_container.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/list_container.js
diff --git a/chrome/browser/resources/md_history/list_container.js b/chrome/browser/resources/md_history/list_container.js
index 94e545e73ac6d1274dd9fb16bc4bb7fbcbdeb6fb..772a9a62ea0ad6aef9992e2ef86f9921f10f1e3d 100644
--- a/chrome/browser/resources/md_history/list_container.js
+++ b/chrome/browser/resources/md_history/list_container.js
@@ -6,15 +6,6 @@ Polymer({
is: 'history-list-container',
properties: {
- // The path of the currently selected page.
- selectedPage_: {
- type: String,
- computed: 'computeSelectedPage_(queryState.range)',
- },
-
- // Whether domain-grouped history is enabled.
- grouped: Boolean,
-
/** @type {!QueryState} */
queryState: Object,
@@ -54,14 +45,7 @@ Polymer({
}
var list = /** @type {HistoryListBehavior} */ this.getSelectedList_();
- // It is possible for results to arrive for the grouped list before the lazy
- // load has finished and the <history-grouped-list> element exists. In this
- // case, add the items to a property on the unresolved element which can be
- // read when it upgrades and is attached.
- if (Polymer.isInstance(list))
- list.addNewResults(results, this.queryState.incremental, info.finished);
- else
- list.initialData = results;
+ list.addNewResults(results, this.queryState.incremental, info.finished);
},
historyDeleted: function() {
@@ -108,15 +92,6 @@ Polymer({
},
/**
- * @param {HistoryRange} range
- * @return {string}
- * @private
- */
- computeSelectedPage_: function(range) {
- return range == HistoryRange.ALL_TIME ? 'infinite-list' : 'grouped-list';
- },
-
- /**
* @param {HistoryQuery} info
* @param {!Array<HistoryEntry>} results
* @private
@@ -233,7 +208,7 @@ Polymer({
* @private
*/
getSelectedList_: function() {
- return this.$$('#' + this.selectedPage_);
+ return this.$['infinite-list'];
},
/** @private */
« no previous file with comments | « chrome/browser/resources/md_history/list_container.html ('k') | chrome/browser/resources/md_history/query_manager.js » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698