Chromium Code Reviews| Index: chrome/browser/resources/md_history/app.js |
| diff --git a/chrome/browser/resources/md_history/app.js b/chrome/browser/resources/md_history/app.js |
| index 0d68ff73cf2a85e2e6e4b62ed74c743d8a9261c9..1a6c0916067ec17450e2facef02e6a5339588a1a 100644 |
| --- a/chrome/browser/resources/md_history/app.js |
| +++ b/chrome/browser/resources/md_history/app.js |
| @@ -92,16 +92,14 @@ Polymer({ |
| * @param {boolean} isTabSyncEnabled Is tab sync enabled for this profile? |
| */ |
| setForeignSessions: function(sessionList, isTabSyncEnabled) { |
| - // TODO(calamity): Add a 'no synced devices' message when sessions are |
|
tsergeant
2016/05/05 17:59:09
This TODO is still relevant.
calamity
2016/05/05 18:49:33
Reinstated.
|
| - // empty. |
| - this.$['history-side-bar'].hidden = !isTabSyncEnabled; |
| + if (!isTabSyncEnabled) |
| + return; |
| + |
| var syncedDeviceElem = this.$['history-synced-device-manager']; |
| var syncedDeviceManager = |
| /** @type {HistorySyncedDeviceManagerElement} */(syncedDeviceElem); |
| - if (isTabSyncEnabled) { |
| - syncedDeviceManager.setSyncedHistory(sessionList); |
| - /** @type {HistoryToolbarElement} */(this.$.toolbar).hasSidebar = true; |
| - } |
| + syncedDeviceManager.setSyncedHistory(sessionList); |
| + /** @type {HistoryToolbarElement} */(this.$.toolbar).hasSidebar = true; |
|
tsergeant
2016/05/05 17:59:09
Can you please change the implementation of histor
calamity
2016/05/05 18:49:33
Good point. Done. I'll remove more of the extraneo
|
| }, |
| deleteComplete: function() { |