diff options
author | vollick@chromium.org <vollick@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2013-10-15 19:48:26 +0000 |
---|---|---|
committer | vollick@chromium.org <vollick@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2013-10-15 19:48:26 +0000 |
commit | 09eaa4c8b409f81cfc8c18bed8e1eadffd1615dd (patch) | |
tree | 5a9adca836f9c61c8b417595990b8e1b20115806 /content/renderer/render_view_impl.cc | |
parent | b4ec084839630cb18b8071d1bf4f94cfc92cc047 (diff) | |
download | chromium_src-09eaa4c8b409f81cfc8c18bed8e1eadffd1615dd.zip chromium_src-09eaa4c8b409f81cfc8c18bed8e1eadffd1615dd.tar.gz chromium_src-09eaa4c8b409f81cfc8c18bed8e1eadffd1615dd.tar.bz2 |
Revert "Add a flag for enabling/disabling the new accelerated scrolling path"
This reverts commit 429530bcff087403123cecafedbd93022fb74746.
> Add a flag for enabling/disabling the new accelerated scrolling path
>
> We now have two accelerated scrolling paths. One accelerates scrolling whenever
> it's safe to promote the overflow scrolling div to a stacking container /
> containing block and not break stacking or clipping.
>
> The new path, the 'universal' path, enables accelerated scrolling in those cases
> where the old path would not opt-in.
>
> Eventually, we'd like to eliminate the old path, but this will require layer
> squashing (the new path can result in many composited layers that ought to be
> combined -- i.e., a 'layer explosion'). Until these paths are mature, we need to
> be able to enable and disable them both, so we have two separate sets of flags.
>
> Once we've eliminated the old path, we can replace both with
> --[enable/disable]-accelerated-overflow-scroll (and eventually get rid of both
> sets of flags).
>
> BUG=None
>
> Review URL: https://codereview.chromium.org/26809004
TBR=jamser@chromium.org
BUG=307486
Review URL: https://codereview.chromium.org/27393002
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@228743 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'content/renderer/render_view_impl.cc')
-rw-r--r-- | content/renderer/render_view_impl.cc | 16 |
1 files changed, 0 insertions, 16 deletions
diff --git a/content/renderer/render_view_impl.cc b/content/renderer/render_view_impl.cc index f71c8eaa5..aef0ac7 100644 --- a/content/renderer/render_view_impl.cc +++ b/content/renderer/render_view_impl.cc @@ -606,20 +606,6 @@ static bool ShouldUseAcceleratedCompositingForOverflowScroll( return DeviceScaleEnsuresTextQuality(device_scale_factor); } -static bool ShouldUseUniversalAcceleratedCompositingForOverflowScroll() { - const CommandLine& command_line = *CommandLine::ForCurrentProcess(); - - if (command_line.HasSwitch( - switches::kDisableUniversalAcceleratedOverflowScroll)) - return false; - - if (command_line.HasSwitch( - switches::kEnableUniversalAcceleratedOverflowScroll)) - return true; - - return false; -} - static bool ShouldUseTransitionCompositing(float device_scale_factor) { const CommandLine& command_line = *CommandLine::ForCurrentProcess(); @@ -909,8 +895,6 @@ void RenderViewImpl::Initialize(RenderViewImplParams* params) { ShouldUseFixedPositionCompositing(device_scale_factor_)); webview()->settings()->setAcceleratedCompositingForOverflowScrollEnabled( ShouldUseAcceleratedCompositingForOverflowScroll(device_scale_factor_)); - webview()->settings()->setCompositorDrivenAcceleratedScrollingEnabled( - ShouldUseUniversalAcceleratedCompositingForOverflowScroll()); webview()->settings()->setAcceleratedCompositingForTransitionEnabled( ShouldUseTransitionCompositing(device_scale_factor_)); webview()->settings()->setAcceleratedCompositingForFixedRootBackgroundEnabled( |