Solving some telemetry tests' slowness

content/browser/gpu/gpu_data_manager_impl_private.cc

 // Copyright (c) 2013 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 "content/browser/gpu/gpu_data_manager_impl_private.h"
 
 #include <algorithm>
 #include <iterator>
 #include <memory>
 #include <utility>
@@ skipping 410 matching lines @@
   GpuProcessHost::SendOnIO(
 #if defined(OS_WIN)
       GpuProcessHost::GPU_PROCESS_KIND_UNSANDBOXED,
 #else
       GpuProcessHost::GPU_PROCESS_KIND_SANDBOXED,
 #endif
       true /* force_create */, new GpuMsg_CollectGraphicsInfo());
 }
 
 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 {
 #if defined(OS_WIN)
   if (gpu_info_.dx_diagnostics_info_state == gpu::kCollectInfoNone)
     return false;
 #endif
   return IsEssentialGpuInfoAvailable();
 }
 
@@ skipping 916 matching lines @@
   gpu_info_.context_info_state = gpu::kCollectInfoFatalFailure;
 #if defined(OS_WIN)
   gpu_info_.dx_diagnostics_info_state = gpu::kCollectInfoFatalFailure;
 #endif
   complete_gpu_info_already_requested_ = true;
   // Some observers might be waiting.
   NotifyGpuInfoUpdate();
 }
 
 }  // namespace content