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

Side by Side Diff: chrome/browser/resources/md_history/app.js

Issue 2088093004: Add a footer about other forms of history to the MD history page. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 4 years, 6 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 unified diff | Download patch
OLDNEW
1 // Copyright 2016 The Chromium Authors. All rights reserved. 1 // Copyright 2016 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 /** 5 /**
6 * @typedef {{querying: boolean, 6 * @typedef {{querying: boolean,
7 * searchTerm: string, 7 * searchTerm: string,
8 * results: ?Array<!HistoryEntry>, 8 * results: ?Array<!HistoryEntry>,
9 * info: ?HistoryQuery, 9 * info: ?HistoryQuery,
10 * incremental: boolean, 10 * incremental: boolean,
11 * range: HistoryRange, 11 * range: HistoryRange,
12 * groupedOffset: number, 12 * groupedOffset: number,
13 * sessionList: ?Array<!ForeignSession>}} 13 * sessionList: ?Array<!ForeignSession>}}
14 */ 14 */
15 var QueryState; 15 var QueryState;
16 16
17 Polymer({ 17 Polymer({
18 is: 'history-app', 18 is: 'history-app',
19 19
20 properties: { 20 properties: {
21 // Whether to display the footer.
22 showFooter: Boolean,
23
21 // The id of the currently selected page. 24 // The id of the currently selected page.
22 selectedPage_: String, 25 selectedPage_: String,
23 26
24 // Whether domain-grouped history is enabled. 27 // Whether domain-grouped history is enabled.
25 grouped_: { 28 grouped_: {
26 type: Boolean, 29 type: Boolean,
27 reflectToAttribute: true 30 reflectToAttribute: true
28 }, 31 },
29 32
30 // Whether the first set of results have returned. 33 // Whether the first set of results have returned.
(...skipping 218 matching lines...) Expand 10 before | Expand all | Expand 10 after
249 * @param {boolean} incremental 252 * @param {boolean} incremental
250 * @param {string} searchTerm 253 * @param {string} searchTerm
251 * @return {boolean} Whether a loading spinner should be shown (implies the 254 * @return {boolean} Whether a loading spinner should be shown (implies the
252 * backend is querying a new search term). 255 * backend is querying a new search term).
253 * @private 256 * @private
254 */ 257 */
255 shouldShowSpinner_: function(querying, incremental, searchTerm) { 258 shouldShowSpinner_: function(querying, incremental, searchTerm) {
256 return querying && !incremental && searchTerm != ''; 259 return querying && !incremental && searchTerm != '';
257 } 260 }
258 }); 261 });
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698