diff options
author | Robert Phillips <robertphillips@google.com> | 2017-04-24 10:57:28 -0400 |
---|---|---|
committer | Skia Commit-Bot <skia-commit-bot@chromium.org> | 2017-04-25 11:44:20 +0000 |
commit | dd3b3f41829d32d7eaf3eb4903570d49c2ba9ff8 (patch) | |
tree | 10b4a5919c2a7565510c7f6c4e2cbb311bb2822a /tests | |
parent | c1889823de68ffd2ef08b5c1969d41c98034ec6a (diff) |
Rm makeRenderTargetContext in favor of deferred version (take 3)
This is a reland of: https://skia-review.googlesource.com/c/13001/ (Rm makeRenderTargetContext in favor of deferred version)
TBR=bsalomon@google.com
Change-Id: If81f4d9fb889c091cd37ffde133d906fb3e37773
Reviewed-on: https://skia-review.googlesource.com/14027
Commit-Queue: Robert Phillips <robertphillips@google.com>
Reviewed-by: Greg Daniel <egdaniel@google.com>
Diffstat (limited to 'tests')
-rw-r--r-- | tests/ClearTest.cpp | 4 | ||||
-rw-r--r-- | tests/GLProgramsTest.cpp | 4 | ||||
-rw-r--r-- | tests/GpuSampleLocationsTest.cpp | 4 | ||||
-rw-r--r-- | tests/ImageStorageTest.cpp | 4 | ||||
-rw-r--r-- | tests/IntTextureTest.cpp | 2 | ||||
-rw-r--r-- | tests/PreFlushCallbackTest.cpp | 6 | ||||
-rw-r--r-- | tests/PrimitiveProcessorTest.cpp | 2 | ||||
-rw-r--r-- | tests/ProcessorTest.cpp | 4 | ||||
-rw-r--r-- | tests/RectangleTextureTest.cpp | 6 | ||||
-rw-r--r-- | tests/RenderTargetContextTest.cpp | 25 | ||||
-rw-r--r-- | tests/SRGBMipMapTest.cpp | 4 | ||||
-rw-r--r-- | tests/TessellatingPathRendererTests.cpp | 3 |
12 files changed, 29 insertions, 39 deletions
diff --git a/tests/ClearTest.cpp b/tests/ClearTest.cpp index 843c4c371c..45eb2afa2d 100644 --- a/tests/ClearTest.cpp +++ b/tests/ClearTest.cpp @@ -57,8 +57,8 @@ static bool reset_rtc(sk_sp<GrRenderTargetContext>* rtc, GrContext* context, int } context->freeGpuResources(); - *rtc = context->makeRenderTargetContext(SkBackingFit::kExact, w, h, kRGBA_8888_GrPixelConfig, - nullptr); + *rtc = context->makeDeferredRenderTargetContext(SkBackingFit::kExact, w, h, + kRGBA_8888_GrPixelConfig, nullptr); SkASSERT((*rtc)->accessRenderTarget()->uniqueID() != oldID); diff --git a/tests/GLProgramsTest.cpp b/tests/GLProgramsTest.cpp index 5400d92bc6..25cd3125b9 100644 --- a/tests/GLProgramsTest.cpp +++ b/tests/GLProgramsTest.cpp @@ -148,7 +148,7 @@ static sk_sp<GrRenderTargetContext> random_render_target_context(GrContext* cont : kBottomLeft_GrSurfaceOrigin; int sampleCnt = random->nextBool() ? SkTMin(4, caps->maxSampleCount()) : 0; - sk_sp<GrRenderTargetContext> renderTargetContext(context->makeRenderTargetContext( + sk_sp<GrRenderTargetContext> renderTargetContext(context->makeDeferredRenderTargetContext( SkBackingFit::kExact, kRenderTargetWidth, kRenderTargetHeight, @@ -345,7 +345,7 @@ bool GrDrawingManager::ProgramUnitTest(GrContext* context, int maxStages) { drawingManager->flush(nullptr); // Validate that GrFPs work correctly without an input. - sk_sp<GrRenderTargetContext> renderTargetContext(context->makeRenderTargetContext( + sk_sp<GrRenderTargetContext> renderTargetContext(context->makeDeferredRenderTargetContext( SkBackingFit::kExact, kRenderTargetWidth, kRenderTargetHeight, diff --git a/tests/GpuSampleLocationsTest.cpp b/tests/GpuSampleLocationsTest.cpp index 26d7e70148..6e685463b7 100644 --- a/tests/GpuSampleLocationsTest.cpp +++ b/tests/GpuSampleLocationsTest.cpp @@ -122,11 +122,11 @@ void test_sampleLocations(skiatest::Reporter* reporter, TestSampleLocationsInter for (int i = 0; i < numTestPatterns; ++i) { int numSamples = (int)kTestPatterns[i].size(); GrAlwaysAssert(numSamples > 1 && SkIsPow2(numSamples)); - bottomUps[i] = ctx->makeRenderTargetContextWithFallback( + bottomUps[i] = ctx->makeDeferredRenderTargetContextWithFallback( SkBackingFit::kExact, 100, 100, kRGBA_8888_GrPixelConfig, nullptr, rand.nextRangeU(1 + numSamples / 2, numSamples), kBottomLeft_GrSurfaceOrigin); - topDowns[i] = ctx->makeRenderTargetContextWithFallback( + topDowns[i] = ctx->makeDeferredRenderTargetContextWithFallback( SkBackingFit::kExact, 100, 100, kRGBA_8888_GrPixelConfig, nullptr, rand.nextRangeU(1 + numSamples / 2, numSamples), kTopLeft_GrSurfaceOrigin); diff --git a/tests/ImageStorageTest.cpp b/tests/ImageStorageTest.cpp index f3e3482b07..dc4d21294d 100644 --- a/tests/ImageStorageTest.cpp +++ b/tests/ImageStorageTest.cpp @@ -140,8 +140,8 @@ DEF_GPUTEST_FOR_RENDERING_CONTEXTS(ImageStorageLoad, reporter, ctxInfo) { SkBudgeted::kYes, test.fData.get(), 0); sk_sp<GrRenderTargetContext> rtContext = - context->makeRenderTargetContext(SkBackingFit::kExact, kS, kS, - kRGBA_8888_GrPixelConfig, nullptr); + context->makeDeferredRenderTargetContext(SkBackingFit::kExact, kS, kS, + kRGBA_8888_GrPixelConfig, nullptr); GrPaint paint; paint.setPorterDuffXPFactory(SkBlendMode::kSrc); paint.addColorFragmentProcessor(TestFP::Make(context->resourceProvider(), diff --git a/tests/IntTextureTest.cpp b/tests/IntTextureTest.cpp index daeb13e53d..72f05f9591 100644 --- a/tests/IntTextureTest.cpp +++ b/tests/IntTextureTest.cpp @@ -244,7 +244,7 @@ DEF_GPUTEST_FOR_RENDERING_CONTEXTS(IntTexture, reporter, ctxInfo) { kRGBA_8888_sint_GrPixelConfig, nullptr, testData.get(), 0); - sk_sp<GrRenderTargetContext> rtContext = context->makeRenderTargetContext( + sk_sp<GrRenderTargetContext> rtContext = context->makeDeferredRenderTargetContext( SkBackingFit::kExact, kS, kS, kRGBA_8888_GrPixelConfig, nullptr); struct { diff --git a/tests/PreFlushCallbackTest.cpp b/tests/PreFlushCallbackTest.cpp index 7be01fb709..da7f93b583 100644 --- a/tests/PreFlushCallbackTest.cpp +++ b/tests/PreFlushCallbackTest.cpp @@ -419,7 +419,8 @@ private: static sk_sp<GrTextureProxy> make_upstream_image(GrContext* context, AtlasObject* object, int start, sk_sp<GrTextureProxy> fakeAtlas) { - sk_sp<GrRenderTargetContext> rtc(context->makeRenderTargetContext(SkBackingFit::kApprox, + sk_sp<GrRenderTargetContext> rtc(context->makeDeferredRenderTargetContext( + SkBackingFit::kApprox, 3*kDrawnTileSize, kDrawnTileSize, kRGBA_8888_GrPixelConfig, @@ -554,7 +555,8 @@ DEF_GPUTEST_FOR_GL_RENDERING_CONTEXTS(PreFlushCallbackTest, reporter, ctxInfo) { static const int kFinalWidth = 6*kDrawnTileSize; static const int kFinalHeight = kDrawnTileSize; - sk_sp<GrRenderTargetContext> rtc(context->makeRenderTargetContext(SkBackingFit::kApprox, + sk_sp<GrRenderTargetContext> rtc(context->makeDeferredRenderTargetContext( + SkBackingFit::kApprox, kFinalWidth, kFinalHeight, kRGBA_8888_GrPixelConfig, diff --git a/tests/PrimitiveProcessorTest.cpp b/tests/PrimitiveProcessorTest.cpp index 4e4edb7e35..6afc3acec8 100644 --- a/tests/PrimitiveProcessorTest.cpp +++ b/tests/PrimitiveProcessorTest.cpp @@ -105,7 +105,7 @@ private: DEF_GPUTEST_FOR_ALL_CONTEXTS(VertexAttributeCount, reporter, ctxInfo) { GrContext* context = ctxInfo.grContext(); - sk_sp<GrRenderTargetContext> renderTargetContext(context->makeRenderTargetContext( + sk_sp<GrRenderTargetContext> renderTargetContext(context->makeDeferredRenderTargetContext( SkBackingFit::kApprox, 1, 1, kRGBA_8888_GrPixelConfig, nullptr)); diff --git a/tests/ProcessorTest.cpp b/tests/ProcessorTest.cpp index 3eb4e16f20..e5bbc92759 100644 --- a/tests/ProcessorTest.cpp +++ b/tests/ProcessorTest.cpp @@ -137,7 +137,7 @@ DEF_GPUTEST_FOR_ALL_CONTEXTS(ProcessorRefTest, reporter, ctxInfo) { desc.fHeight = 10; for (int parentCnt = 0; parentCnt < 2; parentCnt++) { - sk_sp<GrRenderTargetContext> renderTargetContext(context->makeRenderTargetContext( + sk_sp<GrRenderTargetContext> renderTargetContext(context->makeDeferredRenderTargetContext( SkBackingFit::kApprox, 1, 1, kRGBA_8888_GrPixelConfig, nullptr)); { bool texelBufferSupport = context->caps()->shaderCaps()->texelBufferSupport(); @@ -290,7 +290,7 @@ DEF_GPUTEST_FOR_GL_RENDERING_CONTEXTS(ProcessorOptimizationValidationTest, repor // hard-code that value here: SkRandom random(seed); - sk_sp<GrRenderTargetContext> rtc = context->makeRenderTargetContext( + sk_sp<GrRenderTargetContext> rtc = context->makeDeferredRenderTargetContext( SkBackingFit::kExact, 256, 256, kRGBA_8888_GrPixelConfig, nullptr); GrSurfaceDesc desc; desc.fWidth = 256; diff --git a/tests/RectangleTextureTest.cpp b/tests/RectangleTextureTest.cpp index 967d86c9c4..cdcd8d7ac6 100644 --- a/tests/RectangleTextureTest.cpp +++ b/tests/RectangleTextureTest.cpp @@ -22,9 +22,9 @@ static void test_basic_draw_as_src(skiatest::Reporter* reporter, GrContext* context, sk_sp<GrTextureProxy> rectProxy, uint32_t expectedPixelValues[]) { sk_sp<GrRenderTargetContext> rtContext( - context->makeRenderTargetContext(SkBackingFit::kExact, rectProxy->width(), - rectProxy->height(), rectProxy->config(), - nullptr)); + context->makeDeferredRenderTargetContext(SkBackingFit::kExact, rectProxy->width(), + rectProxy->height(), rectProxy->config(), + nullptr)); for (auto filter : {GrSamplerParams::kNone_FilterMode, GrSamplerParams::kBilerp_FilterMode, GrSamplerParams::kMipMap_FilterMode}) { diff --git a/tests/RenderTargetContextTest.cpp b/tests/RenderTargetContextTest.cpp index 760b258d07..dbb41851c6 100644 --- a/tests/RenderTargetContextTest.cpp +++ b/tests/RenderTargetContextTest.cpp @@ -20,17 +20,10 @@ static const int kSize = 64; -static sk_sp<GrRenderTargetContext> get_rtc(GrContext* ctx, bool wrapped) { - - if (wrapped) { - return ctx->makeRenderTargetContext(SkBackingFit::kExact, - kSize, kSize, - kRGBA_8888_GrPixelConfig, nullptr); - } else { - return ctx->makeDeferredRenderTargetContext(SkBackingFit::kExact, - kSize, kSize, - kRGBA_8888_GrPixelConfig, nullptr); - } +static sk_sp<GrRenderTargetContext> get_rtc(GrContext* ctx) { + return ctx->makeDeferredRenderTargetContext(SkBackingFit::kExact, + kSize, kSize, + kRGBA_8888_GrPixelConfig, nullptr); } static void check_is_wrapped_status(skiatest::Reporter* reporter, @@ -47,16 +40,10 @@ static void check_is_wrapped_status(skiatest::Reporter* reporter, DEF_GPUTEST_FOR_RENDERING_CONTEXTS(RenderTargetContextTest, reporter, ctxInfo) { GrContext* ctx = ctxInfo.grContext(); - // A wrapped rtCtx's textureProxy is also wrapped - { - sk_sp<GrRenderTargetContext> rtCtx(get_rtc(ctx, true)); - check_is_wrapped_status(reporter, rtCtx.get(), true); - } - // Calling instantiate on a GrRenderTargetContext's textureProxy also instantiates the // GrRenderTargetContext { - sk_sp<GrRenderTargetContext> rtCtx(get_rtc(ctx, false)); + sk_sp<GrRenderTargetContext> rtCtx(get_rtc(ctx)); check_is_wrapped_status(reporter, rtCtx.get(), false); @@ -71,7 +58,7 @@ DEF_GPUTEST_FOR_RENDERING_CONTEXTS(RenderTargetContextTest, reporter, ctxInfo) { // readPixels switches a deferred rtCtx to wrapped { - sk_sp<GrRenderTargetContext> rtCtx(get_rtc(ctx, false)); + sk_sp<GrRenderTargetContext> rtCtx(get_rtc(ctx)); check_is_wrapped_status(reporter, rtCtx.get(), false); diff --git a/tests/SRGBMipMapTest.cpp b/tests/SRGBMipMapTest.cpp index 54c68aeb23..33d99f2208 100644 --- a/tests/SRGBMipMapTest.cpp +++ b/tests/SRGBMipMapTest.cpp @@ -136,9 +136,9 @@ DEF_GPUTEST_FOR_GL_RENDERING_CONTEXTS(SRGBMipMaps, reporter, ctxInfo) { // Create two render target contexts (L32 and S32) sk_sp<SkColorSpace> srgbColorSpace = SkColorSpace::MakeSRGB(); - sk_sp<GrRenderTargetContext> l32RenderTargetContext = context->makeRenderTargetContext( + sk_sp<GrRenderTargetContext> l32RenderTargetContext = context->makeDeferredRenderTargetContext( SkBackingFit::kExact, rtS, rtS, kRGBA_8888_GrPixelConfig, nullptr); - sk_sp<GrRenderTargetContext> s32RenderTargetContext = context->makeRenderTargetContext( + sk_sp<GrRenderTargetContext> s32RenderTargetContext = context->makeDeferredRenderTargetContext( SkBackingFit::kExact, rtS, rtS, kSRGBA_8888_GrPixelConfig, std::move(srgbColorSpace)); SkRect rect = SkRect::MakeWH(SkIntToScalar(rtS), SkIntToScalar(rtS)); diff --git a/tests/TessellatingPathRendererTests.cpp b/tests/TessellatingPathRendererTests.cpp index 8579c4dcbd..162dd7ea80 100644 --- a/tests/TessellatingPathRendererTests.cpp +++ b/tests/TessellatingPathRendererTests.cpp @@ -303,7 +303,8 @@ static void test_path(GrContext* ctx, DEF_GPUTEST_FOR_ALL_CONTEXTS(TessellatingPathRendererTests, reporter, ctxInfo) { GrContext* ctx = ctxInfo.grContext(); - sk_sp<GrRenderTargetContext> rtc(ctx->makeRenderTargetContext(SkBackingFit::kApprox, + sk_sp<GrRenderTargetContext> rtc(ctx->makeDeferredRenderTargetContext( + SkBackingFit::kApprox, 800, 800, kRGBA_8888_GrPixelConfig, nullptr, |