diff options
author | ernstm@chromium.org <ernstm@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2013-07-15 23:33:29 +0000 |
---|---|---|
committer | ernstm@chromium.org <ernstm@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2013-07-15 23:33:29 +0000 |
commit | 5c6739c1645ebbd84bb1e39c09a6fd12d4d0256d (patch) | |
tree | e0fcba5345cc770a671b5cca4ac42153354b3204 /cc/resources/picture_pile_unittest.cc | |
parent | b614d2e4a14684973b534043d52651410bed6c2b (diff) | |
download | chromium_src-5c6739c1645ebbd84bb1e39c09a6fd12d4d0256d.zip chromium_src-5c6739c1645ebbd84bb1e39c09a6fd12d4d0256d.tar.gz chromium_src-5c6739c1645ebbd84bb1e39c09a6fd12d4d0256d.tar.bz2 |
cc: Increased robustness of rasterize and record benchmark.
- 'protected' relevent trace events from changes, by putting argument strings
as constants into separate file.
- fixed a bug in the collection of raster times, that would output wrong
statistics for layers with multiple pictures in a pile.
- Added command line parameters to set number of raster and record repeats,
wait time before starting the benchmark, and wait time before taking the
measurements.
- Reduced the default number of raster and record repeats from 100 to 20.
This increases variance, but improves overall robustness (e.g. avoiding
event buffer overflows)
- Added command line option to print statistics after every page run.
R=nduca@chromium.org,tengs@chromium.org
BUG=226489
Review URL: https://chromiumcodereview.appspot.com/18048006
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@211728 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'cc/resources/picture_pile_unittest.cc')
-rw-r--r-- | cc/resources/picture_pile_unittest.cc | 18 |
1 files changed, 6 insertions, 12 deletions
diff --git a/cc/resources/picture_pile_unittest.cc b/cc/resources/picture_pile_unittest.cc index 0ed19f9..1eeb5ad 100644 --- a/cc/resources/picture_pile_unittest.cc +++ b/cc/resources/picture_pile_unittest.cc @@ -46,8 +46,7 @@ TEST(PicturePileTest, SmallInvalidateInflated) { false, gfx::Rect(layer_size), gfx::Rect(layer_size), - &stats_instrumentation, - 0); + &stats_instrumentation); // Invalidate something inside a tile. gfx::Rect invalidate_rect(50, 50, 1, 1); @@ -56,8 +55,7 @@ TEST(PicturePileTest, SmallInvalidateInflated) { false, invalidate_rect, gfx::Rect(layer_size), - &stats_instrumentation, - 0); + &stats_instrumentation); EXPECT_EQ(1, pile->tiling().num_tiles_x()); EXPECT_EQ(1, pile->tiling().num_tiles_y()); @@ -100,8 +98,7 @@ TEST(PicturePileTest, LargeInvalidateInflated) { false, gfx::Rect(layer_size), gfx::Rect(layer_size), - &stats_instrumentation, - 0); + &stats_instrumentation); // Invalidate something inside a tile. gfx::Rect invalidate_rect(50, 50, 100, 100); @@ -110,8 +107,7 @@ TEST(PicturePileTest, LargeInvalidateInflated) { false, invalidate_rect, gfx::Rect(layer_size), - &stats_instrumentation, - 0); + &stats_instrumentation); EXPECT_EQ(1, pile->tiling().num_tiles_x()); EXPECT_EQ(1, pile->tiling().num_tiles_y()); @@ -165,8 +161,7 @@ TEST(PicturePileTest, InvalidateOnTileBoundaryInflated) { false, gfx::Rect(layer_size), gfx::Rect(layer_size), - &stats_instrumentation, - 0); + &stats_instrumentation); // Invalidate something just over a tile boundary by a single pixel. // This will invalidate the tile (1, 1), as well as 1 row of pixels in (1, 0). @@ -180,8 +175,7 @@ TEST(PicturePileTest, InvalidateOnTileBoundaryInflated) { false, invalidate_rect, gfx::Rect(layer_size), - &stats_instrumentation, - 0); + &stats_instrumentation); for (int i = 0; i < pile->tiling().num_tiles_x(); ++i) { for (int j = 0; j < pile->tiling().num_tiles_y(); ++j) { |