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

Issue 683226 link

Starred by 2 users

Issue metadata

Status: Assigned
Owner:
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 2
Type: Bug


Sign in to add a comment

Refactor GpuDataManagerImplPrivate so that it can be used by mus gpu-host

Project Member Reported by sadrul@chromium.org, Jan 20 2017

Issue description

GpuDataManagerImplPrivate [1] currently lives in content. It needs to be refactored (e.g. in gpu/ipc/host, or gpu/ipc/client) so that it can be used by the gpu-host in mus [2]. Among other things, it updates the GpuPreferences used to initialize the gpu process (which mus-gpu currently doesn't do at all [3]), whether to use gpu at all (see issue 669965) etc.


1: https://cs.chromium.org/chromium/src/content/browser/gpu/gpu_data_manager_impl.h?sq=package:chromium&dr=CSs&l=43

2: https://cs.chromium.org/chromium/src/services/ui/ws/gpu_host.h?l=30

3: https://cs.chromium.org/chromium/src/services/ui/ws/gpu_host.cc?l=111
 
Cc: rjkroege@chromium.org fsam...@chromium.org
Adding myself to this bug to track. I would prefer gpu/ipc/host because this is privileged code.

Comment 2 by piman@chromium.org, Mar 8 2017

Cc: zmo@chromium.org
Not sure how much that interacts with the goals of moving the config/blacklisting logic to the GPU process
Blocking: 647421

Comment 4 by zmo@chromium.org, Aug 25 2017

Please see  crbug.com/744658 

I am actively moving most decision making to GPU process startup time, so pretty soon GpuDataManager won't have much functionalities left.

I feel it's better to revisit this after the work on  crbug.com/744658  is done

Comment 5 by sadrul@chromium.org, Aug 25 2017

Blockedon: 744658
Cool! I will block this bug on  issue 744658 

Comment 6 by sadrul@chromium.org, Sep 27 2017

Blocking: 609316

Comment 7 by zmo@chromium.org, Feb 1 2018

Blockedon: 808119
Components: -Internals>MUS Internals>Services>WindowService

Comment 9 by sadrul@chromium.org, Jun 11 2018

Blocking: 709332
Owner: moh...@chromium.org
Status: Assigned (was: Available)
Labels: Proj-Mash-MultiProcess
Labels: -Proj-Mustash-Mus-GPU
Cleaning up old Proj-Mustash labels.

Sign in to add a comment