Bug 214452 – [WebGL] Safari snapshots of WebGL content in the tab picker don't work

• Status: RESOLVED
• Resolution: FIXED
• Component: WebGL

[Justin Fan]

Support displaying WebGL canvases in Safari tab preview and tab overview features.

[Justin Fan]

<rdar://problem/21243082>

[Justin Fan]

Created attachment 405013 [details]
Patch

[Justin Fan]

Sort of WIP patch for now. The snapshot will use the last-painted Image by the canvas if the Page is no longer active when the snapshot is requested. May still try to write an internal test for the snapshotting feature.

[Justin Fan]

To clarify, the code works, but there's no test.

[Darin Adler]

Comment on attachment 405013 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=405013&action=review

> Source/WebCore/ChangeLog:9
> +        No new tests; writing a test may involve adding a new feature to TestRunner to trigger this path.

Understood. Could we please add that feature?

[Tim Horton]

(Probably we should just write an API test similar to the other snapshotting tests)

[Justin Fan]

Should have checked this earlier, but iOS just works. This only occurred on mac.

[Dean Jackson]

Comment on attachment 405013 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=405013&action=review

>> Source/WebCore/ChangeLog:9
>> +        No new tests; writing a test may involve adding a new feature to TestRunner to trigger this path.
> 
> Understood. Could we please add that feature?

We definitely should. Either an API test or something in Internals.

Internals could expose the same thing that the Web Inspector uses with "PageAgent.snapshotNode", which ultimately is WebCore::snapshotNode, and either return the ImageBuffer as an HTMLImageElement, an ArrayBuffer (for drawing into a canvas) or even a DOMString via toDataURL("image/png"_s, WTF::nullopt, PreserveResolution::Yes) on the result.

[Dean Jackson]

I slightly prefer using Internals because then it can be a reftest in LayoutTests, which I think people run more frequently than API tests.

[Dean Jackson]

Committed r265116: <https://trac.webkit.org/changeset/265116>

[Dean Jackson]

I added an API test (although mac only at the moment).