Title: [140313] trunk
Revision
140313
Author
[email protected]
Date
2013-01-21 01:33:05 -0800 (Mon, 21 Jan 2013)

Log Message

Unreviewed, rolling out r140311.
http://trac.webkit.org/changeset/140311
https://bugs.webkit.org/show_bug.cgi?id=107431

Broke selection-related tests (Requested by apavlov on
#webkit).

Patch by Sheriff Bot <[email protected]> on 2013-01-21

Source/WebCore:

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

LayoutTests:

* fast/events/touch/emulate-touch-events.html:

Modified Paths

Diff

Modified: trunk/LayoutTests/ChangeLog (140312 => 140313)


--- trunk/LayoutTests/ChangeLog	2013-01-21 09:11:58 UTC (rev 140312)
+++ trunk/LayoutTests/ChangeLog	2013-01-21 09:33:05 UTC (rev 140313)
@@ -1,3 +1,14 @@
+2013-01-21  Sheriff Bot  <[email protected]>
+
+        Unreviewed, rolling out r140311.
+        http://trac.webkit.org/changeset/140311
+        https://bugs.webkit.org/show_bug.cgi?id=107431
+
+        Broke selection-related tests (Requested by apavlov on
+        #webkit).
+
+        * fast/events/touch/emulate-touch-events.html:
+
 2013-01-20  Alexander Pavlov  <[email protected]>
 
         Web Inspector: Do not dispatch mousemove when emulating touch event and no touch is active

Modified: trunk/LayoutTests/fast/events/touch/emulate-touch-events.html (140312 => 140313)


--- trunk/LayoutTests/fast/events/touch/emulate-touch-events.html	2013-01-21 09:11:58 UTC (rev 140312)
+++ trunk/LayoutTests/fast/events/touch/emulate-touch-events.html	2013-01-21 09:33:05 UTC (rev 140313)
@@ -9,7 +9,6 @@
 var div = document.getElementById("touchtarget");
 var lastEvent = null;
 var touchEventsReceived = 0;
-var mouseMoveEventsReceived = 0;
 var EXPECTED_TOUCH_EVENTS_TOTAL = 3;
 
 function touchEventCallback() {
@@ -23,15 +22,9 @@
         finishJSTest();
 }
 
-function mouseMoveCallback(e) {
-    if (++mouseMoveEventsReceived === 2)
-        debug("Unexpected mousemove event received before touchstart");
-}
-
 div.addEventListener("touchstart", touchEventCallback, false);
 div.addEventListener("touchmove", touchEventCallback, false);
 div.addEventListener("touchend", touchEventCallback, false);
-div.addEventListener("mousemove", mouseMoveCallback, false);
 
 function verifyTouchEvent(type, totalTouchCount, changedTouchCount, targetTouchCount)
 {

Modified: trunk/Source/WebCore/ChangeLog (140312 => 140313)


--- trunk/Source/WebCore/ChangeLog	2013-01-21 09:11:58 UTC (rev 140312)
+++ trunk/Source/WebCore/ChangeLog	2013-01-21 09:33:05 UTC (rev 140313)
@@ -1,3 +1,15 @@
+2013-01-21  Sheriff Bot  <[email protected]>
+
+        Unreviewed, rolling out r140311.
+        http://trac.webkit.org/changeset/140311
+        https://bugs.webkit.org/show_bug.cgi?id=107431
+
+        Broke selection-related tests (Requested by apavlov on
+        #webkit).
+
+        * page/EventHandler.cpp:
+        (WebCore::EventHandler::dispatchSyntheticTouchEventIfEnabled):
+
 2013-01-21  Kentaro Hara  <[email protected]>
 
         Unreviewed. Build fix.

Modified: trunk/Source/WebCore/page/EventHandler.cpp (140312 => 140313)


--- trunk/Source/WebCore/page/EventHandler.cpp	2013-01-21 09:11:58 UTC (rev 140312)
+++ trunk/Source/WebCore/page/EventHandler.cpp	2013-01-21 09:33:05 UTC (rev 140313)
@@ -3981,7 +3981,7 @@
         return false;
 
     if (eventType == PlatformEvent::MouseMoved && !m_touchPressed)
-        return true;
+        return false;
 
     HitTestRequest request(HitTestRequest::Active);
     MouseEventWithHitTestResults mev = prepareMouseEvent(request, event);
_______________________________________________
webkit-changes mailing list
[email protected]
http://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to