gpu: Some cleanups.

content/browser/gpu/gpu_process_host.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 "content/browser/gpu/gpu_process_host.h"
 
 #include <stddef.h>
 
 #include <algorithm>
 #include <list>
@@ skipping 368 matching lines @@
 
   // TODO(sievers): Revisit this behavior. It's not really a crash, but we also
   // want the fallback-to-sw behavior if we cannot initialize the GPU.
   host->RecordProcessCrash();
 
   delete host;
   return NULL;
 }
 
 // static
-void GpuProcessHost::GetProcessHandles(
-    const GpuDataManager::GetGpuProcessHandlesCallback& callback)  {
+void GpuProcessHost::GetHasGpuProcess(
+    const base::Callback<void(bool)>& callback) {
   if (!BrowserThread::CurrentlyOn(BrowserThread::IO)) {
     BrowserThread::PostTask(
-        BrowserThread::IO,
-        FROM_HERE,
-        base::Bind(&GpuProcessHost::GetProcessHandles, callback));
+        BrowserThread::IO, FROM_HERE,
+        base::Bind(&GpuProcessHost::GetHasGpuProcess, callback));
     return;
   }
-  std::list<base::ProcessHandle> handles;
+  bool has_gpu = false;
   for (size_t i = 0; i < arraysize(g_gpu_process_hosts); ++i) {
-    // TODO(rvargas) crbug/417532: don't store ProcessHandles!.
     GpuProcessHost* host = g_gpu_process_hosts[i];
-    if (host && ValidateHost(host))
-      handles.push_back(host->process_->GetProcess().Handle());
+    if (host && ValidateHost(host)) {
+      has_gpu = true;
+      break;
+    }
   }
-  BrowserThread::PostTask(
-      BrowserThread::UI,
-      FROM_HERE,
-      base::Bind(callback, handles));
+  BrowserThread::PostTask(BrowserThread::UI, FROM_HERE,
+                          base::Bind(callback, has_gpu));
 }
 
 // static
 void GpuProcessHost::CallOnIO(
     GpuProcessKind kind,
     bool force_create,
     const base::Callback<void(GpuProcessHost*)>& callback) {
 #if !defined(OS_WIN)
   DCHECK_NE(kind, GpuProcessHost::GPU_PROCESS_KIND_UNSANDBOXED);
 #endif
@@ skipping 760 matching lines @@
   if (!cache.get())
     return;
 
   cache->set_shader_loaded_callback(base::Bind(&GpuProcessHost::LoadedShader,
                                                weak_ptr_factory_.GetWeakPtr()));
 
   client_id_to_shader_cache_[client_id] = cache;
 }
 
 }  // namespace content