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