| Index: cc/trees/layer_tree_impl.cc
|
| diff --git a/cc/trees/layer_tree_impl.cc b/cc/trees/layer_tree_impl.cc
|
| index 8a89c8504f5698093ef436ff2f3afafdc92a5b35..59ce942f5476e440b8a3e4611b1ceba2c11a396c 100644
|
| --- a/cc/trees/layer_tree_impl.cc
|
| +++ b/cc/trees/layer_tree_impl.cc
|
| @@ -243,27 +243,31 @@ void LayerTreeImpl::UpdateScrollbars(int scroll_layer_id, int clip_layer_id) {
|
| }
|
|
|
| bool scrollbar_needs_animation = false;
|
| + bool clip_layer_size_did_change = false;
|
| bool scroll_layer_size_did_change = false;
|
| bool y_offset_did_change = false;
|
| for (ScrollbarLayerImplBase* scrollbar : ScrollbarsFor(scroll_layer_id)) {
|
| if (scrollbar->orientation() == HORIZONTAL) {
|
| scrollbar_needs_animation |= scrollbar->SetCurrentPos(current_offset.x());
|
| - scrollbar_needs_animation |=
|
| + clip_layer_size_did_change |=
|
| scrollbar->SetClipLayerLength(clip_size.width());
|
| - scrollbar_needs_animation |= scroll_layer_size_did_change |=
|
| + scroll_layer_size_did_change |=
|
| scrollbar->SetScrollLayerLength(scroll_size.width());
|
| } else {
|
| scrollbar_needs_animation |= y_offset_did_change |=
|
| scrollbar->SetCurrentPos(current_offset.y());
|
| - scrollbar_needs_animation |=
|
| + clip_layer_size_did_change |=
|
| scrollbar->SetClipLayerLength(clip_size.height());
|
| - scrollbar_needs_animation |= scroll_layer_size_did_change |=
|
| + scroll_layer_size_did_change |=
|
| scrollbar->SetScrollLayerLength(scroll_size.height());
|
| }
|
| scrollbar_needs_animation |=
|
| scrollbar->SetVerticalAdjust(clip_layer->bounds_delta().y());
|
| }
|
|
|
| + scrollbar_needs_animation |=
|
| + (clip_layer_size_did_change || scroll_layer_size_did_change);
|
| +
|
| if (y_offset_did_change && IsViewportLayerId(scroll_layer_id))
|
| TRACE_COUNTER_ID1("cc", "scroll_offset_y", scroll_layer->id(),
|
| current_offset.y());
|
| @@ -272,8 +276,16 @@ void LayerTreeImpl::UpdateScrollbars(int scroll_layer_id, int clip_layer_id) {
|
| ScrollbarAnimationController* controller =
|
| layer_tree_host_impl_->ScrollbarAnimationControllerForId(
|
| scroll_layer_id);
|
| - if (controller)
|
| - controller->DidScrollUpdate(scroll_layer_size_did_change);
|
| + if (!controller)
|
| + return;
|
| +
|
| + // TODO(chaopeng) clip_layer_size_did_change should call DidResize after
|
| + // crbug.com/701810 got fixed.
|
| + if (scroll_layer_size_did_change) {
|
| + controller->DidResize();
|
| + } else {
|
| + controller->DidScrollUpdate();
|
| + }
|
| }
|
| }
|
|
|
|
|