• Mitsuru Oshima's avatar
    Revert "Overview optimizations" · 3f852913
    Mitsuru Oshima authored
    This reverts commit 46a9bbb1.
    
    Reason for revert: <INSERT REASONING HERE>
    
    Original change's description:
    > Overview optimizations
    > 
    > This is 2nd attempt to land: crrev.com/c/1304973
    > [2] are the difference from above CL.
    > 
    > *[2] Added null check in OnStartingAnimationComplete.
    > * Pause occusion tracker during overview animation
    >  [2] wait 5000ms when exiting overview before resume
    >        because a user may re-enter overview
    >  [2] wait 50ms when entering overview before resume
    >        because it can take 2 frame to finish draing the frame
    >        after animation observer is called.
    > * Activate the text filter after animation
    > * [2]Animate shelf after overview animation.
    > * [2] Improve blur animation:
    >  Use Compositor's AnimationObesrver to drive animation
    >  Progress only by 2 to reduce the frequency.
    > * Don't animate backdrop window during overview animation.
    > * Fix the bounds change issue in WindowGrid. It shouldn't change
    > the parent window's bounds.
    > 
    > 
    > Bug: 898077, 897387, b/118675936
    > Test: no functional change. Manually tested visual change by entering/exiting overview mode
    >   in both clamshell and tabletmode.
    >   The web contents should load after animations without backdrop animation.
    >   No gap at the bottom of the screen after exiting overview mode.
    > 
    > Change-Id: I5e5dc17afd62d4dc477d87becdeee91c55bd81ca
    > Reviewed-on: https://chromium-review.googlesource.com/c/1304973
    > Commit-Queue: Mitsuru Oshima <oshima@chromium.org>
    > Reviewed-by: Sammie Quon <sammiequon@chromium.org>
    > Cr-Original-Commit-Position: refs/heads/master@{#604206}
    > Reviewed-on: https://chromium-review.googlesource.com/c/1312614
    > Reviewed-by: Malay Keshav <malaykeshav@chromium.org>
    > Cr-Commit-Position: refs/heads/master@{#604844}
    
    TBR=oshima@chromium.org,sammiequon@chromium.org,malaykeshav@chromium.org
    
    Change-Id: Ia07f696386ce2889a1d6344218be355a3515975c
    No-Presubmit: true
    No-Tree-Checks: true
    No-Try: true
    Bug: 898077, 897387, b/118675936
    Reviewed-on: https://chromium-review.googlesource.com/c/1316169Reviewed-by: default avatarMitsuru Oshima <oshima@chromium.org>
    Commit-Queue: Mitsuru Oshima <oshima@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#605052}
    3f852913
workspace_layout_manager.cc 18.4 KB