Bug 95685 – Move AllowShadowContent flag to HitTestRequest

• Component: Layout and Rendering
• Status: RESOLVED FIXED

[Allan Sandfeld Jensen]

The HitTestResult class currently has a flag that control whether shadow-content should be allowed in the result, but since it is not really part of the result but a parameter to the hit-test itself it doesn't really belong in the HitTestResult class. All other flags controlling the hit-test are stored in HitTestRequest, and I propose moving it there. 

This is a part of the work to also include a flag that controls if child-frame content is allowed. See bug #95204

[Allan Sandfeld Jensen]

Created attachment 161919 [details]
Patch

[Antonio Gomes]

Comment on attachment 161919 [details]
Patch

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

r=me

please fix the method name before landing, and maybe strip out the supposedly unrelated change?

> Source/WebCore/rendering/HitTestRequest.h:56
> +    bool allowShadowContent() const { return m_requestType & AllowShadowContent; }

I think it should be allowsShadowContent

> Source/WebCore/rendering/HitTestResult.cpp:107
> -    , m_region(region)
> +    , m_region(region ? region : other.m_region)

unrelated

> Source/WebCore/rendering/HitTestResult.h:58
> -    HitTestLocation(const HitTestLocation&, const LayoutSize& offset, RenderRegion*);
> +    HitTestLocation(const HitTestLocation&, const LayoutSize& offset, RenderRegion* = 0);

unrelated?

[Allan Sandfeld Jensen]

(In reply to comment #2)
> (From update of attachment 161919 [details])
> View in context: https://bugs.webkit.org/attachment.cgi?id=161919&action=review
> 
> r=me
> 
> please fix the method name before landing, and maybe strip out the supposedly unrelated change?
> 
I will. You are right that the two changes were incorrectly included in this patch.

[Allan Sandfeld Jensen]

Committed r127421: <http://trac.webkit.org/changeset/127421>