Match the Fullscreen spec's CSS as far as currently practical
This change was prompted by the "Fullscreen API bug fixes" blink-dev
thread started by Ali Alabbas at Microsoft:
https://groups.google.com/a/chromium.org/forum/#!topic/blink-dev/f-V2GWatXkA
This is very similar to, and borrows from, the in-progress review to
move Fullscreen to the top layer system:
https://codereview.chromium.org/788073004/
With this change, any fullscreen element (except :root) will have
style applied that makes match the screen size, where previously this
was only done for iframe and video. This will cause a re-layout of
those elements when entering and exiting fullscreen, with the side
effect that the full-screen-render-inline.html test now works. While
the bug does not reproduce now, the underlying cause has not been
fixed, the move to top layer is probably the best cure.
iframe:-webkit-full-screen { border: none; } was previously
!important, but isn't in the spec. This is the cause of the changed
test expectations for full-screen-iframe-zIndex.html.
The flex:1 and display:block rules for audio and video originate from
a Vimeo fullscreen bug: https://bugs.webkit.org/show_bug.cgi?id=58291
With the size and position of the fullscreen element forced to match
the viewport by the UA style sheet, neither of these rules should have
any observable effect, so they are removed.
BUG=402378, 246077
Review URL: https://codereview.chromium.org/974783002
git-svn-id: svn://svn.chromium.org/blink/trunk@191409 bbb929c8-8fbe-4397-9dbb-9b2b20218538
10 files changed
