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

Issue 709709 link

Starred by 3 users
Status: Fixed
Owner:
xing...@intel.com
Closed: Apr 2017
Cc:
samans@chromium.org
█ fsam...@chromium.org
mustash-bugs@google.com
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Feature



Sign in to add a comment

Move existing frame eviction code to display compositor

Project Member Reported by samans@chromium.org, Apr 8 2017 
RendererFrameManager and DelegatedFrameEvictor are the core of current frame eviction system in Chrome. They currently live in content but have surprisingly very little dependency on it. It should be a fairly simple task to move them into the display compositor (components/viz or whatever, I don't even know)

Comment 1 by samans@chromium.org, Apr 8 2017

Status: Available (was: Untriaged)
Project Member

Comment 2 by bugdroid1@chromium.org, Apr 19 2017 

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

commit e012eb7d8849c55a50689509a81eb0a9588485da
Author: xing.xu <xing.xu@intel.com>
Date: Wed Apr 19 03:19:41 2017

Move frame eviction into components

Eventually frame eviction will be solely handled by the display compositor and no need to depend on content, so move it into components.

BUG= 709709 

Review-Url: https://codereview.chromium.org/2811083002
Cr-Commit-Position: refs/heads/master@{#465481}

[modify] https://crrev.com/e012eb7d8849c55a50689509a81eb0a9588485da/components/viz/frame_sinks/BUILD.gn
[modify] https://crrev.com/e012eb7d8849c55a50689509a81eb0a9588485da/components/viz/frame_sinks/DEPS
[rename] https://crrev.com/e012eb7d8849c55a50689509a81eb0a9588485da/components/viz/frame_sinks/frame_eviction_manager.cc
[add] https://crrev.com/e012eb7d8849c55a50689509a81eb0a9588485da/components/viz/frame_sinks/frame_eviction_manager.h
[add] https://crrev.com/e012eb7d8849c55a50689509a81eb0a9588485da/components/viz/frame_sinks/frame_evictor.cc
[add] https://crrev.com/e012eb7d8849c55a50689509a81eb0a9588485da/components/viz/frame_sinks/frame_evictor.h
[modify] https://crrev.com/e012eb7d8849c55a50689509a81eb0a9588485da/components/viz/frame_sinks/mojo_frame_sink_manager.h
[add] https://crrev.com/e012eb7d8849c55a50689509a81eb0a9588485da/components/viz/frame_sinks/viz_export.h
[modify] https://crrev.com/e012eb7d8849c55a50689509a81eb0a9588485da/content/browser/BUILD.gn
[delete] https://crrev.com/faf2ee225610f256279fe77bf1413bb512dec3c2/content/browser/renderer_host/delegated_frame_evictor.cc
[delete] https://crrev.com/faf2ee225610f256279fe77bf1413bb512dec3c2/content/browser/renderer_host/delegated_frame_evictor.h
[modify] https://crrev.com/e012eb7d8849c55a50689509a81eb0a9588485da/content/browser/renderer_host/delegated_frame_host.cc
[modify] https://crrev.com/e012eb7d8849c55a50689509a81eb0a9588485da/content/browser/renderer_host/delegated_frame_host.h
[modify] https://crrev.com/e012eb7d8849c55a50689509a81eb0a9588485da/content/browser/renderer_host/render_widget_host_view_android.cc
[modify] https://crrev.com/e012eb7d8849c55a50689509a81eb0a9588485da/content/browser/renderer_host/render_widget_host_view_android.h
[modify] https://crrev.com/e012eb7d8849c55a50689509a81eb0a9588485da/content/browser/renderer_host/render_widget_host_view_aura_unittest.cc
[delete] https://crrev.com/faf2ee225610f256279fe77bf1413bb512dec3c2/content/browser/renderer_host/renderer_frame_manager.h
[modify] https://crrev.com/e012eb7d8849c55a50689509a81eb0a9588485da/content/test/BUILD.gn

Comment 3 by fsam...@chromium.org, Apr 19 2017

Labels: Type-Feature

Comment 4 by fsamuel@google.com, Apr 20 2017

Owner: xing...@intel.com
Status: Fixed (was: Available)
I'm marking this as FIXED.

Comment 5 by fsam...@chromium.org, Jun 13 2017

Blocking:

Comment 6 by lafo...@chromium.org, Feb 26 2018

Components: -Internals>MUS Internals>Services>WindowService
►

Sign in to add a comment