Kelsey Gilbert [:jgilbert] Reporter
	Description • 11 years ago

We still maintain support for retrieving a webgl context from a canvas via 'moz-webgl'. We should deprecate and then remove this.

	Kelsey Gilbert [:jgilbert] Reporter
	Comment 3 • 11 years ago

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.

	Benoit Jacob [:bjacob] (mostly away)
	Comment 4 • 11 years ago

Nope, that's fine :-)

	Ryan VanderMeulen [:RyanVM]
	Comment 5 • 11 years ago

https://hg.mozilla.org/integration/mozilla-inbound/rev/505e8f8480d2

	Phil Ringnalda (:philor)
	Comment 6 • 11 years ago

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

	Phil Ringnalda (:philor)
	Comment 7 • 11 years ago

Oh, and b2g, which supports neither, https://tbpl.mozilla.org/php/getParsedLog.php?id=28013847&tree=Mozilla-Inbound

	Kelsey Gilbert [:jgilbert] Reporter
	Comment 8 • 11 years ago

Yep, this is true.
Restrict the test to just 'moz-webgl' and 'experimental-webgl', since we already have a test that checks 'webgl'.

	Kelsey Gilbert [:jgilbert] Reporter
	Comment 9 • 11 years ago

Actually, probably just test that if 'experimental-webgl' succeeds, then 'moz-webgl' fails.

	u480271
	Comment 10 • 10 years ago

I'm not actively working on this.

	Masatoshi Kimura [:emk] Assignee
	Comment 11 • 10 years ago

Unbitrotted and removed the test for the unprefixed context name.

	Masatoshi Kimura [:emk] Assignee
	Comment 12 • 10 years ago

https://hg.mozilla.org/integration/mozilla-inbound/rev/07e5bf2cb7c8

	Ryan VanderMeulen [:RyanVM]
	Comment 13 • 10 years ago

Still failing. Backed out.
https://hg.mozilla.org/integration/mozilla-inbound/rev/5cf670407fac

https://tbpl.mozilla.org/php/getParsedLog.php?id=33402846&tree=Mozilla-Inbound

	Ryan VanderMeulen [:RyanVM]
	Comment 14 • 10 years ago

And regular B2G.
https://tbpl.mozilla.org/php/getParsedLog.php?id=33405111&tree=Mozilla-Inbound

	Masatoshi Kimura [:emk] Assignee
	Comment 15 • 10 years ago

Is WebGL totally unavailable on B2G?

	Benoit Jacob [:bjacob] (mostly away)
	Comment 16 • 10 years ago

WebGL is totally _available_ on B2G and even used in the Gallery application :-)

	Masatoshi Kimura [:emk] Assignee
	Comment 17 • 10 years ago

Then there should be a test preventing it from regression.

	Kelsey Gilbert [:jgilbert] Reporter
	Comment 18 • 10 years ago

(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.

	Masatoshi Kimura [:emk] Assignee
	Comment 19 • 10 years ago

According to bug 909903, WebGL context creation was possible until bug 900201 was fixed.

	Masatoshi Kimura [:emk] Assignee
	Comment 20 • 10 years ago

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.

	Masatoshi Kimura [:emk] Assignee
	Comment 21 • 10 years ago

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

	Masatoshi Kimura [:emk] Assignee
	Comment 22 • 10 years ago

https://hg.mozilla.org/integration/mozilla-inbound/rev/e9c39aa248f3

	Carsten Book [:Tomcat]
	Comment 23 • 10 years ago

https://hg.mozilla.org/mozilla-central/rev/e9c39aa248f3

	Kohei Yoshino [:kohei]
	Comment 24 • 10 years ago

https://developer.mozilla.org/en-US/Firefox/Releases/29/Site_Compatibility