Title: [126870] trunk/Source/WebKit/blackberry
Revision
126870
Author
[email protected]
Date
2012-08-28 06:00:00 -0700 (Tue, 28 Aug 2012)

Log Message

[BlackBerry] Pseudo class "active" is broken
https://bugs.webkit.org/show_bug.cgi?id=95142

Patch by Andy Chen <[email protected]> on 2012-08-28
Reviewed by Antonio Gomes.

Check if an element is affected by active rule before cancelling
the touch event.
PR 198544.

* WebKitSupport/TouchEventHandler.cpp:
(BlackBerry::WebKit::TouchEventHandler::touchEventCancel):

Modified Paths

Diff

Modified: trunk/Source/WebKit/blackberry/ChangeLog (126869 => 126870)


--- trunk/Source/WebKit/blackberry/ChangeLog	2012-08-28 12:50:04 UTC (rev 126869)
+++ trunk/Source/WebKit/blackberry/ChangeLog	2012-08-28 13:00:00 UTC (rev 126870)
@@ -1,3 +1,17 @@
+2012-08-28  Andy Chen  <[email protected]>
+
+        [BlackBerry] Pseudo class "active" is broken
+        https://bugs.webkit.org/show_bug.cgi?id=95142
+
+        Reviewed by Antonio Gomes.
+
+        Check if an element is affected by active rule before cancelling
+        the touch event.
+        PR 198544.
+
+        * WebKitSupport/TouchEventHandler.cpp:
+        (BlackBerry::WebKit::TouchEventHandler::touchEventCancel):
+
 2012-08-27  Sheriff Bot  <[email protected]>
 
         Unreviewed, rolling out r126836.

Modified: trunk/Source/WebKit/blackberry/WebKitSupport/TouchEventHandler.cpp (126869 => 126870)


--- trunk/Source/WebKit/blackberry/WebKitSupport/TouchEventHandler.cpp	2012-08-28 12:50:04 UTC (rev 126869)
+++ trunk/Source/WebKit/blackberry/WebKitSupport/TouchEventHandler.cpp	2012-08-28 13:00:00 UTC (rev 126870)
@@ -140,7 +140,8 @@
         if (!elementUnderFatFinger || !elementUnderFatFinger->renderer())
             break;
 
-        if (!elementUnderFatFinger->renderer()->style()->affectedByHoverRules())
+        if (!elementUnderFatFinger->renderer()->style()->affectedByHoverRules()
+            && !elementUnderFatFinger->renderer()->style()->affectedByActiveRules())
             break;
 
         HitTestRequest request(HitTestRequest::FingerUp);
_______________________________________________
webkit-changes mailing list
[email protected]
http://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to