Title: [141929] trunk/Source/WebCore
Revision
141929
Author
[email protected]
Date
2013-02-05 14:04:33 -0800 (Tue, 05 Feb 2013)

Log Message

[chromium] Enable shouldGesturesTriggerActive for Android
https://bugs.webkit.org/show_bug.cgi?id=96948

Patch by Yusuf Ozuysal <[email protected]> on 2013-02-05
Reviewed by James Robinson.

We need this to fix performance issues we are getting because of touchstart
triggering hover/active states.

No new tests. The tests introduced in https://bugs.webkit.org/show_bug.cgi?id=96060
should run with the corrected behavior and would cover this change as well.

* page/EventHandler.cpp:
(WebCore::shouldGesturesTriggerActive):

Modified Paths

Diff

Modified: trunk/Source/WebCore/ChangeLog (141928 => 141929)


--- trunk/Source/WebCore/ChangeLog	2013-02-05 22:02:46 UTC (rev 141928)
+++ trunk/Source/WebCore/ChangeLog	2013-02-05 22:04:33 UTC (rev 141929)
@@ -1,3 +1,19 @@
+2013-02-05  Yusuf Ozuysal  <[email protected]>
+
+        [chromium] Enable shouldGesturesTriggerActive for Android
+        https://bugs.webkit.org/show_bug.cgi?id=96948
+
+        Reviewed by James Robinson.
+
+        We need this to fix performance issues we are getting because of touchstart 
+        triggering hover/active states.
+
+        No new tests. The tests introduced in https://bugs.webkit.org/show_bug.cgi?id=96060
+        should run with the corrected behavior and would cover this change as well.
+
+        * page/EventHandler.cpp:
+        (WebCore::shouldGesturesTriggerActive):
+
 2013-02-05  Mark Lam  <[email protected]>
 
         Introduced back-end database classes + a few small fixes.

Modified: trunk/Source/WebCore/page/EventHandler.cpp (141928 => 141929)


--- trunk/Source/WebCore/page/EventHandler.cpp	2013-02-05 22:02:46 UTC (rev 141928)
+++ trunk/Source/WebCore/page/EventHandler.cpp	2013-02-05 22:04:33 UTC (rev 141929)
@@ -282,7 +282,7 @@
     // If the platform we're on supports GestureTapDown and GestureTapCancel then we'll
     // rely on them to set the active state. Unfortunately there's no generic way to
     // know in advance what event types are supported.
-#if PLATFORM(CHROMIUM) && !OS(ANDROID)
+#if PLATFORM(CHROMIUM)
     return true;
 #else
     return false;
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to