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

Unified Diff: cc/layers/picture_layer_impl_unittest.cc

Issue 563723004: cc: Make low res tilings off by default in LayerTreeSettings. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 6 years, 3 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 | « no previous file | cc/resources/tile_manager_unittest.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: cc/layers/picture_layer_impl_unittest.cc
diff --git a/cc/layers/picture_layer_impl_unittest.cc b/cc/layers/picture_layer_impl_unittest.cc
index 44662d7b2baaec06c2b980d211971ecbc7c7b3b1..d922966854a9cd178da94c8fb28632675019390e 100644
--- a/cc/layers/picture_layer_impl_unittest.cc
+++ b/cc/layers/picture_layer_impl_unittest.cc
@@ -46,13 +46,18 @@ class MockCanvas : public SkCanvas {
std::vector<SkRect> rects_;
};
+class NoLowResTilingsSettings : public ImplSidePaintingSettings {};
+
+class LowResTilingsSettings : public ImplSidePaintingSettings {
+ public:
+ LowResTilingsSettings() { create_low_res_tiling = true; }
+};
+
class PictureLayerImplTest : public testing::Test {
public:
PictureLayerImplTest()
: proxy_(base::MessageLoopProxy::current()),
- host_impl_(ImplSidePaintingSettings(),
- &proxy_,
- &shared_bitmap_manager_),
+ host_impl_(LowResTilingsSettings(), &proxy_, &shared_bitmap_manager_),
id_(7),
pending_layer_(NULL),
old_pending_layer_(NULL),
@@ -2992,11 +2997,6 @@ TEST_F(PictureLayerImplTest, SharedActiveHighResReadyNotEnoughToActivate) {
EXPECT_TRUE(pending_layer_->AllTilesRequiredForActivationAreReadyToDraw());
}
-class NoLowResTilingsSettings : public ImplSidePaintingSettings {
- public:
- NoLowResTilingsSettings() { create_low_res_tiling = false; }
-};
-
class NoLowResPictureLayerImplTest : public PictureLayerImplTest {
public:
NoLowResPictureLayerImplTest()
@@ -3582,7 +3582,7 @@ TEST_F(PictureLayerImplTestWithDelegatingRenderer,
render_pass->quad_list.back()->material);
}
-class OcclusionTrackingSettings : public ImplSidePaintingSettings {
+class OcclusionTrackingSettings : public LowResTilingsSettings {
public:
OcclusionTrackingSettings() { use_occlusion_for_tile_prioritization = true; }
};
« no previous file with comments | « no previous file | cc/resources/tile_manager_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698