Updated GLintptr and GLsizeiptr to be the appropriate types on 64 bit Windows (Take 2)

Updated GLintptr and GLsizeiptr to be the appropriate types on 64 bit Windows (Take 2)

Requires changes from https://codereview.chromium.org/99053007/ before this can be rolled in Chromium DEPS. If this begins breaking the build due to an early roll, please contact bajones@chromium.org before reverting. 

BUG=326382
TBR=bsalomon@chromium.org

Committed: http://code.google.com/p/skia/source/detail?r=12758

[commit-bot: I haz the power, 2013-12-18 20:51:14]

No comments yet.

[bsalomon, 2013-12-18 20:55:11]

On 2013/12/18 20:51:54, bajones wrote:

lgtm. BTW Skia DEPS rolls every morning Eastern but you're always free to do it
before then.

[commit-bot: I haz the power, 2013-12-18 20:56:58]

CQ is trying da patch. Follow status at
https://skia-tree-status.appspot.com/cq/bajones@chromium.org/118533003/1

[commit-bot: I haz the power, 2013-12-18 21:09:23]

Change committed as 12758

[robertphillips, 2013-12-18 23:57:24]

This was reverted in r12762

[robertphillips, 2013-12-19 19:21:37]

Brian PTAL.

[bsalomon, 2013-12-19 19:23:25]

lgtm

[bajones, 2013-12-19 19:30:48]

Wouldn't it be better to use an "Enable" flag instead of an "Ignore" flag? That
way this can roll without any Chromeium-side changes? I was going to update this
with the following:

#if defined(_WIN64) && defined(SK_ENABLE_64BIT_OPENGL_TYPES)
typedef signed long long int GrGLintptr;
typedef signed long long int GrGLsizeiptr;
#else
typedef signed long int GrGLintptr;
typedef signed long int GrGLsizeiptr;
#endif

[robertphillips, 2013-12-19 19:32:29]

We always use "ignore" flags so Chromium feels a tiny bit bad about not using
the good stuff.

[bajones, 2013-12-19 19:34:56]

On 2013/12/19 19:32:29, robertphillips wrote:
> We always use "ignore" flags so Chromium feels a tiny bit bad about not using
> the good stuff.

Fair enough. :) I assume I'll need to land a Chromium CL today adding the ignore
define to allow this to roll cleanly, then?

[robertphillips, 2013-12-19 19:36:10]

Nope - I'll add the flag in tonight's/tomorrow morning's DEPS roll. All you have
to do is then remove the flag in your Chromium-side patch that turns everything
on.

[robertphillips, 2013-12-19 19:37:25]

committed as r12781