Bug 250283 – RenderLayer::setHasVisibleContent shouldn't call dirtyZOrderLists()

• Component: Compositing
• Status: RESOLVED DUPLICATE

[Ahmad Saleem]

Hi Team,

I came across this clean-up patch from Blink and we still have this code, although I know Blink compositing is not same as WebKit but still good to ask for an opportunity to do code clean-up.

Blink Commit - https://chromium.googlesource.com/chromium/blink/+/3ae6522cf95a377050d18a8d8cc11c7db4ccf2b3

WebKit Source - https://searchfox.org/wubkat/source/Source/WebCore/rendering/RenderLayer.cpp#1522

Unfortunately, it does not have any bug or testcase to confirm that it is dead code but just wanted to ask here, if it is something we can drop.

Thanks!

[Radar WebKit Bug Importer]

<rdar://problem/104267662>

[Ahmad Saleem]

PR - https://github.com/WebKit/WebKit/pull/8695

Let's try on EWS..

[Ahmad Saleem]

Based on PR - (In reply to Ahmad Saleem from comment #2)
> PR - https://github.com/WebKit/WebKit/pull/8695
> 
> Let's try on EWS..

Simon replied that this is something still required in WebKit legacy:

'''

I think we may still need this code in WebKitLegacy. RenderLayer::collectLayers() only collects visibility:hidden layers if includeHiddenLayers is true, and that's only true when compositing is enabled. In WebKitLegacy, we enter compositing mode dynamically.

'''

So I am closing my PR for time being but we can explore it in future when we don't support WebKit Legacy. It passes all tests right now even Mac-wk1 as well but it might have potential to introduce future regressions.

[Ahmad Saleem]

Seems like this is taken care here:

https://github.com/WebKit/WebKit/pull/30666

Thanks Matt.

*** This bug has been marked as a duplicate of bug 276451 ***