Initialize memory for BitMask and Clip test.

Initialize memory for BitMask and Clip test.

BUG=skia:

Committed: https://skia.googlesource.com/skia/+/1ac026d71c4637622138adcbfa320e41e61bde53

[herb_g, 2015-11-20 16:50:03]

Description was changed from

==========
Initialize memory for BitMask and Clip test.

BUG=skia:
==========

to

==========
Initialize memory for BitMask and Clip test.

BUG=skia:
==========

[herb_g, 2015-11-20 16:50:03]

herb@google.com changed reviewers:
+ bungeman@google.com

[bungeman-skia, 2015-11-20 16:55:37]

https://codereview.chromium.org/1466713002/diff/1/tests/BlitMaskClip.cpp
File tests/BlitMaskClip.cpp (right):

https://codereview.chromium.org/1466713002/diff/1/tests/BlitMaskClip.cpp#newc...
tests/BlitMaskClip.cpp:10: #include "SkUtils.h"
Is there a reason for adding this?

https://codereview.chromium.org/1466713002/diff/1/tests/BlitMaskClip.cpp#newc...
tests/BlitMaskClip.cpp:42: for (int i = 0; i < rowBytes * height; i++) {
memset?

[herb_g, 2015-11-20 17:01:18]

change to memset.

[herb_g, 2015-11-20 17:05:01]

The CQ bit was checked by herb@google.com

[commit-bot: I haz the power, 2015-11-20 17:05:14]

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

[commit-bot: I haz the power, 2015-11-20 17:05:16]

Note for Reviewers:
The CQ is waiting for an approval.
If you believe that the CL is not ready yet, or if you would like to L-G-T-M
with comments then please uncheck the CQ checkbox.

Waiting for LGTM from valid reviewer(s) till 2015-11-20 23:05 UTC

[herb_g, 2015-11-20 17:09:24]

get includes right

[herb_g, 2015-11-20 17:11:10]

PTAL

https://codereview.chromium.org/1466713002/diff/1/tests/BlitMaskClip.cpp
File tests/BlitMaskClip.cpp (right):

https://codereview.chromium.org/1466713002/diff/1/tests/BlitMaskClip.cpp#newc...
tests/BlitMaskClip.cpp:10: #include "SkUtils.h"
On 2015/11/20 16:55:36, bungeman1 wrote:
> Is there a reason for adding this?

Done.

https://codereview.chromium.org/1466713002/diff/1/tests/BlitMaskClip.cpp#newc...
tests/BlitMaskClip.cpp:42: for (int i = 0; i < rowBytes * height; i++) {
On 2015/11/20 16:55:36, bungeman1 wrote:
> memset?

Done.

[bungeman-skia, 2015-11-20 17:11:37]

The CQ bit was checked by bungeman@google.com to run a CQ dry run

[commit-bot: I haz the power, 2015-11-20 17:11:43]

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

[bungeman-skia, 2015-11-20 18:04:19]

The CQ bit was unchecked by bungeman@google.com

[bungeman-skia, 2015-11-20 18:04:39]

The CQ bit was checked by bungeman@google.com

[commit-bot: I haz the power, 2015-11-20 18:04:46]

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

[bungeman-skia, 2015-11-20 18:05:06]

lgtm

[commit-bot: I haz the power, 2015-11-20 21:37:39]

Committed patchset #3 (id:40001) as
https://skia.googlesource.com/skia/+/1ac026d71c4637622138adcbfa320e41e61bde53