958369 - Make GLContext enums typed, and drop ContextFlags

Status: RESOLVED FIXED

(Core :: Graphics, defect)

Comment 1

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

Attachment: 1/5 - Make GLFeature a typed enum

Comment 2

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

Attachment: 2/5 - Make GLVendor and GLRenderer typed enums

Comment 3

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

Attachment: 3/5 - Make SharedTextureShareType and SharedTextureBufferType typed enums

Comment 4

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

Attachment: 4/5 - remove the ContextFlags enum

Comment 5

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

Attachment: 5/5 - Make ContextType a typed enum

Comment 6

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

$ hg diff --stat -r drop-mesa-llvmpipe-switch:ContextType
 content/canvas/src/CanvasRenderingContext2D.cpp |   2 +-
 content/canvas/src/WebGL2Context.cpp            |   2 +-
 content/canvas/src/WebGLContext.cpp             |   5 +-
 content/canvas/src/WebGLContextGL.cpp           |   8 +-
 content/canvas/src/WebGLContextValidate.cpp     |   2 +-
 content/canvas/src/WebGLContextVertices.cpp     |   2 +-
 content/canvas/src/WebGLRenderbuffer.cpp        |   2 +-
 dom/plugins/base/nsNPAPIPluginInstance.cpp      |   8 +-
 dom/plugins/base/nsPluginInstanceOwner.cpp      |   6 +-
 gfx/gl/GLContext.cpp                            |  40 +++++-----
 gfx/gl/GLContext.h                              |  77 +++++++++-----------
 gfx/gl/GLContextCGL.h                           |   4 +-
 gfx/gl/GLContextEGL.h                           |   4 +-
 gfx/gl/GLContextFeatures.cpp                    |  26 +++---
 gfx/gl/GLContextGLX.h                           |   4 +-
 gfx/gl/GLContextProviderCGL.mm                  |   5 +-
 gfx/gl/GLContextProviderEGL.cpp                 |   5 +-
 gfx/gl/GLContextProviderGLX.cpp                 |  12 +-
 gfx/gl/GLContextProviderImpl.h                  |   7 +-
 gfx/gl/GLContextProviderWGL.cpp                 |  19 ++---
 gfx/gl/GLContextTypes.h                         |  38 ++++------
 gfx/gl/GLContextWGL.h                           |   4 +-
 gfx/gl/GLReadTexImageHelper.cpp                 |   2 +-
 gfx/gl/GLSharedHandleHelpers.cpp                |  16 ++--
 gfx/gl/GLTextureImage.cpp                       |  10 +-
 gfx/gl/GLUploadHelpers.cpp                      |  12 +-
 gfx/layers/client/ClientCanvasLayer.cpp         |   2 +-
 gfx/layers/opengl/TextureHostOGL.h              |   2 +-
 28 files changed, 151 insertions(+), 175 deletions(-)

Comment 7

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

Attachment: Combined diff

Comment 8

[Kelsey Gilbert [:jgilbert]]

Comment on attachment 8358120 [details] [diff] [review]
1/5 - Make GLFeature a typed enum

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

::: gfx/gl/GLContext.h
@@ +85,1 @@
>          bind_buffer_offset,

This is over-indented now.

Comment 9

[Kelsey Gilbert [:jgilbert]]

Holding off on reviewing 4 and 5 until we've resolved whether/how to allow LLVMpipe usage.

Comment 10

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

https://hg.mozilla.org/integration/mozilla-inbound/rev/08f0663b62bb
https://hg.mozilla.org/integration/mozilla-inbound/rev/4cd12a362c61
https://hg.mozilla.org/integration/mozilla-inbound/rev/b930db813245
https://hg.mozilla.org/integration/mozilla-inbound/rev/9797b751a8a3
https://hg.mozilla.org/integration/mozilla-inbound/rev/c2dd397d92c4

Comment 11

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/mozilla-central/rev/08f0663b62bb
https://hg.mozilla.org/mozilla-central/rev/4cd12a362c61
https://hg.mozilla.org/mozilla-central/rev/b930db813245
https://hg.mozilla.org/mozilla-central/rev/9797b751a8a3
https://hg.mozilla.org/mozilla-central/rev/c2dd397d92c4