911879 - Give WebGLContext.h a good round of removing unneeded inline method bodies

Status: RESOLVED FIXED

(Core :: Graphics: CanvasWebGL, defect)

Description

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

Attachment: Give WebGLContext.h a good round of removing unneeded inline method bodies

Note in particular:
 - many of the WebGL entry points were moved to WebGLContext.h during the switch to new DOM bindings. For many of them, that wasn't really needed or useful.
 - some of the WebGL entry points taking DOM elements (like TexImage2D) are templated. Those are nontrivial to take out of the header. Left in for now.
 - created a new file, WebGLContextLossTimer.cpp, for that stuff.
 - one method was substantially modified: UpdateWebGLErrorAndClearGLError. It used to have 2 signatures, one taking an output pointer param to get the error code. Changed to one signature with optional argument.

Comment 1

[Kelsey Gilbert [:jgilbert]]

Comment on attachment 798671 [details] [diff] [review]
Give WebGLContext.h a good round of removing unneeded inline method bodies

Review of attachment 798671 [details] [diff] [review]:
-----------------------------------------------------------------

::: content/canvas/src/WebGLContext.h
@@ +198,5 @@
>  
>      void DummyFramebufferOperation(const char *info);
>  
> +    WebGLTexture *activeBoundTextureForTarget(GLenum target) const {
> +      return target == LOCAL_GL_TEXTURE_2D ? mBound2DTextures[mActiveTexture]

You two-space'd this indent on accident.

Comment 2

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

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

Comment 3

[Wes Kocher (:KWierso) (Not reading bugmail; email directly if needed)]

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