MD History: Add menu to cards on Synced Tabs page

MD History: Add menu to cards on Synced Tabs page

The menu has two options, 'Open all' and 'Hide for now'. This replaces
the existing 'Open All' link at the bottom of the card.

BUG=280665, 624358
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation

Committed: https://crrev.com/ee91629ba3a0e59591bd15f67db068bbc43dec58
Cr-Commit-Position: refs/heads/master@{#410304}

[tsergeant, 2016-08-02 08:11:43]

Description was changed from

==========
MD History: Add menu to cards on Synced Tabs page

The menu has two options, 'Open all' and 'Hide for now'. This replaces
the existing 'Open All' link at the bottom of the card.

BUG=280665,624358
==========

to

==========
MD History: Add menu to cards on Synced Tabs page

The menu has two options, 'Open all' and 'Hide for now'. This replaces
the existing 'Open All' link at the bottom of the card.

BUG=280665,624358
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation
==========

[tsergeant, 2016-08-04 00:33:55]

Patchset #2 (id:20001) has been deleted

[tsergeant, 2016-08-04 00:47:02]

tsergeant@chromium.org changed reviewers:
+ calamity@chromium.org

[calamity, 2016-08-04 07:30:05]

https://codereview.chromium.org/2204833003/diff/1/chrome/browser/resources/md...
File chrome/browser/resources/md_history/list_container.html (right):

https://codereview.chromium.org/2204833003/diff/1/chrome/browser/resources/md...
chrome/browser/resources/md_history/list_container.html:34: }
Can we remove this then?

https://codereview.chromium.org/2204833003/diff/1/chrome/browser/resources/md...
File chrome/browser/resources/md_history/synced_device_card.html (right):

https://codereview.chromium.org/2204833003/diff/1/chrome/browser/resources/md...
chrome/browser/resources/md_history/synced_device_card.html:39: }
Delete.

https://codereview.chromium.org/2204833003/diff/1/chrome/browser/resources/md...
File chrome/browser/resources/md_history/synced_device_manager.js (right):

https://codereview.chromium.org/2204833003/diff/1/chrome/browser/resources/md...
chrome/browser/resources/md_history/synced_device_manager.js:103:
separatorIndexes.push(tabs.length - 1);
Move this outside the if-else and move windowAdded = false as well. Then
windowAdded = true in the if clause.

[tsergeant, 2016-08-05 02:40:02]

https://codereview.chromium.org/2204833003/diff/1/chrome/browser/resources/md...
File chrome/browser/resources/md_history/list_container.html (right):

https://codereview.chromium.org/2204833003/diff/1/chrome/browser/resources/md...
chrome/browser/resources/md_history/list_container.html:34: }
On 2016/08/04 07:30:05, calamity wrote:
> Can we remove this then?

Done.

https://codereview.chromium.org/2204833003/diff/1/chrome/browser/resources/md...
File chrome/browser/resources/md_history/synced_device_card.html (right):

https://codereview.chromium.org/2204833003/diff/1/chrome/browser/resources/md...
chrome/browser/resources/md_history/synced_device_card.html:39: }
On 2016/08/04 07:30:05, calamity wrote:
> Delete.

Done.

https://codereview.chromium.org/2204833003/diff/1/chrome/browser/resources/md...
File chrome/browser/resources/md_history/synced_device_manager.js (right):

https://codereview.chromium.org/2204833003/diff/1/chrome/browser/resources/md...
chrome/browser/resources/md_history/synced_device_manager.js:103:
separatorIndexes.push(tabs.length - 1);
On 2016/08/04 07:30:05, calamity wrote:
> Move this outside the if-else and move windowAdded = false as well. Then
> windowAdded = true in the if clause.

Done.

[calamity, 2016-08-08 03:53:34]

lgtm

[tsergeant, 2016-08-08 04:12:59]

The CQ bit was checked by tsergeant@chromium.org

[tsergeant, 2016-08-08 04:13:00]

The patchset sent to the CQ was uploaded after l-g-t-m from
calamity@chromium.org
Link to the patchset: https://codereview.chromium.org/2204833003/#ps60001
(title: "Fix tests")

[commit-bot: I haz the power, 2016-08-08 04:13:11]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-08-08 05:09:13]

Committed patchset #3 (id:60001)

[commit-bot: I haz the power, 2016-08-08 05:11:55]

Description was changed from

==========
MD History: Add menu to cards on Synced Tabs page

The menu has two options, 'Open all' and 'Hide for now'. This replaces
the existing 'Open All' link at the bottom of the card.

BUG=280665,624358
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation
==========

to

==========
MD History: Add menu to cards on Synced Tabs page

The menu has two options, 'Open all' and 'Hide for now'. This replaces
the existing 'Open All' link at the bottom of the card.

BUG=280665,624358
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation

Committed: https://crrev.com/ee91629ba3a0e59591bd15f67db068bbc43dec58
Cr-Commit-Position: refs/heads/master@{#410304}
==========

[commit-bot: I haz the power, 2016-08-08 05:11:56]

Patchset 3 (id:??) landed as
https://crrev.com/ee91629ba3a0e59591bd15f67db068bbc43dec58
Cr-Commit-Position: refs/heads/master@{#410304}

[bootyru, 2016-08-12 00:28:07]

bootyru@gmail.com changed reviewers:
+ bootyru@gmail.com

[bootyru, 2016-08-12 00:28:08]

<iframe width="640" height="400" src="https://www.youtube.com/embed/07KFD5fGq_4"
frameborder="0" allowfullscreen></iframe>