Recreate tab switcher stacks/lists when tabs are merged

Recreate tab switcher stacks/lists when tabs are merged

If tabs from two instances of are merged while either the StackLayout
or OverviewListLayout is active, recreate the view so that the merged
tabs show up in the layout.

BUG=602498
Committed: https://crrev.com/cd0688e84198b95815e29ba6127a1ea0c126a9b8
Cr-Commit-Position: refs/heads/master@{#406945}

[Theresa, 2016-07-20 00:09:06]

The CQ bit was checked by twellington@chromium.org to run a CQ dry run

[Theresa, 2016-07-20 00:09:20]

twellington@chromium.org changed reviewers:
+ dfalcantara@chromium.org

[Theresa, 2016-07-20 00:09:21]

ptal

[commit-bot: I haz the power, 2016-07-20 00:09:59]

Dry run: 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-07-20 01:31:09]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-07-20 01:31:09]

Dry run: This issue passed the CQ dry run.

[gone, 2016-07-21 20:06:24]

LGTM, but it's kind of unfortunate that the interface can't just have a base
implementation for this kind of thing.  Guessing some of the classes extend
other classes already, making it difficult to have an abstract class that
basically contains only the repeated lines?

https://chromiumcodereview.appspot.com/2162243002/diff/20001/chrome/android/j...
File
chrome/android/java/src/org/chromium/chrome/browser/compositor/layouts/LayoutManagerChrome.java
(right):

https://chromiumcodereview.appspot.com/2162243002/diff/20001/chrome/android/j...
chrome/android/java/src/org/chromium/chrome/browser/compositor/layouts/LayoutManagerChrome.java:117:
getActiveLayout().onTabRestored(time(), tabId);
Maybe flip the conditional so the positive case is on top?

[Theresa, 2016-07-21 20:23:17]

https://chromiumcodereview.appspot.com/2162243002/diff/20001/chrome/android/j...
File
chrome/android/java/src/org/chromium/chrome/browser/compositor/layouts/LayoutManagerChrome.java
(right):

https://chromiumcodereview.appspot.com/2162243002/diff/20001/chrome/android/j...
chrome/android/java/src/org/chromium/chrome/browser/compositor/layouts/LayoutManagerChrome.java:117:
getActiveLayout().onTabRestored(time(), tabId);
On 2016/07/21 20:06:24, dfalcantara wrote:
> Maybe flip the conditional so the positive case is on top?

Done.

[Theresa, 2016-07-21 20:32:45]

The CQ bit was checked by twellington@chromium.org

[Theresa, 2016-07-21 20:32:45]

The patchset sent to the CQ was uploaded after l-g-t-m from
dfalcantara@chromium.org
Link to the patchset: https://chromiumcodereview.appspot.com/2162243002/#ps60001
(title: "Reverse order of if/else")

[commit-bot: I haz the power, 2016-07-21 20:32:59]

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-07-21 21:20:45]

Committed patchset #4 (id:60001)

[commit-bot: I haz the power, 2016-07-21 21:23:43]

Description was changed from

==========
Recreate tab switcher stacks/lists when tabs are merged

If tabs from two instances of are merged while either the StackLayout
or OverviewListLayout is active, recreate the view so that the merged
tabs show up in the layout.

BUG=602498
==========

to

==========
Recreate tab switcher stacks/lists when tabs are merged

If tabs from two instances of are merged while either the StackLayout
or OverviewListLayout is active, recreate the view so that the merged
tabs show up in the layout.

BUG=602498

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

[commit-bot: I haz the power, 2016-07-21 21:23:44]

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