TileGrid: earliest need not be a pointer anymore

TileGrid: earliest need not be a pointer anymore

BUG=skia:

Committed: https://skia.googlesource.com/skia/+/15c7ceb6aabec88ee4950135dbc74011b7446193

[mtklein, 2014-10-02 14:50:12]

mtklein@google.com changed reviewers:
+ mtklein@google.com, robertphillips@google.com

[mtklein, 2014-10-02 14:50:12]

Turned out to be simpler than I feared...

[mtklein, 2014-10-02 14:50:16]

The CQ bit was checked by mtklein@google.com

[commit-bot: I haz the power, 2014-10-02 14:51:00]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/625573003/1

[commit-bot: I haz the power, 2014-10-02 14:51:01]

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-10-02 20:51 UTC

[robertphillips, 2014-10-02 15:17:11]

The CQ bit was unchecked by robertphillips@google.com

[robertphillips, 2014-10-02 15:17:25]

lgtm + a suggestion

https://codereview.chromium.org/625573003/diff/1/src/core/SkTileGrid.cpp
File src/core/SkTileGrid.cpp (right):

https://codereview.chromium.org/625573003/diff/1/src/core/SkTileGrid.cpp#newc...
src/core/SkTileGrid.cpp:126: // tile. It may be at the front of several, even
all, tiles.
SK_MaxU32 ?

https://codereview.chromium.org/625573003/diff/1/src/core/SkTileGrid.cpp#newc...
src/core/SkTileGrid.cpp:134: // If we didn't find an earliest op, there isn't
anything left to merge.
and here ?

[mtklein, 2014-10-02 15:40:59]

https://codereview.chromium.org/625573003/diff/1/src/core/SkTileGrid.cpp
File src/core/SkTileGrid.cpp (right):

https://codereview.chromium.org/625573003/diff/1/src/core/SkTileGrid.cpp#newc...
src/core/SkTileGrid.cpp:126: // tile. It may be at the front of several, even
all, tiles.
On 2014/10/02 15:17:25, robertphillips wrote:
> SK_MaxU32 ?

Done.

https://codereview.chromium.org/625573003/diff/1/src/core/SkTileGrid.cpp#newc...
src/core/SkTileGrid.cpp:134: // If we didn't find an earliest op, there isn't
anything left to merge.
On 2014/10/02 15:17:25, robertphillips wrote:
> and here ?

Done.

[mtklein, 2014-10-02 15:41:04]

The CQ bit was checked by mtklein@google.com

[commit-bot: I haz the power, 2014-10-02 15:42:07]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/625573003/20001

[commit-bot: I haz the power, 2014-10-02 15:50:44]

Committed patchset #2 (id:20001) as 15c7ceb6aabec88ee4950135dbc74011b7446193