Bug 211494 – Factor EventHandler code that sends mouseEnteredContentArea/mouseExitedContentArea into its own function

• Component: New Bugs
• Status: RESOLVED FIXED

[Simon Fraser (smfr)]

Factor EventHandler code that sends mouseEnteredContentArea/mouseExitedContentArea into its own function

[Simon Fraser (smfr)]

Created attachment 398588 [details]
Patch

[Simon Fraser (smfr)]

Created attachment 398593 [details]
Patch

[Antti Koivisto]

Comment on attachment 398593 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=398593&action=review

> Source/WebCore/page/EventHandler.cpp:2627
> +        if (elementUnderMouse) {
> +            frameView->mouseEnteredContentArea();
> +            if (scrollableAreaForNodeUnderMouse)
> +                scrollableAreaForNodeUnderMouse->mouseEnteredContentArea();
> +        } else {
> +            if (scrollableAreaForLastNode)
> +                scrollableAreaForLastNode->mouseExitedContentArea();
> +            frameView->mouseExitedContentArea();
> +        }
> +        return;

You could early return instead of the else branch.

[EWS]

Committed r261229: <https://trac.webkit.org/changeset/261229>

All reviewed patches have been landed. Closing bug and clearing flags on attachment 398593 [details].

[Radar WebKit Bug Importer]

<rdar://problem/62931618>