Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(3537)

Issues Search
    My Issues | Starred     Open | Closed | All

Unified Diff: cc/playback/display_list_raster_source_unittest.cc

Issue 1531013004: cc: Do solid color analysis before scheduling tiles. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: add test. Created 4 years, 11 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« cc/playback/display_list_raster_source.h ('K') | « cc/playback/display_list_raster_source.cc ('k') | cc/raster/tile_task_worker_pool_unittest.cc » ('j') | cc/tiles/tile_manager_unittest.cc » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: cc/playback/display_list_raster_source_unittest.cc
diff --git a/cc/playback/display_list_raster_source_unittest.cc b/cc/playback/display_list_raster_source_unittest.cc
index c7b3af9706144e799ac1ed121ed4d3dcf255f117..ba8ba7fb87d32de20514a414b34c209fe965d648 100644
--- a/cc/playback/display_list_raster_source_unittest.cc
+++ b/cc/playback/display_list_raster_source_unittest.cc
@@ -27,7 +27,9 @@ TEST(DisplayListRasterSourceTest, AnalyzeIsSolidUnscaled) {
solid_paint.setColor(solid_color);
SkColor non_solid_color = SkColorSetARGB(128, 45, 56, 67);
+ SkColor color = SK_ColorTRANSPARENT;
SkPaint non_solid_paint;
+ bool is_solid_color = false;
non_solid_paint.setColor(non_solid_color);
recording_source->add_draw_rect_with_paint(gfx::Rect(layer_bounds),
@@ -41,11 +43,10 @@ TEST(DisplayListRasterSourceTest, AnalyzeIsSolidUnscaled) {
// Ensure everything is solid.
for (int y = 0; y <= 300; y += 100) {
for (int x = 0; x <= 300; x += 100) {
- DisplayListRasterSource::SolidColorAnalysis analysis;
gfx::Rect rect(x, y, 100, 100);
- raster->PerformSolidColorAnalysis(rect, 1.0, &analysis);
- EXPECT_TRUE(analysis.is_solid_color) << rect.ToString();
- EXPECT_EQ(solid_color, analysis.solid_color) << rect.ToString();
+ is_solid_color = raster->PerformSolidColorAnalysis(rect, 1.0, &color);
+ EXPECT_TRUE(is_solid_color) << rect.ToString();
+ EXPECT_EQ(solid_color, color) << rect.ToString();
}
}
@@ -56,33 +57,39 @@ TEST(DisplayListRasterSourceTest, AnalyzeIsSolidUnscaled) {
raster = DisplayListRasterSource::CreateFromDisplayListRecordingSource(
recording_source.get(), false);
- DisplayListRasterSource::SolidColorAnalysis analysis;
- raster->PerformSolidColorAnalysis(gfx::Rect(0, 0, 100, 100), 1.0, &analysis);
- EXPECT_FALSE(analysis.is_solid_color);
+ color = SK_ColorTRANSPARENT;
+ is_solid_color =
+ raster->PerformSolidColorAnalysis(gfx::Rect(0, 0, 100, 100), 1.0, &color);
+ EXPECT_FALSE(is_solid_color);
- raster->PerformSolidColorAnalysis(gfx::Rect(100, 0, 100, 100), 1.0,
- &analysis);
- EXPECT_TRUE(analysis.is_solid_color);
- EXPECT_EQ(solid_color, analysis.solid_color);
+ is_solid_color = false;
+ color = SK_ColorTRANSPARENT;
+ is_solid_color = raster->PerformSolidColorAnalysis(
+ gfx::Rect(100, 0, 100, 100), 1.0, &color);
+ EXPECT_TRUE(is_solid_color);
+ EXPECT_EQ(solid_color, color);
// Boundaries should be clipped.
- analysis.is_solid_color = false;
- raster->PerformSolidColorAnalysis(gfx::Rect(350, 0, 100, 100), 1.0,
- &analysis);
- EXPECT_TRUE(analysis.is_solid_color);
- EXPECT_EQ(solid_color, analysis.solid_color);
-
- analysis.is_solid_color = false;
- raster->PerformSolidColorAnalysis(gfx::Rect(0, 350, 100, 100), 1.0,
- &analysis);
- EXPECT_TRUE(analysis.is_solid_color);
- EXPECT_EQ(solid_color, analysis.solid_color);
-
- analysis.is_solid_color = false;
- raster->PerformSolidColorAnalysis(gfx::Rect(350, 350, 100, 100), 1.0,
- &analysis);
- EXPECT_TRUE(analysis.is_solid_color);
- EXPECT_EQ(solid_color, analysis.solid_color);
+ is_solid_color = false;
+ color = SK_ColorTRANSPARENT;
+ is_solid_color = raster->PerformSolidColorAnalysis(
+ gfx::Rect(350, 0, 100, 100), 1.0, &color);
+ EXPECT_TRUE(is_solid_color);
+ EXPECT_EQ(solid_color, color);
+
+ is_solid_color = false;
+ color = SK_ColorTRANSPARENT;
+ is_solid_color = raster->PerformSolidColorAnalysis(
+ gfx::Rect(0, 350, 100, 100), 1.0, &color);
+ EXPECT_TRUE(is_solid_color);
+ EXPECT_EQ(solid_color, color);
+
+ is_solid_color = false;
+ color = SK_ColorTRANSPARENT;
+ is_solid_color = raster->PerformSolidColorAnalysis(
+ gfx::Rect(350, 350, 100, 100), 1.0, &color);
+ EXPECT_TRUE(is_solid_color);
+ EXPECT_EQ(solid_color, color);
}
TEST(DisplayListRasterSourceTest, AnalyzeIsSolidScaled) {
@@ -92,7 +99,9 @@ TEST(DisplayListRasterSourceTest, AnalyzeIsSolidScaled) {
FakeDisplayListRecordingSource::CreateFilledRecordingSource(layer_bounds);
SkColor solid_color = SkColorSetARGB(255, 12, 23, 34);
+ SkColor color = SK_ColorTRANSPARENT;
SkPaint solid_paint;
+ bool is_solid_color = false;
solid_paint.setColor(solid_color);
SkColor non_solid_color = SkColorSetARGB(128, 45, 56, 67);
@@ -110,11 +119,10 @@ TEST(DisplayListRasterSourceTest, AnalyzeIsSolidScaled) {
// Ensure everything is solid.
for (int y = 0; y <= 30; y += 10) {
for (int x = 0; x <= 30; x += 10) {
- DisplayListRasterSource::SolidColorAnalysis analysis;
gfx::Rect rect(x, y, 10, 10);
- raster->PerformSolidColorAnalysis(rect, 0.1f, &analysis);
- EXPECT_TRUE(analysis.is_solid_color) << rect.ToString();
- EXPECT_EQ(analysis.solid_color, solid_color) << rect.ToString();
+ is_solid_color = raster->PerformSolidColorAnalysis(rect, 0.1f, &color);
+ EXPECT_TRUE(is_solid_color) << rect.ToString();
+ EXPECT_EQ(color, solid_color) << rect.ToString();
}
}
@@ -125,29 +133,40 @@ TEST(DisplayListRasterSourceTest, AnalyzeIsSolidScaled) {
raster = DisplayListRasterSource::CreateFromDisplayListRecordingSource(
recording_source.get(), false);
- DisplayListRasterSource::SolidColorAnalysis analysis;
- raster->PerformSolidColorAnalysis(gfx::Rect(0, 0, 10, 10), 0.1f, &analysis);
- EXPECT_FALSE(analysis.is_solid_color);
+ is_solid_color = false;
+ color = SK_ColorTRANSPARENT;
+ is_solid_color =
+ raster->PerformSolidColorAnalysis(gfx::Rect(0, 0, 10, 10), 0.1f, &color);
+ EXPECT_FALSE(is_solid_color);
- raster->PerformSolidColorAnalysis(gfx::Rect(10, 0, 10, 10), 0.1f, &analysis);
- EXPECT_TRUE(analysis.is_solid_color);
- EXPECT_EQ(analysis.solid_color, solid_color);
+ is_solid_color = false;
+ color = SK_ColorTRANSPARENT;
+ is_solid_color =
+ raster->PerformSolidColorAnalysis(gfx::Rect(10, 0, 10, 10), 0.1f, &color);
+ EXPECT_TRUE(is_solid_color);
+ EXPECT_EQ(color, solid_color);
// Boundaries should be clipped.
- analysis.is_solid_color = false;
- raster->PerformSolidColorAnalysis(gfx::Rect(35, 0, 10, 10), 0.1f, &analysis);
- EXPECT_TRUE(analysis.is_solid_color);
- EXPECT_EQ(analysis.solid_color, solid_color);
-
- analysis.is_solid_color = false;
- raster->PerformSolidColorAnalysis(gfx::Rect(0, 35, 10, 10), 0.1f, &analysis);
- EXPECT_TRUE(analysis.is_solid_color);
- EXPECT_EQ(analysis.solid_color, solid_color);
-
- analysis.is_solid_color = false;
- raster->PerformSolidColorAnalysis(gfx::Rect(35, 35, 10, 10), 0.1f, &analysis);
- EXPECT_TRUE(analysis.is_solid_color);
- EXPECT_EQ(analysis.solid_color, solid_color);
+ is_solid_color = false;
+ color = SK_ColorTRANSPARENT;
+ is_solid_color =
+ raster->PerformSolidColorAnalysis(gfx::Rect(35, 0, 10, 10), 0.1f, &color);
+ EXPECT_TRUE(is_solid_color);
+ EXPECT_EQ(color, solid_color);
+
+ is_solid_color = false;
+ color = SK_ColorTRANSPARENT;
+ is_solid_color =
+ raster->PerformSolidColorAnalysis(gfx::Rect(0, 35, 10, 10), 0.1f, &color);
+ EXPECT_TRUE(is_solid_color);
+ EXPECT_EQ(color, solid_color);
+
+ is_solid_color = false;
+ color = SK_ColorTRANSPARENT;
+ is_solid_color = raster->PerformSolidColorAnalysis(gfx::Rect(35, 35, 10, 10),
+ 0.1f, &color);
+ EXPECT_TRUE(is_solid_color);
+ EXPECT_EQ(color, solid_color);
}
TEST(DisplayListRasterSourceTest, AnalyzeIsSolidEmpty) {
@@ -160,13 +179,13 @@ TEST(DisplayListRasterSourceTest, AnalyzeIsSolidEmpty) {
scoped_refptr<DisplayListRasterSource> raster =
DisplayListRasterSource::CreateFromDisplayListRecordingSource(
recording_source.get(), false);
- DisplayListRasterSource::SolidColorAnalysis analysis;
- EXPECT_FALSE(analysis.is_solid_color);
- raster->PerformSolidColorAnalysis(gfx::Rect(0, 0, 400, 400), 1.f, &analysis);
+ SkColor color = SK_ColorTRANSPARENT;
+ bool is_solid_color =
+ raster->PerformSolidColorAnalysis(gfx::Rect(0, 0, 400, 400), 1.f, &color);
- EXPECT_TRUE(analysis.is_solid_color);
- EXPECT_EQ(analysis.solid_color, SkColorSetARGB(0, 0, 0, 0));
+ EXPECT_TRUE(is_solid_color);
+ EXPECT_EQ(color, SkColorSetARGB(0, 0, 0, 0));
}
TEST(DisplayListRasterSourceTest, PixelRefIteratorDiscardableRefsOneTile) {
« cc/playback/display_list_raster_source.h ('K') | « cc/playback/display_list_raster_source.cc ('k') | cc/raster/tile_task_worker_pool_unittest.cc » ('j') | cc/tiles/tile_manager_unittest.cc » ('J')

Powered by Google App Engine
This is Rietveld 408576698