Start cleaning up 64bit Win warnings

Start cleaning up 64bit Win warnings

In service of Skia issue 1695 (64-bit Mac and Windows bots reporting warnings -  https://code.google.com/p/skia/issues/detail?id=1695)

[bsalomon, 2013-10-14 15:17:02]

https://codereview.chromium.org/27192003/diff/3001/src/gpu/GrDrawState.h
File src/gpu/GrDrawState.h (right):

https://codereview.chromium.org/27192003/diff/3001/src/gpu/GrDrawState.h#newc...
src/gpu/GrDrawState.h:1017: int coverageStageCnt = int(fStages.count()) -
fColorStageCnt;
I think we conventionally write this cast as (int)fStages.count() or
static_cast<int>(fStages.count()).

Also, I think SkAutoSTArray is violating the bytes-are-size_t-and-counts-are-int
style and that its count() function should return an int.

[robertphillips, 2013-10-14 17:39:26]

I recognize a mandate to cause trouble when I see one. PTAL.

https://codereview.chromium.org/27192003/diff/3001/src/gpu/GrDrawState.h
File src/gpu/GrDrawState.h (right):

https://codereview.chromium.org/27192003/diff/3001/src/gpu/GrDrawState.h#newc...
src/gpu/GrDrawState.h:1017: int coverageStageCnt = int(fStages.count()) -
fColorStageCnt;
On 2013/10/14 15:17:02, bsalomon wrote:
> I think we conventionally write this cast as (int)fStages.count() or
> static_cast<int>(fStages.count()).
> 
> Also, I think SkAutoSTArray is violating the
bytes-are-size_t-and-counts-are-int
> style and that its count() function should return an int.

Done.

[bsalomon, 2013-10-14 19:27:36]

https://codereview.chromium.org/27192003/diff/10001/include/gpu/GrTypes.h
File include/gpu/GrTypes.h (right):

https://codereview.chromium.org/27192003/diff/10001/include/gpu/GrTypes.h#new...
include/gpu/GrTypes.h:318: static inline int GrBytesPerPixel(GrPixelConfig
config) {
Hmm.. seems like this should be a size_t

[robertphillips, 2013-10-14 19:51:11]

https://codereview.chromium.org/27192003/diff/10001/include/gpu/GrTypes.h
File include/gpu/GrTypes.h (right):

https://codereview.chromium.org/27192003/diff/10001/include/gpu/GrTypes.h#new...
include/gpu/GrTypes.h:318: static inline int GrBytesPerPixel(GrPixelConfig
config) {
On 2013/10/14 19:27:37, bsalomon wrote:
> Hmm.. seems like this should be a size_t

Done.

[bsalomon, 2013-10-14 19:53:16]

lgtm

[robertphillips, 2013-10-14 21:53:51]

committed as r11764