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

Unified Diff: cc/raster/raster_source_unittest.cc

Issue 2797583002: cc: Add color space to image decode caches (Closed)
Patch Set: Remove dead code 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
« no previous file with comments | « cc/raster/raster_source.cc ('k') | cc/tiles/checker_image_tracker_unittest.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: cc/raster/raster_source_unittest.cc
diff --git a/cc/raster/raster_source_unittest.cc b/cc/raster/raster_source_unittest.cc
index e15107814d6b20bdb0a9bc8e63e791171c618cb6..344205714c4b8b6242f423a01f8ff28d1e2f952b 100644
--- a/cc/raster/raster_source_unittest.cc
+++ b/cc/raster/raster_source_unittest.cc
@@ -215,34 +215,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());
}
}
« no previous file with comments | « cc/raster/raster_source.cc ('k') | cc/tiles/checker_image_tracker_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698