[webkit-reviews] review granted: [Bug 40753] Hit testing on margins of body and head elements doesn't recur : [Attachment 107943] fixes the bug

[bugzilla-daemon at webkit.org]

Darin Adler <darin at apple.com> has granted Ryosuke Niwa <rniwa at webkit.org>'s
request for review:
Bug 40753: Hit testing on margins of body and head elements doesn't recur
https://bugs.webkit.org/show_bug.cgi?id=40753

Attachment 107943: fixes the bug
https://bugs.webkit.org/attachment.cgi?id=107943&action=review

------- Additional Comments from Darin Adler <darin at apple.com>
View in context: https://bugs.webkit.org/attachment.cgi?id=107943&action=review


> Source/WebCore/rendering/RenderBlock.cpp:4235
> +    if (!ancestor || ancestor->node()->hasTagName(htmlTag) ||
ancestor->node()->rendererIsEditable() == childNode->rendererIsEditable())

I’m not sure that htmlTag is the best way to check for the document element.
It’s probably neither the most efficient way nor the most reliable. The best
way I can think of would be to tie this as closely as possible to the code that
makes body and head elements have special margins. I was unable to quickly
research that because I don’t know what that code is.

Two other ways that come to mind that may be better:

    ancestor->isRenderView()
    ancestor->node()->parent() && ancestor->node()->parent()->isDocumentNode()