Treat (private, internal) grid bounds as doubly-inclusive in SkTileGrid.
authormtklein <mtklein@chromium.org>
Wed, 15 Oct 2014 21:20:00 +0000 (14:20 -0700)
committerCommit bot <commit-bot@chromium.org>
Wed, 15 Oct 2014 21:20:01 +0000 (14:20 -0700)
commit826a50333612e81d1de87a58e32e4f9a3baddefd
treec81de59c3a71a5080f31e502b2649a8aa7468db3
parent8dfdfff98dd14b0f3bef465411941e1bab8f0261
Treat (private, internal) grid bounds as doubly-inclusive in SkTileGrid.

The net effect is that two "+1" instructions are removed from insert().
search() nets no change: two +1 removed, two +1 added.

When vectorized, this removes the need to add in userToGrid() at all and
so the need to read an awkward {0, 0, 1, 1} constant from memory.  Mostly
the benefit is less vector code to look at and think about.

BUG=skia:

Review URL: https://codereview.chromium.org/659823004
src/core/SkTileGrid.cpp