Initial CL to create GrBatchTest infrastructure

Initial CL to create GrBatchTest infrastructure

BUG=skia:

Committed: https://skia.googlesource.com/skia/+/3f655f34a234aceaa0c2947ce999a2127c729dea

[joshualitt, 2015-04-28 20:21:45]

joshualitt@google.com changed reviewers:
+ bsalomon@google.com, joshualitt@google.com

[joshualitt, 2015-04-28 20:21:46]

ptal

[joshualitt, 2015-04-28 20:59:25]

On 2015/04/28 20:21:46, joshualitt wrote:
> ptal

adding stroke rect

[joshualitt, 2015-04-29 13:00:17]

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

[joshualitt, 2015-04-29 13:00:17]

adding Robert for stroke rect stuff

[robertphillips, 2015-04-29 13:23:01]

lgtm but I would like Brian to also have a look

https://codereview.chromium.org/1109153004/diff/60001/src/gpu/GrTestUtils.cpp
File src/gpu/GrTestUtils.cpp (right):

https://codereview.chromium.org/1109153004/diff/60001/src/gpu/GrTestUtils.cpp...
src/gpu/GrTestUtils.cpp:35: gRects[0] = SkRect::MakeWH(1.f, 1.f);
gRects[1] = SkRect::MakeWH(1.0f, 256.0f);
gRects[2] = SkRect::MakeWH(256.0f, 1.0f);
gRects[3] = SkRect::MakeEmpty();
gRects[4] = SkRect::MakeLargest();
gRects[5] = SkRect::MakeLTRB(-65535.0f, -65535.0f, 65535.0f, 65535.0f);
gRects[6] = SkRect::MakeLTRB(10.0f, 10.0f, -10.0f, -10.0f);
?

https://codereview.chromium.org/1109153004/diff/80001/include/gpu/GrTestUtils.h
File include/gpu/GrTestUtils.h (right):

https://codereview.chromium.org/1109153004/diff/80001/include/gpu/GrTestUtils...
include/gpu/GrTestUtils.h:51: case kRandom_ColorMode:
{}s around this (since it has a variable) ?

[bsalomon, 2015-04-29 16:03:58]

lgtm but I think we should probably have our own test macro rather than piggy
backing on the static initializer macro.

[joshualitt, 2015-04-29 16:55:54]

The CQ bit was checked by joshualitt@google.com

[joshualitt, 2015-04-29 16:55:54]

The patchset sent to the CQ was uploaded after l-g-t-m from
robertphillips@google.com, bsalomon@google.com
Link to the patchset: https://codereview.chromium.org/1109153004/#ps200001
(title: "windows warnings")

[commit-bot: I haz the power, 2015-04-29 16:56:00]

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

[commit-bot: I haz the power, 2015-04-29 17:01:27]

Committed patchset #11 (id:200001) as
https://skia.googlesource.com/skia/+/3f655f34a234aceaa0c2947ce999a2127c729dea

[brucedawson, 2015-04-30 20:19:34]

brucedawson@chromium.org changed reviewers:
+ brucedawson@chromium.org

[brucedawson, 2015-04-30 20:19:35]

https://codereview.chromium.org/1109153004/diff/200001/src/gpu/GrTestUtils.cpp
File src/gpu/GrTestUtils.cpp (right):

https://codereview.chromium.org/1109153004/diff/200001/src/gpu/GrTestUtils.cp...
src/gpu/GrTestUtils.cpp:33: static SkRect gRects[1];
Why is this array declared as having just one element when seven elements are
referenced down below? This triggers a series of /analyze warnings, including
this one:

src\third_party\skia\src\gpu\grtestutils.cpp(67) : warning C6201: Index '6' is
out of valid index range '0' to '0' for possibly stack allocated buffer 'struct
SkRect * `struct SkRect const & __cdecl GrTest::TestRect(class SkRandom
*)'::`2'::gRects'.