Title: [138979] trunk/Source/WebKit/blackberry
Revision
138979
Author
commit-qu...@webkit.org
Date
2013-01-07 12:58:46 -0800 (Mon, 07 Jan 2013)

Log Message

[BlackBerry] Touch Events are falling through fullscreen video
https://bugs.webkit.org/show_bug.cgi?id=106112

Patch by Genevieve Mak <g...@rim.com> on 2013-01-07
Reviewed by Rob Buis.

Reviewed Internally by Jakob Petsovits.
PR #274326
Do not do viewport centering calculations if in fullscreen video.
The offset should be zero.

* Api/WebPage.cpp:
(BlackBerry::WebKit::WebPagePrivate::setViewportSize):

Modified Paths

Diff

Modified: trunk/Source/WebKit/blackberry/Api/WebPage.cpp (138978 => 138979)


--- trunk/Source/WebKit/blackberry/Api/WebPage.cpp	2013-01-07 20:53:58 UTC (rev 138978)
+++ trunk/Source/WebKit/blackberry/Api/WebPage.cpp	2013-01-07 20:58:46 UTC (rev 138979)
@@ -3662,7 +3662,7 @@
     IntSize viewportSizeAfter = actualVisibleSize();
 
     IntSize offset;
-    if (hasPendingOrientation) {
+    if (hasPendingOrientation && !m_fullscreenVideoNode) {
         offset = IntSize(roundf((viewportSizeBefore.width() - viewportSizeAfter.width()) / 2.0),
             roundf((viewportSizeBefore.height() - viewportSizeAfter.height()) / 2.0));
     }

Modified: trunk/Source/WebKit/blackberry/ChangeLog (138978 => 138979)


--- trunk/Source/WebKit/blackberry/ChangeLog	2013-01-07 20:53:58 UTC (rev 138978)
+++ trunk/Source/WebKit/blackberry/ChangeLog	2013-01-07 20:58:46 UTC (rev 138979)
@@ -1,3 +1,18 @@
+2013-01-07  Genevieve Mak  <g...@rim.com>
+
+        [BlackBerry] Touch Events are falling through fullscreen video
+        https://bugs.webkit.org/show_bug.cgi?id=106112
+
+        Reviewed by Rob Buis.
+
+        Reviewed Internally by Jakob Petsovits.
+        PR #274326
+        Do not do viewport centering calculations if in fullscreen video.
+        The offset should be zero.
+
+        * Api/WebPage.cpp:
+        (BlackBerry::WebKit::WebPagePrivate::setViewportSize):
+
 2013-01-07  Jacky Jiang  <zhaji...@rim.com>
 
         [BlackBerry] Roll out r136173 as there are alternative approaches from the client sides now
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
http://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to