Commit 07529ff8 by Yuly Novikov Committed by Commit Bot

Remove const qualifier from generateConfigs

Requested in https://chromium-review.googlesource.com/349353 BUG=angleproject:1362 Change-Id: I989757c2fb8f65d5afcfb177b7f8da1414871e6f Reviewed-on: https://chromium-review.googlesource.com/349694Reviewed-by: 's avatarCorentin Wallez <cwallez@chromium.org> Reviewed-by: 's avatarJamie Madill <jmadill@chromium.org> Commit-Queue: Yuly Novikov <ynovikov@chromium.org>
parent c3c14099
...@@ -52,7 +52,7 @@ class DisplayImpl : public EGLImplFactory ...@@ -52,7 +52,7 @@ class DisplayImpl : public EGLImplFactory
virtual egl::Error makeCurrent(egl::Surface *drawSurface, egl::Surface *readSurface, gl::Context *context) = 0; virtual egl::Error makeCurrent(egl::Surface *drawSurface, egl::Surface *readSurface, gl::Context *context) = 0;
virtual egl::ConfigSet generateConfigs() const = 0; virtual egl::ConfigSet generateConfigs() = 0;
virtual bool isDeviceLost() const = 0; virtual bool isDeviceLost() const = 0;
virtual bool testDeviceLost() = 0; virtual bool testDeviceLost() = 0;
......
...@@ -241,7 +241,7 @@ void DisplayD3D::terminate() ...@@ -241,7 +241,7 @@ void DisplayD3D::terminate()
SafeDelete(mRenderer); SafeDelete(mRenderer);
} }
egl::ConfigSet DisplayD3D::generateConfigs() const egl::ConfigSet DisplayD3D::generateConfigs()
{ {
ASSERT(mRenderer != nullptr); ASSERT(mRenderer != nullptr);
return mRenderer->generateConfigs(); return mRenderer->generateConfigs();
......
...@@ -53,7 +53,7 @@ class DisplayD3D : public DisplayImpl ...@@ -53,7 +53,7 @@ class DisplayD3D : public DisplayImpl
egl::Error makeCurrent(egl::Surface *drawSurface, egl::Surface *readSurface, gl::Context *context) override; egl::Error makeCurrent(egl::Surface *drawSurface, egl::Surface *readSurface, gl::Context *context) override;
egl::ConfigSet generateConfigs() const override; egl::ConfigSet generateConfigs() override;
bool isDeviceLost() const override; bool isDeviceLost() const override;
bool testDeviceLost() override; bool testDeviceLost() override;
......
...@@ -106,7 +106,7 @@ class RendererD3D : public BufferFactoryD3D ...@@ -106,7 +106,7 @@ class RendererD3D : public BufferFactoryD3D
virtual egl::Error initialize() = 0; virtual egl::Error initialize() = 0;
virtual egl::ConfigSet generateConfigs() const = 0; virtual egl::ConfigSet generateConfigs() = 0;
virtual void generateDisplayExtensions(egl::DisplayExtensions *outExtensions) const = 0; virtual void generateDisplayExtensions(egl::DisplayExtensions *outExtensions) const = 0;
virtual ContextImpl *createContext(const gl::ContextState &state) = 0; virtual ContextImpl *createContext(const gl::ContextState &state) = 0;
......
...@@ -916,7 +916,7 @@ void Renderer11::populateRenderer11DeviceCaps() ...@@ -916,7 +916,7 @@ void Renderer11::populateRenderer11DeviceCaps()
SafeRelease(dxgiAdapter2); SafeRelease(dxgiAdapter2);
} }
egl::ConfigSet Renderer11::generateConfigs() const egl::ConfigSet Renderer11::generateConfigs()
{ {
std::vector<GLenum> colorBufferFormats; std::vector<GLenum> colorBufferFormats;
......
...@@ -107,7 +107,7 @@ class Renderer11 : public RendererD3D ...@@ -107,7 +107,7 @@ class Renderer11 : public RendererD3D
egl::Error initialize() override; egl::Error initialize() override;
virtual bool resetDevice(); virtual bool resetDevice();
egl::ConfigSet generateConfigs() const override; egl::ConfigSet generateConfigs() override;
void generateDisplayExtensions(egl::DisplayExtensions *outExtensions) const override; void generateDisplayExtensions(egl::DisplayExtensions *outExtensions) const override;
ContextImpl *createContext(const gl::ContextState &state) override; ContextImpl *createContext(const gl::ContextState &state) override;
......
...@@ -397,7 +397,7 @@ D3DPRESENT_PARAMETERS Renderer9::getDefaultPresentParameters() ...@@ -397,7 +397,7 @@ D3DPRESENT_PARAMETERS Renderer9::getDefaultPresentParameters()
return presentParameters; return presentParameters;
} }
egl::ConfigSet Renderer9::generateConfigs() const egl::ConfigSet Renderer9::generateConfigs()
{ {
static const GLenum colorBufferFormats[] = static const GLenum colorBufferFormats[] =
{ {
......
...@@ -71,7 +71,7 @@ class Renderer9 : public RendererD3D ...@@ -71,7 +71,7 @@ class Renderer9 : public RendererD3D
egl::Error initialize() override; egl::Error initialize() override;
virtual bool resetDevice(); virtual bool resetDevice();
egl::ConfigSet generateConfigs() const override; egl::ConfigSet generateConfigs() override;
void generateDisplayExtensions(egl::DisplayExtensions *outExtensions) const override; void generateDisplayExtensions(egl::DisplayExtensions *outExtensions) const override;
void startScene(); void startScene();
......
...@@ -42,7 +42,7 @@ class DisplayCGL : public DisplayGL ...@@ -42,7 +42,7 @@ class DisplayCGL : public DisplayGL
NativePixmapType nativePixmap, NativePixmapType nativePixmap,
const egl::AttributeMap &attribs) override; const egl::AttributeMap &attribs) override;
egl::ConfigSet generateConfigs() const override; egl::ConfigSet generateConfigs() override;
bool isDeviceLost() const override; bool isDeviceLost() const override;
bool testDeviceLost() override; bool testDeviceLost() override;
......
...@@ -150,7 +150,7 @@ egl::Error DisplayCGL::getDevice(DeviceImpl **device) ...@@ -150,7 +150,7 @@ egl::Error DisplayCGL::getDevice(DeviceImpl **device)
return egl::Error(EGL_BAD_DISPLAY); return egl::Error(EGL_BAD_DISPLAY);
} }
egl::ConfigSet DisplayCGL::generateConfigs() const egl::ConfigSet DisplayCGL::generateConfigs()
{ {
// TODO(cwallez): generate more config permutations // TODO(cwallez): generate more config permutations
egl::ConfigSet configs; egl::ConfigSet configs;
......
...@@ -923,7 +923,7 @@ EGLContext DisplayOzone::initializeContext(EGLConfig config, const egl::Attribut ...@@ -923,7 +923,7 @@ EGLContext DisplayOzone::initializeContext(EGLConfig config, const egl::Attribut
return EGL_NO_CONTEXT; return EGL_NO_CONTEXT;
} }
egl::ConfigSet DisplayOzone::generateConfigs() const egl::ConfigSet DisplayOzone::generateConfigs()
{ {
egl::ConfigSet configs; egl::ConfigSet configs;
......
...@@ -125,7 +125,7 @@ class DisplayOzone final : public DisplayGL ...@@ -125,7 +125,7 @@ class DisplayOzone final : public DisplayGL
NativePixmapType nativePixmap, NativePixmapType nativePixmap,
const egl::AttributeMap &attribs) override; const egl::AttributeMap &attribs) override;
egl::ConfigSet generateConfigs() const override; egl::ConfigSet generateConfigs() override;
bool isDeviceLost() const override; bool isDeviceLost() const override;
bool testDeviceLost() override; bool testDeviceLost() override;
......
...@@ -550,7 +550,7 @@ egl::Error DisplayGLX::initializeContext(glx::FBConfig config, ...@@ -550,7 +550,7 @@ egl::Error DisplayGLX::initializeContext(glx::FBConfig config,
return egl::Error(EGL_NOT_INITIALIZED, "Could not create a backing OpenGL context."); return egl::Error(EGL_NOT_INITIALIZED, "Could not create a backing OpenGL context.");
} }
egl::ConfigSet DisplayGLX::generateConfigs() const egl::ConfigSet DisplayGLX::generateConfigs()
{ {
egl::ConfigSet configs; egl::ConfigSet configs;
configIdToGLXConfig.clear(); configIdToGLXConfig.clear();
......
...@@ -60,7 +60,7 @@ class DisplayGLX : public DisplayGL ...@@ -60,7 +60,7 @@ class DisplayGLX : public DisplayGL
NativePixmapType nativePixmap, NativePixmapType nativePixmap,
const egl::AttributeMap &attribs) override; const egl::AttributeMap &attribs) override;
egl::ConfigSet generateConfigs() const override; egl::ConfigSet generateConfigs() override;
bool isDeviceLost() const override; bool isDeviceLost() const override;
bool testDeviceLost() override; bool testDeviceLost() override;
...@@ -113,8 +113,7 @@ class DisplayGLX : public DisplayGL ...@@ -113,8 +113,7 @@ class DisplayGLX : public DisplayGL
FunctionsGL *mFunctionsGL; FunctionsGL *mFunctionsGL;
//TODO(cwallez) yuck, change generateConfigs to be non-const or add a userdata member to egl::Config? std::map<int, glx::FBConfig> configIdToGLXConfig;
mutable std::map<int, glx::FBConfig> configIdToGLXConfig;
EGLint mRequestedVisual; EGLint mRequestedVisual;
glx::FBConfig mContextConfig; glx::FBConfig mContextConfig;
......
...@@ -466,7 +466,7 @@ egl::Error DisplayWGL::getDevice(DeviceImpl **device) ...@@ -466,7 +466,7 @@ egl::Error DisplayWGL::getDevice(DeviceImpl **device)
return egl::Error(EGL_BAD_DISPLAY); return egl::Error(EGL_BAD_DISPLAY);
} }
egl::ConfigSet DisplayWGL::generateConfigs() const egl::ConfigSet DisplayWGL::generateConfigs()
{ {
egl::ConfigSet configs; egl::ConfigSet configs;
......
...@@ -44,7 +44,7 @@ class DisplayWGL : public DisplayGL ...@@ -44,7 +44,7 @@ class DisplayWGL : public DisplayGL
NativePixmapType nativePixmap, NativePixmapType nativePixmap,
const egl::AttributeMap &attribs) override; const egl::AttributeMap &attribs) override;
egl::ConfigSet generateConfigs() const override; egl::ConfigSet generateConfigs() override;
bool isDeviceLost() const override; bool isDeviceLost() const override;
bool testDeviceLost() override; bool testDeviceLost() override;
......
...@@ -41,7 +41,7 @@ egl::Error DisplayVk::makeCurrent(egl::Surface *drawSurface, ...@@ -41,7 +41,7 @@ egl::Error DisplayVk::makeCurrent(egl::Surface *drawSurface,
return egl::Error(EGL_BAD_ACCESS); return egl::Error(EGL_BAD_ACCESS);
} }
egl::ConfigSet DisplayVk::generateConfigs() const egl::ConfigSet DisplayVk::generateConfigs()
{ {
UNIMPLEMENTED(); UNIMPLEMENTED();
return egl::ConfigSet(); return egl::ConfigSet();
......
...@@ -28,7 +28,7 @@ class DisplayVk : public DisplayImpl ...@@ -28,7 +28,7 @@ class DisplayVk : public DisplayImpl
egl::Surface *readSurface, egl::Surface *readSurface,
gl::Context *context) override; gl::Context *context) override;
egl::ConfigSet generateConfigs() const override; egl::ConfigSet generateConfigs() override;
bool isDeviceLost() const override; bool isDeviceLost() const override;
bool testDeviceLost() override; bool testDeviceLost() override;
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment