Title: [247023] trunk/Source/WebCore
Revision
247023
Author
[email protected]
Date
2019-07-01 14:23:44 -0700 (Mon, 01 Jul 2019)

Log Message

Unreviewed, rolling out r246849.

12 tests broken in r246844 require this to be rolled out.

Reverted changeset:

"[Pointer Events] Respect pointer capture when dispatching
mouse boundary events and updating :hover"
https://bugs.webkit.org/show_bug.cgi?id=198999
https://trac.webkit.org/changeset/246849

Modified Paths

Diff

Modified: trunk/Source/WebCore/ChangeLog (247022 => 247023)


--- trunk/Source/WebCore/ChangeLog	2019-07-01 21:17:00 UTC (rev 247022)
+++ trunk/Source/WebCore/ChangeLog	2019-07-01 21:23:44 UTC (rev 247023)
@@ -1,3 +1,16 @@
+2019-07-01  Truitt Savell  <[email protected]>
+
+        Unreviewed, rolling out r246849.
+
+        12 tests broken in r246844 require this to be rolled out.
+
+        Reverted changeset:
+
+        "[Pointer Events] Respect pointer capture when dispatching
+        mouse boundary events and updating :hover"
+        https://bugs.webkit.org/show_bug.cgi?id=198999
+        https://trac.webkit.org/changeset/246849
+
 2019-07-01  Alex Christensen  <[email protected]>
 
         Null check provisionalItem in FrameLoader::continueLoadAfterNavigationPolicy

Modified: trunk/Source/WebCore/page/PointerCaptureController.cpp (247022 => 247023)


--- trunk/Source/WebCore/page/PointerCaptureController.cpp	2019-07-01 21:17:00 UTC (rev 247022)
+++ trunk/Source/WebCore/page/PointerCaptureController.cpp	2019-07-01 21:23:44 UTC (rev 247023)
@@ -409,15 +409,15 @@
 
 void PointerCaptureController::processPendingPointerCapture(PointerID pointerId)
 {
-    auto iterator = m_activePointerIdsToCapturingData.find(pointerId);
-    if (iterator == m_activePointerIdsToCapturingData.end())
-        return;
-
     if (m_processingPendingPointerCapture)
         return;
 
     m_processingPendingPointerCapture = true;
 
+    auto iterator = m_activePointerIdsToCapturingData.find(pointerId);
+    if (iterator == m_activePointerIdsToCapturingData.end())
+        return;
+
     auto& capturingData = iterator->value;
 
     // Cache the pending target override since it could be modified during the dispatch of events in this function.
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to