New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 756693 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Oct 2017
Cc:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 2
Type: Bug

Blocking:
issue 753224



Sign in to add a comment

Cache render surface in hide animation.

Project Member Reported by wutao@chromium.org, Aug 18 2017

Issue description

Cache render surface in hide animation by using the new  CacheRenderSurface() function in ScopedHidingAnimationSettings.
 

Comment 1 by wutao@chromium.org, Aug 18 2017

Blocking: 753224
Project Member

Comment 2 by bugdroid1@chromium.org, Sep 7 2017

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

commit fb6613a035e615799d30f141cc4d140dc06d134f
Author: wutao <wutao@chromium.org>
Date: Thu Sep 07 21:43:36 2017

CrOS: Cache render surface in hide animation.

Cache render surface in animation can improve the UI performance
significantly. We have built up fundamental blocks already. In this
cl, we are caching the render surface in hide animation.

Performance improvement:
Linux build: ~30%
KEVIN: ~15%

Bug:  756693 ,  753224 
Test: Tested improvement with/out cache.
Change-Id: I539a8407ae9742593a4a44de8f2542fb5f2630f9
Reviewed-on: https://chromium-review.googlesource.com/620132
Commit-Queue: Tao Wu <wutao@chromium.org>
Reviewed-by: Antoine Labour <piman@chromium.org>
Reviewed-by: Scott Violet <sky@chromium.org>
Reviewed-by: David Reveman <reveman@chromium.org>
Cr-Commit-Position: refs/heads/master@{#500398}
[modify] https://crrev.com/fb6613a035e615799d30f141cc4d140dc06d134f/ui/wm/core/window_animations.cc

Comment 3 by wutao@chromium.org, Oct 11 2017

Status: Fixed (was: Available)

Sign in to add a comment