619485 - add WebGL info to about:support

Status: RESOLVED FIXED

(Core :: Graphics: CanvasWebGL, defect)

Description

[Vladimir Vukicevic [:vlad] [:vladv] (needinfo me, slow to respond)]

Attachment: add webgl renderer info to about:support

It's a bit roundabout, since I want it to go through the same creation dance that WebGL contexts normally go through.

Comment 1

[Jeff Muizelaar [:jrmuizel]]

Comment on attachment 497894 [details] [diff] [review]
add webgl renderer info to about:support

I'd rather we not have two copies of GetWebGLParameter

Comment 2

[Vladimir Vukicevic [:vlad] [:vladv] (needinfo me, slow to respond)]

Yeah, me too, though the way to avoid that was going to be to introduce GfxInfoBase.cpp in xpwidgets, which means that every gfxinfo method has to be declared manually in the child gfxinfo classes.  Kind of ugly.

Maybe a GfxInfoWebGLParameterImpl.cpp that can be #included?

Comment 4

[Vladimir Vukicevic [:vlad] [:vladv] (needinfo me, slow to respond)]

Attachment: updated

Updated, with more shared code and a string bundle (needs late-l10n I guess?)

Comment 5

[Mike Beltzner [:beltzner, not reading bugmail]]

Comment on attachment 498156 [details] [diff] [review]
updated

a=beltzner

Comment 6

[Vladimir Vukicevic [:vlad] [:vladv] (needinfo me, slow to respond)]

http://hg.mozilla.org/mozilla-central/rev/075bb085d755

Comment 7

[Daniel C]

Mozilla/5.0 (X11; Linux i686; rv:2.0b13pre) Gecko/20110226 Firefox/4.0b13pre

Should this always display something? I'm using the latest trunk nightly on Linux and all I see in the Graphics section is:

"Graphics

GPU Accelerated Windows 0/1"

WebGL is working, as verified by running a few demos.

Comment 8

[Benoit Jacob [:bjacob] (mostly away)]

This is not yet implemented on X11; will be in Firefox 5.