[Webkit-unassigned] [Bug 94107] Refactoring: move EventHandler::targetNode into HitTestResult

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=94107





--- Comment #4 from Iain Merrick <husky at chromium.org>  2012-08-15 10:00:36 PST ---
(From update of attachment 158566)
View in context: https://bugs.webkit.org/attachment.cgi?id=158566&action=review

>> Source/WebCore/rendering/HitTestResult.cpp:808
>> +        return element;
> 
> I'm trying to understanding how this is different from innerNode.  In the context of an event, it makes sense to speak about the targetNode of an event, but outside the context of an event the word "target" doesn't mean that much.
> 
> Specifically, how can a node not be inDocument but its parentElement be inDocument?

That's a great question, and I don't know! I'm trying to understand this code by teasing apart the bits that look unnecessarily tangled-up.

Does inDocument() refer to the shadow DOM?

>> Source/WebCore/rendering/HitTestResult.h:180
>> +    Node* targetNode() const;
> 
> I would move this function next to innerNode() so that it's clear that there is a choice between the two.

Good idea, will do.

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.