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

Issue 654564 link

Starred by 5 users

Issue metadata

Status: Fixed
Owner:
Closed: Feb 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Bug



Sign in to add a comment

ARC++ makes Task Manager much less useful

Project Member Reported by esprehn@chromium.org, Oct 10 2016

Issue description

Google Chrome	53.0.2785.144 (Official Build) (64-bit)
Revision	79ce8e9f7864f5dbb6e07f3ba7b9a05943acb912-refs/branch-heads/2785@{#931}

What steps will reproduce the problem?
(1) Enable the play store (ARC++).
(2) Open the Chrome Task Manager (Search+Esc)

What is the expected output?

I see chrome tabs.

What do you see instead?

I see 30+ android processes, and a few chrome ones.

I think we need some kind of filter that hides the system level arc++ stuff. It's really hard to use the task manager once you enable arc because all your tabs are hidden inside the sea of android tasks.
 
Components: Platform>ARC

Comment 2 by uekawa@chromium.org, Nov 28 2016

Cc: hctsai@chromium.org mitsuji@chromium.org nya@chromium.org
I disagree somewhat that we already have chrome tabs and all the background processes and android only adds so many.
But I agree we are piling on to a situation where we already have an issue that we have too many processes to comfortably browse.

Hiding Android System processes makes more sense to me.
We don't list ChromeOS system processes like frecon, dbus-daemon, cryptohomed, .... Android vold, surfaceflinger, zygote, ... looks to fall in the same category to me.

That sounds good to me. I disagree with #2, on a brand new Chromebook there's maybe 3 things in the task list, as soon as I enable ARC++ there's 20-30 things, even when no Android apps are open there's a huge list of Android system services running in there.

If there's no tabs open the task list should have a tiny number of things (Browser, GPU, maybe 1-2 Android system things).
The purpose to add these daemon processes is to track hidden cpu or memory usage. I agree that they seem to be too many in numbers and think of several options for improvement.

1. An option checkbox for hiding or showing these processes, default hiding all
  pros: Intuitive. Users could select what they want.
  cons: Make the user interface more complex.

2. Show the processes with cpu / memory usage higher than some threshold.
  pros: Show only what we want to know. Keep the process list clean at most of the time.
  cons: Some processes might appear and disappear due to resource usage change and the behavior might confuse the user. 

3. Keep a process names white-list for display
  pros: The process list is stable. No processes would randomly appear and disappear. We can decide what to monitor.
  cons: Need efforts to review and maintain the list. Developers might get confused why some of them are shown while some are not.

I personally like idea 3 most. The android processes are still needed because some of them would consume much resources in certain situation. For example, the surfaceflinger. Most of them are quiet. I think the users would like to know about what processes consume the system resources no matter where they come from. 

Comment 6 by uekawa@chromium.org, Dec 13 2016

Owner: mitsuji@chromium.org
Status: Assigned (was: Untriaged)
Hi Hiro, could you triage this UI request.
Owner: sgabr...@chromium.org
Per triage: sgabriel@ are there any plans to change the task manager in the near future? I'm ok with option 3 for now. 


Owner: mitsuji@chromium.org
No plan as far as I know to change the task manager. Ok with option 3 as well if eng is cool with it.
Owner: uekawa@chromium.org
uekawa@ is this something your team can work on? If not please kick it back to me. 
Cc: edcourtney@chromium.org
Cc: -edcourtney@chromium.org
Owner: edcourtney@chromium.org
I'm having a look at this now. Which processes should be white-listed? Another concern I have is that if a not shown process misbehaves and eats CPU, it might be confusing. Personally I would want to be able to kill the process in that situation.
Status: Fixed (was: Assigned)
This removes the majority of the useless processes, with an exception for surface flinger (as suggested earlier in the bug). I wasn't entirely sure exactly what to whitelist for, so feel free to modify the whitelist and/or change the ProcessStates that are implicitly allowed.

Sign in to add a comment