Title: [124009] trunk/Source/WebCore
- Revision
- 124009
- Author
- [email protected]
- Date
- 2012-07-30 02:45:47 -0700 (Mon, 30 Jul 2012)
Log Message
Remove unnecessary code which set event's target from EventDispatcher::dispatchEvent.
https://bugs.webkit.org/show_bug.cgi?id=92613
Reviewed by Hajime Morita.
EventContext::handleLocalEvents does all required jobs so we don't
need to set target in EventDispacher::dispatchEvent() for TARGET phase.
No new tests, no behavior change.
* dom/EventDispatcher.cpp:
(WebCore::EventDispatcher::dispatchEvent):
Modified Paths
Diff
Modified: trunk/Source/WebCore/ChangeLog (124008 => 124009)
--- trunk/Source/WebCore/ChangeLog 2012-07-30 09:29:45 UTC (rev 124008)
+++ trunk/Source/WebCore/ChangeLog 2012-07-30 09:45:47 UTC (rev 124009)
@@ -1,3 +1,18 @@
+2012-07-30 Hayato Ito <[email protected]>
+
+ Remove unnecessary code which set event's target from EventDispatcher::dispatchEvent.
+ https://bugs.webkit.org/show_bug.cgi?id=92613
+
+ Reviewed by Hajime Morita.
+
+ EventContext::handleLocalEvents does all required jobs so we don't
+ need to set target in EventDispacher::dispatchEvent() for TARGET phase.
+
+ No new tests, no behavior change.
+
+ * dom/EventDispatcher.cpp:
+ (WebCore::EventDispatcher::dispatchEvent):
+
2012-07-30 Sheriff Bot <[email protected]>
Unreviewed, rolling out r124004.
Modified: trunk/Source/WebCore/dom/EventDispatcher.cpp (124008 => 124009)
--- trunk/Source/WebCore/dom/EventDispatcher.cpp 2012-07-30 09:29:45 UTC (rev 124008)
+++ trunk/Source/WebCore/dom/EventDispatcher.cpp 2012-07-30 09:45:47 UTC (rev 124009)
@@ -277,8 +277,6 @@
}
event->setEventPhase(Event::AT_TARGET);
- event->setTarget(originalTarget.get());
- event->setCurrentTarget(eventTargetRespectingSVGTargetRules(m_node.get()));
m_ancestors[0].handleLocalEvents(event.get());
if (event->propagationStopped())
goto doneDispatching;
_______________________________________________
webkit-changes mailing list
[email protected]
http://lists.webkit.org/mailman/listinfo/webkit-changes
