Chromium Code Reviews| Index: chrome/browser/resources/md_history/history.js |
| diff --git a/chrome/browser/resources/md_history/history.js b/chrome/browser/resources/md_history/history.js |
| index a81338b4be58bb0d0c77b356e643670d0c62f1d9..ae80df98265459ebea98186a9e2c9065b09f6613 100644 |
| --- a/chrome/browser/resources/md_history/history.js |
| +++ b/chrome/browser/resources/md_history/history.js |
| @@ -47,8 +47,16 @@ function historyResult(info, results) { |
| */ |
| function showNotification( |
| hasSyncedResults, includeOtherFormsOfBrowsingHistory) { |
| - // TODO(msramek): Implement the joint notification about web history and other |
| - // forms of browsing history for the MD history page. |
| + // TODO(msramek): |hasSyncedResults| was used in the old WebUI to show |
| + // the message about other signed-in devices. This message does not exist |
| + // in the MD history anymore, so the parameter is not needed. Remove it |
| + // when WebUI is removed and this becomes the only client of |
| + // BrowsingHistoryHandler. |
| + var appElem = $('history-app'); |
| + waitForUpgrade(appElem).then(function() { |
| + /** @type {HistoryAppElement} */(appElem) |
| + .set('showFooter', includeOtherFormsOfBrowsingHistory); |
|
Dan Beam
2016/06/30 23:30:03
nit: you can just do
.showFooter = includeOther
msramek
2016/07/01 11:42:06
Done.
|
| + }); |
| } |
| /** |