Bug 93020 – REGRESSION (tiled scrolling): Full-screen video is broken if page is scrolled

• Status: RESOLVED
• Resolution: FIXED
• Component: Layout and Rendering

[Beth Dakin]

<rdar://problem/11629778>

* SUMMARY
Full-screen video appears with linen on the bottom, and controls don't work, if the page is scrolled.

* STEPS TO REPRODUCE
1. Go to http://www.apple.com/apple-events/education-january-2012/
2. Scroll down the page some
3. Click the full-screen button in the video controls

* RESULTS
The video zooms up, but rather than filling the screen, it appears with linen at the bottom of the window equal to the distance down the page that you're scrolled. Controls do not work. Escape will take you back out of full-screen.

This bug is not unique to video. It also reproduces with the attached test case following these steps:
1. Make the window small enough so that they test case has a scroll bar
2. Scroll to the bottom.
3. Press the "toggle fixed" button
restult-> a bunch of linen shows at the bottom, but it should not.

[Beth Dakin]

Created attachment 156148 [details]
Reduction

[Beth Dakin]

Created attachment 156149 [details]
Patch

[Anders Carlsson]

Comment on attachment 156149 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=156149&action=review

> Source/WebCore/page/scrolling/ScrollingCoordinator.cpp:419
> +    // at this point. So we'll upate it before we switch back to main thread scrolling

Typo, 'upate'.

[Beth Dakin]

Thanks Anders! Committed http://trac.webkit.org/changeset/124489