| Index: cc/layers/layer_impl_unittest.cc
|
| diff --git a/cc/layers/layer_impl_unittest.cc b/cc/layers/layer_impl_unittest.cc
|
| index 7b16710e20556e1b3e9bf01a4649f5d05c52d47f..c0b2aa7691b194c9d021b620010e9604bdceca7b 100644
|
| --- a/cc/layers/layer_impl_unittest.cc
|
| +++ b/cc/layers/layer_impl_unittest.cc
|
| @@ -511,14 +511,14 @@ TEST_F(LayerImplScrollTest, ScrollByWithNonZeroOffset) {
|
|
|
| class ScrollDelegateIgnore : public LayerImpl::ScrollOffsetDelegate {
|
| public:
|
| - void SetTotalScrollOffset(const gfx::ScrollOffset& new_value) override {
|
| + void SetCurrentScrollOffset(const gfx::ScrollOffset& new_value) override {
|
| last_attempted_set_offset_ = new_value;
|
| }
|
| gfx::ScrollOffset last_attempted_set_offset() const {
|
| return last_attempted_set_offset_;
|
| }
|
|
|
| - gfx::ScrollOffset GetTotalScrollOffset() override {
|
| + gfx::ScrollOffset GetCurrentScrollOffset() override {
|
| return gfx::ScrollOffset(fixed_offset_);
|
| }
|
| bool IsExternalFlingActive() const override { return false; }
|
| @@ -545,6 +545,7 @@ TEST_F(LayerImplScrollTest, ScrollByWithIgnoringDelegate) {
|
| gfx::Vector2dF fixed_offset(32, 12);
|
| delegate.set_fixed_offset(fixed_offset);
|
| layer()->SetScrollOffsetDelegate(&delegate);
|
| + layer()->RefreshFromScrollDelegate();
|
|
|
| EXPECT_VECTOR_EQ(fixed_offset, layer()->TotalScrollOffset());
|
| EXPECT_VECTOR_EQ(scroll_offset, layer()->scroll_offset());
|
| @@ -568,10 +569,12 @@ TEST_F(LayerImplScrollTest, ScrollByWithIgnoringDelegate) {
|
|
|
| class ScrollDelegateAccept : public LayerImpl::ScrollOffsetDelegate {
|
| public:
|
| - void SetTotalScrollOffset(const gfx::ScrollOffset& new_value) override {
|
| + void SetCurrentScrollOffset(const gfx::ScrollOffset& new_value) override {
|
| current_offset_ = new_value;
|
| }
|
| - gfx::ScrollOffset GetTotalScrollOffset() override { return current_offset_; }
|
| + gfx::ScrollOffset GetCurrentScrollOffset() override {
|
| + return current_offset_;
|
| + }
|
| bool IsExternalFlingActive() const override { return false; }
|
| void Update() const override { }
|
|
|
| @@ -616,15 +619,16 @@ TEST_F(LayerImplScrollTest, ApplySentScrollsNoDelegate) {
|
| gfx::Vector2dF scroll_delta(20.5f, 8.5f);
|
| gfx::Vector2d sent_scroll_delta(12, -3);
|
|
|
| - layer()->SetScrollOffset(scroll_offset);
|
| - layer()->ScrollBy(scroll_delta);
|
| - layer()->SetSentScrollDelta(sent_scroll_delta);
|
| + layer()->PushScrollOffsetFromMainThread(scroll_offset);
|
| + layer()->ScrollBy(sent_scroll_delta);
|
| + layer()->PullDeltaForMainThread();
|
| + layer()->SetCurrentScrollOffset(scroll_offset +
|
| + gfx::ScrollOffset(scroll_delta));
|
|
|
| EXPECT_VECTOR_EQ(gfx::ScrollOffsetWithDelta(scroll_offset, scroll_delta),
|
| layer()->TotalScrollOffset());
|
| EXPECT_VECTOR_EQ(scroll_delta, layer()->ScrollDelta());
|
| EXPECT_VECTOR_EQ(scroll_offset, layer()->scroll_offset());
|
| - EXPECT_VECTOR_EQ(sent_scroll_delta, layer()->sent_scroll_delta());
|
|
|
| layer()->ApplySentScrollDeltasFromAbortedCommit();
|
|
|
| @@ -633,7 +637,6 @@ TEST_F(LayerImplScrollTest, ApplySentScrollsNoDelegate) {
|
| EXPECT_VECTOR_EQ(scroll_delta - sent_scroll_delta, layer()->ScrollDelta());
|
| EXPECT_VECTOR_EQ(gfx::ScrollOffsetWithDelta(scroll_offset, sent_scroll_delta),
|
| layer()->scroll_offset());
|
| - EXPECT_VECTOR_EQ(gfx::Vector2d(), layer()->sent_scroll_delta());
|
| }
|
|
|
| TEST_F(LayerImplScrollTest, ApplySentScrollsWithIgnoringDelegate) {
|
| @@ -641,15 +644,17 @@ TEST_F(LayerImplScrollTest, ApplySentScrollsWithIgnoringDelegate) {
|
| gfx::Vector2d sent_scroll_delta(12, -3);
|
| gfx::Vector2dF fixed_offset(32, 12);
|
|
|
| - layer()->SetScrollOffset(scroll_offset);
|
| + layer()->PushScrollOffsetFromMainThread(scroll_offset);
|
| + layer()->ScrollBy(sent_scroll_delta);
|
| + layer()->PullDeltaForMainThread();
|
| + layer()->SetCurrentScrollOffset(scroll_offset);
|
| ScrollDelegateIgnore delegate;
|
| delegate.set_fixed_offset(fixed_offset);
|
| layer()->SetScrollOffsetDelegate(&delegate);
|
| - layer()->SetSentScrollDelta(sent_scroll_delta);
|
| + layer()->RefreshFromScrollDelegate();
|
|
|
| EXPECT_VECTOR_EQ(fixed_offset, layer()->TotalScrollOffset());
|
| EXPECT_VECTOR_EQ(scroll_offset, layer()->scroll_offset());
|
| - EXPECT_VECTOR_EQ(sent_scroll_delta, layer()->sent_scroll_delta());
|
|
|
| layer()->ApplySentScrollDeltasFromAbortedCommit();
|
|
|
| @@ -658,7 +663,6 @@ TEST_F(LayerImplScrollTest, ApplySentScrollsWithIgnoringDelegate) {
|
| EXPECT_VECTOR_EQ(fixed_offset, layer()->TotalScrollOffset());
|
| EXPECT_VECTOR_EQ(gfx::ScrollOffsetWithDelta(scroll_offset, sent_scroll_delta),
|
| layer()->scroll_offset());
|
| - EXPECT_VECTOR_EQ(gfx::Vector2d(), layer()->sent_scroll_delta());
|
| }
|
|
|
| TEST_F(LayerImplScrollTest, ApplySentScrollsWithAcceptingDelegate) {
|
| @@ -666,16 +670,17 @@ TEST_F(LayerImplScrollTest, ApplySentScrollsWithAcceptingDelegate) {
|
| gfx::Vector2d sent_scroll_delta(12, -3);
|
| gfx::Vector2dF scroll_delta(20.5f, 8.5f);
|
|
|
| - layer()->SetScrollOffset(scroll_offset);
|
| + layer()->PushScrollOffsetFromMainThread(scroll_offset);
|
| + layer()->ScrollBy(sent_scroll_delta);
|
| + layer()->PullDeltaForMainThread();
|
| ScrollDelegateAccept delegate;
|
| layer()->SetScrollOffsetDelegate(&delegate);
|
| - layer()->ScrollBy(scroll_delta);
|
| - layer()->SetSentScrollDelta(sent_scroll_delta);
|
| + layer()->SetCurrentScrollOffset(scroll_offset +
|
| + gfx::ScrollOffset(scroll_delta));
|
|
|
| EXPECT_VECTOR_EQ(gfx::ScrollOffsetWithDelta(scroll_offset, scroll_delta),
|
| layer()->TotalScrollOffset());
|
| EXPECT_VECTOR_EQ(scroll_offset, layer()->scroll_offset());
|
| - EXPECT_VECTOR_EQ(sent_scroll_delta, layer()->sent_scroll_delta());
|
|
|
| layer()->ApplySentScrollDeltasFromAbortedCommit();
|
|
|
| @@ -683,7 +688,6 @@ TEST_F(LayerImplScrollTest, ApplySentScrollsWithAcceptingDelegate) {
|
| layer()->TotalScrollOffset());
|
| EXPECT_VECTOR_EQ(gfx::ScrollOffsetWithDelta(scroll_offset, sent_scroll_delta),
|
| layer()->scroll_offset());
|
| - EXPECT_VECTOR_EQ(gfx::Vector2d(), layer()->sent_scroll_delta());
|
| }
|
|
|
| TEST_F(LayerImplScrollTest, ScrollUserUnscrollableLayer) {
|
| @@ -710,10 +714,10 @@ TEST_F(LayerImplScrollTest, PushPropertiesToMirrorsTotalScrollOffset) {
|
| EXPECT_VECTOR_EQ(gfx::Vector2dF(0, 0), unscrolled);
|
| EXPECT_VECTOR_EQ(gfx::Vector2dF(22, 23), layer()->TotalScrollOffset());
|
|
|
| - layer()->SetSentScrollDelta(scroll_delta);
|
| + layer()->PullDeltaForMainThread();
|
|
|
| - scoped_ptr<LayerImpl> pending_layer =
|
| - LayerImpl::Create(host_impl().sync_tree(), layer()->id());
|
| + scoped_ptr<LayerImpl> pending_layer = LayerImpl::Create(
|
| + host_impl().sync_tree(), layer()->id(), layer()->synced_scroll_offset());
|
| pending_layer->SetScrollOffset(layer()->TotalScrollOffset());
|
|
|
| pending_layer->PushPropertiesTo(layer());
|
|
|