Disable GPU info collection on GPU process startup on Win/Mac.

content/gpu/gpu_main.cc

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include <stdlib.h>
 
 #if defined(OS_WIN)
 #include <windows.h>
 #endif
 
 #include "base/environment.h"
 #include "base/message_loop.h"
 #include "base/rand_util.h"
+#include "base/string_number_conversions.h"
 #include "base/stringprintf.h"
 #include "base/threading/platform_thread.h"
 #include "base/win/scoped_com_initializer.h"
 #include "build/build_config.h"
 #include "content/common/gpu/gpu_config.h"
 #include "content/gpu/gpu_child_thread.h"
 #include "content/gpu/gpu_info_collector.h"
 #include "content/gpu/gpu_process.h"
 #include "content/public/common/content_client.h"
 #include "content/public/common/content_switches.h"
@@ skipping 41 matching lines @@
   // will need to tear down this process. However, we can not do so
   // safely until the IPC channel is set up, because the detection of
   // early return of a child process is implemented using an IPC
   // channel error. If the IPC channel is not fully set up between the
   // browser and GPU process, and the GPU process crashes or exits
   // early, the browser process will never detect it.  For this reason
   // we defer tearing down the GPU process until receiving the
   // GpuMsg_Initialize message from the browser.
   bool dead_on_arrival = false;
 
+  content::GPUInfo gpu_info;
+  // Get vendor_id, device_id, driver_version from browser process through
+  // commandline switches.
+  DCHECK(command_line.HasSwitch(switches::kGpuVendorID) &&
+         command_line.HasSwitch(switches::kGpuDeviceID) &&
+         command_line.HasSwitch(switches::kGpuDriverVersion));
+  bool success = base::HexStringToInt(
+      command_line.GetSwitchValueASCII(switches::kGpuVendorID),
+      reinterpret_cast<int*>(&(gpu_info.gpu.vendor_id)));
+  DCHECK(success);
+  success = base::HexStringToInt(
+      command_line.GetSwitchValueASCII(switches::kGpuDeviceID),
+      reinterpret_cast<int*>(&(gpu_info.gpu.device_id)));
+  gpu_info.driver_version =
+      command_line.GetSwitchValueASCII(switches::kGpuDriverVersion);
+  content::GetContentClient()->SetGpuInfo(gpu_info);
+
   // Load and initialize the GL implementation and locate the GL entry points.
-  content::GPUInfo gpu_info;
   if (gfx::GLSurface::InitializeOneOff()) {
-    // Collect information about the GPU.
-    if (!gpu_info_collector::CollectGraphicsInfo(&gpu_info)) {
-      LOG(INFO) << "gpu_info_collector::CollectGraphicsInfo failed";
-    }
+#if defined(OS_LINUX)
+    // We collect full GPU info on demand in Win/Mac, i.e., when about:gpu
+    // page opens.  This is because we can make blacklist decisions based on
+    // preliminary GPU info.
+    // However, on Linux, blacklist decisions are based on full GPU info.
+    if (!gpu_info_collector::CollectGraphicsInfo(&gpu_info))
+      VLOG(1) << "gpu_info_collector::CollectGraphicsInfo failed";
+    content::GetContentClient()->SetGpuInfo(gpu_info);
 
-#if defined(OS_LINUX)
     if (gpu_info.gpu.vendor_id == 0x10de &&  // NVIDIA
         gpu_info.driver_vendor == "NVIDIA") {
       base::ThreadRestrictions::AssertIOAllowed();
       if (access("/dev/nvidiactl", R_OK) != 0) {
-        LOG(INFO) << "NVIDIA device file /dev/nvidiactl access denied";
+        VLOG(1) << "NVIDIA device file /dev/nvidiactl access denied";
         gpu_info.gpu_accessible = false;
         dead_on_arrival = true;
       }
     }
 #endif
-
-    // Set the GPU info even if it failed.
-    content::GetContentClient()->SetGpuInfo(gpu_info);
   } else {
-    LOG(INFO) << "gfx::GLSurface::InitializeOneOff failed";
+    VLOG(1) << "gfx::GLSurface::InitializeOneOff failed";
     gpu_info.gpu_accessible = false;
+    gpu_info.finalized = true;
     dead_on_arrival = true;
   }
 
   // Warm up the random subsystem, which needs to be done pre-sandbox on all
   // platforms.
   (void) base::RandUint64();
 
   // Warm up the crypto subsystem, which needs to done pre-sandbox on all
   // platforms.
   crypto::HMAC hmac(crypto::HMAC::SHA256);
@@ skipping 47 matching lines @@
   child_thread->Init(start_time);
 
   gpu_process.set_main_thread(child_thread);
 
   main_message_loop.Run();
 
   child_thread->StopWatchdog();
 
   return 0;
 }