Bug 233123 - Canvas rendering context should not be tied to PlatformLayer
Summary: Canvas rendering context should not be tied to PlatformLayer
Status: RESOLVED CONFIGURATION CHANGED
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebGL (show other bugs)
Version: WebKit Nightly Build
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Kimmo Kinnunen
URL:
Keywords:
Depends on: 233175
Blocks: 231009
  Show dependency treegraph
 
Reported: 2021-11-15 05:42 PST by Kimmo Kinnunen
Modified: 2021-11-18 12:38 PST (History)
7 users (show)

See Also:


Attachments
Patch (77.96 KB, patch)
2021-11-15 06:20 PST, Kimmo Kinnunen
no flags Details | Formatted Diff | Diff
Patch (72.41 KB, patch)
2021-11-16 07:39 PST, Kimmo Kinnunen
no flags Details | Formatted Diff | Diff
Patch (72.28 KB, patch)
2021-11-16 07:43 PST, Kimmo Kinnunen
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Kimmo Kinnunen 2021-11-15 05:42:54 PST
Canvas rendering context should not be tied to PlatformLayer

Currently CanvasRenderingContext "draws to" PlatformLayer, i.e. has CanvasRenderingContext::platformLayer().
This does not work for UI-side compositing, as we do not want the WebGL graphics context need CALayer in the UI process.

Instead there should be some sort of abstraction that can be used in conjunction with GraphicsLayer contents property, so that what CanvasRenderingContext draws can be shown as GraphicsLayer contents.
Comment 1 Kimmo Kinnunen 2021-11-15 06:20:56 PST
Created attachment 444245 [details]
Patch
Comment 2 Kimmo Kinnunen 2021-11-16 07:39:00 PST
Created attachment 444380 [details]
Patch
Comment 3 Kimmo Kinnunen 2021-11-16 07:43:05 PST
Created attachment 444381 [details]
Patch
Comment 4 Kimmo Kinnunen 2021-11-18 10:51:33 PST
Currently implemented as part of the parent patch in bug 231009