check for too-large rowBytes

check for too-large rowBytes

BUG=446164
Committed: https://skia.googlesource.com/skia/+/2ff257bd95c732b9cebc3aac03fbed72d6e6082a

[reed1, 2015-01-23 15:00:03]

reed@google.com changed reviewers:
+ caryclark@google.com

[reed1, 2015-01-23 15:00:26]

The CQ bit was checked by reed@google.com

[commit-bot: I haz the power, 2015-01-23 15:00:50]

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

[commit-bot: I haz the power, 2015-01-23 15:00:50]

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-01-23 21:00 UTC

[caryclark, 2015-01-23 15:10:10]

https://codereview.chromium.org/871993003/diff/20001/src/core/SkMallocPixelRe...
File src/core/SkMallocPixelRef.cpp (right):

https://codereview.chromium.org/871993003/diff/20001/src/core/SkMallocPixelRe...
src/core/SkMallocPixelRef.cpp:59: int64_t minRB = info.minRowBytes64();
signature returns uint64_t

https://codereview.chromium.org/871993003/diff/20001/src/core/SkMallocPixelRe...
src/core/SkMallocPixelRef.cpp:60: if (!sk_64_isS32(minRB)) {
... then this would need to be sk_64_asS32(uint64_t ) ... (which doesn't exist
yet)

[reed1, 2015-01-23 15:35:39]

https://codereview.chromium.org/871993003/diff/20001/src/core/SkMallocPixelRe...
File src/core/SkMallocPixelRef.cpp (right):

https://codereview.chromium.org/871993003/diff/20001/src/core/SkMallocPixelRe...
src/core/SkMallocPixelRef.cpp:59: int64_t minRB = info.minRowBytes64();
On 2015/01/23 15:10:09, caryclark wrote:
> signature returns uint64_t

trying a cast instead, to stay consistent with the following code

[reed1, 2015-01-23 15:35:43]

The CQ bit was checked by reed@google.com

[commit-bot: I haz the power, 2015-01-23 15:35:50]

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

[commit-bot: I haz the power, 2015-01-23 15:35:53]

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-01-23 21:35 UTC

[caryclark, 2015-01-23 15:49:32]

lgtm

[commit-bot: I haz the power, 2015-01-23 15:51:17]

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