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

Unified Diff: cc/trees/layer_tree_host_unittest.cc

Issue 787803004: Update from https://crrev.com/307664 (Closed) Base URL: https://github.com/domokit/mojo.git@master
Patch Set: Rebase. 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
« no previous file with comments | « cc/trees/layer_tree_host_impl_unittest.cc ('k') | cc/trees/layer_tree_host_unittest_no_message_loop.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
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 d70d83223c4cc4d14773db2b58969a818dc33fc4..06c522ad2d49dd0917529f6381897b20621c9aee 100644
--- a/cc/trees/layer_tree_host_unittest.cc
+++ b/cc/trees/layer_tree_host_unittest.cc
@@ -5496,11 +5496,11 @@ class LayerTreeHostAcceptsDeltasFromImplWithoutRootLayer
EndTest();
}
- void ApplyViewportDeltas(
- const gfx::Vector2d& inner,
- const gfx::Vector2d& outer,
- float scale_delta,
- float top_controls_delta) override {
+ void ApplyViewportDeltas(const gfx::Vector2d& inner,
+ const gfx::Vector2d& outer,
+ const gfx::Vector2dF& elastic_overscroll_delta,
+ float scale_delta,
+ float top_controls_delta) override {
EXPECT_EQ(info_.page_scale_delta, scale_delta);
EXPECT_EQ(info_.top_controls_delta, top_controls_delta);
deltas_sent_to_client_ = true;
@@ -5910,10 +5910,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);
@@ -5942,27 +5943,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
« no previous file with comments | « cc/trees/layer_tree_host_impl_unittest.cc ('k') | cc/trees/layer_tree_host_unittest_no_message_loop.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698