AuroraMiddleware/skia2
1
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity
47b48bc2be
skia2/tests/AdvancedBlendTest.cpp

73 lines
3.2 KiB
C++
Raw Normal View History

Enforce non-overlap constraint from GrRenderTargetOpList Enforces this constraint from one central location, rather than relying on each Op to remember to check if overlap is allowed from its onCombineIfPossible method. Fixes an issue where we need to check the total bounds of both chains for overlap (not the bounds of individual Ops). Bug: skia:8671 Change-Id: I163651c868847884459acfc00d13ffdfca3a27c3 Reviewed-on: https://skia-review.googlesource.com/c/185815 Commit-Queue: Chris Dalton <csmartdalton@google.com> Reviewed-by: Brian Salomon <bsalomon@google.com>
2019-01-23 16:10:36 +00:00
/*
* Copyright 2019 Google Inc.
*
* Use of this source code is governed by a BSD-style license that can be
* found in the LICENSE file.
*/
IWYU for tests A-D. Change-Id: I0cb383eb6ea5a302627f93b90182a7286c2c0995 Reviewed-on: https://skia-review.googlesource.com/c/skia/+/212280 Reviewed-by: Hal Canary <halcanary@google.com> Commit-Queue: Ben Wagner <bungeman@google.com>
2019-05-06 21:17:19 +00:00
#include "include/core/SkBlendMode.h"
Update unit tests to accept GrDirectContext This CL makes it explicit that the unit tests always get a direct context. It is mainly a mechanical CL. Change-Id: I49e0628851d9c81eb47386ef978edf905c6469d8 Reviewed-on: https://skia-review.googlesource.com/c/skia/+/299866 Reviewed-by: Adlai Holler <adlai@google.com> Commit-Queue: Robert Phillips <robertphillips@google.com>
2020-07-06 14:56:46 +00:00
#include "include/gpu/GrDirectContext.h"
IWYU for tests A-D. Change-Id: I0cb383eb6ea5a302627f93b90182a7286c2c0995 Reviewed-on: https://skia-review.googlesource.com/c/skia/+/212280 Reviewed-by: Hal Canary <halcanary@google.com> Commit-Queue: Ben Wagner <bungeman@google.com>
2019-05-06 21:17:19 +00:00
#include "include/private/GrTypesPriv.h"
#include "include/private/SkColorData.h"
Make GrBlend.h private Change-Id: I77f3dbc80d85d3a19e8986dae42450ee5b8c07a7 Reviewed-on: https://skia-review.googlesource.com/c/skia/+/234668 Commit-Queue: Brian Salomon <bsalomon@google.com> Reviewed-by: Robert Phillips <robertphillips@google.com>
2019-08-14 20:56:13 +00:00
#include "src/gpu/GrBlend.h"
IWYU for tests A-D. Change-Id: I0cb383eb6ea5a302627f93b90182a7286c2c0995 Reviewed-on: https://skia-review.googlesource.com/c/skia/+/212280 Reviewed-by: Hal Canary <halcanary@google.com> Commit-Queue: Ben Wagner <bungeman@google.com>
2019-05-06 21:17:19 +00:00
#include "src/gpu/GrCaps.h"
Rename GrContextPriv to GrDirectContextPriv Change-Id: I3fccadd8a2860dbee73f93f995738146373f8a39 Reviewed-on: https://skia-review.googlesource.com/c/skia/+/326196 Commit-Queue: Adlai Holler <adlai@google.com> Auto-Submit: Adlai Holler <adlai@google.com> Reviewed-by: Robert Phillips <robertphillips@google.com>
2020-10-14 15:23:11 +00:00
#include "src/gpu/GrDirectContextPriv.h"
rewrite includes to not need so much -Ifoo Current strategy: everything from the top Things to look at first are the manual changes: - added tools/rewrite_includes.py - removed -Idirectives from BUILD.gn - various compile.sh simplifications - tweak tools/embed_resources.py - update gn/find_headers.py to write paths from the top - update gn/gn_to_bp.py SkUserConfig.h layout so that #include "include/config/SkUserConfig.h" always gets the header we want. No-Presubmit: true Change-Id: I73a4b181654e0e38d229bc456c0d0854bae3363e Reviewed-on: https://skia-review.googlesource.com/c/skia/+/209706 Commit-Queue: Mike Klein <mtklein@google.com> Reviewed-by: Hal Canary <halcanary@google.com> Reviewed-by: Brian Osman <brianosman@google.com> Reviewed-by: Florin Malita <fmalita@chromium.org>
2019-04-23 17:05:21 +00:00
#include "src/gpu/GrPaint.h"
IWYU for tests A-D. Change-Id: I0cb383eb6ea5a302627f93b90182a7286c2c0995 Reviewed-on: https://skia-review.googlesource.com/c/skia/+/212280 Reviewed-by: Hal Canary <halcanary@google.com> Commit-Queue: Ben Wagner <bungeman@google.com>
2019-05-06 21:17:19 +00:00
#include "src/gpu/GrProcessorAnalysis.h"
rewrite includes to not need so much -Ifoo Current strategy: everything from the top Things to look at first are the manual changes: - added tools/rewrite_includes.py - removed -Idirectives from BUILD.gn - various compile.sh simplifications - tweak tools/embed_resources.py - update gn/find_headers.py to write paths from the top - update gn/gn_to_bp.py SkUserConfig.h layout so that #include "include/config/SkUserConfig.h" always gets the header we want. No-Presubmit: true Change-Id: I73a4b181654e0e38d229bc456c0d0854bae3363e Reviewed-on: https://skia-review.googlesource.com/c/skia/+/209706 Commit-Queue: Mike Klein <mtklein@google.com> Reviewed-by: Hal Canary <halcanary@google.com> Reviewed-by: Brian Osman <brianosman@google.com> Reviewed-by: Florin Malita <fmalita@chromium.org>
2019-04-23 17:05:21 +00:00
#include "src/gpu/GrProcessorSet.h"
IWYU for tests A-D. Change-Id: I0cb383eb6ea5a302627f93b90182a7286c2c0995 Reviewed-on: https://skia-review.googlesource.com/c/skia/+/212280 Reviewed-by: Hal Canary <halcanary@google.com> Commit-Queue: Ben Wagner <bungeman@google.com>
2019-05-06 21:17:19 +00:00
#include "src/gpu/GrUserStencilSettings.h"
#include "src/gpu/GrXferProcessor.h"
rewrite includes to not need so much -Ifoo Current strategy: everything from the top Things to look at first are the manual changes: - added tools/rewrite_includes.py - removed -Idirectives from BUILD.gn - various compile.sh simplifications - tweak tools/embed_resources.py - update gn/find_headers.py to write paths from the top - update gn/gn_to_bp.py SkUserConfig.h layout so that #include "include/config/SkUserConfig.h" always gets the header we want. No-Presubmit: true Change-Id: I73a4b181654e0e38d229bc456c0d0854bae3363e Reviewed-on: https://skia-review.googlesource.com/c/skia/+/209706 Commit-Queue: Mike Klein <mtklein@google.com> Reviewed-by: Hal Canary <halcanary@google.com> Reviewed-by: Brian Osman <brianosman@google.com> Reviewed-by: Florin Malita <fmalita@chromium.org>
2019-04-23 17:05:21 +00:00
#include "src/gpu/effects/GrCustomXfermode.h"
IWYU for tests A-D. Change-Id: I0cb383eb6ea5a302627f93b90182a7286c2c0995 Reviewed-on: https://skia-review.googlesource.com/c/skia/+/212280 Reviewed-by: Hal Canary <halcanary@google.com> Commit-Queue: Ben Wagner <bungeman@google.com>
2019-05-06 21:17:19 +00:00
#include "tests/Test.h"
#include "tools/gpu/GrContextFactory.h"
#include <utility>
Enforce non-overlap constraint from GrRenderTargetOpList Enforces this constraint from one central location, rather than relying on each Op to remember to check if overlap is allowed from its onCombineIfPossible method. Fixes an issue where we need to check the total bounds of both chains for overlap (not the bounds of individual Ops). Bug: skia:8671 Change-Id: I163651c868847884459acfc00d13ffdfca3a27c3 Reviewed-on: https://skia-review.googlesource.com/c/185815 Commit-Queue: Chris Dalton <csmartdalton@google.com> Reviewed-by: Brian Salomon <bsalomon@google.com>
2019-01-23 16:10:36 +00:00
DEF_GPUTEST_FOR_RENDERING_CONTEXTS(AdvancedBlendTest, reporter, ctxInfo) {
static constexpr auto opaque = GrProcessorAnalysisColor::Opaque::kYes;
static constexpr auto coverage = GrProcessorAnalysisCoverage::kSingleChannel;
Update unit tests to accept GrDirectContext This CL makes it explicit that the unit tests always get a direct context. It is mainly a mechanical CL. Change-Id: I49e0628851d9c81eb47386ef978edf905c6469d8 Reviewed-on: https://skia-review.googlesource.com/c/skia/+/299866 Reviewed-by: Adlai Holler <adlai@google.com> Commit-Queue: Robert Phillips <robertphillips@google.com>
2020-07-06 14:56:46 +00:00
const GrCaps& caps = *ctxInfo.directContext()->priv().caps();
Enforce non-overlap constraint from GrRenderTargetOpList Enforces this constraint from one central location, rather than relying on each Op to remember to check if overlap is allowed from its onCombineIfPossible method. Fixes an issue where we need to check the total bounds of both chains for overlap (not the bounds of individual Ops). Bug: skia:8671 Change-Id: I163651c868847884459acfc00d13ffdfca3a27c3 Reviewed-on: https://skia-review.googlesource.com/c/185815 Commit-Queue: Chris Dalton <csmartdalton@google.com> Reviewed-by: Brian Salomon <bsalomon@google.com>
2019-01-23 16:10:36 +00:00
for (int mode = (int)SkBlendMode::kLastMode; mode > (int)SkBlendMode::kLastCoeffMode; --mode) {
const SkBlendMode blendMode = (SkBlendMode)mode;
const GrBlendEquation blendEquation =
(GrBlendEquation)(mode + (kOverlay_GrBlendEquation - (int)SkBlendMode::kOverlay));
const GrXPFactory* xpf = GrCustomXfermode::Get(blendMode);
GrXPFactory::AnalysisProperties xpfAnalysis =
Plumb mixed samples through GrXPFactory::GetAnalysisProperties Bug: skia:10419 Change-Id: I43898b0131231e2a4d1d6153e60530ba372b1c19 Reviewed-on: https://skia-review.googlesource.com/c/skia/+/340516 Reviewed-by: Brian Salomon <bsalomon@google.com> Reviewed-by: Greg Daniel <egdaniel@google.com> Commit-Queue: Chris Dalton <csmartdalton@google.com>
2020-12-03 08:43:08 +00:00
GrXPFactory::GetAnalysisProperties(xpf, opaque, coverage, false, caps,
GrClampType::kAuto);
Enforce non-overlap constraint from GrRenderTargetOpList Enforces this constraint from one central location, rather than relying on each Op to remember to check if overlap is allowed from its onCombineIfPossible method. Fixes an issue where we need to check the total bounds of both chains for overlap (not the bounds of individual Ops). Bug: skia:8671 Change-Id: I163651c868847884459acfc00d13ffdfca3a27c3 Reviewed-on: https://skia-review.googlesource.com/c/185815 Commit-Queue: Chris Dalton <csmartdalton@google.com> Reviewed-by: Brian Salomon <bsalomon@google.com>
2019-01-23 16:10:36 +00:00
GrPaint paint;
paint.setXPFactory(xpf);
GrProcessorSet procs(std::move(paint));
Eliminate mixed samples as an FBO type or AA type From now on, sample counts always refer to the number of actual color samples, and render targets don't have separate color and stencil sample counts. If mixed samples support is available when making a "GrAAType::kCoverage" draw, then an op may attach and use a mixed sampled stencil buffer internally. But this will all be invisible to the client. After this CL, we temporarily won't have a mode to use nvpr with mixed samples. That will soon be fixed by a follow-on CL that enables nvpr with mixed samples in the normal "gl" and "gles" configs. Bug: skia: Change-Id: I1cb8277f0d2d0d371f24bb9f39cd473ed5c5c83b Reviewed-on: https://skia-review.googlesource.com/c/skia/+/221878 Commit-Queue: Chris Dalton <csmartdalton@google.com> Reviewed-by: Brian Salomon <bsalomon@google.com>
2019-06-24 00:07:38 +00:00
bool hasMixedSampledCoverage = false;
Enforce non-overlap constraint from GrRenderTargetOpList Enforces this constraint from one central location, rather than relying on each Op to remember to check if overlap is allowed from its onCombineIfPossible method. Fixes an issue where we need to check the total bounds of both chains for overlap (not the bounds of individual Ops). Bug: skia:8671 Change-Id: I163651c868847884459acfc00d13ffdfca3a27c3 Reviewed-on: https://skia-review.googlesource.com/c/185815 Commit-Queue: Chris Dalton <csmartdalton@google.com> Reviewed-by: Brian Salomon <bsalomon@google.com>
2019-01-23 16:10:36 +00:00
SkPMColor4f overrideColor;
Eliminate mixed samples as an FBO type or AA type From now on, sample counts always refer to the number of actual color samples, and render targets don't have separate color and stencil sample counts. If mixed samples support is available when making a "GrAAType::kCoverage" draw, then an op may attach and use a mixed sampled stencil buffer internally. But this will all be invisible to the client. After this CL, we temporarily won't have a mode to use nvpr with mixed samples. That will soon be fixed by a follow-on CL that enables nvpr with mixed samples in the normal "gl" and "gles" configs. Bug: skia: Change-Id: I1cb8277f0d2d0d371f24bb9f39cd473ed5c5c83b Reviewed-on: https://skia-review.googlesource.com/c/skia/+/221878 Commit-Queue: Chris Dalton <csmartdalton@google.com> Reviewed-by: Brian Salomon <bsalomon@google.com>
2019-06-24 00:07:38 +00:00
GrProcessorSet::Analysis processorAnalysis = procs.finalize(
opaque, coverage, nullptr, &GrUserStencilSettings::kUnused, hasMixedSampledCoverage,
caps, GrClampType::kAuto, &overrideColor);
Enforce non-overlap constraint from GrRenderTargetOpList Enforces this constraint from one central location, rather than relying on each Op to remember to check if overlap is allowed from its onCombineIfPossible method. Fixes an issue where we need to check the total bounds of both chains for overlap (not the bounds of individual Ops). Bug: skia:8671 Change-Id: I163651c868847884459acfc00d13ffdfca3a27c3 Reviewed-on: https://skia-review.googlesource.com/c/185815 Commit-Queue: Chris Dalton <csmartdalton@google.com> Reviewed-by: Brian Salomon <bsalomon@google.com>
2019-01-23 16:10:36 +00:00
if (caps.advancedBlendEquationSupport() &&
Remove uses of non-inclusive terminology from Ganesh code. Change-Id: Ice75d0caa7b4beb2d982be094d62b54e71b45045 Bug: chromium:1101491 Reviewed-on: https://skia-review.googlesource.com/c/skia/+/301456 Reviewed-by: Brian Salomon <bsalomon@google.com> Commit-Queue: Jim Van Verth <jvanverth@google.com>
2020-07-09 12:09:13 +00:00
!caps.isAdvancedBlendEquationDisabled(blendEquation)) {
Enforce non-overlap constraint from GrRenderTargetOpList Enforces this constraint from one central location, rather than relying on each Op to remember to check if overlap is allowed from its onCombineIfPossible method. Fixes an issue where we need to check the total bounds of both chains for overlap (not the bounds of individual Ops). Bug: skia:8671 Change-Id: I163651c868847884459acfc00d13ffdfca3a27c3 Reviewed-on: https://skia-review.googlesource.com/c/185815 Commit-Queue: Chris Dalton <csmartdalton@google.com> Reviewed-by: Brian Salomon <bsalomon@google.com>
2019-01-23 16:10:36 +00:00
REPORTER_ASSERT(reporter,
!(xpfAnalysis & GrXPFactory::AnalysisProperties::kReadsDstInShader));
if (GrCaps::kAdvancedCoherent_BlendEquationSupport == caps.blendEquationSupport()) {
REPORTER_ASSERT(reporter, !processorAnalysis.requiresNonOverlappingDraws());
} else {
REPORTER_ASSERT(reporter,
GrCaps::kAdvanced_BlendEquationSupport
== caps.blendEquationSupport());
REPORTER_ASSERT(reporter, processorAnalysis.requiresNonOverlappingDraws());
}
} else {
REPORTER_ASSERT(reporter,
(xpfAnalysis & GrXPFactory::AnalysisProperties::kReadsDstInShader));
if (xpfAnalysis & GrXPFactory::AnalysisProperties::kRequiresDstTexture) {
REPORTER_ASSERT(reporter, processorAnalysis.requiresNonOverlappingDraws());
} else {
REPORTER_ASSERT(reporter, !processorAnalysis.requiresNonOverlappingDraws());
}
}
}
}
Reference in New Issue Copy Permalink