Commit c74ff571 by Jamie Madill Committed by Commit Bot

Consolidate TypedResourceManager::isXGenerated.

BUG=angleproject:1458 Change-Id: Ib5c561610ab6dbcf0f65d915dfe8a8ca21ebe7f3 Reviewed-on: https://chromium-review.googlesource.com/544683Reviewed-by: 's avatarJamie Madill <jmadill@chromium.org> Commit-Queue: Jamie Madill <jmadill@chromium.org>
parent 84a6c5b2
...@@ -724,22 +724,22 @@ Shader *ValidationContext::getShader(GLuint handle) const ...@@ -724,22 +724,22 @@ Shader *ValidationContext::getShader(GLuint handle) const
bool ValidationContext::isTextureGenerated(GLuint texture) const bool ValidationContext::isTextureGenerated(GLuint texture) const
{ {
return mState.mTextures->isTextureGenerated(texture); return mState.mTextures->isHandleGenerated(texture);
} }
bool ValidationContext::isBufferGenerated(GLuint buffer) const bool ValidationContext::isBufferGenerated(GLuint buffer) const
{ {
return mState.mBuffers->isBufferGenerated(buffer); return mState.mBuffers->isHandleGenerated(buffer);
} }
bool ValidationContext::isRenderbufferGenerated(GLuint renderbuffer) const bool ValidationContext::isRenderbufferGenerated(GLuint renderbuffer) const
{ {
return mState.mRenderbuffers->isRenderbufferGenerated(renderbuffer); return mState.mRenderbuffers->isHandleGenerated(renderbuffer);
} }
bool ValidationContext::isFramebufferGenerated(GLuint framebuffer) const bool ValidationContext::isFramebufferGenerated(GLuint framebuffer) const
{ {
return mState.mFramebuffers->isFramebufferGenerated(framebuffer); return mState.mFramebuffers->isHandleGenerated(framebuffer);
} }
bool ValidationContext::usingDisplayTextureShareGroup() const bool ValidationContext::usingDisplayTextureShareGroup() const
......
...@@ -182,11 +182,6 @@ Buffer *BufferManager::getBuffer(GLuint handle) const ...@@ -182,11 +182,6 @@ Buffer *BufferManager::getBuffer(GLuint handle) const
return GetObject(mObjectMap, handle); return GetObject(mObjectMap, handle);
} }
bool BufferManager::isBufferGenerated(GLuint buffer) const
{
return buffer == 0 || mObjectMap.find(buffer) != mObjectMap.end();
}
// ShaderProgramManager Implementation. // ShaderProgramManager Implementation.
ShaderProgramManager::~ShaderProgramManager() ShaderProgramManager::~ShaderProgramManager()
...@@ -297,11 +292,6 @@ Texture *TextureManager::getTexture(GLuint handle) const ...@@ -297,11 +292,6 @@ Texture *TextureManager::getTexture(GLuint handle) const
return GetObject(mObjectMap, handle); return GetObject(mObjectMap, handle);
} }
bool TextureManager::isTextureGenerated(GLuint texture) const
{
return texture == 0 || mObjectMap.find(texture) != mObjectMap.end();
}
void TextureManager::invalidateTextureComplenessCache() void TextureManager::invalidateTextureComplenessCache()
{ {
for (auto &texture : mObjectMap) for (auto &texture : mObjectMap)
...@@ -339,11 +329,6 @@ Renderbuffer *RenderbufferManager::getRenderbuffer(GLuint handle) ...@@ -339,11 +329,6 @@ Renderbuffer *RenderbufferManager::getRenderbuffer(GLuint handle)
return GetObject(mObjectMap, handle); return GetObject(mObjectMap, handle);
} }
bool RenderbufferManager::isRenderbufferGenerated(GLuint renderbuffer) const
{
return renderbuffer == 0 || mObjectMap.find(renderbuffer) != mObjectMap.end();
}
// SamplerManager Implementation. // SamplerManager Implementation.
// static // static
...@@ -501,12 +486,6 @@ void FramebufferManager::setDefaultFramebuffer(Framebuffer *framebuffer) ...@@ -501,12 +486,6 @@ void FramebufferManager::setDefaultFramebuffer(Framebuffer *framebuffer)
mObjectMap[0] = framebuffer; mObjectMap[0] = framebuffer;
} }
bool FramebufferManager::isFramebufferGenerated(GLuint framebuffer)
{
ASSERT(mObjectMap.find(0) != mObjectMap.end());
return mObjectMap.find(framebuffer) != mObjectMap.end();
}
void FramebufferManager::invalidateFramebufferComplenessCache() void FramebufferManager::invalidateFramebufferComplenessCache()
{ {
for (auto &framebuffer : mObjectMap) for (auto &framebuffer : mObjectMap)
......
...@@ -63,6 +63,11 @@ class TypedResourceManager : public ResourceManagerBase<HandleAllocatorType> ...@@ -63,6 +63,11 @@ class TypedResourceManager : public ResourceManagerBase<HandleAllocatorType>
TypedResourceManager() {} TypedResourceManager() {}
void deleteObject(const Context *context, GLuint handle); void deleteObject(const Context *context, GLuint handle);
bool isHandleGenerated(GLuint handle) const
{
// Zero is always assumed to have been generated implicitly.
return handle == 0 || mObjectMap.find(handle) != mObjectMap.end();
}
protected: protected:
~TypedResourceManager() override; ~TypedResourceManager() override;
...@@ -102,7 +107,6 @@ class BufferManager : public TypedResourceManager<Buffer, HandleAllocator, Buffe ...@@ -102,7 +107,6 @@ class BufferManager : public TypedResourceManager<Buffer, HandleAllocator, Buffe
public: public:
GLuint createBuffer(); GLuint createBuffer();
Buffer *getBuffer(GLuint handle) const; Buffer *getBuffer(GLuint handle) const;
bool isBufferGenerated(GLuint buffer) const;
Buffer *checkBufferAllocation(rx::GLImplFactory *factory, GLuint handle) Buffer *checkBufferAllocation(rx::GLImplFactory *factory, GLuint handle)
{ {
...@@ -148,7 +152,6 @@ class TextureManager : public TypedResourceManager<Texture, HandleAllocator, Tex ...@@ -148,7 +152,6 @@ class TextureManager : public TypedResourceManager<Texture, HandleAllocator, Tex
public: public:
GLuint createTexture(); GLuint createTexture();
Texture *getTexture(GLuint handle) const; Texture *getTexture(GLuint handle) const;
bool isTextureGenerated(GLuint texture) const;
void invalidateTextureComplenessCache(); void invalidateTextureComplenessCache();
...@@ -170,7 +173,6 @@ class RenderbufferManager ...@@ -170,7 +173,6 @@ class RenderbufferManager
public: public:
GLuint createRenderbuffer(); GLuint createRenderbuffer();
Renderbuffer *getRenderbuffer(GLuint handle); Renderbuffer *getRenderbuffer(GLuint handle);
bool isRenderbufferGenerated(GLuint renderbuffer) const;
Renderbuffer *checkRenderbufferAllocation(rx::GLImplFactory *factory, GLuint handle) Renderbuffer *checkRenderbufferAllocation(rx::GLImplFactory *factory, GLuint handle)
{ {
...@@ -238,7 +240,6 @@ class FramebufferManager ...@@ -238,7 +240,6 @@ class FramebufferManager
GLuint createFramebuffer(); GLuint createFramebuffer();
Framebuffer *getFramebuffer(GLuint handle) const; Framebuffer *getFramebuffer(GLuint handle) const;
void setDefaultFramebuffer(Framebuffer *framebuffer); void setDefaultFramebuffer(Framebuffer *framebuffer);
bool isFramebufferGenerated(GLuint framebuffer);
void invalidateFramebufferComplenessCache(); void invalidateFramebufferComplenessCache();
......
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