Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(27)

Unified Diff: content/gpu/gpu_main.cc

Issue 1463823002: Add DriverBugWorkaroundsInGpuProcessPage to gpu_process_test.py (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: To fix win bot failure, call IdentifyActiveGPU when receiving GpuInfi back from the GPU process thr… Created 4 years, 8 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « content/common/gpu/gpu_channel.cc ('k') | content/renderer/gpu/gpu_benchmarking_extension.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: content/gpu/gpu_main.cc
diff --git a/content/gpu/gpu_main.cc b/content/gpu/gpu_main.cc
index 6c65f5c5cdd947883983ee996f704f6e7fc2fb8f..e463379a693fdc934c01e929f5811c1c791b05c3 100644
--- a/content/gpu/gpu_main.cc
+++ b/content/gpu/gpu_main.cc
@@ -306,7 +306,8 @@ int GpuMain(const MainFunctionParams& parameters) {
if (!CollectGraphicsInfo(gpu_info))
dead_on_arrival = true;
-#if defined(OS_CHROMEOS) || defined(OS_ANDROID) || defined(OS_LINUX)
+#if defined(OS_CHROMEOS) || defined(OS_ANDROID) || defined(OS_LINUX) || \
+ defined(OS_WIN)
// Recompute gpu driver bug workarounds.
// This is necessary on systems where vendor_id/device_id aren't available
// (Chrome OS, Android) or where workarounds may be dependent on GL_VENDOR
@@ -316,6 +317,12 @@ int GpuMain(const MainFunctionParams& parameters) {
// initialized in the bindings. This should be moved before bindings
// initialization. However, populating GPUInfo fully works only on
// Android. Other platforms would need the bindings to query GL strings.
+ // TODO(j.isorce): Pass secondary gpus through the command line so that
+ // the local function GetGpuInfoFromCommandLine can generate a complete
+ // GpuInfo. This will allow for the call CollectGraphicsInfo above to
+ // to detect that a secondary gpu is active instead of the primary.
+ // Then the following call will make sure to get the workarounds for the
+ // active gpu.
gpu::ApplyGpuDriverBugWorkarounds(
gpu_info, const_cast<base::CommandLine*>(&command_line));
}
« no previous file with comments | « content/common/gpu/gpu_channel.cc ('k') | content/renderer/gpu/gpu_benchmarking_extension.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698