bump the size of the atlas id to 64 bits

bump the size of the atlas id to 64 bits

BUG=skia:

Committed: https://skia.googlesource.com/skia/+/8db6fdc465e0f76b1f8eb0d5f799d42b7c2246f5

[joshualitt, 2015-07-31 14:04:23]

joshualitt@google.com changed reviewers:
+ bsalomon@chromium.org, joshualitt@google.com, robertphillips@google.com

[joshualitt, 2015-07-31 14:04:23]

ptal

[bsalomon, 2015-07-31 14:25:28]

On 2015/07/31 14:04:23, joshualitt wrote:
> ptal

lgtm

[joshualitt, 2015-07-31 14:33:28]

On 2015/07/31 14:25:28, bsalomon wrote:
> On 2015/07/31 14:04:23, joshualitt wrote:
> > ptal
> 
> lgtm

please take another quick look, I actually messed something up in the old CL
with casting to int and such

[robertphillips, 2015-07-31 15:00:54]

https://codereview.chromium.org/1253003005/diff/40001/src/gpu/GrBatchAtlas.cpp
File src/gpu/GrBatchAtlas.cpp (right):

https://codereview.chromium.org/1253003005/diff/40001/src/gpu/GrBatchAtlas.cp...
src/gpu/GrBatchAtlas.cpp:21: SkASSERT(index < (1 << 16));
SkASSERT(generation < (1 << 48)); ?

https://codereview.chromium.org/1253003005/diff/40001/src/gpu/GrBatchAtlas.cp...
src/gpu/GrBatchAtlas.cpp:122: 
Do we need to be explicit here ?

https://codereview.chromium.org/1253003005/diff/40001/src/gpu/GrBatchAtlas.cp...
src/gpu/GrBatchAtlas.cpp:180: unsigned char* fData;
Do we need to be explicit here ?

https://codereview.chromium.org/1253003005/diff/40001/src/gpu/GrBatchAtlas.h
File src/gpu/GrBatchAtlas.h (right):

https://codereview.chromium.org/1253003005/diff/40001/src/gpu/GrBatchAtlas.h#...
src/gpu/GrBatchAtlas.h:109: private:
uint16_t ? do we care ?

https://codereview.chromium.org/1253003005/diff/40001/src/gpu/GrBatchAtlas.h#...
src/gpu/GrBatchAtlas.h:115: static uint64_t GetGenerationFromID(AtlasID id) {
Aren't we missing an 'ff' here ?

https://codereview.chromium.org/1253003005/diff/40001/src/gpu/GrBatchAtlas.h#...
src/gpu/GrBatchAtlas.h:125: GrTexture* fTexture;
Why do we have to be more explicit about these types ?

[joshualitt, 2015-07-31 15:05:38]

The CQ bit was checked by joshualitt@google.com

[joshualitt, 2015-07-31 15:05:38]

The patchset sent to the CQ was uploaded after l-g-t-m from bsalomon@google.com
Link to the patchset: https://codereview.chromium.org/1253003005/#ps60001
(title: "tweaks")

[commit-bot: I haz the power, 2015-07-31 15:05:46]

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

[commit-bot: I haz the power, 2015-07-31 15:07:02]

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

[commit-bot: I haz the power, 2015-07-31 15:07:03]

Try jobs failed on following builders:
  Build-Ubuntu-Clang-x86_64-Debug-Trybot on client.skia.compile (JOB_FAILED,
http://build.chromium.org/p/client.skia.compile/builders/Build-Ubuntu-Clang-x...)
  Build-Ubuntu-GCC-Arm7-Debug-Android-Trybot on client.skia.compile (JOB_FAILED,
http://build.chromium.org/p/client.skia.compile/builders/Build-Ubuntu-GCC-Arm...)
  Build-Ubuntu-GCC-Mips-Debug-Android-Trybot on client.skia.compile (JOB_FAILED,
http://build.chromium.org/p/client.skia.compile/builders/Build-Ubuntu-GCC-Mip...)

[joshualitt, 2015-07-31 15:17:34]

The CQ bit was checked by joshualitt@google.com

[joshualitt, 2015-07-31 15:17:34]

The patchset sent to the CQ was uploaded after l-g-t-m from bsalomon@google.com
Link to the patchset: https://codereview.chromium.org/1253003005/#ps80001
(title: "tweaks")

[commit-bot: I haz the power, 2015-07-31 15:17:39]

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

[commit-bot: I haz the power, 2015-07-31 15:25:11]

Committed patchset #5 (id:80001) as
https://skia.googlesource.com/skia/+/8db6fdc465e0f76b1f8eb0d5f799d42b7c2246f5

[joshualitt, 2015-08-04 16:34:23]

Patchset #6 (id:100001) has been deleted