Even more win64 warning fixes

Even more win64 warning fixes

Committed: https://skia.googlesource.com/skia/+/9880607151ca7c2dc6b1a4b9756938bd71913ab5

[bsalomon, 2014-12-12 20:03:30]

bsalomon@google.com changed reviewers:
+ mtklein@google.com, reed@google.com, robertphillips@google.com

[mtklein, 2014-12-12 20:34:08]

lgtm

https://codereview.chromium.org/800993002/diff/20001/src/utils/SkTextureCompr...
File src/utils/SkTextureCompressor.cpp (right):

https://codereview.chromium.org/800993002/diff/20001/src/utils/SkTextureCompr...
src/utils/SkTextureCompressor.cpp:28: return 0 == etc1_encode_image(src, width,
height, 2, SkToInt(rowBytes), dst);
This looks like one that might want to plumb further through.

[bsalomon, 2014-12-12 20:39:31]

https://codereview.chromium.org/800993002/diff/20001/src/utils/SkTextureCompr...
File src/utils/SkTextureCompressor.cpp (right):

https://codereview.chromium.org/800993002/diff/20001/src/utils/SkTextureCompr...
src/utils/SkTextureCompressor.cpp:28: return 0 == etc1_encode_image(src, width,
height, 2, SkToInt(rowBytes), dst);
On 2014/12/12 20:34:08, mtklein wrote:
> This looks like one that might want to plumb further through.

That function is in third_party. If it's ok to change it then I'm happy to.

[bsalomon, 2014-12-12 20:39:37]

The CQ bit was checked by bsalomon@google.com

[commit-bot: I haz the power, 2014-12-12 20:40:35]

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

[commit-bot: I haz the power, 2014-12-12 20:42:56]

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

[commit-bot: I haz the power, 2014-12-12 20:42:57]

Try jobs failed on following builders:
  Build-Ubuntu13.10-GCC4.8-Arm7-Debug-Android-Trybot on client.skia.compile
(http://build.chromium.org/p/client.skia.compile/builders/Build-Ubuntu13.10-GC...)

[mtklein, 2014-12-12 20:43:49]

https://codereview.chromium.org/800993002/diff/20001/src/utils/SkTextureCompr...
File src/utils/SkTextureCompressor.cpp (right):

https://codereview.chromium.org/800993002/diff/20001/src/utils/SkTextureCompr...
src/utils/SkTextureCompressor.cpp:28: return 0 == etc1_encode_image(src, width,
height, 2, SkToInt(rowBytes), dst);
On 2014/12/12 20:39:31, bsalomon wrote:
> On 2014/12/12 20:34:08, mtklein wrote:
> > This looks like one that might want to plumb further through.
> 
> That function is in third_party. If it's ok to change it then I'm happy to.

Ah, right.  I think I was confusing it with ktx.{h,cpp}, which are in
third_party but aren't really third_party.

[bsalomon, 2014-12-12 22:44:02]

The CQ bit was checked by bsalomon@google.com

[commit-bot: I haz the power, 2014-12-12 22:44:45]

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

[commit-bot: I haz the power, 2014-12-12 22:46:26]

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

[commit-bot: I haz the power, 2014-12-12 22:46:27]

Try jobs failed on following builders:
  Build-Ubuntu13.10-Clang-x86_64-Debug-Trybot on client.skia.compile
(http://build.chromium.org/p/client.skia.compile/builders/Build-Ubuntu13.10-Cl...)
  Build-Ubuntu13.10-GCC4.8-x86_64-Release-Trybot on client.skia.compile
(http://build.chromium.org/p/client.skia.compile/builders/Build-Ubuntu13.10-GC...)

[bsalomon, 2014-12-12 23:02:30]

The CQ bit was checked by bsalomon@google.com

[commit-bot: I haz the power, 2014-12-12 23:02:45]

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

[commit-bot: I haz the power, 2014-12-12 23:11:22]

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