New issue
Advanced search Search tips

Issue 904902 link

Starred by 3 users

Issue metadata

Status: Fixed
Owner: ----
Closed: Jan 16
Components:
EstimatedDays: ----
NextAction: ----
OS: Windows
Pri: 3
Type: Bug



Sign in to add a comment

Running GCPW on winlogon requires --disable-gpu

Project Member Reported by tienmai@chromium.org, Nov 13

Issue description

When running Chrome as a windows credential provider, it needs to run the gpu process in an alternate desktop because the default sandbox integrity level of the gpu process (LOW) is too low to access the winlogon desktop, which requires an integrity level of MEDIUM. However, since the gpu process is running on an alternate desktop it cannot render to the winlogon desktop causing all gpu rendering to be disabled. 

To prevent this potential problem, Chrome is run with the flag --disable-gpu so that it can still render to screen correctly even without the gpu-process. If we need to enable the gpu-process in the future, further investigation will be needed. At minimum if we change the integrity level of the GPU process to MEDIUM or set a delayed integrity level (so that the process starts at MEDIUM and then later gets lowered to LOW), this will resolve the problem. 
 
Status: Available (was: Untriaged)
Labels: Enterprise-Triaged
Project Member

Comment 3 by bugdroid1@chromium.org, Jan 16 (6 days ago)

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

commit 74dd62006d7f4f95203f96b46286fdf57e837c67
Author: Tien Mai <tienmai@chromium.org>
Date: Wed Jan 16 21:35:05 2019

Remove unnecessary disable-gpu switch on startup of Chrome for GCPW

With the new fix to gpu-process running under the winlogon desktop
it is no longer needed to disable the gpu to run Chrome for GCPW.
This change will also now allow us to correctly render videos and other
GPU related content when signing on with GCPW.

Bug:  904902 
Change-Id: Iaf3c10527e9a2de4be3086df0c253e48a2efcdb9
Reviewed-on: https://chromium-review.googlesource.com/c/1415692
Reviewed-by: Roger Tawa <rogerta@chromium.org>
Commit-Queue: Tien Mai <tienmai@chromium.org>
Cr-Commit-Position: refs/heads/master@{#623376}
[modify] https://crrev.com/74dd62006d7f4f95203f96b46286fdf57e837c67/chrome/credential_provider/gaiacp/gaia_credential_base.cc

Comment 4 by tienmai@google.com, Jan 16 (6 days ago)

Status: Fixed (was: Available)

Sign in to add a comment