Bug 122500 – Simplify the loop in EventRetargeter::calculateEventPath

• Status: RESOLVED
• Resolution: FIXED
• Component: UI Events

[Ryosuke Niwa]

The loop in EventRetargeter::calculateEventPath is better expressed as two nested loops along the line of:

for (Node* node = nodeOrHostIfPseudoElement(targetNode); node; node = toShadowRoot(node)->hostElement()) {
    if (!target || !isSVGElement)
        target = &eventTargetRespectingTargetRules(*node);
    for (; node; node = node->parentNode()) {
        ...
        if (!inDocument)
            return;
        if (node->isShadowRoot())
            break;
    }
    if (!shouldEventCrossShadowBoundary(event, toShadowRoot(node), targetStack.last()))
        return;
}

This codifies the semantics much more clearly.

[Ryosuke Niwa]

Created attachment 213706 [details]
Cleanup

[Ryosuke Niwa]

Committed r157127: <http://trac.webkit.org/changeset/157127>