cc: Cull DrawingDisplayItems outside of the raster playback rect

cc: Cull DrawingDisplayItems outside of the raster playback rect

Avoid playing an SkPicture into the raster canvas at all if its
bounds are not going to intersect what is being rastered. This is
for the non-cached picture path which is only used by the ui
compositor at this time.

This improves the time to raster a single tab loading spinner in
the ui compositor from 0.0807235 ms to 0.0597263 for 26% less
raster time.

BUG=487387
Committed: https://crrev.com/fb311cbcfcc06427a037ee01c7f0a3098a060e03
Cr-Commit-Position: refs/heads/master@{#332730}

[danakj, 2015-06-03 22:20:34]

danakj@chromium.org changed reviewers:
+ enne@chromium.org, vmpstr@chromium.org

[danakj, 2015-06-03 23:05:27]

Idk how to test this, I need to know that some DisplayItem did not raster.. it's
not going to appear either way. I could add bools to DrawingDisplayItem??? Ew.
Um.. Uh.. I could make Raster return a bool, and a FakeDrawingDisplayItem that
records that bool?

[enne (OOO), 2015-06-03 23:14:53]

lgtm

I'm not super keen on having that sort of wart just for testing.  I think it's
hard to test this sort of scenario, and I think I'm ok landing without a test.

[danakj, 2015-06-03 23:23:24]

The CQ bit was checked by danakj@chromium.org

[danakj, 2015-06-03 23:24:38]

The CQ bit was unchecked by danakj@chromium.org

[danakj, 2015-06-03 23:24:41]

The CQ bit was checked by danakj@chromium.org

[commit-bot: I haz the power, 2015-06-03 23:25:22]

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

[commit-bot: I haz the power, 2015-06-03 23:42:10]

Committed patchset #4 (id:60001)

[commit-bot: I haz the power, 2015-06-03 23:44:10]

Patchset 4 (id:??) landed as
https://crrev.com/fb311cbcfcc06427a037ee01c7f0a3098a060e03
Cr-Commit-Position: refs/heads/master@{#332730}