Commit baa25fd7 by Jamie Madill Committed by Commit Bot

Framebuffer: Remove ext check for depth/stencil textures.

The absence of the extension should not prevent depth/stencil textures in FBOs in ES3, since they are core. In fact it shouldn't be possible to make depth/stencil attachments at all without support, so the check shouldn't be necessary. BUG=angleproject:1828 Change-Id: Ibec1e95c78b79cec9654f06ba11beec447017f1c Reviewed-on: https://chromium-review.googlesource.com/438604 Commit-Queue: Jamie Madill <jmadill@chromium.org> Reviewed-by: 's avatarYuly Novikov <ynovikov@chromium.org>
parent 5f56ddb9
...@@ -635,12 +635,6 @@ GLenum Framebuffer::checkStatusImpl(const ContextState &state) ...@@ -635,12 +635,6 @@ GLenum Framebuffer::checkStatusImpl(const ContextState &state)
const TextureCaps &formatCaps = state.getTextureCap(format.asSized()); const TextureCaps &formatCaps = state.getTextureCap(format.asSized());
if (depthAttachment.type() == GL_TEXTURE) if (depthAttachment.type() == GL_TEXTURE)
{ {
// depth texture attachments require OES/ANGLE_depth_texture
if (!state.getExtensions().depthTextures)
{
return GL_FRAMEBUFFER_INCOMPLETE_ATTACHMENT;
}
if (!formatCaps.renderable) if (!formatCaps.renderable)
{ {
return GL_FRAMEBUFFER_INCOMPLETE_ATTACHMENT; return GL_FRAMEBUFFER_INCOMPLETE_ATTACHMENT;
...@@ -693,13 +687,6 @@ GLenum Framebuffer::checkStatusImpl(const ContextState &state) ...@@ -693,13 +687,6 @@ GLenum Framebuffer::checkStatusImpl(const ContextState &state)
const TextureCaps &formatCaps = state.getTextureCap(format.asSized()); const TextureCaps &formatCaps = state.getTextureCap(format.asSized());
if (stencilAttachment.type() == GL_TEXTURE) if (stencilAttachment.type() == GL_TEXTURE)
{ {
// texture stencil attachments come along as part
// of OES_packed_depth_stencil + OES/ANGLE_depth_texture
if (!state.getExtensions().depthTextures)
{
return GL_FRAMEBUFFER_INCOMPLETE_ATTACHMENT;
}
if (!formatCaps.renderable) if (!formatCaps.renderable)
{ {
return GL_FRAMEBUFFER_INCOMPLETE_ATTACHMENT; return GL_FRAMEBUFFER_INCOMPLETE_ATTACHMENT;
......
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