Move scissor state to GrAppliedClip

src/gpu/GrClipMaskManager.h

Index: src/gpu/GrClipMaskManager.h
diff --git a/src/gpu/GrClipMaskManager.h b/src/gpu/GrClipMaskManager.h
index a57391a8e51325413a9bf67f5e6edd7afb8a0dc7..53c66cef0ab7cca5ff1544c4e315cdf46035528c 100644
--- a/src/gpu/GrClipMaskManager.h
+++ b/src/gpu/GrClipMaskManager.h
@@ -28,9 +28,11 @@ class GrAppliedClip : public SkNoncopyable {
 public:
     GrAppliedClip() {}
     const GrFragmentProcessor* clipCoverageFragmentProcessor() const { return fClipCoverageFP; }
+    const GrScissorState& scissorState() const { return fScissorState; }
 

[robertphillips, 2015/10/07 12:28:04]

void setScissorState(const GrScissorState& newState);

?

[bsalomon, 2015/10/07 13:31:23]

The way I see this evolving is GrCMM become a factory/bulider for creating an
immutable one of these given a clip and a few other pieces of info. I don't
really think we'll need setters or need to hide anything in GrAC from GrCMM.

 private:
     SkAutoTUnref<const GrFragmentProcessor> fClipCoverageFP;
+    GrScissorState                          fScissorState;
     friend GrClipMaskManager;
 
     typedef SkNoncopyable INHERITED;
@@ -57,7 +59,6 @@ public:
      */
     bool setupClipping(const GrPipelineBuilder&,
                        GrPipelineBuilder::AutoRestoreStencil*,
-                       GrScissorState*,
                        const SkRect* devBounds,
                        GrAppliedClip*);