Closed Bug 913597 Opened 8 years ago Closed 7 years ago
Remove support for 'moz-webgl' context requests
We still maintain support for retrieving a webgl context from a canvas via 'moz-webgl'. We should deprecate and then remove this.
Assignee: nobody → dglastonbury
Comment on attachment 806004 [details] [diff] [review] fix-913597.patch Review of attachment 806004 [details] [diff] [review]: ----------------------------------------------------------------- Awesome, thanks. bjacob can review if he wants, but it really only needs one of us.
Nope, that's fine :-)
Backed out in https://hg.mozilla.org/integration/mozilla-inbound/rev/ae31c55333b3 - from the looks of it, we must only support experimental-webgl, and not webgl, on Android, but the test was running there, so https://tbpl.mozilla.org/php/getParsedLog.php?id=28010372&tree=Mozilla-Inbound
Oh, and b2g, which supports neither, https://tbpl.mozilla.org/php/getParsedLog.php?id=28013847&tree=Mozilla-Inbound
Yep, this is true. Restrict the test to just 'moz-webgl' and 'experimental-webgl', since we already have a test that checks 'webgl'.
Actually, probably just test that if 'experimental-webgl' succeeds, then 'moz-webgl' fails.
I'm not actively working on this.
Assignee: dglastonbury → nobody
Unbitrotted and removed the test for the unprefixed context name.
Attachment #8362520 - Flags: review?(jgilbert) → review+
Is WebGL totally unavailable on B2G?
WebGL is totally _available_ on B2G and even used in the Gallery application :-)
Then there should be a test preventing it from regression.
(In reply to Benoit Jacob [:bjacob] from comment #16) > WebGL is totally _available_ on B2G and even used in the Gallery application > :-) Yeah, but we have awful coverage on try. I don't remember if we successfully run any webgl on B2G at the moment.
The workaround written in bug 900201 comment #19 did not work anymore. https://tbpl.mozilla.org/?tree=Try&rev=7c9f1b59daca We will have to disable the test on B2G atm.
Added SpecialPowers.pushPrevEnv to force-enable WebGL on B2G desktop build, and B2G check for exemption. (force-enable had no effect on B2G emulator.) Try run: https://tbpl.mozilla.org/?tree=Try&rev=06babe90c8bc
Attachment #8365627 - Flags: review?(jgilbert) → review+
Status: ASSIGNED → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla29
You need to log in before you can comment on or make changes to this bug.