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..0cf1617008a0231191fb9e3d69e9aa69ceeebe15 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( |
calamity
2016/07/05 04:03:13
Does this get called when the user signs in for th
msramek
2016/07/06 19:24:10
No - and it probably should.
This is called from
|
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) |
+ .showFooter = includeOtherFormsOfBrowsingHistory; |
calamity
2016/07/05 04:03:13
Just access the sidebar through (appElem).$['side-
msramek
2016/07/06 19:24:10
Done. Also removed the property in history-app.
|
+ }); |
} |
/** |