ADHOC/replicant-frameworks_native
3
0
Fork 0
Code Issues 6 Pull Requests Releases Wiki Activity

libgui_test: fix a couple minor test issues

Browse Source 
This change fixes how the MultiTextureConsumerTest.EGLImageTargetWorks checks
for pixels.  It removes the call to eglSwapBuffers so that the test does not
rely on EGL swap-preserve behavior, and it makes the test use checkPixel()
rather than doing glReadPixels itself.

Bug: 8349336
Change-Id: I0f446a0083eebd07af6fd208762878b4e367725a
This commit is contained in:
Jamie Gennis 2013-04-23 15:00:45 -07:00
parent e692ab9a6b
commit ea2d942dda
1 changed files with 2 additions and 9 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

11
libs/gui/tests/SurfaceTexture_test.cpp
View File

@ -463,19 +463,12 @@ TEST_F(MultiTextureConsumerTest, EGLImageTargetWorks) {
glVertexPointer(2, GL_FLOAT, 0, vertices);
glDrawArrays(GL_TRIANGLE_FAN, 0, 4);
ASSERT_EQ( glGetError(), GL_NO_ERROR );
ASSERT_EQ(GLenum(GL_NO_ERROR), glGetError());
}
ASSERT_TRUE( eglSwapBuffers(mEglDisplay, mEglSurface) );
uint32_t* pixels = new uint32_t[8*16*16];
glReadPixels(0, 0, 8*16, 16, GL_RGBA, GL_UNSIGNED_BYTE, pixels);
for (int i=0 ; i<8 ; i++) {
uint32_t p = pixels[i*16 + 8 + 8*(8*16)]; // center of each square
EXPECT_EQ(p, (i&7) * 0x10101010);
EXPECT_TRUE(checkPixel(i*16 + 8, 8, i*16, i*16, i*16, i*16, 0));
}
delete [] pixels;
}