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

Unified Diff: cc/trees/layer_tree_host_unittest.cc

Issue 767083005: cc: Rename TileManager::ManageTiles to PrepareTiles. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 6 years 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/trees/layer_tree_host_unittest.cc
diff --git a/cc/trees/layer_tree_host_unittest.cc b/cc/trees/layer_tree_host_unittest.cc
index 2a1915c5259471196d55d61c90a3442863f2229e..ab0da37b54df8f25e9a99ccd9ecf340f5be2c5b4 100644
--- a/cc/trees/layer_tree_host_unittest.cc
+++ b/cc/trees/layer_tree_host_unittest.cc
@@ -5934,10 +5934,11 @@ class LayerTreeHostTestContinuousDrawWhenCreatingVisibleTiles
MULTI_THREAD_TEST_F(LayerTreeHostTestContinuousDrawWhenCreatingVisibleTiles);
-class LayerTreeHostTestOneActivatePerManageTiles : public LayerTreeHostTest {
+class LayerTreeHostTestOneActivatePerPrepareTiles : public LayerTreeHostTest {
public:
- LayerTreeHostTestOneActivatePerManageTiles()
- : notify_ready_to_activate_count_(0u), scheduled_manage_tiles_count_(0) {}
+ LayerTreeHostTestOneActivatePerPrepareTiles()
+ : notify_ready_to_activate_count_(0u),
+ scheduled_prepare_tiles_count_(0) {}
void SetupTree() override {
client_.set_fill_with_nonsolid_color(true);
@@ -5966,27 +5967,27 @@ class LayerTreeHostTestOneActivatePerManageTiles : public LayerTreeHostTest {
EndTestAfterDelayMs(100);
}
- void ScheduledActionManageTiles() override {
- ++scheduled_manage_tiles_count_;
+ void ScheduledActionPrepareTiles() override {
+ ++scheduled_prepare_tiles_count_;
}
void AfterTest() override {
- // Expect at most a notification for each scheduled manage tiles, plus one
+ // Expect at most a notification for each scheduled prepare tiles, plus one
// for the initial commit (which doesn't go through scheduled actions).
// The reason this is not an equality is because depending on timing, we
- // might get a manage tiles but not yet get a notification that we're
+ // might get a prepare tiles but not yet get a notification that we're
// ready to activate. The intent of a test is to ensure that we don't
- // get more than one notification per manage tiles, so this is OK.
+ // get more than one notification per prepare tiles, so this is OK.
EXPECT_LE(notify_ready_to_activate_count_,
- 1u + scheduled_manage_tiles_count_);
+ 1u + scheduled_prepare_tiles_count_);
}
protected:
FakeContentLayerClient client_;
size_t notify_ready_to_activate_count_;
- size_t scheduled_manage_tiles_count_;
+ size_t scheduled_prepare_tiles_count_;
};
-MULTI_THREAD_IMPL_TEST_F(LayerTreeHostTestOneActivatePerManageTiles);
+MULTI_THREAD_IMPL_TEST_F(LayerTreeHostTestOneActivatePerPrepareTiles);
} // namespace cc

Powered by Google App Engine
This is Rietveld 408576698