No need to short circuit the query rect here anymore.

No need to short circuit the query rect here anymore.

Whatever assertion was triggering in SkRTree.cpp seems to be gone now.

BUG=432991
Committed: https://skia.googlesource.com/skia/+/72f92acd47a884b9a1cc961a822a2d4e586b54c1

[mtklein_C, 2014-11-19 15:19:50]

The CQ bit was checked by mtklein@chromium.org

[commit-bot: I haz the power, 2014-11-19 15:20:27]

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

[commit-bot: I haz the power, 2014-11-19 15:20:28]

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 valid reviewer(s) till 2014-11-19 21:20 UTC

[mtklein, 2014-11-19 15:34:13]

mtklein@google.com changed reviewers:
+ junov@chromium.org, mtklein@google.com

[mtklein, 2014-11-19 15:34:13]

I'm going to follow this up with tests to make sure clear() always runs, even
when the clip goes empty.  I'm starting to think sktilegrid accidentally
guarantees this by outsetting the query... would be better to pull that up a
level to work for all BBHs.

[Justin Novosad, 2014-11-19 18:37:10]

On 2014/11/19 15:34:13, mtklein wrote:
> I'm going to follow this up with tests to make sure clear() always runs, even
> when the clip goes empty.  I'm starting to think sktilegrid accidentally
> guarantees this by outsetting the query... would be better to pull that up a
> level to work for all BBHs.

lgtm

[commit-bot: I haz the power, 2014-11-19 18:37:25]

Committed patchset #1 (id:1) as
https://skia.googlesource.com/skia/+/72f92acd47a884b9a1cc961a822a2d4e586b54c1