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..220488aaa338023409effb37309fd6143868f2b9 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) |
+ .$['side-bar'].showFooter = includeOtherFormsOfBrowsingHistory; |
Dan Beam
2016/07/12 02:05:57
can we not poke into the local DOM of #history-app
msramek
2016/07/12 13:56:36
It was specifically calamity@ who suggested this i
|
+ }); |
} |
/** |