User Agent: Mozilla/5.0 (Windows NT 6.1; WOW64) AppleWebKit/536.5 (KHTML, like Gecko) Chrome/19.0.1084.56 Safari/536.5 Steps to reproduce: Following on from https://bugzilla.mozilla.org/show_bug.cgi?id=728017 * Run nightly 16.0a1 on a machine that uses Angle for WebGL (eg Windows 7 ATI 6800 with catalyist 12.3) * Look at the WebGL supported extension and it list "Extensions: OES_texture_float OES_standard_derivatives MOZ_WEBGL_lose_context" * If I force native GL using about:config and it then lists "Extensions: OES_texture_float OES_standard_derivatives MOZ_EXT_texture_filter_anisotropic MOZ_WEBGL_lose_context MOZ_WEBGL_compressed_texture_s3tc" Actual results: No MOZ_WEBGL_compressed_texture_s3tc support Expected results: MOZ_WEBGL_compressed_texture_s3tc support as Angle supports the required features.

Many thanks for the report, this is really important and had escaped us. Chromium is doing the same thing here.

Oh right --- Michael: the category for WebGL bugs is Core -> Canvas:WebGL, correct categorization helps a lot avoiding bugs getting forgotten about. Gavin: what can we do to help with the problem that most people don't understand what 'Core' is and just file all their bugs into Firefox, and then obviously can't find the Gecko categories they're looking for under Firefox, so they end up filing in Firefox->General or Untriaged? The bugzilla homepage already says this about Core: "Shared components used by Firefox and other Mozilla software, including handling of Web content (HTML, CSS, scripts, images, networking, etc.)." Maybe we could have a notice under Firefox reminding people that "handling of Web content" does NOT belong there?

Comment on attachment 633214 [details] [diff] [review] fix s3tc extension on ANGLE Review of attachment 633214 [details] [diff] [review]: ----------------------------------------------------------------- A nit for consideration. ::: content/canvas/src/WebGLContext.cpp @@ +859,5 @@ > case WebGL_WEBGL_compressed_texture_s3tc: > + isSupported = gl->IsExtensionSupported(GLContext::EXT_texture_compression_s3tc) || > + (gl->IsExtensionSupported(GLContext::EXT_texture_compression_dxt1) && > + gl->IsExtensionSupported(GLContext::ANGLE_texture_compression_dxt3) && > + gl->IsExtensionSupported(GLContext::ANGLE_texture_compression_dxt5)); This would probably be more readable as two separate if() statements, but it's...acceptable as-is.

jgilbert: Chrome did it first! http://code.google.com/searchframe#OAMlx_jo-ck/src/third_party/WebKit/Source/WebCore/html/canvas/WebGLCompressedTextureS3TC.cpp&type=cs&l=65 :)

Should we try and get this in fx15?

Martin: Yes of course!

Looks like 15 may still be on trunk but I'll check with Alex to see if we need to nominated it.

Trunk has been 16 for a week now and 15 is aurora. But this is such a low-risk and high-importance fix that I have no doubt that we'll get aurora approval. Aurora approval early in the cycle isn't that hard to get.

Updated to follow Jeff's recommendation.

Comment on attachment 633369 [details] [diff] [review] fix s3tc extension on ANGLE (updated for landing) [Approval Request Comment] Bug caused by (feature/regressing bug #): not a regression, but it makes the benefits of S3TC compressed textures (landed in bug 728017) unavailable to Windows users. User impact if declined: No s3tc compressed texture support for Windows users in default configuration. That really sucks: s3tc compressed textures is THE most important missing WebGL feature, it's required to be serious players in the Games area, and we were looking forward to announce support to that in Firefox 15. Testing completed (on m-c, etc.): Just landed on m-i Risk to taking this patch (and alternatives if risky): Not risky at all. Look at the patch, it's really trivial. String or UUID changes made by this patch: none

Sorry, this is the patch that I actually landed.

Comment on attachment 633375 [details] [diff] [review] fix s3tc extension on ANGLE (updated for landing) [Approval Request Comment] Bug caused by (feature/regressing bug #): not a regression, but it makes the benefits of S3TC compressed textures (landed in bug 728017) unavailable to Windows users. User impact if declined: No s3tc compressed texture support for Windows users in default configuration. That really sucks: s3tc compressed textures is THE most important missing WebGL feature, it's required to be serious players in the Games area, and we were looking forward to announce support to that in Firefox 15. Testing completed (on m-c, etc.): Just landed on m-i Risk to taking this patch (and alternatives if risky): Not risky at all. Look at the patch, it's really trivial. String or UUID changes made by this patch: none

Comment on attachment 633375 [details] [diff] [review] fix s3tc extension on ANGLE (updated for landing) [Triage Comment] With an approval request like that, how could we say no? Low risk, gets us better gaming on Windows - approved for Aurora.

Thanks! http://hg.mozilla.org/releases/mozilla-aurora/rev/ac587e102ae8

How is this dev-doc-needed? This landed in 15 just like s3tc support itself.

(In reply to comment #17) > How is this dev-doc-needed? This landed in 15 just like s3tc support itself. Is it covered by the same documentation?

My point is from the point of view of dev-docs, there is only 1 feature to document: s3tc textures. There is no need to enter into platform-specific details as this support landed on all OSes in the same version.

Understood and agreed!