Cleanup: Turn GrReducedClip into a class with a static function.

Cleanup: Turn GrReducedClip into a class with a static function.

Clean up namespace usage.

Similar to what was done in
https://skia.googlesource.com/skia/+/a5414c4a8efc3119ee20fcee96c0bf68a04909c7

BUG=None
TEST=None
R=bsalomon@google.com

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

[tfarina, 2014-10-23 04:03:08]

ptal.

https://codereview.chromium.org/653393003/diff/1/src/gpu/GrReducedClip.cpp
File src/gpu/GrReducedClip.cpp (right):

https://codereview.chromium.org/653393003/diff/1/src/gpu/GrReducedClip.cpp#ne...
src/gpu/GrReducedClip.cpp:326: void GrReducedClip::ReduceClipStack(const
SkClipStack& stack,
Sorry for this big diff. I can revert if you want. What I did was to move the
helper function above and make it static, plus the necessary changes to get it
compiling.

[robertphillips, 2014-10-23 12:24:17]

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

[robertphillips, 2014-10-23 12:24:18]

lgtm + some formatting nits

https://codereview.chromium.org/653393003/diff/1/src/gpu/GrClipMaskManager.cpp
File src/gpu/GrClipMaskManager.cpp (right):

https://codereview.chromium.org/653393003/diff/1/src/gpu/GrClipMaskManager.cp...
src/gpu/GrClipMaskManager.cpp:737: 
overlength line ?

https://codereview.chromium.org/653393003/diff/1/src/gpu/GrReducedClip.cpp
File src/gpu/GrReducedClip.cpp (right):

https://codereview.chromium.org/653393003/diff/1/src/gpu/GrReducedClip.cpp#ne...
src/gpu/GrReducedClip.cpp:23: 
overlength ?

https://codereview.chromium.org/653393003/diff/1/src/gpu/GrReducedClip.h
File src/gpu/GrReducedClip.h (right):

https://codereview.chromium.org/653393003/diff/1/src/gpu/GrReducedClip.h#newc...
src/gpu/GrReducedClip.h:23: /**
overlength lines due to new indentation ?

[tfarina, 2014-10-24 00:37:51]

Patchset #2 (id:20001) has been deleted

[tfarina, 2014-10-24 00:38:57]

https://codereview.chromium.org/653393003/diff/1/src/gpu/GrClipMaskManager.cpp
File src/gpu/GrClipMaskManager.cpp (right):

https://codereview.chromium.org/653393003/diff/1/src/gpu/GrClipMaskManager.cp...
src/gpu/GrClipMaskManager.cpp:737: 
On 2014/10/23 12:24:17, robertphillips wrote:
> overlength line ?

Done.

https://codereview.chromium.org/653393003/diff/1/src/gpu/GrReducedClip.cpp
File src/gpu/GrReducedClip.cpp (right):

https://codereview.chromium.org/653393003/diff/1/src/gpu/GrReducedClip.cpp#ne...
src/gpu/GrReducedClip.cpp:23: 
On 2014/10/23 12:24:17, robertphillips wrote:
> overlength ?

Done.

https://codereview.chromium.org/653393003/diff/1/src/gpu/GrReducedClip.h
File src/gpu/GrReducedClip.h (right):

https://codereview.chromium.org/653393003/diff/1/src/gpu/GrReducedClip.h#newc...
src/gpu/GrReducedClip.h:23: /**
On 2014/10/23 12:24:17, robertphillips wrote:
> overlength lines due to new indentation ?

Done.

[tfarina, 2014-10-24 00:39:03]

The CQ bit was checked by tfarina@chromium.org

[commit-bot: I haz the power, 2014-10-24 00:39:20]

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

[commit-bot: I haz the power, 2014-10-24 00:47:31]

Committed patchset #2 (id:40001) as bf54e49e30cdc92f9ce3f336cc48cf6b236ddeab