New API for computing optimization invariants.

src/gpu/batches/GrDrawBatch.cpp

Index: src/gpu/batches/GrDrawBatch.cpp
diff --git a/src/gpu/batches/GrDrawBatch.cpp b/src/gpu/batches/GrDrawBatch.cpp
index 43ef2ec64bb107372794d7d0efa6623e2be9debc..aa5ac5ee449e4612d72225b06c35ea9923604ab4 100644
--- a/src/gpu/batches/GrDrawBatch.cpp
+++ b/src/gpu/batches/GrDrawBatch.cpp
@@ -15,13 +15,21 @@ GrDrawBatch::~GrDrawBatch() {
     }
 }
 
+void GrDrawBatch::getPipelineOptimizations(GrPipelineOptimizations &opt) const {

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

I think we generally use pointers when the parameter is going to change

+    GrInitInvariantOutput color;
+    GrInitInvariantOutput coverage;

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

this->computePipelineOptimizations(...)

+    computePipelineOptimizations(&color, &coverage, &opt.fOverrides);
+    opt.fColorPOI.initUsingInvariantOutput(color);
+    opt.fCoveragePOI.initUsingInvariantOutput(coverage);
+}
+
 bool GrDrawBatch::installPipeline(const GrPipeline::CreateArgs& args) {
-    GrPipelineOptimizations opts;
+    GrXPOverridesForBatch overrides;
     void* location = fPipelineStorage.get();
-    if (!GrPipeline::CreateAt(location, args, &opts)) {
+    if (!GrPipeline::CreateAt(location, args, &overrides)) {
         return false;
     }
-    this->initBatchTracker(opts);
+    this->initBatchTracker(overrides);
     fPipelineInstalled = true;
     return true;
 }