Solving some telemetry tests' slowness

content/browser/gpu/gpu_data_manager_impl_private.cc

Index: content/browser/gpu/gpu_data_manager_impl_private.cc
diff --git a/content/browser/gpu/gpu_data_manager_impl_private.cc b/content/browser/gpu/gpu_data_manager_impl_private.cc
index 319a130a4c0c68108af5018bf44776b414c81fdb..260a8969f6a891dbcc09dd791541f616ca16a698 100644
--- a/content/browser/gpu/gpu_data_manager_impl_private.cc
+++ b/content/browser/gpu/gpu_data_manager_impl_private.cc
@@ -428,11 +428,9 @@ void GpuDataManagerImplPrivate::RequestCompleteGpuInfoIfNeeded() {
 }
 
 bool GpuDataManagerImplPrivate::IsEssentialGpuInfoAvailable() const {
-  if (gpu_info_.basic_info_state == gpu::kCollectInfoNone ||
-      gpu_info_.context_info_state == gpu::kCollectInfoNone) {
-    return false;
-  }
-  return true;
+  return (gpu_info_.basic_info_state != gpu::kCollectInfoNone &&
+          gpu_info_.context_info_state != gpu::kCollectInfoNone) ||
+         use_swiftshader_;

[Ken Russell (switch to Gerrit), 2017/03/24 21:57:52]

What does GpuDataManager::GetGPUInfo() return now in this case?

If it returns a GPUInfo with no information at all in it -- I think that may
violate the contract of the API.

We should consider populating the GPUInfo, if empty, with information indicating
SwiftShader's being used. (The context_info in particular.)

 }
 
 bool GpuDataManagerImplPrivate::IsCompleteGpuInfoAvailable() const {