Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(489)

Unified Diff: content/gpu/gpu_child_thread.cc

Issue 11574049: Only collect GL/D3D context specific GPUInfo in GPU process (Closed) Base URL: svn://chrome-svn/chrome/trunk/src/
Patch Set: Created 8 years ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: content/gpu/gpu_child_thread.cc
===================================================================
--- content/gpu/gpu_child_thread.cc (revision 173184)
+++ content/gpu/gpu_child_thread.cc (working copy)
@@ -74,7 +74,6 @@
}
}
-
GpuChildThread::~GpuChildThread() {
logging::SetLogMessageHandler(NULL);
}
@@ -134,7 +133,6 @@
// take a significant amount of time.
gpu_info_.initialization_time = base::Time::Now() - process_start_time_;
-
// Defer creation of the render thread. This is to prevent it from handling
// IPC messages before the sandbox has been enabled and all other necessary
// initialization has succeeded.
@@ -144,11 +142,9 @@
ChildProcess::current()->io_message_loop_proxy(),
ChildProcess::current()->GetShutDownEvent()));
-#if defined(OS_LINUX)
// Ensure the browser process receives the GPU info before a reply to any
// subsequent IPC it might send.
Send(new GpuHostMsg_GraphicsInfoCollected(gpu_info_));
-#endif
}
void GpuChildThread::StopWatchdog() {
@@ -169,10 +165,8 @@
// Sandbox state is not part of the gpu info collection. It is determined
// in GpuMain() and passed down to GpuChildThread.
Ken Russell (switch to Gerrit) 2012/12/15 00:33:56 Is this comment still relevant?
- bool sandboxed = gpu_info_.sandboxed;
- if (!gpu_info_collector::CollectGraphicsInfo(&gpu_info_))
+ if (!gpu_info_collector::CollectContextGraphicsInfo(&gpu_info_))
VLOG(1) << "gpu_info_collector::CollectGraphicsInfo failed";
- gpu_info_.sandboxed = sandboxed;
GetContentClient()->SetGpuInfo(gpu_info_);
#if defined(OS_WIN)

Powered by Google App Engine
This is Rietveld 408576698