Aggregate damage rects in surface aggregator

Aggregate damage rects in surface aggregator

The surface aggregator should keep track of the last frame from each surface that was aggregated, and use that to determine what the appropriate damage rect is for the next aggregation.

BUG=
TEST=chrome --use-surfaces --disable-gpu

Committed: https://crrev.com/2eb7fbb502958e0a5d34e1a0c43f998419117217
Cr-Commit-Position: refs/heads/master@{#292227}

[jbauman, 2014-08-26 21:29:39]

jbauman@chromium.org changed reviewers:
+ jamesr@chromium.org

[jbauman, 2014-08-26 21:29:39]

Ok, have a new version that keeps track of the last frame that was drawn (using
an integer).

[jamesr, 2014-08-26 22:06:22]

lgtm

https://codereview.chromium.org/506273002/diff/20001/cc/surfaces/surface.cc
File cc/surfaces/surface.cc (right):

https://codereview.chromium.org/506273002/diff/20001/cc/surfaces/surface.cc#n...
cc/surfaces/surface.cc:13: : surface_id_(id), size_(size), factory_(factory),
frame_index_(2) {
why 2? just for debugging?

https://codereview.chromium.org/506273002/diff/20001/cc/surfaces/surface_aggr...
File cc/surfaces/surface_aggregator.h (right):

https://codereview.chromium.org/506273002/diff/20001/cc/surfaces/surface_aggr...
cc/surfaces/surface_aggregator.h:83: base::hash_map<SurfaceId, int>
previous_contained_surfaces_;
could use a typedef for this container type, it's used a lot

[jbauman, 2014-08-27 20:27:32]

The CQ bit was checked by jbauman@chromium.org

[commit-bot: I haz the power, 2014-08-27 20:29:14]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/jbauman@chromium.org/506273002/40001

[commit-bot: I haz the power, 2014-08-27 21:24:55]

Committed patchset #3 (id:40001) as 2bb7359598121486ddfb3e056404334ac783fe13

[commit-bot: I haz the power, 2014-09-10 02:54:10]

Patchset 3 (id:??) landed as
https://crrev.com/2eb7fbb502958e0a5d34e1a0c43f998419117217
Cr-Commit-Position: refs/heads/master@{#292227}