Changeset 138979 in webkit

Timestamp:

Jan 7, 2013 12:58:46 PM (11 years ago)

Author:

commit-queue@webkit.org

Message:

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

Patch by Genevieve Mak <​gmak@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):

Location:

trunk/Source/WebKit/blackberry

Files:

• Api/WebPage.cpp (modified) (1 diff)
• ChangeLog (modified) (1 diff)

trunk/Source/WebKit/blackberry/Api/WebPage.cpp

@@ -3663 +3663 @@
 
     IntSize offset;
-    if (hasPendingOrientation) {
+    if (hasPendingOrientation && !m_fullscreenVideoNode) {
         offset = IntSize(roundf((viewportSizeBefore.width() - viewportSizeAfter.width()) / 2.0),
             roundf((viewportSizeBefore.height() - viewportSizeAfter.height()) / 2.0));

trunk/Source/WebKit/blackberry/ChangeLog

@@ +1 @@
+2013-01-07  Genevieve Mak  <gmak@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  <zhajiang@rim.com>