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

Side by Side Diff: src/gpu/GrPipelineBuilder.cpp

Issue 1228763005: small cleanups after fixing const of GrPipelineBuilder on DrawTarget (Closed) Base URL: https://skia.googlesource.com/skia.git@const-fix3
Patch Set: tweaks Created 5 years, 5 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 unified diff | Download patch
« no previous file with comments | « src/gpu/GrPipelineBuilder.h ('k') | src/gpu/GrSWMaskHelper.cpp » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 /* 1 /*
2 * Copyright 2015 Google Inc. 2 * Copyright 2015 Google Inc.
3 * 3 *
4 * Use of this source code is governed by a BSD-style license that can be 4 * Use of this source code is governed by a BSD-style license that can be
5 * found in the LICENSE file. 5 * found in the LICENSE file.
6 */ 6 */
7 7
8 #include "GrPipelineBuilder.h" 8 #include "GrPipelineBuilder.h"
9 9
10 #include "GrBatch.h" 10 #include "GrBatch.h"
(...skipping 81 matching lines...) Expand 10 before | Expand all | Expand 10 after
92 92
93 //////////////////////////////////////////////////////////////////////////////s 93 //////////////////////////////////////////////////////////////////////////////s
94 94
95 bool GrPipelineBuilder::willXPNeedDstTexture(const GrCaps& caps, 95 bool GrPipelineBuilder::willXPNeedDstTexture(const GrCaps& caps,
96 const GrProcOptInfo& colorPOI, 96 const GrProcOptInfo& colorPOI,
97 const GrProcOptInfo& coveragePOI) c onst { 97 const GrProcOptInfo& coveragePOI) c onst {
98 return this->getXPFactory()->willNeedDstTexture(caps, colorPOI, coveragePOI, 98 return this->getXPFactory()->willNeedDstTexture(caps, colorPOI, coveragePOI,
99 this->hasMixedSamples()); 99 this->hasMixedSamples());
100 } 100 }
101 101
102 void GrPipelineBuilder::AutoRestoreFragmentProcessors::set( 102 void GrPipelineBuilder::AutoRestoreFragmentProcessorState::set(
103 const GrPipelineBuilder * pipelineBuilder) { 103 const GrPipelineBuilder * pipelineBuilder) {
104 if (fPipelineBuilder) { 104 if (fPipelineBuilder) {
105 int m = fPipelineBuilder->numColorFragmentStages() - fColorEffectCnt; 105 int m = fPipelineBuilder->numColorFragmentStages() - fColorEffectCnt;
106 SkASSERT(m >= 0); 106 SkASSERT(m >= 0);
107 fPipelineBuilder->fColorStages.pop_back_n(m); 107 fPipelineBuilder->fColorStages.pop_back_n(m);
108 108
109 int n = fPipelineBuilder->numCoverageFragmentStages() - fCoverageEffectC nt; 109 int n = fPipelineBuilder->numCoverageFragmentStages() - fCoverageEffectC nt;
110 SkASSERT(n >= 0); 110 SkASSERT(n >= 0);
111 fPipelineBuilder->fCoverageStages.pop_back_n(n); 111 fPipelineBuilder->fCoverageStages.pop_back_n(n);
112 if (m + n > 0) { 112 if (m + n > 0) {
113 fPipelineBuilder->fColorProcInfoValid = false; 113 fPipelineBuilder->fColorProcInfoValid = false;
114 fPipelineBuilder->fCoverageProcInfoValid = false; 114 fPipelineBuilder->fCoverageProcInfoValid = false;
115 } 115 }
116 SkDEBUGCODE(--fPipelineBuilder->fBlockEffectRemovalCnt;) 116 SkDEBUGCODE(--fPipelineBuilder->fBlockEffectRemovalCnt;)
117 fPipelineBuilder->getProcessorDataManager()->restoreToSaveMarker(/*fSave Marker*/);
117 } 118 }
118 fPipelineBuilder = const_cast<GrPipelineBuilder*>(pipelineBuilder); 119 fPipelineBuilder = const_cast<GrPipelineBuilder*>(pipelineBuilder);
119 if (NULL != pipelineBuilder) { 120 if (NULL != pipelineBuilder) {
120 fColorEffectCnt = pipelineBuilder->numColorFragmentStages(); 121 fColorEffectCnt = pipelineBuilder->numColorFragmentStages();
121 fCoverageEffectCnt = pipelineBuilder->numCoverageFragmentStages(); 122 fCoverageEffectCnt = pipelineBuilder->numCoverageFragmentStages();
122 SkDEBUGCODE(++pipelineBuilder->fBlockEffectRemovalCnt;) 123 SkDEBUGCODE(++pipelineBuilder->fBlockEffectRemovalCnt;)
124 fSaveMarker = pipelineBuilder->processorDataManager()->currentSaveMarker ();
123 } 125 }
124 } 126 }
125 127
126 //////////////////////////////////////////////////////////////////////////////// 128 ////////////////////////////////////////////////////////////////////////////////
127 129
128 GrPipelineBuilder::~GrPipelineBuilder() { 130 GrPipelineBuilder::~GrPipelineBuilder() {
129 SkASSERT(0 == fBlockEffectRemovalCnt); 131 SkASSERT(0 == fBlockEffectRemovalCnt);
130 } 132 }
131 133
132 //////////////////////////////////////////////////////////////////////////////// 134 ////////////////////////////////////////////////////////////////////////////////
(...skipping 43 matching lines...) Expand 10 before | Expand all | Expand 10 after
176 void GrPipelineBuilder::calcCoverageInvariantOutput(GrColor coverage) const { 178 void GrPipelineBuilder::calcCoverageInvariantOutput(GrColor coverage) const {
177 if (!fCoverageProcInfoValid || coverage != fCoverageCache) { 179 if (!fCoverageProcInfoValid || coverage != fCoverageCache) {
178 GrColorComponentFlags flags = kRGBA_GrColorComponentFlags; 180 GrColorComponentFlags flags = kRGBA_GrColorComponentFlags;
179 fCoverageProcInfo.calcWithInitialValues(fCoverageStages.begin(), 181 fCoverageProcInfo.calcWithInitialValues(fCoverageStages.begin(),
180 this->numCoverageFragmentStages( ), coverage, flags, 182 this->numCoverageFragmentStages( ), coverage, flags,
181 true); 183 true);
182 fCoverageProcInfoValid = true; 184 fCoverageProcInfoValid = true;
183 fCoverageCache = coverage; 185 fCoverageCache = coverage;
184 } 186 }
185 } 187 }
OLDNEW
« no previous file with comments | « src/gpu/GrPipelineBuilder.h ('k') | src/gpu/GrSWMaskHelper.cpp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698