cc: Simply invalidation regions after each union.

This patch makes the region simplification happen after each union,
to make sure that if there are a lot of invalidations, the performance
doesn't degrade too much.

BUG=303757
R=enne@chromium.org

Review URL: https://codereview.chromium.org/25828004

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@226853 0039d316-1c4b-4281-b951-d872f2087c98

1 files changed, 1 insertions, 10 deletions

diff --git a/cc/base/invalidation_region.cc b/cc/base/invalidation_region.cc
index 2c39d25..6a8f97f 100644
--- a/cc/base/invalidation_region.cc
+++ b/cc/base/invalidation_region.cc
@@ -19,14 +19,6 @@ InvalidationRegion::InvalidationRegion() {}
 InvalidationRegion::~InvalidationRegion() {}
 
 void InvalidationRegion::Swap(Region* region) {
-  UMA_HISTOGRAM_CUSTOM_COUNTS(
-      "Renderer4.InvalidationRegionApproximateRectCount",
-      region_.GetRegionComplexity(),
-      1,
-      5000,
-      50);
-
-  SimplifyIfNeeded();
   region_.Swap(region);
 }
 
@@ -35,9 +27,8 @@ void InvalidationRegion::Clear() {
 }
 
 void InvalidationRegion::Union(gfx::Rect rect) {
-  // TODO(vmpstr): We should simplify the region after Union() after we get a
-  // good idea of what kind of regions are typical (from the UMA histogram).
   region_.Union(rect);
+  SimplifyIfNeeded();
 }
 
 void InvalidationRegion::SimplifyIfNeeded() {