Title: [173584] trunk/Source/WebCore
- Revision
- 173584
- Author
- [email protected]
- Date
- 2014-09-12 16:02:42 -0700 (Fri, 12 Sep 2014)
Log Message
[Fullscreen] Taking embedded youtube video full screen takes the whole page full screen
https://bugs.webkit.org/show_bug.cgi?id=136792
Reviewed by Beth Dakin.
Recalculate style after beginning the enter fullscreen animation, but before sending the
"webkitfullscreenchange" event to listeners.
* dom/Document.cpp:
(WebCore::Document::webkitWillEnterFullScreenForElement):
Modified Paths
Diff
Modified: trunk/Source/WebCore/ChangeLog (173583 => 173584)
--- trunk/Source/WebCore/ChangeLog 2014-09-12 22:48:06 UTC (rev 173583)
+++ trunk/Source/WebCore/ChangeLog 2014-09-12 23:02:42 UTC (rev 173584)
@@ -1,3 +1,16 @@
+2014-09-12 Jer Noble <[email protected]>
+
+ [Fullscreen] Taking embedded youtube video full screen takes the whole page full screen
+ https://bugs.webkit.org/show_bug.cgi?id=136792
+
+ Reviewed by Beth Dakin.
+
+ Recalculate style after beginning the enter fullscreen animation, but before sending the
+ "webkitfullscreenchange" event to listeners.
+
+ * dom/Document.cpp:
+ (WebCore::Document::webkitWillEnterFullScreenForElement):
+
2014-09-12 Commit Queue <[email protected]>
Unreviewed, rolling out r173575.
Modified: trunk/Source/WebCore/dom/Document.cpp (173583 => 173584)
--- trunk/Source/WebCore/dom/Document.cpp 2014-09-12 22:48:06 UTC (rev 173583)
+++ trunk/Source/WebCore/dom/Document.cpp 2014-09-12 23:02:42 UTC (rev 173584)
@@ -5379,10 +5379,10 @@
m_fullScreenElement->setContainsFullScreenElementOnAncestorsCrossingFrameBoundaries(true);
+ recalcStyle(Style::Force);
+
if (settings() && settings()->needsSiteSpecificQuirks() && hostIsYouTube(url().host()))
fullScreenChangeDelayTimerFired(m_fullScreenChangeDelayTimer);
-
- recalcStyle(Style::Force);
}
void Document::webkitDidEnterFullScreenForElement(Element*)
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes
