Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(281)

Unified Diff: third_party/WebKit/Source/core/css/fullscreen.css

Issue 2168373002: Revert of Reland "Implement FullScreen using top layer." (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: resolve WebFrameTest.cpp conflict and appease presubmit Created 4 years, 5 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « third_party/WebKit/Source/core/core.gypi ('k') | third_party/WebKit/Source/core/css/html.css » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: third_party/WebKit/Source/core/css/fullscreen.css
diff --git a/third_party/WebKit/Source/core/css/fullscreen.css b/third_party/WebKit/Source/core/css/fullscreen.css
index eb2e9369eaec016d1455027071e2a01e9fead7a9..f6a773a73e17bf7a0a5399429012a6d612dd3803 100644
--- a/third_party/WebKit/Source/core/css/fullscreen.css
+++ b/third_party/WebKit/Source/core/css/fullscreen.css
@@ -1,44 +1,61 @@
-:not(:root):-webkit-full-screen {
- position: fixed !important;
- top: 0 !important;
- right: 0 !important;
- bottom: 0 !important;
+:-webkit-full-screen {
+ background-color: white;
+ z-index: 2147483647 !important;
+}
+
+:root:-webkit-full-screen-ancestor {
+ overflow: hidden !important;
+}
+
+:-webkit-full-screen-ancestor:not(iframe) {
+ z-index: auto !important;
+ position: static !important;
+ opacity: 1 !important;
+ transform: none !important;
+ -webkit-mask: none !important;
+ clip: none !important;
+ -webkit-filter: none !important;
+ transition: none !important;
+ -webkit-box-reflect: none !important;
+ -webkit-perspective: none !important;
+ -webkit-transform-style: flat !important;
+ will-change: auto !important;
+}
+
+video:-webkit-full-screen, audio:-webkit-full-screen {
+ background-color: transparent !important;
+ position: relative !important;
left: 0 !important;
+ top: 0 ! important;
margin: 0 !important;
- box-sizing: border-box !important;
min-width: 0 !important;
max-width: none !important;
min-height: 0 !important;
max-height: none !important;
width: 100% !important;
height: 100% !important;
- object-fit: contain !important;
+ flex: 1 !important;
+ display: block !important;
transform: none !important;
}
-iframe:-webkit-full-screen {
- border: none !important;
- padding: 0 !important;
-}
-
-/* TODO(foolip): In the spec, there's a ::backdrop block with the properties
- shared with dialog::backdrop (see html.css). */
-:not(:root):-webkit-full-screen::backdrop {
- position: fixed;
- top: 0;
- right: 0;
- bottom: 0;
- left: 0;
- background: black;
+img:-webkit-full-screen {
+ width: auto;
+ height: 100%;
+ max-width: 100%;
}
-/* Anything below are extensions over what the Fullscreen API (3 May 2016) mandates. */
-
-/* This prevents video from overflowing the viewport in
- virtual/android/fullscreen/video-scrolled-iframe.html (crbug.com/441890) and
- removes scrollbars caused by html { overflow: scroll } (crbug.com/543946).
- TODO(foolip): This is done differently in Gecko, find a fix not involving the
- ancestor selector in https://github.com/whatwg/fullscreen/issues/19. */
-:root:-webkit-full-screen-ancestor {
- overflow: hidden !important;
+iframe:-webkit-full-screen {
+ margin: 0 !important;
+ padding: 0 !important;
+ border: 0 !important;
+ position: fixed !important;
+ min-width: 0 !important;
+ max-width: none !important;
+ min-height: 0 !important;
+ max-height: none !important;
+ width: 100% !important;
+ height: 100% !important;
+ left: 0 !important;
+ top: 0 !important;
}
« no previous file with comments | « third_party/WebKit/Source/core/core.gypi ('k') | third_party/WebKit/Source/core/css/html.css » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698