Move atlas creation to internal to GrResourceProvider

Move atlas creation to internal to GrResourceProvider

BUG=skia:

Committed: https://skia.googlesource.com/skia/+/b356cbc765468e2594fca9ec6e5476a3da145d26

[joshualitt, 2015-08-04 17:10:02]

joshualitt@google.com changed reviewers:
+ bsalomon@google.com, joshualitt@google.com

[joshualitt, 2015-08-04 17:10:02]

ptal

[bsalomon, 2015-08-04 17:39:41]

https://codereview.chromium.org/1260183006/diff/1/src/gpu/GrResourceProvider.h
File src/gpu/GrResourceProvider.h (right):

https://codereview.chromium.org/1260183006/diff/1/src/gpu/GrResourceProvider....
src/gpu/GrResourceProvider.h:118: /**  Returns a GrBatchAtlas.  Should only be
used inside of GrBatch::generateGeometry
This reads as though the function should only be used inside GG but neither
caller does that. Should it say that the returned atlas should only be used
inside GG?

[joshualitt, 2015-08-04 17:44:54]

On 2015/08/04 17:39:41, bsalomon wrote:
> https://codereview.chromium.org/1260183006/diff/1/src/gpu/GrResourceProvider.h
> File src/gpu/GrResourceProvider.h (right):
> 
>
https://codereview.chromium.org/1260183006/diff/1/src/gpu/GrResourceProvider....
> src/gpu/GrResourceProvider.h:118: /**  Returns a GrBatchAtlas.  Should only be
> used inside of GrBatch::generateGeometry
> This reads as though the function should only be used inside GG but neither
> caller does that. Should it say that the returned atlas should only be used
> inside GG?

comment updated

[bsalomon, 2015-08-04 18:18:04]

On 2015/08/04 17:44:54, joshualitt wrote:
> On 2015/08/04 17:39:41, bsalomon wrote:
> >
https://codereview.chromium.org/1260183006/diff/1/src/gpu/GrResourceProvider.h
> > File src/gpu/GrResourceProvider.h (right):
> > 
> >
>
https://codereview.chromium.org/1260183006/diff/1/src/gpu/GrResourceProvider....
> > src/gpu/GrResourceProvider.h:118: /**  Returns a GrBatchAtlas.  Should only
be
> > used inside of GrBatch::generateGeometry
> > This reads as though the function should only be used inside GG but neither
> > caller does that. Should it say that the returned atlas should only be used
> > inside GG?
> 
> comment updated

lgtm

[joshualitt, 2015-08-04 21:18:15]

The CQ bit was checked by joshualitt@google.com

[joshualitt, 2015-08-04 21:18:15]

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/1260183006/#ps40001
(title: "tweak")

[joshualitt, 2015-08-04 21:18:19]

The CQ bit was unchecked by joshualitt@google.com

[joshualitt, 2015-08-04 21:18:21]

The CQ bit was checked by joshualitt@google.com

[commit-bot: I haz the power, 2015-08-04 21:18:33]

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

[commit-bot: I haz the power, 2015-08-04 21:29:32]

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

[commit-bot: I haz the power, 2015-08-04 21:29:33]

Failed to commit the patch.

[commit-bot: I haz the power, 2015-08-04 21:29:37]

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

[joshualitt, 2015-08-05 13:35:56]

The CQ bit was checked by joshualitt@google.com

[commit-bot: I haz the power, 2015-08-05 13:36:07]

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

[commit-bot: I haz the power, 2015-08-05 13:36:42]

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