Bug 180863 - HLS WebGL video textures, UNPACK_FLIP_Y_WEBGL breaks rendering with Safari 11 macOS 10.12.6
Summary: HLS WebGL video textures, UNPACK_FLIP_Y_WEBGL breaks rendering with Safari 11...
Status: NEW
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebGL (show other bugs)
Version: Safari 11
Hardware: Macintosh macOS 10.12
: P2 Normal
Assignee: Nobody
URL:
Keywords: InRadar
Depends on:
Blocks:
 
Reported: 2017-12-15 04:28 PST by Daniel Rossi
Modified: 2018-04-11 12:39 PDT (History)
6 users (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Daniel Rossi 2017-12-15 04:28:40 PST
I am confirming that UNPACK_FLIP_Y_WEBGL is still breaking HLS rendering with Safari 11 on macOS 10.12.6. I have reported these issues before but no resolution. I have seperated this out with bugs in IOS 11 which flipy now has no effect at all. 

The Flipy flags cause black rendering like the IOS 11 HLS problem. No flag change fixes it apart from disabling FlipY. 

It has to be worked around by disabling Flipy and inverting the y coordinate on the vertex shader. 


With Flipy Flags:

http://dev.electroteque.org/webgl/three-hls.html
http://dev.electroteque.org/webgl/webgl-hls.html

Without FlipY flags and using shader Y inverse. Works.  

http://dev.electroteque.org/webgl/three-hls-flipy.html
http://dev.electroteque.org/webgl/webgl-hls-flipy.html

Mp4 with FlipY is fine

http://dev.electroteque.org/webgl/three-mp4.html

Please see these tickets for reference

https://bugs.webkit.org/show_bug.cgi?id=163866
https://bugs.webkit.org/show_bug.cgi?id=179417
Comment 1 Radar WebKit Bug Importer 2017-12-18 01:16:14 PST
<rdar://problem/36101946>
Comment 2 Daniel Rossi 2017-12-19 02:11:09 PST
I've had to go back to my original ticket and findings. Someone is complaining that Safari 10 and Yosemite has the same problem as IOS 11 with HLS. As in the flipY fix does nothing. 

Just another problem to look out for and enable canvas rendering for instead as a work around.
Comment 3 Daniel Rossi 2017-12-24 17:44:29 PST
I'm sorry but I just stumbled across a comment in regards to macOS 10.13. Is this out yet ? 

So I will have to create yet another platform version check to not use flipY work arounds for 10.13 ! 

I am using a cloud based macOS as I don't have hardware and vmware was very complicated to maintain and hacky. It is difficult to keep things up to date in that automatically. Is 10.13 out yet ? 

https://bugs.webkit.org/show_bug.cgi?id=179417#c8

A flipY for textures says is  fixed but this was never an issue with mp4 ? Was it tested with HLS textures ? 

https://bugs.webkit.org/show_bug.cgi?id=176491
Comment 4 Daniel Rossi 2017-12-26 03:50:48 PST
I now have reports from someone on High Sierra 10.13 the FlipY work around which I am reporting here is totally broken. Just like going from IOS 10 to IOs 11 the FlipY work around stopped working and there is a complete breakage. And it seemed just after that FlipY "fix" which was never a problem with Mp4 to begin with totally broke in HLS which I believe HLS wasn't even tested. 

Shall we forget about 10.12, people will have to do the countless platform checks and provide the workarounds for backwards compatibility. 

I believe we have to roll this issue back with the IOS 11 ticket.
Comment 5 Daniel Rossi 2017-12-26 03:52:00 PST
I believe this "fix" created a "regression" or something. 

https://bugs.webkit.org/show_bug.cgi?id=176491
Comment 6 Daniel Rossi 2017-12-26 08:33:13 PST
Scratch that. I can't edit my comments. They now say the flipy work around is ok.  

This problem is still relevant to 10.13 then. Requires FlipY disabled.
Comment 7 Justin Fan 2018-04-11 12:39:04 PDT
(In reply to Daniel Rossi from comment #6)
> Scratch that. I can't edit my comments. They now say the flipy work around
> is ok.  
> 
> This problem is still relevant to 10.13 then. Requires FlipY disabled.

Hi Daniel, I cannot replicate the black screen issue on Safari Tech Preview on 10.13.14, but I noticed that you have both FLIP_Y set to true and the shader workaround enabled on your webgl-hls.html example. Can you confirm your experience on the latest Safari Tech Preview?