Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1477)

Unified Diff: tests/GLProgramsTest.cpp

Issue 947443003: Move clip off of draw target (Closed) Base URL: https://skia.googlesource.com/skia.git@master
Patch Set: feedback inc Created 5 years, 10 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « tests/ClipCacheTest.cpp ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: tests/GLProgramsTest.cpp
diff --git a/tests/GLProgramsTest.cpp b/tests/GLProgramsTest.cpp
index aeb97c46358389f3b846a9d1b6f1aa5755a334c6..d96c56c5e235e282f7b8eb4b126069fcf9440ddf 100644
--- a/tests/GLProgramsTest.cpp
+++ b/tests/GLProgramsTest.cpp
@@ -248,9 +248,8 @@ bool GrDrawTarget::programUnitTest(int maxStages) {
stack.clipDevRect(screen, SkRegion::kReplace_Op, false);
// wrap the SkClipStack in a GrClipData
- GrClipData clipData;
- clipData.fClipStack.reset(SkRef(&stack));
- this->setClip(&clipData);
+ GrClip clip;
+ clip.setClipStack(&stack);
SkRandom random;
static const int NUM_TESTS = 512;
@@ -264,6 +263,7 @@ bool GrDrawTarget::programUnitTest(int maxStages) {
GrPipelineBuilder pipelineBuilder;
pipelineBuilder.setRenderTarget(rt.get());
+ pipelineBuilder.setClip(clip);
// if path rendering we have to setup a couple of things like the draw type
bool usePathRendering = gpu->glCaps().pathRenderingSupport() && random.nextBool();
« no previous file with comments | « tests/ClipCacheTest.cpp ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698