New issue
Advanced search Search tips

Issue 707800 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Dec 11
Cc:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Bug

Blocked on:
issue 678763



Sign in to add a comment

Clean up the flag of kAshEnableSmoothScreenRotationName[].

Project Member Reported by wutao@chromium.org, Apr 3 2017

Issue description

After we fully launch the new screen rotation animation, we should clean up the flag of kAshEnableSmoothScreenRotationName[].

Create this bug for tracking.
 

Comment 1 by wutao@chromium.org, Apr 24 2017

Upload a cl for review to flip the flag to make the screen rotation animation enabled by default.
Changed the flag to kAshDisableSmoothScreenRotationName.

https://codereview.chromium.org/2837773003/

Comment 2 by wutao@chromium.org, Aug 24 2017

The flag has been changed to kAshDisableSmoothScreenRotation.

Comment 3 by wutao@chromium.org, Aug 28 2017

Default is sync in test code. Many tests are setup with the switch in ash_test_helper, so in the test will call sync by default. Any test for new animation, we need to remove the switch or attach empty switch.


When removing the flag, we need to put the tests using async animation. For any test requires sync animation, will use the new enum to choose the sync mode.




Hi oshima@, is it ok to remove this flag now.

Thanks!
sgtm
Project Member

Comment 6 by bugdroid1@chromium.org, Dec 11

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/b1041214265f46e6bd84299fc7205f12c47b90bb

commit b1041214265f46e6bd84299fc7205f12c47b90bb
Author: wutao <wutao@chromium.org>
Date: Tue Dec 11 19:12:43 2018

ash: Remove flag kAshDisableSmoothScreenRotation

This feature has been enabled by default since M60. It is stable enough
to be always enabled.

Bug:  707800 
Test: manual and coverred by unittest.
Change-Id: I03e88f0b3a95d4f76ebd3a51f9f9241f257c82c3
Reviewed-on: https://chromium-review.googlesource.com/c/1370953
Reviewed-by: Mitsuru Oshima (gardener - slow) <oshima@chromium.org>
Commit-Queue: Tao Wu <wutao@chromium.org>
Cr-Commit-Position: refs/heads/master@{#615616}
[modify] https://crrev.com/b1041214265f46e6bd84299fc7205f12c47b90bb/ash/display/display_configuration_controller_unittest.cc
[modify] https://crrev.com/b1041214265f46e6bd84299fc7205f12c47b90bb/ash/public/cpp/ash_switches.cc
[modify] https://crrev.com/b1041214265f46e6bd84299fc7205f12c47b90bb/ash/public/cpp/ash_switches.h
[modify] https://crrev.com/b1041214265f46e6bd84299fc7205f12c47b90bb/ash/rotator/screen_rotation_animator.cc
[modify] https://crrev.com/b1041214265f46e6bd84299fc7205f12c47b90bb/ash/rotator/screen_rotation_animator.h
[modify] https://crrev.com/b1041214265f46e6bd84299fc7205f12c47b90bb/ash/test/ash_test_helper.cc
[modify] https://crrev.com/b1041214265f46e6bd84299fc7205f12c47b90bb/chrome/browser/about_flags.cc
[modify] https://crrev.com/b1041214265f46e6bd84299fc7205f12c47b90bb/chrome/browser/flag_descriptions.cc
[modify] https://crrev.com/b1041214265f46e6bd84299fc7205f12c47b90bb/chrome/browser/flag_descriptions.h

Status: Fixed (was: Assigned)

Sign in to add a comment