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

Unified Diff: cc/raster/raster_source_unittest.cc

Issue 2797583002: cc: Add color space to image decode caches (Closed)
Patch Set: Fix perf test compile Created 3 years, 8 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
Index: cc/raster/raster_source_unittest.cc
diff --git a/cc/raster/raster_source_unittest.cc b/cc/raster/raster_source_unittest.cc
index 4ec8c94b6f2d5852b12cd2b7a65cb17bbe093ee2..c9713bb5bc728035ed6e3313bcd3eaaf0f0bbbad 100644
--- a/cc/raster/raster_source_unittest.cc
+++ b/cc/raster/raster_source_unittest.cc
@@ -214,34 +214,45 @@ TEST(RasterSourceTest, PixelRefIteratorDiscardableRefsOneTile) {
// Tile sized iterators. These should find only one pixel ref.
{
+ gfx::ColorSpace target_color_space = gfx::ColorSpace::CreateSRGB();
std::vector<DrawImage> images;
- raster->GetDiscardableImagesInRect(gfx::Rect(0, 0, 256, 256), 1.f, &images);
+ raster->GetDiscardableImagesInRect(gfx::Rect(0, 0, 256, 256), 1.f,
+ target_color_space, &images);
EXPECT_EQ(1u, images.size());
EXPECT_EQ(discardable_image[0][0], images[0].image());
+ EXPECT_EQ(target_color_space, images[0].target_color_space());
}
// Shifted tile sized iterators. These should find only one pixel ref.
{
+ gfx::ColorSpace target_color_space = gfx::ColorSpace::CreateXYZD50();
std::vector<DrawImage> images;
raster->GetDiscardableImagesInRect(gfx::Rect(260, 260, 256, 256), 1.f,
- &images);
+ target_color_space, &images);
EXPECT_EQ(1u, images.size());
EXPECT_EQ(discardable_image[1][1], images[0].image());
+ EXPECT_EQ(target_color_space, images[0].target_color_space());
}
// Ensure there's no discardable pixel refs in the empty cell
{
+ gfx::ColorSpace target_color_space = gfx::ColorSpace::CreateSRGB();
std::vector<DrawImage> images;
raster->GetDiscardableImagesInRect(gfx::Rect(0, 256, 256, 256), 1.f,
- &images);
+ target_color_space, &images);
EXPECT_EQ(0u, images.size());
}
// Layer sized iterators. These should find three pixel ref.
{
+ gfx::ColorSpace target_color_space;
std::vector<DrawImage> images;
- raster->GetDiscardableImagesInRect(gfx::Rect(0, 0, 512, 512), 1.f, &images);
+ raster->GetDiscardableImagesInRect(gfx::Rect(0, 0, 512, 512), 1.f,
+ target_color_space, &images);
EXPECT_EQ(3u, images.size());
EXPECT_EQ(discardable_image[0][0], images[0].image());
EXPECT_EQ(discardable_image[0][1], images[1].image());
EXPECT_EQ(discardable_image[1][1], images[2].image());
+ EXPECT_EQ(target_color_space, images[0].target_color_space());
+ EXPECT_EQ(target_color_space, images[1].target_color_space());
+ EXPECT_EQ(target_color_space, images[2].target_color_space());
}
}

Powered by Google App Engine
This is Rietveld 408576698