Summary: | Sampled Page Top Color: tie into existing TCADA/RLTT systems so that the value is kept in sync with painting | ||||||
---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Devin Rousso <hi> | ||||
Component: | New Bugs | Assignee: | Devin Rousso <hi> | ||||
Status: | RESOLVED FIXED | ||||||
Severity: | Normal | CC: | cdumez, esprehn+autocc, ews-watchlist, hi, kangil.han, thorton, webkit-bug-importer | ||||
Priority: | P2 | Keywords: | InRadar | ||||
Version: | WebKit Nightly Build | ||||||
Hardware: | Unspecified | ||||||
OS: | Unspecified | ||||||
Attachments: |
|
Description
Devin Rousso
2021-05-05 12:41:44 PDT
Created attachment 427792 [details]
Patch
Comment on attachment 427792 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=427792&action=review > Source/WebKit/WebProcess/WebPage/WebPage.cpp:6469 > + send(Messages::WebPageProxy::SampledPageTopColorChanged(m_page->sampledPageTopColor())); I don't understand why there's a non-transactional version of this? It should just dirty the transaction and be sent that way (so that it, again, is synchronized with the layer content in the transaction). Comment on attachment 427792 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=427792&action=review >> Source/WebKit/WebProcess/WebPage/WebPage.cpp:6469 >> + send(Messages::WebPageProxy::SampledPageTopColorChanged(m_page->sampledPageTopColor())); > > I don't understand why there's a non-transactional version of this? It should just dirty the transaction and be sent that way (so that it, again, is synchronized with the layer content in the transaction). This is only called inside `TiledCoreAnimationDrawingArea::updateRendering`. This is also how `themeColor` and `pageExtendedBackgroundColor` do it. It was my understanding that this is how it's done for macOS? Am I mistaken? (In reply to Devin Rousso from comment #3) > Comment on attachment 427792 [details] > Patch > > View in context: > https://bugs.webkit.org/attachment.cgi?id=427792&action=review > > >> Source/WebKit/WebProcess/WebPage/WebPage.cpp:6469 > >> + send(Messages::WebPageProxy::SampledPageTopColorChanged(m_page->sampledPageTopColor())); > > > > I don't understand why there's a non-transactional version of this? It should just dirty the transaction and be sent that way (so that it, again, is synchronized with the layer content in the transaction). > > This is only called inside `TiledCoreAnimationDrawingArea::updateRendering`. > This is also how `themeColor` and `pageExtendedBackgroundColor` do it. It > was my understanding that this is how it's done for macOS? Am I mistaken? Oh oh oh, yes, this is for non-transactional WebKit :) Committed r277045 (237357@main): <https://commits.webkit.org/237357@main> All reviewed patches have been landed. Closing bug and clearing flags on attachment 427792 [details]. |