Summary: | [GPU Process][Resource caching 1/7]: Replace "ImageBufferIdentifier" by a generic identifier named "RemoteResourceIdentifier" | ||||||
---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Said Abou-Hallawa <sabouhallawa> | ||||
Component: | Layout and Rendering | Assignee: | Said Abou-Hallawa <sabouhallawa> | ||||
Status: | RESOLVED FIXED | ||||||
Severity: | Normal | CC: | annulen, bfulgham, ews-watchlist, gyuyoung.kim, ryuan.choi, sergio, simon.fraser, webkit-bug-importer, youennf, zalan | ||||
Priority: | P2 | Keywords: | InRadar | ||||
Version: | WebKit Nightly Build | ||||||
Hardware: | Unspecified | ||||||
OS: | Unspecified | ||||||
Bug Depends on: | |||||||
Bug Blocks: | 217342, 217554 | ||||||
Attachments: |
|
Description
Said Abou-Hallawa
2020-10-09 18:34:36 PDT
Created attachment 410994 [details]
Patch
Comment on attachment 410994 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=410994&action=review > Source/WebCore/ChangeLog:10 > + be used drawing a remote NativeImage to a remote ImageBuffer or DOM layer. RemoteResourceIdentifier seems very generic and it seems the identifier is all about images. Are there plans to use this identifier in non images cases? How about some more specific names like RemoteImageIdentifier instead? Comment on attachment 410994 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=410994&action=review >> Source/WebCore/ChangeLog:10 >> + be used drawing a remote NativeImage to a remote ImageBuffer or DOM layer. > > RemoteResourceIdentifier seems very generic and it seems the identifier is all about images. > Are there plans to use this identifier in non images cases? > How about some more specific names like RemoteImageIdentifier instead? It's going to get used for fonts as well. > Source/WebKit/WebProcess/GPU/graphics/RemoteImageBufferMessageHandler.h:75 > + WebCore::RemoteResourceIdentifier m_remoteResourceIdentifier { WebCore::RemoteResourceIdentifier::generate() }; It's weird for RemoteImageBufferMessageHandler to be the one that creates the identifiers and positively wrong for the WebProcess to be creating identifiers because they have to be unique in the GPU process. Comment on attachment 410994 [details]
Patch
r+ but let's fix the identifier creation in a followup.
Comment on attachment 410994 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=410994&action=review >> Source/WebKit/WebProcess/GPU/graphics/RemoteImageBufferMessageHandler.h:75 >> + WebCore::RemoteResourceIdentifier m_remoteResourceIdentifier { WebCore::RemoteResourceIdentifier::generate() }; > > It's weird for RemoteImageBufferMessageHandler to be the one that creates the identifiers and positively wrong for the WebProcess to be creating identifiers because they have to be unique in the GPU process. Logged bug 217638 to track this issue. Committed r268372: <https://trac.webkit.org/changeset/268372> All reviewed patches have been landed. Closing bug and clearing flags on attachment 410994 [details]. |