Minimize animation is disabled on fullscreen/maximized exo window |
||||||||||||||
Issue descriptionbecause we disable all animations for maximize/fullscreen state.
,
May 3 2017
,
May 3 2017
,
May 4 2017
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/ff3a93f666037548cc7120e82fdea9197d1caf8f commit ff3a93f666037548cc7120e82fdea9197d1caf8f Author: Mitsuru Oshima <oshima@chromium.org> Date: Thu May 04 14:44:57 2017 Do not disable minimize animation for maximized/fullscreened exo windows BUG= 717761 TEST=Fullscreen/Maximize Play Store app, then minimize/unminmize. Review-Url: https://codereview.chromium.org/2861523002 Cr-Commit-Position: refs/heads/master@{#469171} (cherry picked from commit 30b31ad5f8d2cbfebe42a9288dd372edcaf0eca0) Review-Url: https://codereview.chromium.org/2864433004 . Cr-Commit-Position: refs/branch-heads/3071@{#398} Cr-Branched-From: a106f0abbf69dad349d4aaf4bcc4f5d376dd2377-refs/heads/master@{#464641} [modify] https://crrev.com/ff3a93f666037548cc7120e82fdea9197d1caf8f/components/exo/shell_surface.cc
,
May 4 2017
bhthompson@ for m58. This is fairly safe and worth considering 58 merge.
,
May 4 2017
LGTM for 58, assuming it has been verified on a higher channel.
,
May 5 2017
Your change meets the bar and is auto-approved for M59. Please go ahead and merge the CL to branch 3071 manually. Please contact milestone owner if you have questions. Owners: amineer@(Android), cmasso@(iOS), gkihumba@(ChromeOS), Abdul Syed@(Desktop) For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
,
May 6 2017
,
May 6 2017
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/b849071b201e27f26a5cb7a87298793ad2edd48b commit b849071b201e27f26a5cb7a87298793ad2edd48b Author: Mitsuru Oshima <oshima@chromium.org> Date: Sat May 06 01:53:55 2017 Do not disable minimize animation for maximized/fullscreened exo windows BUG= 717761 TEST=Fullscreen/Maximize Play Store app, then minimize/unminmize. Review-Url: https://codereview.chromium.org/2861523002 Cr-Commit-Position: refs/heads/master@{#469171} (cherry picked from commit 30b31ad5f8d2cbfebe42a9288dd372edcaf0eca0) Review-Url: https://codereview.chromium.org/2863683006 . Cr-Commit-Position: refs/branch-heads/3029@{#811} Cr-Branched-From: 939b32ee5ba05c396eef3fd992822fcca9a2e262-refs/heads/master@{#454471} [modify] https://crrev.com/b849071b201e27f26a5cb7a87298793ad2edd48b/components/exo/shell_surface.cc
,
May 8 2017
This issue has been approved for a merge. Please merge the fix to any appropriate branches as soon as possible! If all merges have been completed, please remove any remaining Merge-Approved labels from this issue. Thanks for your time! To disable nags, add the Disable-Nags label. For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
,
May 12 2017
This issue has been approved for a merge. Please merge the fix to any appropriate branches as soon as possible! If all merges have been completed, please remove any remaining Merge-Approved labels from this issue. Thanks for your time! To disable nags, add the Disable-Nags label. For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
,
May 12 2017
,
May 24 2017
,
Jan 22 2018
,
Feb 26 2018
|
||||||||||||||
►
Sign in to add a comment |
||||||||||||||
Comment 1 by bugdroid1@chromium.org
, May 3 2017