diff options
author | mtklein <mtklein@chromium.org> | 2015-01-23 05:47:55 -0800 |
---|---|---|
committer | Commit bot <commit-bot@chromium.org> | 2015-01-23 05:47:55 -0800 |
commit | 2f6bb6b85246314dd016ecf68bbfd9b48319117a (patch) | |
tree | 08106dfa9d5a434bfb8965039f35ea0ece91a70d /tests/FloatingPointTextureTest.cpp | |
parent | 7983bf914a6df040befe2aafb3657c469b678257 (diff) |
minor readability / refactors
I have been hacking at this test trying to understand why the N5 is
still sometimes crashy (it seems, less but not zero now). No luck so far.
But, while I've been reading and hacking at it, I think I've made a
few small improvements, mostly to readability.
BUG=skia:
Review URL: https://codereview.chromium.org/870803002
Diffstat (limited to 'tests/FloatingPointTextureTest.cpp')
-rw-r--r-- | tests/FloatingPointTextureTest.cpp | 64 |
1 files changed, 23 insertions, 41 deletions
diff --git a/tests/FloatingPointTextureTest.cpp b/tests/FloatingPointTextureTest.cpp index 15f2e2eafa..3bc87f6102 100644 --- a/tests/FloatingPointTextureTest.cpp +++ b/tests/FloatingPointTextureTest.cpp @@ -22,7 +22,7 @@ #include "SkHalf.h" static const int DEV_W = 100, DEV_H = 100; -static const int FP_CONTROL_ARRAY_SIZE = DEV_W * DEV_H * 4; +static const int FP_CONTROL_ARRAY_SIZE = DEV_W * DEV_H * 4/*RGBA*/; static const float kMaxIntegerRepresentableInSPFloatingPoint = 16777216; // 2 ^ 24 static const SkIRect DEV_RECT = SkIRect::MakeWH(DEV_W, DEV_H); @@ -33,55 +33,46 @@ DEF_GPUTEST(FloatingPointTextureTest, reporter, factory) { readBuffer.setCount(FP_CONTROL_ARRAY_SIZE); for (int i = 0; i < FP_CONTROL_ARRAY_SIZE; i += 4) { - controlPixelData[i] = FLT_MIN; + controlPixelData[i + 0] = FLT_MIN; controlPixelData[i + 1] = FLT_MAX; controlPixelData[i + 2] = FLT_EPSILON; controlPixelData[i + 3] = kMaxIntegerRepresentableInSPFloatingPoint; } for (int origin = 0; origin < 2; ++origin) { - int glCtxTypeCnt = 1; - glCtxTypeCnt = GrContextFactory::kGLContextTypeCnt; - for (int glCtxType = 0; glCtxType < glCtxTypeCnt; ++glCtxType) { + for (int glCtxType = 0; glCtxType < GrContextFactory::kGLContextTypeCnt; ++glCtxType) { GrSurfaceDesc desc; - desc.fFlags = kRenderTarget_GrSurfaceFlag; - desc.fWidth = DEV_W; + desc.fFlags = kRenderTarget_GrSurfaceFlag; + desc.fWidth = DEV_W; desc.fHeight = DEV_H; desc.fConfig = kRGBA_float_GrPixelConfig; desc.fOrigin = 0 == origin ? kTopLeft_GrSurfaceOrigin : kBottomLeft_GrSurfaceOrigin; - GrContext* context = NULL; GrContextFactory::GLContextType type = - static_cast<GrContextFactory::GLContextType>(glCtxType); + static_cast<GrContextFactory::GLContextType>(glCtxType); if (!GrContextFactory::IsRenderingGLContext(type)) { continue; } - context = factory->get(type); + GrContext* context = factory->get(type); if (NULL == context){ continue; } - SkAutoTUnref<GrTexture> fpTexture(context->createUncachedTexture(desc, - NULL, - 0)); - + SkAutoTUnref<GrTexture> fpTexture( + context->createUncachedTexture(desc, controlPixelData.begin(), 0)); // Floating point textures are NOT supported everywhere if (NULL == fpTexture) { continue; } - - // write square - fpTexture->writePixels(0, 0, DEV_W, DEV_H, desc.fConfig, controlPixelData.begin(), 0); fpTexture->readPixels(0, 0, DEV_W, DEV_H, desc.fConfig, readBuffer.begin(), 0); - for (int j = 0; j < FP_CONTROL_ARRAY_SIZE; ++j) { - REPORTER_ASSERT(reporter, readBuffer[j] == controlPixelData[j]); - } + REPORTER_ASSERT(reporter, + 0 == memcmp(readBuffer.begin(), controlPixelData.begin(), readBuffer.bytes())); } } } -static const int HALF_CONTROL_ARRAY_SIZE = DEV_W * DEV_H; +static const int HALF_CONTROL_ARRAY_SIZE = DEV_W * DEV_H * 1 /*alpha-only*/; DEF_GPUTEST(HalfFloatTextureTest, reporter, factory) { SkTDArray<SkHalf> controlPixelData, readBuffer; @@ -89,50 +80,41 @@ DEF_GPUTEST(HalfFloatTextureTest, reporter, factory) { readBuffer.setCount(HALF_CONTROL_ARRAY_SIZE); for (int i = 0; i < HALF_CONTROL_ARRAY_SIZE; i += 4) { - controlPixelData[i] = SK_HalfMin; + controlPixelData[i + 0] = SK_HalfMin; controlPixelData[i + 1] = SK_HalfMax; controlPixelData[i + 2] = SK_HalfEpsilon; controlPixelData[i + 3] = 0x6800; // 2^11 } for (int origin = 0; origin < 2; ++origin) { - int glCtxTypeCnt = 1; - glCtxTypeCnt = GrContextFactory::kGLContextTypeCnt; - for (int glCtxType = 0; glCtxType < glCtxTypeCnt; ++glCtxType) { + for (int glCtxType = 0; glCtxType < GrContextFactory::kGLContextTypeCnt; ++glCtxType) { GrSurfaceDesc desc; - desc.fFlags = kRenderTarget_GrSurfaceFlag; - desc.fWidth = DEV_W; + desc.fFlags = kRenderTarget_GrSurfaceFlag; + desc.fWidth = DEV_W; desc.fHeight = DEV_H; desc.fConfig = kAlpha_half_GrPixelConfig; desc.fOrigin = 0 == origin ? - kTopLeft_GrSurfaceOrigin : kBottomLeft_GrSurfaceOrigin; + kTopLeft_GrSurfaceOrigin : kBottomLeft_GrSurfaceOrigin; - GrContext* context = NULL; GrContextFactory::GLContextType type = - static_cast<GrContextFactory::GLContextType>(glCtxType); + static_cast<GrContextFactory::GLContextType>(glCtxType); if (!GrContextFactory::IsRenderingGLContext(type)) { continue; } - context = factory->get(type); + GrContext* context = factory->get(type); if (NULL == context){ continue; } - SkAutoTUnref<GrTexture> fpTexture(context->createUncachedTexture(desc, - NULL, - 0)); - + SkAutoTUnref<GrTexture> fpTexture( + context->createUncachedTexture(desc, controlPixelData.begin(), 0)); // 16-bit floating point textures are NOT supported everywhere if (NULL == fpTexture) { continue; } - - // write square - fpTexture->writePixels(0, 0, DEV_W, DEV_H, desc.fConfig, controlPixelData.begin(), 0); fpTexture->readPixels(0, 0, DEV_W, DEV_H, desc.fConfig, readBuffer.begin(), 0); - for (int j = 0; j < HALF_CONTROL_ARRAY_SIZE; ++j) { - REPORTER_ASSERT(reporter, readBuffer[j] == controlPixelData[j]); - } + REPORTER_ASSERT(reporter, + 0 == memcmp(readBuffer.begin(), controlPixelData.begin(), readBuffer.bytes())); } } } |