New API for computing optimization invariants.

src/gpu/GrDrawTarget.cpp

 
 /*
  * Copyright 2010 Google Inc.
  *
  * Use of this source code is governed by a BSD-style license that can be
  * found in the LICENSE file.
  */
 
 #include "GrDrawTarget.h"
 
@@ skipping 520 matching lines @@
 
 ///////////////////////////////////////////////////////////////////////////////
 
 bool GrDrawTarget::installPipelineInDrawBatch(const GrPipelineBuilder* pipelineBuilder,
                                               const GrScissorState* scissor,
                                               GrDrawBatch* batch) {
     GrPipeline::CreateArgs args;
     args.fPipelineBuilder = pipelineBuilder;
     args.fCaps = this->caps();
     args.fScissor = scissor;
-    args.fColorPOI = pipelineBuilder->colorProcInfo(batch);
-    args.fCoveragePOI = pipelineBuilder->coverageProcInfo(batch);
+    GrPipelineOptimizations opt;
+    batch->getPipelineOptimizations(opt);
+    opt.fColorPOI.completeCalculations(pipelineBuilder->fColorFragmentProcessors.begin(),
+                                       pipelineBuilder->numColorFragmentProcessors());
+    opt.fCoveragePOI.completeCalculations(pipelineBuilder->fCoverageFragmentProcessors.begin(),
+                                          pipelineBuilder->numCoverageFragmentProcessors());
+    args.fColorPOI = opt.fColorPOI;

[egdaniel, 2015/11/24 00:08:13]

shouldn't the CreateArgs just take a GrPipelineOptimizations instead of the POIs
directly?

+    args.fCoveragePOI = opt.fCoveragePOI;
     if (!this->setupDstReadIfNecessary(*pipelineBuilder, args.fColorPOI,
                                        args.fCoveragePOI, &args.fDstTexture,
                                        batch->bounds())) {
         return false;
     }
 
     if (!batch->installPipeline(args)) {
         return false;
     }
 
     return true;
 }
 
 void GrDrawTarget::clearStencilClip(const SkIRect& rect, bool insideClip, GrRenderTarget* rt) {
     GrBatch* batch = new GrClearStencilClipBatch(rect, insideClip, rt);
     this->recordBatch(batch);
     batch->unref();
 }