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

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

Issue 1607403004: MD History: Display synced tabs history (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@second_patch
Patch Set: Style fixes & having no synced history will hide the sidebar Created 4 years, 10 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
(Empty)
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
3 // found in the LICENSE file.
4
5 Polymer({
6 is: 'synced-device-manager',
7
8 properties: {
9 // An array of synced devices with synced tab data.
10 syncedTabs: {
calamity 2016/02/02 04:09:48 syncedDevices?
yingran 2016/02/09 04:21:34 Done.
11 type: Array,
12 value: []
13 }
14 },
15
16 /**
17 * Re-organises the sessionList information.
18 */
19 addSyncedHistory: function(sessionList) {
20 // Does not add more items onto the page when the sessionList updates.
21 // TODO(yingran) : update the cards dynamically by refreshing the tab list
22 // and last update time for each synced tab card.
23 if (this.syncedTabs.length > 0) {
24 return;
25 }
26
27 for (var i = 0; i < sessionList.length && sessionList[i].windows[0]; i++) {
28 this.push('syncedTabs', {
29 device: sessionList[i].name,
30 lastUpdateTime: '- ' + sessionList[i].modifiedTime,
31 tabs: sessionList[i].windows[0].tabs,
calamity 2016/02/02 04:09:48 Only 1 window? We should also have a horizontal li
yingran 2016/02/09 04:21:34 Coming soon, in another CL!
calamity 2016/02/11 00:23:31 Can we have tabs from all windows, unsplit? That w
yingran 2016/02/11 02:06:35 they already appear... as in all tabs from all win
32 });
33 }
34 }
35 });
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698