SkTileGrid: store insertion order, return results sorted by that.

SkTileGrid: store insertion order, return results sorted by that.

This removes the need to assume the void* are SkPictureStateTree::Data*.

BUG=skia:

Committed: https://skia.googlesource.com/skia/+/03bde3e6fa8deaf635bd19b45de69ebea71ec299

[mtklein, 2014-08-11 21:21:48]

The CQ bit was checked by mtklein@google.com

[commit-bot: I haz the power, 2014-08-11 21:22:38]

CQ is trying da patch. Follow status at
 https://skia-tree-status.appspot.com/cq/mtklein@chromium.org/466503002/40001

[commit-bot: I haz the power, 2014-08-11 21:22:39]

Note for Reviewers:
The CQ is waiting for an approval.
If you believe that the CL is not ready yet, or if you would like to L-G-T-M
with comments then please uncheck the CQ checkbox.

Waiting for LGTM from a valid reviewer till 2014-08-12 03:22 UTC

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

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

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

No LGTM from a valid reviewer yet. Please ask for an LGTM from a
full Skia committer from
https://skia.googlesource.com/skia/+/master/CQ_COMMITTERS

[robertphillips, 2014-08-12 12:44:38]

lgtm + 2 comment suggestions

https://codereview.chromium.org/466503002/diff/40001/src/core/SkTileGrid.h
File src/core/SkTileGrid.h (right):

https://codereview.chromium.org/466503002/diff/40001/src/core/SkTileGrid.h#ne...
src/core/SkTileGrid.h:53: struct Entry {
// The data's order of insertion into the tile grid. Needed when coalescing
multiple tiles ?

https://codereview.chromium.org/466503002/diff/40001/src/core/SkTileGrid.h#ne...
src/core/SkTileGrid.h:60: size_t fCount;
// fXTiles by fYTiles - with each array holding the data objects overlapping
that tile ?

[mtklein, 2014-08-12 13:52:38]

On 2014/08/12 12:44:38, robertphillips wrote:
> lgtm + 2 comment suggestions
> 
> https://codereview.chromium.org/466503002/diff/40001/src/core/SkTileGrid.h
> File src/core/SkTileGrid.h (right):
> 
>
https://codereview.chromium.org/466503002/diff/40001/src/core/SkTileGrid.h#ne...
> src/core/SkTileGrid.h:53: struct Entry {
> // The data's order of insertion into the tile grid. Needed when coalescing
> multiple tiles ?
> 
>
https://codereview.chromium.org/466503002/diff/40001/src/core/SkTileGrid.h#ne...
> src/core/SkTileGrid.h:60: size_t fCount;
> // fXTiles by fYTiles - with each array holding the data objects overlapping
> that tile ?

Both done.  Also update the comment about the void* argument to insert(): it can
now be NULL safely.

[mtklein, 2014-08-12 13:53:10]

The CQ bit was checked by mtklein@google.com

[commit-bot: I haz the power, 2014-08-12 13:53:17]

CQ is trying da patch. Follow status at
 https://skia-tree-status.appspot.com/cq/mtklein@chromium.org/466503002/60001

[commit-bot: I haz the power, 2014-08-12 14:01:31]

Change committed as 03bde3e6fa8deaf635bd19b45de69ebea71ec299