[Webkit-unassigned] [Bug 235845] REGRESSION: iOS 15.4b1 – gl.texture2D uploads not working

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=235845

Simon Taylor <simontaylor1 at ntlworld.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |simontaylor1 at ntlworld.com

--- Comment #15 from Simon Taylor <simontaylor1 at ntlworld.com> ---
That makes sense - applying the logic to skip unnecessary uploads to user media without fixing Bug 203148 too may have made this visible on sites where it wasn't before.

Kimmo's suggested texSubImage2d workaround does seem to force a "full" upload of the video, at least in the mp4 case from 203148 (tested on 15.2 only for now). Checking with Systrace in Instruments it looks to add minimal overhead.

// A 1-pixel array buffer
let greenPixel = new Uint8Array([0, 128, 0, 255]);

// Inside rAF loop
  gl.bindTexture(gl.TEXTURE_2D, tex2);
+ gl.texSubImage2D(gl.TEXTURE_2D, 0, 0, 0, 1, 1, gl.RGBA, gl.UNSIGNED_BYTE, greenPixel);
  gl.texImage2D(gl.TEXTURE_2D, 0, gl.RGB, gl.RGB, gl.UNSIGNED_BYTE, video);

I'm going to add support for user media in my test case and then update to 15.4b2 to see what's changed. Will also check out requestVideoFrameCallback, that would save unnecessary uploads at all, as well as helping to avoid 203148.

-- 
You are receiving this mail because:
You are the assignee for the bug.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.webkit.org/pipermail/webkit-unassigned/attachments/20220215/e497581b/attachment.htm>