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

Side by Side Diff: content/browser/compositor/gpu_process_transport_factory.cc

Issue 1336733002: Re-land: cc: Implement shared worker contexts. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Fix LayerTreeHostClientTakeAwayOutputSurface test. Content provider is always destroyed on the clie… Created 5 years, 2 months 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 unified diff | Download patch
OLDNEW
1 // Copyright 2014 The Chromium Authors. All rights reserved. 1 // Copyright 2014 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "content/browser/compositor/gpu_process_transport_factory.h" 5 #include "content/browser/compositor/gpu_process_transport_factory.h"
6 6
7 #include <string> 7 #include <string>
8 8
9 #include "base/bind.h" 9 #include "base/bind.h"
10 #include "base/command_line.h" 10 #include "base/command_line.h"
(...skipping 226 matching lines...) Expand 10 before | Expand all | Expand 10 after
237 #if defined(OS_CHROMEOS) 237 #if defined(OS_CHROMEOS)
238 LOG(FATAL) << "Unable to create a UI graphics context, and cannot use " 238 LOG(FATAL) << "Unable to create a UI graphics context, and cannot use "
239 << "software compositing on ChromeOS."; 239 << "software compositing on ChromeOS.";
240 #endif 240 #endif
241 create_gpu_output_surface = false; 241 create_gpu_output_surface = false;
242 } 242 }
243 243
244 scoped_refptr<ContextProviderCommandBuffer> context_provider; 244 scoped_refptr<ContextProviderCommandBuffer> context_provider;
245 if (create_gpu_output_surface) { 245 if (create_gpu_output_surface) {
246 // Try to reuse existing worker context provider. 246 // Try to reuse existing worker context provider.
247 bool shared_worker_context_provider_lost = false;
247 if (shared_worker_context_provider_) { 248 if (shared_worker_context_provider_) {
249 // Note: If context is lost, we delete reference after releasing the lock.
248 base::AutoLock lock(*shared_worker_context_provider_->GetLock()); 250 base::AutoLock lock(*shared_worker_context_provider_->GetLock());
249 if (shared_worker_context_provider_->ContextGL() 251 if (shared_worker_context_provider_->ContextGL()
250 ->GetGraphicsResetStatusKHR() != GL_NO_ERROR) 252 ->GetGraphicsResetStatusKHR() != GL_NO_ERROR) {
251 shared_worker_context_provider_ = nullptr; 253 shared_worker_context_provider_lost = true;
254 }
252 } 255 }
253 scoped_refptr<GpuChannelHost> gpu_channel_host = 256 scoped_refptr<GpuChannelHost> gpu_channel_host =
254 BrowserGpuChannelHostFactory::instance()->GetGpuChannel(); 257 BrowserGpuChannelHostFactory::instance()->GetGpuChannel();
255 if (gpu_channel_host.get()) { 258 if (gpu_channel_host.get()) {
256 context_provider = ContextProviderCommandBuffer::Create( 259 context_provider = ContextProviderCommandBuffer::Create(
257 GpuProcessTransportFactory::CreateContextCommon(gpu_channel_host, 260 GpuProcessTransportFactory::CreateContextCommon(gpu_channel_host,
258 data->surface_id), 261 data->surface_id),
259 BROWSER_COMPOSITOR_ONSCREEN_CONTEXT); 262 BROWSER_COMPOSITOR_ONSCREEN_CONTEXT);
260 if (context_provider && !context_provider->BindToCurrentThread()) 263 if (context_provider && !context_provider->BindToCurrentThread())
261 context_provider = nullptr; 264 context_provider = nullptr;
262 if (!shared_worker_context_provider_) { 265 if (!shared_worker_context_provider_ ||
266 shared_worker_context_provider_lost) {
263 shared_worker_context_provider_ = ContextProviderCommandBuffer::Create( 267 shared_worker_context_provider_ = ContextProviderCommandBuffer::Create(
264 GpuProcessTransportFactory::CreateContextCommon(gpu_channel_host, 268 GpuProcessTransportFactory::CreateContextCommon(gpu_channel_host,
265 0), 269 0),
266 BROWSER_WORKER_CONTEXT); 270 BROWSER_WORKER_CONTEXT);
267 if (shared_worker_context_provider_ && 271 if (shared_worker_context_provider_ &&
268 !shared_worker_context_provider_->BindToCurrentThread()) 272 !shared_worker_context_provider_->BindToCurrentThread())
269 shared_worker_context_provider_ = nullptr; 273 shared_worker_context_provider_ = nullptr;
274 if (shared_worker_context_provider_)
275 shared_worker_context_provider_->SetupLock();
270 } 276 }
271 } 277 }
272 278
273 bool created_gpu_browser_compositor = 279 bool created_gpu_browser_compositor =
274 !!context_provider && !!shared_worker_context_provider_; 280 !!context_provider && !!shared_worker_context_provider_;
275 281
276 UMA_HISTOGRAM_BOOLEAN("Aura.CreatedGpuBrowserCompositor", 282 UMA_HISTOGRAM_BOOLEAN("Aura.CreatedGpuBrowserCompositor",
277 created_gpu_browser_compositor); 283 created_gpu_browser_compositor);
278 284
279 if (!created_gpu_browser_compositor) { 285 if (!created_gpu_browser_compositor) {
(...skipping 346 matching lines...) Expand 10 before | Expand all | Expand 10 after
626 FOR_EACH_OBSERVER(ImageTransportFactoryObserver, 632 FOR_EACH_OBSERVER(ImageTransportFactoryObserver,
627 observer_list_, 633 observer_list_,
628 OnLostResources()); 634 OnLostResources());
629 635
630 // Kill things that use the shared context before killing the shared context. 636 // Kill things that use the shared context before killing the shared context.
631 lost_gl_helper.reset(); 637 lost_gl_helper.reset();
632 lost_shared_main_thread_contexts = NULL; 638 lost_shared_main_thread_contexts = NULL;
633 } 639 }
634 640
635 } // namespace content 641 } // namespace content
OLDNEW
« no previous file with comments | « content/browser/android/in_process/synchronous_compositor_output_surface.cc ('k') | content/child/child_thread_impl.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698