The default bug view has changed. See this FAQ.

Enable WEBGL_compressed_texture_s3tc for machines using Angle

RESOLVED FIXED in Firefox 15

Status

()

Core
Canvas: WebGL
RESOLVED FIXED
5 years ago
5 years ago

People

(Reporter: michaelbraithwaite, Assigned: bjacob)

Tracking

(Blocks: 1 bug)

16 Branch
mozilla16
x86_64
Windows 7
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(firefox15 fixed)

Details

Attachments

(2 attachments, 1 obsolete attachment)

(Reporter)

Description

5 years ago
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.
(Assignee)

Updated

5 years ago
Blocks: 710398
(Assignee)

Comment 1

5 years ago
Created attachment 633214 [details] [diff] [review]
fix s3tc extension on ANGLE

Many thanks for the report, this is really important and had escaped us. Chromium is doing the same thing here.
Attachment #633214 - Flags: review?(jgilbert)
Component: Untriaged → Graphics
Product: Firefox → Core
QA Contact: untriaged → thebes
(Assignee)

Updated

5 years ago
Component: Graphics → Canvas: WebGL
QA Contact: thebes → canvas.webgl
(Assignee)

Comment 2

5 years ago
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.
Attachment #633214 - Flags: review?(jgilbert) → review+
Status: UNCONFIRMED → NEW
Ever confirmed: true

Updated

5 years ago
Depends on: 728017

Comment 4

5 years ago
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

:)

Comment 5

5 years ago
Should we try and get this in fx15?
(Assignee)

Comment 6

5 years ago
Martin: Yes of course!

Comment 7

5 years ago
Looks like 15 may still be on trunk but I'll check with Alex to see if we need to nominated it.
(Assignee)

Comment 8

5 years ago
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.
(Assignee)

Comment 9

5 years ago
Created attachment 633369 [details] [diff] [review]
fix s3tc extension on ANGLE (updated for landing)

Updated to follow Jeff's recommendation.
Attachment #633369 - Flags: review+
http://hg.mozilla.org/integration/mozilla-inbound/rev/3159f0528301
Assignee: nobody → bjacob
Target Milestone: --- → mozilla16
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
Attachment #633369 - Flags: approval-mozilla-aurora?
Created attachment 633375 [details] [diff] [review]
fix s3tc extension on ANGLE (updated for landing)

Sorry, this is the patch that I actually landed.
Attachment #633369 - Attachment is obsolete: true
Attachment #633369 - Flags: approval-mozilla-aurora?
Attachment #633375 - Flags: review+
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
Attachment #633375 - Flags: approval-mozilla-aurora?
https://hg.mozilla.org/mozilla-central/rev/3159f0528301
Status: NEW → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED
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.
Attachment #633375 - Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
Thanks!

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

Updated

5 years ago
status-firefox15: --- → fixed

Updated

5 years ago
Keywords: dev-doc-needed
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!
Keywords: dev-doc-needed
You need to log in before you can comment on or make changes to this bug.