Test both viewports' scrollability during fullscreen-video

Test both viewports' scrollability during fullscreen-video

Background:
To disable scrolling (and pinch-zoom) during <video>-
fullscreen ScrollingCoordinator makes both viewports
unscrollable in updateAfterCompositingChangeIfNeeded().

Problem:
Currently only the outer viewport's scrollability-
setting is tested during fullscreen-video.

Solution:
Extend test case to also verify the inner viewport.

BUG=448738
Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=201007

[Hugo Holgersson, 2015-08-14 16:59:16]

hugo.holgersson@sonymobile.com changed reviewers:
+ bokan@chromium.org

[Hugo Holgersson, 2015-08-14 16:59:17]

@bokan PTAL.

[bokan, 2015-08-18 12:35:49]

lgtm!

https://codereview.chromium.org/1287253005/diff/1/Source/web/tests/WebFrameTe...
File Source/web/tests/WebFrameTest.cpp (right):

https://codereview.chromium.org/1287253005/diff/1/Source/web/tests/WebFrameTe...
Source/web/tests/WebFrameTest.cpp:6529: WebLayer* layoutViewportScrollLayer =
webViewImpl->compositor()->scrollLayer()->platformLayer();
It's probably slightly more clear to get this via
frameView->layoutViewportScrollableArea()->scrollLayer()

[Hugo Holgersson, 2015-08-20 21:12:56]

Thanks!

https://codereview.chromium.org/1287253005/diff/1/Source/web/tests/WebFrameTe...
File Source/web/tests/WebFrameTest.cpp (right):

https://codereview.chromium.org/1287253005/diff/1/Source/web/tests/WebFrameTe...
Source/web/tests/WebFrameTest.cpp:6529: WebLayer* layoutViewportScrollLayer =
webViewImpl->compositor()->scrollLayer()->platformLayer();
On 2015/08/18 12:35:49, bokan wrote:
> It's probably slightly more clear to get this via
> frameView->layoutViewportScrollableArea()->scrollLayer()
Hmm, I got a compile error when I tried that.

[Hugo Holgersson, 2015-08-21 17:28:48]

https://codereview.chromium.org/1287253005/diff/1/Source/web/tests/WebFrameTe...
File Source/web/tests/WebFrameTest.cpp (right):

https://codereview.chromium.org/1287253005/diff/1/Source/web/tests/WebFrameTe...
Source/web/tests/WebFrameTest.cpp:6529: WebLayer* layoutViewportScrollLayer =
webViewImpl->compositor()->scrollLayer()->platformLayer();
On 2015/08/20 21:12:56, Hugo Holgersson wrote:
> On 2015/08/18 12:35:49, bokan wrote:
> > It's probably slightly more clear to get this via
> > frameView->layoutViewportScrollableArea()->scrollLayer()
> Hmm, I got a compile error when I tried that.
Let's keep this lookup unchanged for now.

[Hugo Holgersson, 2015-08-21 17:29:21]

The CQ bit was checked by hugo.holgersson@sonymobile.com

[commit-bot: I haz the power, 2015-08-21 17:29:33]

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

[commit-bot: I haz the power, 2015-08-21 19:56:43]

Committed patchset #1 (id:1) as
https://src.chromium.org/viewvc/blink?view=rev&revision=201007