diff options
author | brettw@chromium.org <brettw@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2010-12-22 22:38:50 +0000 |
---|---|---|
committer | brettw@chromium.org <brettw@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2010-12-22 22:38:50 +0000 |
commit | 7524cd89b0dda14dfcc0d477a576da5e1dee59d1 (patch) | |
tree | 982e310bde8179d6061bdc3bd011ac07bbebdec5 /chrome/renderer/paint_aggregator_unittest.cc | |
parent | ab2679b27175635aca837817fb70d24617564d92 (diff) | |
download | chromium_src-7524cd89b0dda14dfcc0d477a576da5e1dee59d1.zip chromium_src-7524cd89b0dda14dfcc0d477a576da5e1dee59d1.tar.gz chromium_src-7524cd89b0dda14dfcc0d477a576da5e1dee59d1.tar.bz2 |
Increase the % coverage for coalescing for paint rects.
The previous number was made up, and so is my number, it's just a larger made
up number.
We want this number to be as high as possible (less coalescing) without causing
page cycler regressions. This means less painting and gives some other
optimizations like the always-on-top Pepper plugin optimization to kick in more
often.
If this causes a PLT regression, it should be reverted.
This is a reland of 69982 with an updated unit test.
Review URL: http://codereview.chromium.org/6083005
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@69993 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chrome/renderer/paint_aggregator_unittest.cc')
-rw-r--r-- | chrome/renderer/paint_aggregator_unittest.cc | 7 |
1 files changed, 5 insertions, 2 deletions
diff --git a/chrome/renderer/paint_aggregator_unittest.cc b/chrome/renderer/paint_aggregator_unittest.cc index 3473dea..03f605f 100644 --- a/chrome/renderer/paint_aggregator_unittest.cc +++ b/chrome/renderer/paint_aggregator_unittest.cc @@ -50,8 +50,11 @@ TEST(PaintAggregator, DoubleDisjointInvalidation) { TEST(PaintAggregator, DisjointInvalidationsCombined) { PaintAggregator greg; - gfx::Rect r1(2, 4, 2, 2); - gfx::Rect r2(4, 2, 2, 2); + // Make the rectangles such that they don't overlap but cover a very large + // percentage of the area of covered by their union. This is so we're not + // very sensitive to the combining heuristic in the paint aggregator. + gfx::Rect r1(2, 4, 2, 1000); + gfx::Rect r2(5, 2, 2, 1000); greg.InvalidateRect(r1); greg.InvalidateRect(r2); |