Bug 217550 – [GPU Process][Resource caching 1/7]: Replace "ImageBufferIdentifier" by a generic identifier named "RemoteResourceIdentifier"

• Component: Layout and Rendering
• Status: RESOLVED FIXED

[Said Abou-Hallawa]

GPU Process needs to manage all the drawing resources and the drawing buffers/layers the same way. For example an HTMLCanvasElement is backed by a drawing buffer called ImageBuffer. All the drawing to the context of this HTMLCanvasElement is supposed to be done in this ImageBuffer. But at the same time this HTMLCanvasElement can be treated as a drawing resources when it is drawn to another HTMLCanvasElement through the method drawImage().

So RemoteResourceIdentifier will be the identifier of the ImageBuffer when drawing is happening to its context in the GPU process. And it will be the identifier of the image which will be drawn to anther buffer or layer in the GPU Process. The goal is to avoid copying the image from Web Process to GPU Process and use RemoteResourceIdentifier instead.

[Said Abou-Hallawa]

Created attachment 410994 [details]
Patch

[youenn fablet]

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?

[Simon Fraser (smfr)]

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.

[Simon Fraser (smfr)]

Comment on attachment 410994 [details]
Patch

r+ but let's fix the identifier creation in a followup.

[Said Abou-Hallawa]

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.

[EWS]

Committed r268372: <https://trac.webkit.org/changeset/268372>

All reviewed patches have been landed. Closing bug and clearing flags on attachment 410994 [details].

[Radar WebKit Bug Importer]

<rdar://problem/70226333>