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

Side by Side Diff: cc/trees/layer_tree_impl.cc

Issue 608223002: [Android]Increase Scrollbar fade delay on Resize. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: rebased Created 6 years, 2 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 unified diff | Download patch
« no previous file with comments | « cc/trees/layer_tree_host_impl.cc ('k') | cc/trees/layer_tree_settings.h » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright 2011 The Chromium Authors. All rights reserved. 1 // Copyright 2011 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "cc/trees/layer_tree_impl.h" 5 #include "cc/trees/layer_tree_impl.h"
6 6
7 #include <limits> 7 #include <limits>
8 #include <set> 8 #include <set>
9 9
10 #include "base/debug/trace_event.h" 10 #include "base/debug/trace_event.h"
(...skipping 278 matching lines...) Expand 10 before | Expand all | Expand 10 after
289 scrolling_layer_id_from_previous_tree_ = 0; 289 scrolling_layer_id_from_previous_tree_ = 0;
290 } 290 }
291 291
292 namespace { 292 namespace {
293 293
294 void ForceScrollbarParameterUpdateAfterScaleChange(LayerImpl* current_layer) { 294 void ForceScrollbarParameterUpdateAfterScaleChange(LayerImpl* current_layer) {
295 if (!current_layer) 295 if (!current_layer)
296 return; 296 return;
297 297
298 while (current_layer) { 298 while (current_layer) {
299 current_layer->ScrollbarParametersDidChange(); 299 current_layer->ScrollbarParametersDidChange(false);
300 current_layer = current_layer->parent(); 300 current_layer = current_layer->parent();
301 } 301 }
302 } 302 }
303 303
304 } // namespace 304 } // namespace
305 305
306 void LayerTreeImpl::SetPageScaleFactorAndLimits(float page_scale_factor, 306 void LayerTreeImpl::SetPageScaleFactorAndLimits(float page_scale_factor,
307 float min_page_scale_factor, float max_page_scale_factor) { 307 float min_page_scale_factor, float max_page_scale_factor) {
308 SetPageScaleValues(page_scale_factor, min_page_scale_factor, 308 SetPageScaleValues(page_scale_factor, min_page_scale_factor,
309 max_page_scale_factor, page_scale_delta_); 309 max_page_scale_factor, page_scale_delta_);
(...skipping 475 matching lines...) Expand 10 before | Expand all | Expand 10 after
785 const gfx::Rect LayerTreeImpl::ViewportRectForTilePriority() const { 785 const gfx::Rect LayerTreeImpl::ViewportRectForTilePriority() const {
786 return layer_tree_host_impl_->ViewportRectForTilePriority(); 786 return layer_tree_host_impl_->ViewportRectForTilePriority();
787 } 787 }
788 788
789 scoped_ptr<ScrollbarAnimationController> 789 scoped_ptr<ScrollbarAnimationController>
790 LayerTreeImpl::CreateScrollbarAnimationController(LayerImpl* scrolling_layer) { 790 LayerTreeImpl::CreateScrollbarAnimationController(LayerImpl* scrolling_layer) {
791 DCHECK(settings().scrollbar_fade_delay_ms); 791 DCHECK(settings().scrollbar_fade_delay_ms);
792 DCHECK(settings().scrollbar_fade_duration_ms); 792 DCHECK(settings().scrollbar_fade_duration_ms);
793 base::TimeDelta delay = 793 base::TimeDelta delay =
794 base::TimeDelta::FromMilliseconds(settings().scrollbar_fade_delay_ms); 794 base::TimeDelta::FromMilliseconds(settings().scrollbar_fade_delay_ms);
795 base::TimeDelta resize_delay = base::TimeDelta::FromMilliseconds(
796 settings().scrollbar_fade_resize_delay_ms);
795 base::TimeDelta duration = 797 base::TimeDelta duration =
796 base::TimeDelta::FromMilliseconds(settings().scrollbar_fade_duration_ms); 798 base::TimeDelta::FromMilliseconds(settings().scrollbar_fade_duration_ms);
797 switch (settings().scrollbar_animator) { 799 switch (settings().scrollbar_animator) {
798 case LayerTreeSettings::LinearFade: { 800 case LayerTreeSettings::LinearFade: {
799 return ScrollbarAnimationControllerLinearFade::Create( 801 return ScrollbarAnimationControllerLinearFade::Create(
800 scrolling_layer, layer_tree_host_impl_, delay, duration); 802 scrolling_layer,
803 layer_tree_host_impl_,
804 delay,
805 resize_delay,
806 duration);
801 } 807 }
802 case LayerTreeSettings::Thinning: { 808 case LayerTreeSettings::Thinning: {
803 return ScrollbarAnimationControllerThinning::Create( 809 return ScrollbarAnimationControllerThinning::Create(scrolling_layer,
804 scrolling_layer, layer_tree_host_impl_, delay, duration); 810 layer_tree_host_impl_,
811 delay,
812 resize_delay,
813 duration);
805 } 814 }
806 case LayerTreeSettings::NoAnimator: 815 case LayerTreeSettings::NoAnimator:
807 NOTREACHED(); 816 NOTREACHED();
808 break; 817 break;
809 } 818 }
810 return nullptr; 819 return nullptr;
811 } 820 }
812 821
813 void LayerTreeImpl::DidAnimateScrollOffset() { 822 void LayerTreeImpl::DidAnimateScrollOffset() {
814 layer_tree_host_impl_->DidAnimateScrollOffset(); 823 layer_tree_host_impl_->DidAnimateScrollOffset();
(...skipping 681 matching lines...) Expand 10 before | Expand all | Expand 10 after
1496 page_scale, 1505 page_scale,
1497 duration.InSecondsF()); 1506 duration.InSecondsF());
1498 } 1507 }
1499 } 1508 }
1500 1509
1501 scoped_ptr<PageScaleAnimation> LayerTreeImpl::TakePageScaleAnimation() { 1510 scoped_ptr<PageScaleAnimation> LayerTreeImpl::TakePageScaleAnimation() {
1502 return page_scale_animation_.Pass(); 1511 return page_scale_animation_.Pass();
1503 } 1512 }
1504 1513
1505 } // namespace cc 1514 } // namespace cc
OLDNEW
« no previous file with comments | « cc/trees/layer_tree_host_impl.cc ('k') | cc/trees/layer_tree_settings.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698