My favorites | Sign in
Project Home Downloads Wiki Issues Code Search
New issue   Search
for
  Advanced search   Search tips   Subscriptions
Issue 159980: Upstream code related to making the foreground renderer process have same priority as browser
5 people starred this issue and may be notified of changes. Back to list
Status:  Fixed
Owner:  cjhopman@chromium.org
Closed:  Nov 2012
Cc:  dfalcant...@chromium.org


Sign in to add a comment
 
Project Member Reported by yfriedman@chromium.org, Nov 7, 2012
In content_core_view_impl.cc, there's code downstream for NOTIFICATION_RENDER_VIEW_HOST_CHANGED and NOTIFICATION_RENDERER_PROCESS_CREATED that call into ContentViewCore.onRendererProcessSwap.

The point of this code is to ensure that if memory is low, a non-visible renderer is killed before the visible one. This needs to be upstreamed/verified.
Nov 8, 2012
#1 yfriedman@chromium.org
(No comment was entered for this change.)
Status: Assigned
Owner: cjhopman@chromium.org
Cc: dfalcant...@chromium.org
Nov 13, 2012
#2 vino...@google.com
(No comment was entered for this change.)
Labels: -Mstone-24 Mstone-25
Nov 15, 2012
#3 bugdro...@chromium.org
The following revision refers to this bug:
    http://src.chromium.org/viewvc/chrome?view=rev&revision=168007

------------------------------------------------------------------------
r168007 | cjhopman@chromium.org | 2012-11-15T20:26:43.242593Z

Changed paths:
   M http://src.chromium.org/viewvc/chrome/trunk/src/content/browser/android/content_view_core_impl.cc?r1=168007&r2=168006&pathrev=168007
   M http://src.chromium.org/viewvc/chrome/trunk/src/content/public/android/java/src/org/chromium/content/browser/ContentViewCore.java?r1=168007&r2=168006&pathrev=168007

Ensure that the foreground tab's process has high priority

To reclaim memory, Android will kill renderer processes before killing
the browser process (See
http://developer.android.com/guide/components/processes-and-threads.html).
That is, the browser process is a foreground process while the renderer
processes are service processes. A foreground process can bind to a
service process so that that process has the same priority as the
foreground process (in terms of what should be killed).

BUG=159980


Review URL: https://chromiumcodereview.appspot.com/11361252
------------------------------------------------------------------------
Nov 15, 2012
#4 cjhopman@chromium.org
(No comment was entered for this change.)
Status: Fixed
Mar 10, 2013
#5 bugdro...@chromium.org
(No comment was entered for this change.)
Labels: -Mstone-25 M-25
Sign in to add a comment

Powered by Google Project Hosting