Move scissor state to GrAppliedClip

Move scissor state to GrAppliedClip

Committed: https://skia.googlesource.com/skia/+/e91f7b5ceca22e2e68e252b55d59804d1ce8ae4d

[bsalomon, 2015-10-06 19:50:35]

bsalomon@google.com changed reviewers:
+ robertphillips@google.com

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

lgtm

https://codereview.chromium.org/1385233002/diff/20001/src/gpu/GrClipMaskManag...
File src/gpu/GrClipMaskManager.h (right):

https://codereview.chromium.org/1385233002/diff/20001/src/gpu/GrClipMaskManag...
src/gpu/GrClipMaskManager.h:32: 
void setScissorState(const GrScissorState& newState);

?

[bsalomon, 2015-10-07 13:31:24]

https://codereview.chromium.org/1385233002/diff/20001/src/gpu/GrClipMaskManag...
File src/gpu/GrClipMaskManager.h (right):

https://codereview.chromium.org/1385233002/diff/20001/src/gpu/GrClipMaskManag...
src/gpu/GrClipMaskManager.h:32: 
On 2015/10/07 12:28:04, robertphillips wrote:
> void setScissorState(const GrScissorState& newState);
> 
> ?

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.

[bsalomon, 2015-10-07 17:01:20]

The CQ bit was checked by bsalomon@google.com

[bsalomon, 2015-10-07 17:01:20]

The patchset sent to the CQ was uploaded after l-g-t-m from
robertphillips@google.com
Link to the patchset: https://codereview.chromium.org/1385233002/#ps40001
(title: "rebase")

[commit-bot: I haz the power, 2015-10-07 17:01:32]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1385233002/40001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1385233002/40001

[commit-bot: I haz the power, 2015-10-07 17:02:14]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2015-10-07 17:02:14]

Try jobs failed on following builders:
  Test-Ubuntu-GCC-GCE-CPU-AVX2-x86_64-Debug-Trybot on client.skia (JOB_FAILED,
http://build.chromium.org/p/client.skia/builders/Test-Ubuntu-GCC-GCE-CPU-AVX2...)
  Test-Ubuntu-GCC-GCE-CPU-AVX2-x86_64-Release-Shared-Trybot on client.skia
(JOB_FAILED,
http://build.chromium.org/p/client.skia/builders/Test-Ubuntu-GCC-GCE-CPU-AVX2...)
  Build-Ubuntu-Clang-x86_64-Debug-Trybot on client.skia.compile (JOB_FAILED,
http://build.chromium.org/p/client.skia.compile/builders/Build-Ubuntu-Clang-x...)
  Build-Ubuntu-GCC-Arm64-Debug-Android-Trybot on client.skia.compile
(JOB_FAILED,
http://build.chromium.org/p/client.skia.compile/builders/Build-Ubuntu-GCC-Arm...)
  Build-Ubuntu-GCC-Mips-Debug-Android-Trybot on client.skia.compile (JOB_FAILED,
http://build.chromium.org/p/client.skia.compile/builders/Build-Ubuntu-GCC-Mip...)
  Build-Ubuntu-GCC-x86_64-Release-Trybot on client.skia.compile (JOB_FAILED,
http://build.chromium.org/p/client.skia.compile/builders/Build-Ubuntu-GCC-x86...)

[bsalomon, 2015-10-27 13:31:14]

The CQ bit was checked by bsalomon@google.com

[bsalomon, 2015-10-27 13:31:14]

The patchset sent to the CQ was uploaded after l-g-t-m from
robertphillips@google.com
Link to the patchset: https://codereview.chromium.org/1385233002/#ps60001
(title: "rebase")

[commit-bot: I haz the power, 2015-10-27 13:31:24]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1385233002/60001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1385233002/60001

[commit-bot: I haz the power, 2015-10-27 13:42:54]

Committed patchset #4 (id:60001) as
https://skia.googlesource.com/skia/+/e91f7b5ceca22e2e68e252b55d59804d1ce8ae4d