Add bits to indicate which parts of GPUInfo are collected.

content/public/browser/gpu_data_manager.h

Index: content/public/browser/gpu_data_manager.h
diff --git a/content/public/browser/gpu_data_manager.h b/content/public/browser/gpu_data_manager.h
index 7ffc9f18402a86cc082738bac939854dc394476f..16b8ce074a1b03be47c4858b774693048910945d 100644
--- a/content/public/browser/gpu_data_manager.h
+++ b/content/public/browser/gpu_data_manager.h
@@ -60,7 +60,10 @@ class GpuDataManager {
   // Requests complete GPUinfo if it has not already been requested
   virtual void RequestCompleteGpuInfoIfNeeded() = 0;
 
-  virtual bool IsCompleteGpuInfoAvailable() const = 0;
+  // |includes_extra| indicates whether missing DxDiagnostics on Windows
+  // is considered as incomplete; it doesn't make a difference on other
+  // platforms.
+  virtual bool IsCompleteGpuInfoAvailable(bool includes_extra) const = 0;

[Evan Stade, 2014/09/03 00:48:11]

Every time this function is called, it's called with a constant value for the
parameter, never a variable. IsCompleteGpuInfoAvailable(true) isn't very self
documenting, instead I think you should add a new function
IsCompleteGpuInfoAvailableIncludingExtra().

[Zhenyao Mo, 2014/09/03 01:31:23]

Done.

 
   // Requests that the GPU process report its current video memory usage stats,
   // which can be retrieved via the GPU data manager's on-update function.