[Android] Add OverviewListLayout to main CoordinatorLayout

[Android] Add OverviewListLayout to main CoordinatorLayout

Rather than adding the accessibility tab switcher layout to
R.id.content, add it to a FrameLayout in the main CoordinatorLayout
so that the Chrome Home bottom sheet can be drawn on top of it.

BUG=724700
Review-Url: https://codereview.chromium.org/2898243002
Cr-Commit-Position: refs/heads/master@{#474417}
Committed: https://chromium.googlesource.com/chromium/src/+/7dfe6af12787999c6f219ae1df78d44608d5dc45

[Theresa, 2017-05-23 17:32:17]

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

[Theresa, 2017-05-23 17:33:03]

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

[Theresa, 2017-05-23 17:33:04]

ptal

[commit-bot: I haz the power, 2017-05-23 17:33:18]

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, 2017-05-23 18:35:26]

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

[commit-bot: I haz the power, 2017-05-23 18:35:28]

Dry run: This issue passed the CQ dry run.

[Theresa, 2017-05-24 18:46:34]

24-hour ping. I'd love to get this small CL landed by EOD since I'm OOO
tomorrow.

[David Trainor- moved to gerrit, 2017-05-24 18:58:23]

lgtm % todo nit.  Sorry for the delay!

https://codereview.chromium.org/2898243002/diff/1/chrome/android/java/src/org...
File
chrome/android/java/src/org/chromium/chrome/browser/widget/OverviewListLayout.java
(right):

https://codereview.chromium.org/2898243002/diff/1/chrome/android/java/src/org...
chrome/android/java/src/org/chromium/chrome/browser/widget/OverviewListLayout.java:60:
((ViewGroup) container.findViewById(R.id.overview_list_layout_holder))
Hmm should "container" just be this new ViewGroup?  Might make sense have all of
these things attach/animate behind Chrome Home?  Fine with landing this for now
because of the deadline, but maybe file a bug/add a TODO to investigate the
larger change?

[Theresa, 2017-05-24 19:16:35]

Thanks!

https://codereview.chromium.org/2898243002/diff/1/chrome/android/java/src/org...
File
chrome/android/java/src/org/chromium/chrome/browser/widget/OverviewListLayout.java
(right):

https://codereview.chromium.org/2898243002/diff/1/chrome/android/java/src/org...
chrome/android/java/src/org/chromium/chrome/browser/widget/OverviewListLayout.java:60:
((ViewGroup) container.findViewById(R.id.overview_list_layout_holder))
On 2017/05/24 18:58:23, David Trainor-ping if over 24h wrote:
> Hmm should "container" just be this new ViewGroup?  Might make sense have all
of
> these things attach/animate behind Chrome Home?  Fine with landing this for
now
> because of the deadline, but maybe file a bug/add a TODO to investigate the
> larger change?

Possibly. I'd need to look at each thing that extends Layout to determine if it
should go behind or in front of Chrome Home. I filed crbug.com/726033 to track
investigating.

[Theresa, 2017-05-24 19:17:04]

The CQ bit was checked by twellington@chromium.org

[commit-bot: I haz the power, 2017-05-24 19:17:32]

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, 2017-05-24 20:55:58]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1495653424050790, "parent_rev":
"f073ae8c53e657b2dbbd1387f747d79cacefc71c", "commit_rev":
"7dfe6af12787999c6f219ae1df78d44608d5dc45"}

[commit-bot: I haz the power, 2017-05-24 20:56:15]

Description was changed from

==========
[Android] Add OverviewListLayout to main CoordinatorLayout

Rather than adding the accessibility tab switcher layout to
R.id.content, add it to a FrameLayout in the main CoordinatorLayout
so that the Chrome Home bottom sheet can be drawn on top of it.

BUG=724700
==========

to

==========
[Android] Add OverviewListLayout to main CoordinatorLayout

Rather than adding the accessibility tab switcher layout to
R.id.content, add it to a FrameLayout in the main CoordinatorLayout
so that the Chrome Home bottom sheet can be drawn on top of it.

BUG=724700

Review-Url: https://codereview.chromium.org/2898243002
Cr-Commit-Position: refs/heads/master@{#474417}
Committed:
https://chromium.googlesource.com/chromium/src/+/7dfe6af12787999c6f219ae1df78...
==========

[commit-bot: I haz the power, 2017-05-24 20:56:16]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/7dfe6af12787999c6f219ae1df78...