Title: [113498] trunk
Revision
113498
Author
jer.no...@apple.com
Date
2012-04-06 14:38:33 -0700 (Fri, 06 Apr 2012)

Log Message

Foreground of apple.com/iphone video page visible during full screen animation.
https://bugs.webkit.org/show_bug.cgi?id=83080

Reviewed by Simon Fraser.

Source/WebCore:

No new tests; updated fullscreen/full-screen-stacking-context.html

The apple.com/iphone video page uses a -webkit-mask: CSS style, which creates a stacking
context and causes the page to pop in front of the full screen renderer.  Add all the
styles suggested by the W3C full screen spec to the -webkit-full-screen-ancestor rule
to keep these stacking contexts from being created.

* css/fullscreen.css:
(:-webkit-full-screen-ancestor:not(iframe)):

LayoutTests:

* fullscreen/full-screen-stacking-context.html:
* platform/mac/fullscreen/full-screen-stacking-context-expected.png:

Modified Paths

Diff

Modified: trunk/LayoutTests/ChangeLog (113497 => 113498)


--- trunk/LayoutTests/ChangeLog	2012-04-06 21:32:55 UTC (rev 113497)
+++ trunk/LayoutTests/ChangeLog	2012-04-06 21:38:33 UTC (rev 113498)
@@ -1,3 +1,13 @@
+2012-04-03  Jer Noble  <jer.no...@apple.com>
+
+        Foreground of apple.com/iphone video page visible during full screen animation.
+        https://bugs.webkit.org/show_bug.cgi?id=83080
+
+        Reviewed by Simon Fraser.
+
+        * fullscreen/full-screen-stacking-context.html:
+        * platform/mac/fullscreen/full-screen-stacking-context-expected.png:
+
 2012-04-06  Tim Horton  <timothy_hor...@apple.com>
 
         [cg] REGRESSION (r101517): Animating the transform of a <rect> with shape-rendering: crispEdges leaves behind garbage

Modified: trunk/LayoutTests/fullscreen/full-screen-stacking-context.html (113497 => 113498)


--- trunk/LayoutTests/fullscreen/full-screen-stacking-context.html	2012-04-06 21:32:55 UTC (rev 113497)
+++ trunk/LayoutTests/fullscreen/full-screen-stacking-context.html	2012-04-06 21:38:33 UTC (rev 113498)
@@ -28,9 +28,17 @@
                 top: 50px;
                 left: 50px;
                 z-index: 1;
+                background-color: red;
+            }
+            #one, #two {
                 opacity: 0.5;
                 -webkit-transform: rotate(180);
-                background-color: red;
+                -webkit-mask: -webkit-linear-gradient(left, alpha 0%, white 100%);
+                -webkit-filter: contrast(100%);
+                clip:rect(0px,100px,100px,0px);
+                -webkit-box-reflect: below;
+                -webkit-transform-style: preserve-3d;
+                -webkit-perspective: 300;
             }
             #zero {
                 position: relative;

Modified: trunk/LayoutTests/platform/mac/fullscreen/full-screen-stacking-context-expected.png


(Binary files differ)

Modified: trunk/Source/WebCore/ChangeLog (113497 => 113498)


--- trunk/Source/WebCore/ChangeLog	2012-04-06 21:32:55 UTC (rev 113497)
+++ trunk/Source/WebCore/ChangeLog	2012-04-06 21:38:33 UTC (rev 113498)
@@ -1,3 +1,20 @@
+2012-04-03  Jer Noble  <jer.no...@apple.com>
+
+        Foreground of apple.com/iphone video page visible during full screen animation.
+        https://bugs.webkit.org/show_bug.cgi?id=83080
+
+        Reviewed by Simon Fraser.
+
+        No new tests; updated fullscreen/full-screen-stacking-context.html
+
+        The apple.com/iphone video page uses a -webkit-mask: CSS style, which creates a stacking
+        context and causes the page to pop in front of the full screen renderer.  Add all the
+        styles suggested by the W3C full screen spec to the -webkit-full-screen-ancestor rule
+        to keep these stacking contexts from being created.
+
+        * css/fullscreen.css:
+        (:-webkit-full-screen-ancestor:not(iframe)):
+
 2012-04-06  Abhishek Arya  <infe...@chromium.org>
 
         Virtualize createAnonymousBoxWithSameTypeAs.

Modified: trunk/Source/WebCore/css/fullscreen.css (113497 => 113498)


--- trunk/Source/WebCore/css/fullscreen.css	2012-04-06 21:32:55 UTC (rev 113497)
+++ trunk/Source/WebCore/css/fullscreen.css	2012-04-06 21:38:33 UTC (rev 113498)
@@ -11,6 +11,13 @@
     z-index: auto !important;
     opacity: 1 !important;
     -webkit-transform: none !important;
+    -webkit-mask: none !important;
+    clip: none !important;
+    -webkit-filter: none !important;
+    -webkit-transition: none !important;
+    -webkit-box-reflect: none !important;
+    -webkit-perspective: none !important;
+    -webkit-transform-style: flat !important;
 }
 
 video:-webkit-full-screen {
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
http://lists.webkit.org/mailman/listinfo.cgi/webkit-changes

Reply via email to