diff options
author | Chris Dalton <csmartdalton@google.com> | 2018-01-29 14:32:43 -0700 |
---|---|---|
committer | Skia Commit-Bot <skia-commit-bot@chromium.org> | 2018-01-29 23:37:51 +0000 |
commit | 7df27465c4ecc8a4a0cdd95e9785c342903c2de9 (patch) | |
tree | 84f84cffe041645533e897e720fab722881202ba /tests | |
parent | 180f0c183a02cffc165d3bad4d4e4dd121752418 (diff) |
Drop support for unused MSAA extensions
Bug: skia:
Change-Id: I113b80e3f991f195155148625ceb29242ea82776
Reviewed-on: https://skia-review.googlesource.com/101403
Reviewed-by: Brian Salomon <bsalomon@google.com>
Commit-Queue: Chris Dalton <csmartdalton@google.com>
Diffstat (limited to 'tests')
-rw-r--r-- | tests/GpuSampleLocationsTest.cpp | 199 | ||||
-rw-r--r-- | tests/GrMeshTest.cpp | 2 | ||||
-rw-r--r-- | tests/GrPipelineDynamicStateTest.cpp | 2 | ||||
-rw-r--r-- | tests/PrimitiveProcessorTest.cpp | 2 |
4 files changed, 3 insertions, 202 deletions
diff --git a/tests/GpuSampleLocationsTest.cpp b/tests/GpuSampleLocationsTest.cpp deleted file mode 100644 index de15e03389..0000000000 --- a/tests/GpuSampleLocationsTest.cpp +++ /dev/null @@ -1,199 +0,0 @@ -/* - * Copyright 2016 Google Inc. - * - * Use of this source code is governed by a BSD-style license that can be - * found in the LICENSE file. - */ - -#include "SkTypes.h" -#include "SkPoint.h" -#include "Test.h" -#include <vector> - -#if SK_SUPPORT_GPU - -#include "GrAppliedClip.h" -#include "GrRenderTargetContext.h" -#include "GrRenderTargetPriv.h" -#include "GrTypesPriv.h" -#include "gl/GrGLGpu.h" -#include "gl/debug/DebugGLTestContext.h" - -typedef std::vector<SkPoint> SamplePattern; - -static const SamplePattern kTestPatterns[] = { - SamplePattern{ // Intel on mac, msaa8, offscreen. - {0.562500, 0.312500}, - {0.437500, 0.687500}, - {0.812500, 0.562500}, - {0.312500, 0.187500}, - {0.187500, 0.812500}, - {0.062500, 0.437500}, - {0.687500, 0.937500}, - {0.937500, 0.062500} - }, - - SamplePattern{ // Intel on mac, msaa8, on-screen. - {0.562500, 0.687500}, - {0.437500, 0.312500}, - {0.812500, 0.437500}, - {0.312500, 0.812500}, - {0.187500, 0.187500}, - {0.062500, 0.562500}, - {0.687500, 0.062500}, - {0.937500, 0.937500} - }, - - SamplePattern{ // NVIDIA, msaa16. - {0.062500, 0.000000}, - {0.250000, 0.125000}, - {0.187500, 0.375000}, - {0.437500, 0.312500}, - {0.500000, 0.062500}, - {0.687500, 0.187500}, - {0.750000, 0.437500}, - {0.937500, 0.250000}, - {0.000000, 0.500000}, - {0.312500, 0.625000}, - {0.125000, 0.750000}, - {0.375000, 0.875000}, - {0.562500, 0.562500}, - {0.812500, 0.687500}, - {0.625000, 0.812500}, - {0.875000, 0.937500} - }, - - SamplePattern{ // NVIDIA, mixed samples, 16:1. - {0.250000, 0.125000}, - {0.625000, 0.812500}, - {0.500000, 0.062500}, - {0.812500, 0.687500}, - {0.187500, 0.375000}, - {0.875000, 0.937500}, - {0.125000, 0.750000}, - {0.750000, 0.437500}, - {0.937500, 0.250000}, - {0.312500, 0.625000}, - {0.437500, 0.312500}, - {0.000000, 0.500000}, - {0.375000, 0.875000}, - {0.687500, 0.187500}, - {0.062500, 0.000000}, - {0.562500, 0.562500} - } -}; -constexpr int numTestPatterns = SK_ARRAY_COUNT(kTestPatterns); - -class TestSampleLocationsInterface : public SkNoncopyable { -public: - virtual void overrideSamplePattern(const SamplePattern&) = 0; - virtual ~TestSampleLocationsInterface() {} -}; - -void assert_equal(skiatest::Reporter* reporter, const SamplePattern& pattern, - const GrGpu::MultisampleSpecs& specs, bool flipY) { - GrAlwaysAssert(specs.fSampleLocations); - if ((int)pattern.size() != specs.fEffectiveSampleCnt) { - REPORT_FAILURE(reporter, "", SkString("Sample pattern has wrong number of samples.")); - return; - } - for (int i = 0; i < specs.fEffectiveSampleCnt; ++i) { - SkPoint expectedLocation = specs.fSampleLocations[i]; - if (flipY) { - expectedLocation.fY = 1 - expectedLocation.fY; - } - if (pattern[i] != expectedLocation) { - REPORT_FAILURE(reporter, "", SkString("Sample pattern has wrong sample location.")); - return; - } - } -} - -void test_sampleLocations(skiatest::Reporter* reporter, TestSampleLocationsInterface* testInterface, - GrContext* ctx) { - SkRandom rand; - sk_sp<GrRenderTargetContext> bottomUps[numTestPatterns]; - sk_sp<GrRenderTargetContext> topDowns[numTestPatterns]; - for (int i = 0; i < numTestPatterns; ++i) { - int numSamples = (int)kTestPatterns[i].size(); - GrAlwaysAssert(numSamples > 1 && SkIsPow2(numSamples)); - bottomUps[i] = ctx->makeDeferredRenderTargetContext( - SkBackingFit::kExact, 100, 100, kRGBA_8888_GrPixelConfig, nullptr, - rand.nextRangeU(1 + numSamples / 2, numSamples), GrMipMapped::kNo, - kBottomLeft_GrSurfaceOrigin); - topDowns[i] = ctx->makeDeferredRenderTargetContext( - SkBackingFit::kExact, 100, 100, kRGBA_8888_GrPixelConfig, nullptr, - rand.nextRangeU(1 + numSamples / 2, numSamples), GrMipMapped::kNo, - kTopLeft_GrSurfaceOrigin); - } - - // Ensure all sample locations get queried and/or cached properly. - for (int repeat = 0; repeat < 2; ++repeat) { - for (int i = 0; i < numTestPatterns; ++i) { - testInterface->overrideSamplePattern(kTestPatterns[i]); - for (GrRenderTargetContext* rtc : {bottomUps[i].get(), topDowns[i].get()}) { - GrPipeline dummyPipeline(rtc->asRenderTargetProxy(), - GrPipeline::ScissorState::kDisabled, - SkBlendMode::kSrcOver); - GrRenderTarget* rt = rtc->accessRenderTarget(); - assert_equal(reporter, kTestPatterns[i], - rt->renderTargetPriv().getMultisampleSpecs(dummyPipeline), - kBottomLeft_GrSurfaceOrigin == rtc->asSurfaceProxy()->origin()); - } - } - } - -} - -//////////////////////////////////////////////////////////////////////////////////////////////////// - -class GLTestSampleLocationsInterface : public TestSampleLocationsInterface, public GrGLInterface { -public: - GLTestSampleLocationsInterface() : fTestContext(sk_gpu_test::CreateDebugGLTestContext()) { - fStandard = fTestContext->gl()->fStandard; - fExtensions = fTestContext->gl()->fExtensions; - fFunctions = fTestContext->gl()->fFunctions; - - fFunctions.fGetIntegerv = [&](GrGLenum pname, GrGLint* params) { - GrAlwaysAssert(GR_GL_EFFECTIVE_RASTER_SAMPLES != pname); - if (GR_GL_SAMPLES == pname) { - GrAlwaysAssert(!fSamplePattern.empty()); - *params = (int)fSamplePattern.size(); - } else { - fTestContext->gl()->fFunctions.fGetIntegerv(pname, params); - } - }; - - fFunctions.fGetMultisamplefv = [&](GrGLenum pname, GrGLuint index, GrGLfloat* val) { - GrAlwaysAssert(GR_GL_SAMPLE_POSITION == pname); - val[0] = fSamplePattern[index].fX; - val[1] = fSamplePattern[index].fY; - }; - } - - operator GrBackendContext() { - return reinterpret_cast<GrBackendContext>(static_cast<GrGLInterface*>(this)); - } - - void overrideSamplePattern(const SamplePattern& newPattern) override { - fSamplePattern = newPattern; - } - -private: - std::unique_ptr<sk_gpu_test::GLTestContext> fTestContext; - SamplePattern fSamplePattern; -}; - -DEF_GPUTEST(GLSampleLocations, reporter, /* options */) { - auto testInterface = sk_make_sp<GLTestSampleLocationsInterface>(); - sk_sp<GrContext> ctx(GrContext::MakeGL(testInterface)); - - // This test relies on at least 2 samples. - int supportedSample = ctx->caps()->getSampleCount(2, kRGBA_8888_GrPixelConfig); - if (supportedSample < 2) { - return; - } - test_sampleLocations(reporter, testInterface.get(), ctx.get()); -} - -#endif diff --git a/tests/GrMeshTest.cpp b/tests/GrMeshTest.cpp index ccc9a73e8f..86437d5dfb 100644 --- a/tests/GrMeshTest.cpp +++ b/tests/GrMeshTest.cpp @@ -341,7 +341,7 @@ class GLSLMeshTestProcessor : public GrGLSLGeometryProcessor { } gpArgs->fPositionVar.set(kFloat2_GrSLType, "vertex"); - GrGLSLPPFragmentBuilder* f = args.fFragBuilder; + GrGLSLFragmentBuilder* f = args.fFragBuilder; f->codeAppendf("%s = half4(1);", args.fOutputCoverage); } }; diff --git a/tests/GrPipelineDynamicStateTest.cpp b/tests/GrPipelineDynamicStateTest.cpp index cd1ce1e053..00da84e1a9 100644 --- a/tests/GrPipelineDynamicStateTest.cpp +++ b/tests/GrPipelineDynamicStateTest.cpp @@ -93,7 +93,7 @@ class GLSLPipelineDynamicStateTestProcessor : public GrGLSLGeometryProcessor { v->codeAppendf("float2 vertex = %s;", mp.fVertex.fName); gpArgs->fPositionVar.set(kFloat2_GrSLType, "vertex"); - GrGLSLPPFragmentBuilder* f = args.fFragBuilder; + GrGLSLFragmentBuilder* f = args.fFragBuilder; f->codeAppendf("%s = half4(1);", args.fOutputCoverage); } }; diff --git a/tests/PrimitiveProcessorTest.cpp b/tests/PrimitiveProcessorTest.cpp index 406d5e2104..ffe8175ba7 100644 --- a/tests/PrimitiveProcessorTest.cpp +++ b/tests/PrimitiveProcessorTest.cpp @@ -73,7 +73,7 @@ private: const GP& gp = args.fGP.cast<GP>(); args.fVaryingHandler->emitAttributes(gp); this->writeOutputPosition(args.fVertBuilder, gpArgs, gp.getAttrib(0).fName); - GrGLSLPPFragmentBuilder* fragBuilder = args.fFragBuilder; + GrGLSLFragmentBuilder* fragBuilder = args.fFragBuilder; fragBuilder->codeAppendf("%s = half4(1);", args.fOutputColor); fragBuilder->codeAppendf("%s = half4(1);", args.fOutputCoverage); } |