RESOLVED FIXED 189610
WebGL 2 conformance: rgb-format-support.html 
https://bugs.webkit.org/show_bug.cgi?id=189610
Summary WebGL 2 conformance: rgb-format-support.html
Justin Fan
Reported 2018-09-13 19:15:28 PDT
WebGL 2 conformance: rbg-format-support.html
Attachments
Patch (19.27 KB, patch)
2018-09-13 19:29 PDT, Justin Fan 		no flags
DetailsFormatted DiffDiff
Patch (19.31 KB, patch)
2018-09-13 19:33 PDT, Justin Fan 		no flags
DetailsFormatted DiffDiff
Patch (19.29 KB, patch)
2018-09-14 15:05 PDT, Justin Fan 		no flags
DetailsFormatted DiffDiff
Show Obsolete (2) View All Add attachment proposed patch, testcase, etc.
Justin Fan
Comment 1 2018-09-13 19:16:03 PDT
<rdar://problem/44403343>
Justin Fan
Comment 2 2018-09-13 19:29:45 PDT
Created attachment 349726 [details] Patch
Justin Fan
Comment 3 2018-09-13 19:33:34 PDT
Created attachment 349727 [details] Patch
Dean Jackson
Comment 4 2018-09-14 11:18:24 PDT
Comment on attachment 349727 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=349727&action=review > Source/WebCore/html/canvas/WebGL2RenderingContext.cpp:380 > + // On macOS (OpenGL 4.1) we must emulate GL ES 3/OpenGL 4.2 glGetInternalformativ. This path might not be just macOS, so just say "On desktop OpenGL" > Source/WebCore/html/canvas/WebGL2RenderingContext.cpp:396 > + // FIXME: The way I understand this is that this will return a MINIMUM numSamples for all accepeted internalformats. > + // However, the true value of this on supported GL versions is gleaned via a getInternalformativ call that depends on internalformat. This isn't really a FIXME.
Justin Fan
Comment 5 2018-09-14 15:05:19 PDT
Created attachment 349813 [details] Patch
WebKit Commit Bot
Comment 6 2018-09-14 17:05:30 PDT
Comment on attachment 349813 [details] Patch Clearing flags on attachment: 349813 Committed r236025: <https://trac.webkit.org/changeset/236025>
WebKit Commit Bot
Comment 7 2018-09-14 17:05:32 PDT
All reviewed patches have been landed. Closing bug.
Note You need to log in before you can comment on or make changes to this bug.