Bug 208680

Summary: RemoteImageBufferMessageHandler should not manage the lifetime of remote buffers
Product: WebKit Reporter: Dean Jackson <dino>
Component: CanvasAssignee: Dean Jackson <dino>
Status: NEW    
Severity: Normal CC: dino, sabouhallawa, webkit-bug-importer
Priority: P2 Keywords: InRadar
Version: WebKit Nightly Build   
Hardware: Unspecified   
OS: Unspecified   

Dean Jackson
Reported 2020-03-05 17:22:10 PST
RemoteImageBufferMessageHandler is supposed to be a message handler. However it actually manages the lifetime of the remote image buffer, by sending a CreateImageBuffer message in its constructor, and a ReleaseImageBuffer in its destructor. I think these operations should be explicit, and called from the actual class that represents the local version of the remote buffer (which is a subclass of RemoteImageBufferMessageHandler, but is really using that as a protocol, not for inheritance). The create and release operations should be methods on RemoteImageBufferMessageHandler.
Attachments
Radar WebKit Bug Importer
Comment 1 2020-03-05 17:22:18 PST
Note You need to log in before you can comment on or make changes to this bug.