RESOLVED FIXED Bug 219067
[Concurrent display lists] Add a way for display lists to partially replay 
https://bugs.webkit.org/show_bug.cgi?id=219067
Summary [Concurrent display lists] Add a way for display lists to partially replay
Wenson Hsieh
Reported 2020-11-17 21:03:55 PST
SSIA
Attachments
Patch (34.21 KB, patch)
2020-11-17 21:47 PST, Wenson Hsieh 		ews-feeder: commit-queue-
DetailsFormatted DiffDiff
Fix debug builds (30.48 KB, patch)
2020-11-17 23:12 PST, Wenson Hsieh 		no flags
DetailsFormatted DiffDiff
Rebase on trunk (30.51 KB, patch)
2020-11-17 23:22 PST, Wenson Hsieh 		no flags
DetailsFormatted DiffDiff
Slightly simpler approach (28.76 KB, patch)
2020-11-18 14:42 PST, Wenson Hsieh 		no flags
DetailsFormatted DiffDiff
Show Obsolete (3) View All Add attachment proposed patch, testcase, etc.
Wenson Hsieh
Comment 1 2020-11-17 21:47:01 PST Comment hidden (obsolete)
Wenson Hsieh
Comment 2 2020-11-17 23:12:10 PST Comment hidden (obsolete)
Wenson Hsieh
Comment 3 2020-11-17 23:22:49 PST Comment hidden (obsolete)
Wenson Hsieh
Comment 4 2020-11-18 14:42:55 PST
Created attachment 414486 [details] Slightly simpler approach
Simon Fraser (smfr)
Comment 5 2020-11-18 16:25:50 PST
Comment on attachment 414486 [details] Slightly simpler approach View in context: https://bugs.webkit.org/attachment.cgi?id=414486&action=review > Source/WebKit/GPUProcess/graphics/RemoteRenderingBackend.cpp:154 > + if (destination.isAccelerated()) > + static_cast<AcceleratedRemoteImageBuffer&>(destination).submitDisplayList(*displayList); > + else > + static_cast<UnacceleratedRemoteImageBuffer&>(destination).submitDisplayList(*displayList); This is sad-making.
Tim Horton
Comment 6 2020-11-18 16:47:37 PST
Yeah, it's the same problem in a bunch of places (but, hopefully we can fix them all simultaneously).
Wenson Hsieh
Comment 7 2020-11-18 16:49:18 PST
Comment on attachment 414486 [details] Slightly simpler approach View in context: https://bugs.webkit.org/attachment.cgi?id=414486&action=review >> Source/WebKit/GPUProcess/graphics/RemoteRenderingBackend.cpp:154 >> + static_cast<UnacceleratedRemoteImageBuffer&>(destination).submitDisplayList(*displayList); > > This is sad-making. Yeah, it's not great :( I wanted to try to cast to a RemoteImageBuffer, but that's a templated class. I think that ideally, RemoteRenderingBackend would know about the RemoteImageBuffers it can paint into (instead of always digging it up from the resource cache).
EWS
Comment 8 2020-11-18 17:21:12 PST
Committed r270002: <https://trac.webkit.org/changeset/270002> All reviewed patches have been landed. Closing bug and clearing flags on attachment 414486 [details].
Radar WebKit Bug Importer
Comment 9 2020-11-18 17:22:19 PST
<rdar://problem/71563985>
Note You need to log in before you can comment on or make changes to this bug.