Revert of Remove rt flags that are not necessary (patchset #1 id:1 of https://coderev...
authorbsalomon <bsalomon@google.com>
Thu, 16 Jul 2015 17:00:28 +0000 (10:00 -0700)
committerCommit bot <commit-bot@chromium.org>
Thu, 16 Jul 2015 17:00:28 +0000 (10:00 -0700)
Reason for revert:
breaks tests, e.g.
https://build.chromium.org/p/client.skia.android/builders/Test-Android-GCC-Nexus9-GPU-TegraK1-Arm64-Release/builds/752/steps/dm/logs/stdio

Original issue's description:
> Remove rt flags that are not necessary
>
> Committed: https://skia.googlesource.com/skia/+/b6b0a6ef179f4143c94ba7e0e1be6e7ce4b0a23a

TBR=joshualitt@google.com
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true

Review URL: https://codereview.chromium.org/1234503004

tests/FloatingPointTextureTest.cpp
tests/GLProgramsTest.cpp
tests/SurfaceTest.cpp

index d891b47..1adc744 100644 (file)
@@ -42,6 +42,7 @@ void runFPTest(skiatest::Reporter* reporter, GrContextFactory* factory,
     for (int origin = 0; origin < 2; ++origin) {
         for (int glCtxType = 0; glCtxType < GrContextFactory::kGLContextTypeCnt; ++glCtxType) {
             GrSurfaceDesc desc;
+            desc.fFlags = kRenderTarget_GrSurfaceFlag;
             desc.fWidth = DEV_W;
             desc.fHeight = DEV_H;
             desc.fConfig = config;
index 019ab38..d21ae5d 100644 (file)
@@ -200,6 +200,7 @@ static void set_random_stencil(GrPipelineBuilder* pipelineBuilder, SkRandom* ran
 bool GrDrawTarget::programUnitTest(GrContext* context, int maxStages) {
     // setup dummy textures
     GrSurfaceDesc dummyDesc;
+    dummyDesc.fFlags = kRenderTarget_GrSurfaceFlag;
     dummyDesc.fConfig = kSkia8888_GrPixelConfig;
     dummyDesc.fWidth = 34;
     dummyDesc.fHeight = 18;
index a78af9b..b1e233f 100644 (file)
@@ -952,6 +952,7 @@ DEF_GPUTEST(SkImage_NewFromTexture, reporter, factory) {
     sk_memset32(storage, expected0, w * h);
     
     GrSurfaceDesc desc;
+    desc.fFlags = kRenderTarget_GrSurfaceFlag;  // needs to be a rendertarget for readpixels();
     desc.fOrigin = kDefault_GrSurfaceOrigin;
     desc.fWidth = w;
     desc.fHeight = h;