Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(168)

Unified Diff: gpu/command_buffer/service/framebuffer_manager_unittest.cc

Issue 2448083003: Reland of gpu: Clarify sized texture format is available (https://codereview.chromium.org/211718300… (Closed)
Patch Set: diff to reland Created 4 years, 2 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « gpu/command_buffer/service/feature_info.cc ('k') | gpu/command_buffer/service/gles2_cmd_decoder.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: gpu/command_buffer/service/framebuffer_manager_unittest.cc
diff --git a/gpu/command_buffer/service/framebuffer_manager_unittest.cc b/gpu/command_buffer/service/framebuffer_manager_unittest.cc
index 1884200a53ac48986058a2f7cd5e7282bd625709..bb8e2fb2ca08d29a0ad7864115ea736c159ed324 100644
--- a/gpu/command_buffer/service/framebuffer_manager_unittest.cc
+++ b/gpu/command_buffer/service/framebuffer_manager_unittest.cc
@@ -135,14 +135,18 @@ class FramebufferInfoTestBase : public GpuServiceTest {
protected:
void SetUp() override {
- InitializeContext("2.0", "GL_EXT_framebuffer_object");
+ bool is_es3 = false;
+ if (context_type_ == CONTEXT_TYPE_WEBGL2 ||
+ context_type_ == CONTEXT_TYPE_OPENGLES3)
+ is_es3 = true;
+ InitializeContext(is_es3 ? "3.0" : "2.0", "GL_EXT_framebuffer_object");
}
void InitializeContext(const char* gl_version, const char* extensions) {
GpuServiceTest::SetUpWithGLVersion(gl_version, extensions);
TestHelper::SetupFeatureInfoInitExpectationsWithGLVersion(gl_.get(),
extensions, "", gl_version, context_type_);
- feature_info_->InitializeForTesting();
+ feature_info_->InitializeForTesting(context_type_);
decoder_.reset(new MockGLES2Decoder());
manager_.CreateFramebuffer(kClient1Id, kService1Id);
error_state_.reset(new ::testing::StrictMock<gles2::MockErrorState>());
@@ -490,7 +494,7 @@ TEST_F(FramebufferInfoTest, AttachTexture2D) {
const GLsizei kWidth3 = 75;
const GLsizei kHeight3 = 123;
const GLint kLevel3 = 0;
- const GLenum kFormat3 = GL_RGB565;
+ const GLenum kFormat3 = GL_RGBA;
const GLsizei kSamples3 = 0;
EXPECT_FALSE(framebuffer_->HasUnclearedAttachment(GL_COLOR_ATTACHMENT0));
EXPECT_FALSE(framebuffer_->HasUnclearedAttachment(GL_DEPTH_ATTACHMENT));
@@ -1278,11 +1282,10 @@ TEST_F(FramebufferInfoTest, DrawBufferMasks) {
framebuffer_->ValidateAndAdjustDrawBuffers(0x310u, 0x330u));
}
-class FramebufferInfoFloatTest : public FramebufferInfoTest {
+class FramebufferInfoFloatTest : public FramebufferInfoTestBase {
public:
FramebufferInfoFloatTest()
- : FramebufferInfoTest() {
- }
+ : FramebufferInfoTestBase(CONTEXT_TYPE_OPENGLES2) {}
~FramebufferInfoFloatTest() override {}
protected:
@@ -1547,18 +1550,6 @@ class FramebufferInfoES3Test : public FramebufferInfoTestBase {
void SetUp() override {
InitializeContext("OpenGL ES 3.0", "");
}
-
- void InitializeContext(const char* gl_version, const char* extensions) {
- GpuServiceTest::SetUpWithGLVersion(gl_version, extensions);
- TestHelper::SetupFeatureInfoInitExpectationsWithGLVersion(gl_.get(),
- extensions, "", gl_version, context_type_);
- feature_info_->InitializeForTesting(CONTEXT_TYPE_OPENGLES3);
- decoder_.reset(new MockGLES2Decoder());
- manager_.CreateFramebuffer(kClient1Id, kService1Id);
- error_state_.reset(new ::testing::StrictMock<gles2::MockErrorState>());
- framebuffer_ = manager_.GetFramebuffer(kClient1Id);
- ASSERT_TRUE(framebuffer_ != nullptr);
- }
};
TEST_F(FramebufferInfoES3Test, DifferentDimensions) {
« no previous file with comments | « gpu/command_buffer/service/feature_info.cc ('k') | gpu/command_buffer/service/gles2_cmd_decoder.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698