-
Notifications
You must be signed in to change notification settings - Fork 1.2k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. Weβll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Use a different layer border color with UI-side compositing #9358
Use a different layer border color with UI-side compositing #9358
Conversation
EWS run on previous version of this PR (hash 8ef0b7c) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@hortont424 knows my request for a legend to identify what these colors mean still stands.
8ef0b7c
to
9fbb234
Compare
EWS run on current version of this PR (hash 9fbb234) |
515eb88
to
18d49fc
Compare
https://bugs.webkit.org/show_bug.cgi?id=251406 <rdar://problem/104844082> Reviewed by Sam Weinig and Tim Horton. The value returned by GraphicsLayerCA::pageTiledBackingBorderColor() will be the legacy page tile border color once UI-side compositing is enabled by default. Change it to something more navy/gray, so that it can be distinguished from the UI-side compositing colors in screenshots. Also do a drive-by fix where the RemoteLayerTreeDrawingArea "root" layer should not get sized to the contents size on macOS, where we don't delegate scrolling to a native view. * Source/WebCore/platform/graphics/ca/GraphicsLayerCA.cpp: (WebCore::GraphicsLayerCA::pageTiledBackingBorderColor const): (WebCore::GraphicsLayerCA::getDebugBorderInfo const): * Source/WebCore/platform/graphics/ca/GraphicsLayerCA.h: * Source/WebKit/WebProcess/WebPage/RemoteLayerTree/GraphicsLayerCARemote.cpp: (WebKit::GraphicsLayerCARemote::pageTiledBackingBorderColor const): * Source/WebKit/WebProcess/WebPage/RemoteLayerTree/GraphicsLayerCARemote.h: * Source/WebKit/WebProcess/WebPage/mac/RemoteLayerTreeDrawingAreaMac.h: * Source/WebKit/WebProcess/WebPage/mac/RemoteLayerTreeDrawingAreaMac.mm: (WebKit::RemoteLayerTreeDrawingAreaMac::mainFrameContentSizeChanged): Canonical link: https://commits.webkit.org/259637@main
18d49fc
to
288bbe8
Compare
Committed 259637@main (288bbe8): https://commits.webkit.org/259637@main Reviewed commits have been landed. Closing PR #9358 and removing active labels. |
288bbe8
9fbb234
π iosπ macπ π§ͺ winπ ios-simπ mac-AS-debugπ wincairoπ§ͺ ios-wk2π§ͺ api-macπ§ͺ gtk-wk2π§ͺ api-iosπ§ͺ mac-wk1π§ͺ api-gtkπ§ͺ mac-wk2π tv-simπ§ͺ mac-AS-debug-wk2π watchπ§ͺ mac-wk2-stressπ watch-sim