.
<rdar://problem/77933000>
Created attachment 428442 [details] Patch
Comment on attachment 428442 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=428442&action=review > Source/WebCore/rendering/RenderLayerCompositor.cpp:-3997 > - m_layerForOverhangAreas->setCustomAppearance(GraphicsLayer::CustomAppearance::ScrollingOverhang); "CustomAppearance::ScrollingOverhang" can we get rid of this now?
Comment on attachment 428442 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=428442&action=review > Source/WebCore/rendering/RenderLayerCompositor.cpp:3889 > + if (page().settings().useThemeColorForScrollAreaBackgroundColor()) > + backgroundColor = page().themeColor(); > + if (page().settings().useSampledPageTopColorForScrollAreaBackgroundColor() && !backgroundColor.isValid()) > + backgroundColor = page().sampledPageTopColor(); > + if (!backgroundColor.isValid()) > + backgroundColor = m_rootExtendedBackgroundColor; Would be nicer to wrap this in a lambda that returns a Color. >> Source/WebCore/rendering/RenderLayerCompositor.cpp:-3997 >> - m_layerForOverhangAreas->setCustomAppearance(GraphicsLayer::CustomAppearance::ScrollingOverhang); > > "CustomAppearance::ScrollingOverhang" can we get rid of this now? It seems no? The backgroundShouldExtendBeyondPage setting has a default value of false.
Comment on attachment 428442 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=428442&action=review >>> Source/WebCore/rendering/RenderLayerCompositor.cpp:-3997 >>> - m_layerForOverhangAreas->setCustomAppearance(GraphicsLayer::CustomAppearance::ScrollingOverhang); >> >> "CustomAppearance::ScrollingOverhang" can we get rid of this now? > > It seems no? The backgroundShouldExtendBeyondPage setting has a default value of false. OH, I missed the code in updateLayerForOverhangAreasBackgroundColor that sets it, and thought this removed the setCustomAppearance() :D
Created attachment 428567 [details] Patch
Committed r277459 (237703@main): <https://commits.webkit.org/237703@main> All reviewed patches have been landed. Closing bug and clearing flags on attachment 428567 [details].