Fix DirectX diagnostic collection for about:gpu page.

content/browser/gpu/gpu_data_manager.cc

Index: content/browser/gpu/gpu_data_manager.cc
===================================================================
--- content/browser/gpu/gpu_data_manager.cc	(revision 118516)
+++ content/browser/gpu/gpu_data_manager.cc	(working copy)
@@ -250,8 +250,8 @@
 void GpuDataManager::UpdateGpuInfo(const content::GPUInfo& gpu_info) {
   DCHECK(BrowserThread::CurrentlyOn(BrowserThread::UI));
 
-  complete_gpu_info_available_ = true;
-  complete_gpu_info_already_requested_ = true;
+  complete_gpu_info_available_ = gpu_info.finalized;

[apatrick_chromium, 2012/01/24 19:01:04]

these are probably wrong. perhaps once they have become true, they should stay
that way?

[Zhenyao Mo, 2012/01/24 19:15:33]

Yes, this should be

complete_gpu_info_available_ = complete_gpu_info_available_ ||
gpu_info.finalized;
complete_gpu_info_already_requested_ = complete_gpu_info_already_requested_ ||
gpu_info.finalized;

+  complete_gpu_info_already_requested_ = gpu_info.finalized;
   {
     base::AutoLock auto_lock(gpu_info_lock_);
     if (!Merge(&gpu_info_, gpu_info))