commit 380b90c92bead0c1b869df08b2d2790c302c5c71
author Robert Phillips <robertphillips@google.com> Wed Aug 30 07:41:07 2017 -0400
committer Skia Commit-Bot <skia-commit-bot@chromium.org> Wed Aug 30 11:45:42 2017 +0000
tree 06e2a81cbb53e417c5e483b24537a36d77cb33ce
parent fb4a20c7f54110d6c43dee4b78801c163ad6c8f3

Push full screen clears up to OpList

Change-Id: Icfc09b3b25f4d3834bee93286e2b24a559f44ed8
Reviewed-on: https://skia-review.googlesource.com/40120
Commit-Queue: Robert Phillips <robertphillips@google.com>
Reviewed-by: Greg Daniel <egdaniel@google.com>

src/gpu/GrRenderTargetOpList.h

diff --git a/src/gpu/GrRenderTargetOpList.h b/src/gpu/GrRenderTargetOpList.h
index b62cee5..ef4bdac 100644
--- a/src/gpu/GrRenderTargetOpList.h
+++ b/src/gpu/GrRenderTargetOpList.h
@@ -45,7 +45,6 @@
             return;
         }
 
-        fLastFullClearOp = nullptr;
         this->forwardCombine(caps);
 
         INHERITED::makeClosed(caps);
@@ -78,6 +77,8 @@
         return this->uniqueID();
     }
 
+    void discard();
+
     /** Clears the entire render target */
     void fullClear(const GrCaps& caps, GrColor color);
 
@@ -124,10 +125,8 @@
         GrAppliedClip* fAppliedClip;
     };
 
-    // If the input op is combined with an earlier op, this returns the combined op. Otherwise, it
-    // returns the input op.
-    GrOp* recordOp(std::unique_ptr<GrOp>, const GrCaps& caps,
-                   GrAppliedClip* = nullptr, const DstProxy* = nullptr);
+    void recordOp(std::unique_ptr<GrOp>, const GrCaps& caps,
+                  GrAppliedClip* = nullptr, const DstProxy* = nullptr);
 
     void forwardCombine(const GrCaps&);
 
@@ -135,8 +134,6 @@
     bool combineIfPossible(const RecordedOp& a, GrOp* b, const GrAppliedClip* bClip,
                            const DstProxy* bDstTexture, const GrCaps&);
 
-    GrClearOp*                     fLastFullClearOp = nullptr;
-
     std::unique_ptr<gr_instanced::InstancedRendering> fInstancedRendering;
 
     uint32_t                       fLastClipStackGenID;