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

Issues Search
    My Issues | Starred     Open | Closed | All

Unified Diff: cc/trees/layer_tree_host_unittest_scroll.cc

Issue 2661523003: cc: Merge LayerTree into the LayerTreeHost. (Closed)
Patch Set: auto Created 3 years, 11 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/trees/layer_tree_host_unittest_proxy.cc ('k') | cc/trees/layer_tree_host_unittest_video.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_scroll.cc
diff --git a/cc/trees/layer_tree_host_unittest_scroll.cc b/cc/trees/layer_tree_host_unittest_scroll.cc
index 37d98c17b2f93eccc0f880a7b60b26210bd4385d..59fbfb6a4f9632c9da3e1f7627aa0f60e6902672 100644
--- a/cc/trees/layer_tree_host_unittest_scroll.cc
+++ b/cc/trees/layer_tree_host_unittest_scroll.cc
@@ -72,7 +72,7 @@ class LayerTreeHostScrollTest : public LayerTreeTest {
protected:
void SetupTree() override {
LayerTreeTest::SetupTree();
- Layer* root_layer = layer_tree()->root_layer();
+ Layer* root_layer = layer_tree_host()->root_layer();
// Create an effective max_scroll_offset of (100, 100).
gfx::Size scroll_layer_bounds(root_layer->bounds().width() + 100,
@@ -93,18 +93,20 @@ class LayerTreeHostScrollTestScrollSimple : public LayerTreeHostScrollTest {
num_scrolls_(0) {}
void BeginTest() override {
- outer_viewport_container_layer_id_ =
- layer_tree()->outer_viewport_scroll_layer()->scroll_clip_layer()->id();
- layer_tree()->outer_viewport_scroll_layer()->SetScrollOffset(
+ outer_viewport_container_layer_id_ = layer_tree_host()
+ ->outer_viewport_scroll_layer()
+ ->scroll_clip_layer()
+ ->id();
+ layer_tree_host()->outer_viewport_scroll_layer()->SetScrollOffset(
initial_scroll_);
- layer_tree()->outer_viewport_scroll_layer()->set_did_scroll_callback(
+ layer_tree_host()->outer_viewport_scroll_layer()->set_did_scroll_callback(
base::Bind(&LayerTreeHostScrollTestScrollSimple::DidScrollOuterViewport,
base::Unretained(this)));
PostSetNeedsCommitToMainThread();
}
void UpdateLayerTreeHost() override {
- Layer* scroll_layer = layer_tree()->outer_viewport_scroll_layer();
+ Layer* scroll_layer = layer_tree_host()->outer_viewport_scroll_layer();
if (!layer_tree_host()->SourceFrameNumber()) {
EXPECT_VECTOR_EQ(initial_scroll_, scroll_layer->scroll_offset());
} else {
@@ -166,7 +168,7 @@ class LayerTreeHostScrollTestScrollMultipleRedraw
: initial_scroll_(40, 10), scroll_amount_(-3, 17), num_scrolls_(0) {}
void BeginTest() override {
- scroll_layer_ = layer_tree()->outer_viewport_scroll_layer();
+ scroll_layer_ = layer_tree_host()->outer_viewport_scroll_layer();
scroll_layer_->SetScrollOffset(initial_scroll_);
scroll_layer_->set_did_scroll_callback(base::Bind(
&LayerTreeHostScrollTestScrollMultipleRedraw::DidScrollOuterViewport,
@@ -257,9 +259,9 @@ class LayerTreeHostScrollTestScrollAbortedCommit
num_impl_scrolls_(0) {}
void BeginTest() override {
- layer_tree()->outer_viewport_scroll_layer()->SetScrollOffset(
+ layer_tree_host()->outer_viewport_scroll_layer()->SetScrollOffset(
initial_scroll_);
- layer_tree()->outer_viewport_scroll_layer()->set_did_scroll_callback(
+ layer_tree_host()->outer_viewport_scroll_layer()->set_did_scroll_callback(
base::Bind(
&LayerTreeHostScrollTestScrollAbortedCommit::DidScrollOuterViewport,
base::Unretained(this)));
@@ -270,20 +272,21 @@ class LayerTreeHostScrollTestScrollAbortedCommit
LayerTreeHostScrollTest::SetupTree();
gfx::Size scroll_layer_bounds(200, 200);
- layer_tree()->outer_viewport_scroll_layer()->SetBounds(scroll_layer_bounds);
- layer_tree()->SetPageScaleFactorAndLimits(1.f, 0.01f, 100.f);
+ layer_tree_host()->outer_viewport_scroll_layer()->SetBounds(
+ scroll_layer_bounds);
+ layer_tree_host()->SetPageScaleFactorAndLimits(1.f, 0.01f, 100.f);
}
void WillBeginMainFrame() override {
num_will_begin_main_frames_++;
- Layer* root_scroll_layer = layer_tree()->outer_viewport_scroll_layer();
+ Layer* root_scroll_layer = layer_tree_host()->outer_viewport_scroll_layer();
switch (num_will_begin_main_frames_) {
case 1:
// This will not be aborted because of the initial prop changes.
EXPECT_EQ(0, num_impl_scrolls_);
EXPECT_EQ(0, layer_tree_host()->SourceFrameNumber());
EXPECT_VECTOR_EQ(initial_scroll_, root_scroll_layer->scroll_offset());
- EXPECT_EQ(1.f, layer_tree()->page_scale_factor());
+ EXPECT_EQ(1.f, layer_tree_host()->page_scale_factor());
break;
case 2:
// This commit will be aborted, and another commit will be
@@ -293,7 +296,7 @@ class LayerTreeHostScrollTestScrollAbortedCommit
EXPECT_VECTOR_EQ(
gfx::ScrollOffsetWithDelta(initial_scroll_, impl_scroll_),
root_scroll_layer->scroll_offset());
- EXPECT_EQ(impl_scale_, layer_tree()->page_scale_factor());
+ EXPECT_EQ(impl_scale_, layer_tree_host()->page_scale_factor());
PostSetNeedsRedrawToMainThread();
break;
case 3:
@@ -305,7 +308,8 @@ class LayerTreeHostScrollTestScrollAbortedCommit
gfx::ScrollOffsetWithDelta(initial_scroll_,
impl_scroll_ + impl_scroll_),
root_scroll_layer->scroll_offset());
- EXPECT_EQ(impl_scale_ * impl_scale_, layer_tree()->page_scale_factor());
+ EXPECT_EQ(impl_scale_ * impl_scale_,
+ layer_tree_host()->page_scale_factor());
root_scroll_layer->SetScrollOffset(gfx::ScrollOffsetWithDelta(
root_scroll_layer->scroll_offset(), second_main_scroll_));
break;
@@ -442,7 +446,7 @@ class LayerTreeHostScrollTestFractionalScroll : public LayerTreeHostScrollTest {
void SetupTree() override {
LayerTreeHostScrollTest::SetupTree();
- layer_tree()->SetPageScaleFactorAndLimits(1.f, 0.01f, 100.f);
+ layer_tree_host()->SetPageScaleFactorAndLimits(1.f, 0.01f, 100.f);
}
void BeginTest() override { PostSetNeedsCommitToMainThread(); }
@@ -498,17 +502,17 @@ class LayerTreeHostScrollTestScrollSnapping : public LayerTreeHostScrollTest {
void SetupTree() override {
LayerTreeHostScrollTest::SetupTree();
- layer_tree()
+ layer_tree_host()
->outer_viewport_scroll_layer()
->scroll_clip_layer()
->SetForceRenderSurfaceForTesting(true);
gfx::Transform translate;
translate.Translate(0.25f, 0.f);
- layer_tree()
+ layer_tree_host()
->outer_viewport_scroll_layer()
->scroll_clip_layer()
->SetTransform(translate);
- layer_tree()->SetPageScaleFactorAndLimits(1.f, 0.1f, 100.f);
+ layer_tree_host()->SetPageScaleFactorAndLimits(1.f, 0.1f, 100.f);
}
void BeginTest() override { PostSetNeedsCommitToMainThread(); }
@@ -558,7 +562,7 @@ class LayerTreeHostScrollTestCaseWithChild : public LayerTreeHostScrollTest {
num_scrolls_(0) {}
void SetupTree() override {
- layer_tree()->SetDeviceScaleFactor(device_scale_factor_);
+ layer_tree_host()->SetDeviceScaleFactor(device_scale_factor_);
scoped_refptr<Layer> root_layer = Layer::Create();
root_layer->SetBounds(gfx::Size(10, 10));
@@ -589,7 +593,7 @@ class LayerTreeHostScrollTestCaseWithChild : public LayerTreeHostScrollTest {
}
scoped_refptr<Layer> outer_container_layer =
- layer_tree()->outer_viewport_scroll_layer()->parent();
+ layer_tree_host()->outer_viewport_scroll_layer()->parent();
child_layer_->SetIsDrawable(true);
child_layer_->SetScrollClipLayerId(outer_container_layer->id());
@@ -606,11 +610,11 @@ class LayerTreeHostScrollTestCaseWithChild : public LayerTreeHostScrollTest {
expected_scroll_layer_->SetScrollOffset(initial_offset_);
- layer_tree()->SetRootLayer(root_layer);
+ layer_tree_host()->SetRootLayer(root_layer);
LayerTreeTest::SetupTree();
fake_content_layer_client_.set_bounds(root_layer->bounds());
- layer_tree()->outer_viewport_scroll_layer()->set_did_scroll_callback(
+ layer_tree_host()->outer_viewport_scroll_layer()->set_did_scroll_callback(
base::Bind(
&LayerTreeHostScrollTestCaseWithChild::DidScrollOuterViewport,
base::Unretained(this)));
@@ -823,20 +827,20 @@ class LayerTreeHostScrollTestSimple : public LayerTreeHostScrollTest {
void SetupTree() override {
LayerTreeHostScrollTest::SetupTree();
- layer_tree()->SetPageScaleFactorAndLimits(1.f, 0.01f, 100.f);
+ layer_tree_host()->SetPageScaleFactorAndLimits(1.f, 0.01f, 100.f);
}
void BeginTest() override {
- layer_tree()->outer_viewport_scroll_layer()->SetScrollOffset(
+ layer_tree_host()->outer_viewport_scroll_layer()->SetScrollOffset(
initial_scroll_);
- layer_tree()->outer_viewport_scroll_layer()->set_did_scroll_callback(
+ layer_tree_host()->outer_viewport_scroll_layer()->set_did_scroll_callback(
base::Bind(&LayerTreeHostScrollTestSimple::DidScrollOuterViewport,
base::Unretained(this)));
PostSetNeedsCommitToMainThread();
}
void UpdateLayerTreeHost() override {
- Layer* scroll_layer = layer_tree()->outer_viewport_scroll_layer();
+ Layer* scroll_layer = layer_tree_host()->outer_viewport_scroll_layer();
if (!layer_tree_host()->SourceFrameNumber()) {
EXPECT_VECTOR_EQ(initial_scroll_, scroll_layer->scroll_offset());
} else {
@@ -945,21 +949,21 @@ class LayerTreeHostScrollTestImplOnlyScroll : public LayerTreeHostScrollTest {
void SetupTree() override {
LayerTreeHostScrollTest::SetupTree();
- layer_tree()->SetPageScaleFactorAndLimits(1.f, 0.01f, 100.f);
+ layer_tree_host()->SetPageScaleFactorAndLimits(1.f, 0.01f, 100.f);
}
void BeginTest() override {
- layer_tree()->outer_viewport_scroll_layer()->SetScrollOffset(
+ layer_tree_host()->outer_viewport_scroll_layer()->SetScrollOffset(
initial_scroll_);
PostSetNeedsCommitToMainThread();
}
void WillCommit() override {
- Layer* scroll_layer = layer_tree()->outer_viewport_scroll_layer();
+ Layer* scroll_layer = layer_tree_host()->outer_viewport_scroll_layer();
switch (layer_tree_host()->SourceFrameNumber()) {
case 0:
EXPECT_TRUE(
- scroll_layer->GetLayerTree()->LayerNeedsPushPropertiesForTesting(
+ scroll_layer->layer_tree_host()->LayerNeedsPushPropertiesForTesting(
scroll_layer));
break;
case 1:
@@ -967,7 +971,7 @@ class LayerTreeHostScrollTestImplOnlyScroll : public LayerTreeHostScrollTest {
// still pick up scrolls that happen on the active layer during
// commit.
EXPECT_FALSE(
- scroll_layer->GetLayerTree()->LayerNeedsPushPropertiesForTesting(
+ scroll_layer->layer_tree_host()->LayerNeedsPushPropertiesForTesting(
scroll_layer));
break;
}
@@ -1085,14 +1089,16 @@ class LayerTreeHostScrollTestScrollZeroMaxScrollOffset
LayerTreeHostScrollTestScrollZeroMaxScrollOffset() {}
void BeginTest() override {
- outer_viewport_container_layer_id_ =
- layer_tree()->outer_viewport_scroll_layer()->scroll_clip_layer()->id();
+ outer_viewport_container_layer_id_ = layer_tree_host()
+ ->outer_viewport_scroll_layer()
+ ->scroll_clip_layer()
+ ->id();
PostSetNeedsCommitToMainThread();
}
void UpdateLayerTreeHost() override {
- Layer* root = layer_tree()->root_layer();
- Layer* scroll_layer = layer_tree()->outer_viewport_scroll_layer();
+ Layer* root = layer_tree_host()->root_layer();
+ Layer* scroll_layer = layer_tree_host()->outer_viewport_scroll_layer();
switch (layer_tree_host()->SourceFrameNumber()) {
case 0:
scroll_layer->SetScrollClipLayerId(outer_viewport_container_layer_id_);
@@ -1161,11 +1167,12 @@ class LayerTreeHostScrollTestScrollNonDrawnLayer
void SetupTree() override {
LayerTreeHostScrollTest::SetupTree();
- layer_tree()->outer_viewport_scroll_layer()->SetIsDrawable(false);
- layer_tree()->outer_viewport_scroll_layer()->SetScrollOffset(
+ layer_tree_host()->outer_viewport_scroll_layer()->SetIsDrawable(false);
+ layer_tree_host()->outer_viewport_scroll_layer()->SetScrollOffset(
gfx::ScrollOffset(20.f, 20.f));
- layer_tree()->outer_viewport_scroll_layer()->SetNonFastScrollableRegion(
- gfx::Rect(20, 20, 20, 20));
+ layer_tree_host()
+ ->outer_viewport_scroll_layer()
+ ->SetNonFastScrollableRegion(gfx::Rect(20, 20, 20, 20));
}
void DrawLayersOnThread(LayerTreeHostImpl* impl) override {
@@ -1210,8 +1217,10 @@ class LayerTreeHostScrollTestImplScrollUnderMainThreadScrollingParent
void SetupTree() override {
LayerTreeHostScrollTest::SetupTree();
- layer_tree()->inner_viewport_scroll_layer()->AddMainThreadScrollingReasons(
- MainThreadScrollingReason::kNonFastScrollableRegion);
+ layer_tree_host()
+ ->inner_viewport_scroll_layer()
+ ->AddMainThreadScrollingReasons(
+ MainThreadScrollingReason::kNonFastScrollableRegion);
}
void DrawLayersOnThread(LayerTreeHostImpl* impl) override {
@@ -1352,14 +1361,15 @@ class LayerTreeHostScrollTestLayerStructureChange
void SetupTree() override {
LayerTreeTest::SetupTree();
- Layer* root_layer = layer_tree()->root_layer();
+ Layer* root_layer = layer_tree_host()->root_layer();
root_layer->SetBounds(gfx::Size(10, 10));
CreateVirtualViewportLayers(root_layer, root_layer->bounds(),
root_layer->bounds(), root_layer->bounds(),
layer_tree_host());
- Layer* outer_scroll_layer = layer_tree()->outer_viewport_scroll_layer();
+ Layer* outer_scroll_layer =
+ layer_tree_host()->outer_viewport_scroll_layer();
Layer* root_scroll_layer =
CreateScrollLayer(outer_scroll_layer, &root_scroll_layer_client_);
@@ -1399,8 +1409,8 @@ class LayerTreeHostScrollTestLayerStructureChange
virtual void DidScroll(Layer* layer) {
if (scroll_destroy_whole_tree_) {
- layer_tree()->RegisterViewportLayers(NULL, NULL, NULL, NULL);
- layer_tree()->SetRootLayer(NULL);
+ layer_tree_host()->RegisterViewportLayers(NULL, NULL, NULL, NULL);
+ layer_tree_host()->SetRootLayer(NULL);
EndTest();
return;
}
@@ -1479,11 +1489,13 @@ class LayerTreeHostScrollTestScrollMFBA : public LayerTreeHostScrollTest {
}
void BeginTest() override {
- outer_viewport_container_layer_id_ =
- layer_tree()->outer_viewport_scroll_layer()->scroll_clip_layer()->id();
- layer_tree()->outer_viewport_scroll_layer()->SetScrollOffset(
+ outer_viewport_container_layer_id_ = layer_tree_host()
+ ->outer_viewport_scroll_layer()
+ ->scroll_clip_layer()
+ ->id();
+ layer_tree_host()->outer_viewport_scroll_layer()->SetScrollOffset(
initial_scroll_);
- layer_tree()->outer_viewport_scroll_layer()->set_did_scroll_callback(
+ layer_tree_host()->outer_viewport_scroll_layer()->set_did_scroll_callback(
base::Bind(&LayerTreeHostScrollTestScrollMFBA::DidScrollOuterViewport,
base::Unretained(this)));
PostSetNeedsCommitToMainThread();
@@ -1506,7 +1518,7 @@ class LayerTreeHostScrollTestScrollMFBA : public LayerTreeHostScrollTest {
}
void UpdateLayerTreeHost() override {
- Layer* scroll_layer = layer_tree()->outer_viewport_scroll_layer();
+ Layer* scroll_layer = layer_tree_host()->outer_viewport_scroll_layer();
switch (layer_tree_host()->SourceFrameNumber()) {
case 0:
EXPECT_VECTOR_EQ(initial_scroll_, scroll_layer->scroll_offset());
@@ -1610,9 +1622,9 @@ class LayerTreeHostScrollTestScrollAbortedCommitMFBA
}
void BeginTest() override {
- layer_tree()->outer_viewport_scroll_layer()->SetScrollOffset(
+ layer_tree_host()->outer_viewport_scroll_layer()->SetScrollOffset(
initial_scroll_);
- layer_tree()->outer_viewport_scroll_layer()->set_did_scroll_callback(
+ layer_tree_host()->outer_viewport_scroll_layer()->set_did_scroll_callback(
base::Bind(&LayerTreeHostScrollTestScrollAbortedCommitMFBA::
DidScrollOuterViewport,
base::Unretained(this)));
@@ -1623,13 +1635,14 @@ class LayerTreeHostScrollTestScrollAbortedCommitMFBA
LayerTreeHostScrollTest::SetupTree();
gfx::Size scroll_layer_bounds(200, 200);
- layer_tree()->outer_viewport_scroll_layer()->SetBounds(scroll_layer_bounds);
- layer_tree()->SetPageScaleFactorAndLimits(1.f, 0.01f, 100.f);
+ layer_tree_host()->outer_viewport_scroll_layer()->SetBounds(
+ scroll_layer_bounds);
+ layer_tree_host()->SetPageScaleFactorAndLimits(1.f, 0.01f, 100.f);
}
void WillBeginMainFrame() override {
num_will_begin_main_frames_++;
- Layer* root_scroll_layer = layer_tree()->outer_viewport_scroll_layer();
+ Layer* root_scroll_layer = layer_tree_host()->outer_viewport_scroll_layer();
switch (num_will_begin_main_frames_) {
case 1:
// This will not be aborted because of the initial prop changes.
@@ -1873,7 +1886,8 @@ class LayerTreeHostScrollTestElasticOverscroll
elastic_overscroll_test_cases_[num_begin_main_frames_main_thread_];
current_elastic_overscroll_ += elastic_overscroll_delta;
EXPECT_EQ(expected_elastic_overscroll, current_elastic_overscroll_);
- EXPECT_EQ(expected_elastic_overscroll, layer_tree()->elastic_overscroll());
+ EXPECT_EQ(expected_elastic_overscroll,
+ layer_tree_host()->elastic_overscroll());
}
void WillBeginMainFrame() override { num_begin_main_frames_main_thread_++; }
@@ -1996,14 +2010,15 @@ class LayerTreeHostScrollTestPropertyTreeUpdate
: initial_scroll_(10, 20), second_scroll_(0, 0) {}
void BeginTest() override {
- layer_tree()->inner_viewport_scroll_layer()->SetScrollOffset(
+ layer_tree_host()->inner_viewport_scroll_layer()->SetScrollOffset(
initial_scroll_);
- layer_tree()->inner_viewport_scroll_layer()->SetBounds(gfx::Size(100, 100));
+ layer_tree_host()->inner_viewport_scroll_layer()->SetBounds(
+ gfx::Size(100, 100));
PostSetNeedsCommitToMainThread();
}
void UpdateLayerTreeHost() override {
- Layer* scroll_layer = layer_tree()->inner_viewport_scroll_layer();
+ Layer* scroll_layer = layer_tree_host()->inner_viewport_scroll_layer();
if (layer_tree_host()->SourceFrameNumber() == 0) {
EXPECT_VECTOR_EQ(initial_scroll_, scroll_layer->scroll_offset());
} else {
« no previous file with comments | « cc/trees/layer_tree_host_unittest_proxy.cc ('k') | cc/trees/layer_tree_host_unittest_video.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698