Title: [173632] trunk/Source/WebCore
Revision
173632
Author
[email protected]
Date
2014-09-15 14:52:04 -0700 (Mon, 15 Sep 2014)

Log Message

Unreviewed logic fix from r173631. Logic for includeShadowDOM was reversed.

* page/DragController.cpp:
(WebCore::DragController::startDrag):

Modified Paths

Diff

Modified: trunk/Source/WebCore/ChangeLog (173631 => 173632)


--- trunk/Source/WebCore/ChangeLog	2014-09-15 21:47:18 UTC (rev 173631)
+++ trunk/Source/WebCore/ChangeLog	2014-09-15 21:52:04 UTC (rev 173632)
@@ -1,5 +1,12 @@
 2014-09-15  Jer Noble  <[email protected]>
 
+        Unreviewed logic fix from r173631. Logic for includeShadowDOM was reversed.
+
+        * page/DragController.cpp:
+        (WebCore::DragController::startDrag):
+
+2014-09-15  Jer Noble  <[email protected]>
+
         Videos with controls enabled never receive 'dragstart' events.
         https://bugs.webkit.org/show_bug.cgi?id=136837
 

Modified: trunk/Source/WebCore/page/DragController.cpp (173631 => 173632)


--- trunk/Source/WebCore/page/DragController.cpp	2014-09-15 21:47:18 UTC (rev 173631)
+++ trunk/Source/WebCore/page/DragController.cpp	2014-09-15 21:52:04 UTC (rev 173632)
@@ -725,7 +725,7 @@
     // FIXME(136836): Investigate whether all elements should use the containsIncludingShadowDOM() path here.
     bool includeShadowDOM = state.source->isMediaElement();
     bool sourceContainsHitNode;
-    if (includeShadowDOM)
+    if (!includeShadowDOM)
         sourceContainsHitNode = state.source->contains(hitTestResult.innerNode());
     else
         sourceContainsHitNode = state.source->containsIncludingShadowDOM(hitTestResult.innerNode());
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to