Bug 216031 – EventRegion paint should respect layer's foreground and background painting phases

• Status: RESOLVED
• Resolution: FIXED
• Component: Scrolling

[Antti Koivisto]

We are currently painting event region for the box background to the foreground layer.

[Antti Koivisto]

<rdar://problem/67282372>

[Antti Koivisto]

Created attachment 407668 [details]
patch

[Simon Fraser (smfr)]

Comment on attachment 407668 [details]
patch

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

> Source/WebCore/rendering/RenderBlock.cpp:1140
> +        if (paintInfo.eventRegionContext)
> +            paintInfoForChild.paintBehavior.add(PaintBehavior::EventRegionIncludeBackground);

This is pretty weird. I wish event region painting followed normal painting more closely.

> Source/WebCore/rendering/RenderLayer.cpp:4654
> +            paintBehavior = paintBehavior & PaintBehavior::CompositedOverflowScrollContent;

I know you moved this and I added it in r262337 but I don't understand it.

[Simon Fraser (smfr)]

Comment on attachment 407668 [details]
patch

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

>> Source/WebCore/rendering/RenderBlock.cpp:1140
>> +            paintInfoForChild.paintBehavior.add(PaintBehavior::EventRegionIncludeBackground);
> 
> This is pretty weird. I wish event region painting followed normal painting more closely.

Maybe leave it alone so I can decipher it without too many history jumps.

[Antti Koivisto]

> > Source/WebCore/rendering/RenderBlock.cpp:1140
> > +        if (paintInfo.eventRegionContext)
> > +            paintInfoForChild.paintBehavior.add(PaintBehavior::EventRegionIncludeBackground);
> 
> This is pretty weird. I wish event region painting followed normal painting
> more closely.

It basically just does the same thing the code next to it does with paint phases:

newPhase = (newPhase == PaintPhase::ChildBlockBackgrounds) ? PaintPhase::ChildBlockBackground : newPhase;

(ChildBlockBackgrounds vs ChildBlockBackground is bit subtle)

> I know you moved this and I added it in r262337 but I don't understand it.

I suppose the other flags are not relevant for event regions?

[EWS]

Committed r266403: <https://trac.webkit.org/changeset/266403>

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