[DevTools] Apply emulation root layer transform to innerViewportContainerLayer.

[DevTools] Apply emulation root layer transform to innerViewportContainerLayer.

This should not affect anything since innerViewportContainerLayer is the
only child of root layer. On the other side, cc adds a second child - hud layer,
which should not be affected by emulation transform. This patch fixes the problem
and enables "show fps meter" together with emulation.

BUG=386508
Committed: https://crrev.com/99170fbafe811791bd3666f3b76b647223999445
Cr-Commit-Position: refs/heads/master@{#389983}

[dgozman, 2016-04-25 23:00:51]

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

[commit-bot: I haz the power, 2016-04-25 23:01:11]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1914023002/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1914023002/1

[dgozman, 2016-04-25 23:04:05]

dgozman@chromium.org changed reviewers:
+ bokan@chromium.org

[dgozman, 2016-04-25 23:04:06]

David, could you please take a look?
I'm wondering why do we have both rootLayer and innerViewportContainerLayer? Is
it only for emulation transform?
It turned out to be very convenient for me to move transform deeper, but maybe
I'm doing something wrong?

[commit-bot: I haz the power, 2016-04-26 00:49:18]

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

[commit-bot: I haz the power, 2016-04-26 00:49:19]

Dry run: This issue passed the CQ dry run.

[bokan, 2016-04-26 15:29:12]

lgtm % nit

https://codereview.chromium.org/1914023002/diff/1/third_party/WebKit/Source/w...
File third_party/WebKit/Source/web/WebViewImpl.h (right):

https://codereview.chromium.org/1914023002/diff/1/third_party/WebKit/Source/w...
third_party/WebKit/Source/web/WebViewImpl.h:721: GraphicsLayer*
m_containerLayer;
Please make this more descriptive. i.e. m_visualViewportContainerLayer

[dgozman, 2016-04-27 01:31:19]

Thanks!

https://codereview.chromium.org/1914023002/diff/1/third_party/WebKit/Source/w...
File third_party/WebKit/Source/web/WebViewImpl.h (right):

https://codereview.chromium.org/1914023002/diff/1/third_party/WebKit/Source/w...
third_party/WebKit/Source/web/WebViewImpl.h:721: GraphicsLayer*
m_containerLayer;
On 2016/04/26 15:29:12, bokan wrote:
> Please make this more descriptive. i.e. m_visualViewportContainerLayer

Done.

[dgozman, 2016-04-27 01:31:25]

The CQ bit was checked by dgozman@chromium.org

[dgozman, 2016-04-27 01:31:25]

The patchset sent to the CQ was uploaded after l-g-t-m from bokan@chromium.org
Link to the patchset: https://codereview.chromium.org/1914023002/#ps20001
(title: "addressed review comment")

[commit-bot: I haz the power, 2016-04-27 01:31:40]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1914023002/20001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1914023002/20001

[commit-bot: I haz the power, 2016-04-27 03:36:39]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-04-27 03:38:08]

Description was changed from

==========
[DevTools] Apply emulation root layer transform to innerViewportContainerLayer.

This should not affect anything since innerViewportContainerLayer is the
only child of root layer. On the other side, cc adds a second child - hud layer,
which should not be affected by emulation transform. This patch fixes the
problem
and enables "show fps meter" together with emulation.

BUG=386508
==========

to

==========
[DevTools] Apply emulation root layer transform to innerViewportContainerLayer.

This should not affect anything since innerViewportContainerLayer is the
only child of root layer. On the other side, cc adds a second child - hud layer,
which should not be affected by emulation transform. This patch fixes the
problem
and enables "show fps meter" together with emulation.

BUG=386508

Committed: https://crrev.com/99170fbafe811791bd3666f3b76b647223999445
Cr-Commit-Position: refs/heads/master@{#389983}
==========

[commit-bot: I haz the power, 2016-04-27 03:38:09]

Patchset 2 (id:??) landed as
https://crrev.com/99170fbafe811791bd3666f3b76b647223999445
Cr-Commit-Position: refs/heads/master@{#389983}