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

Side by Side Diff: content/renderer/gpu/render_widget_compositor.cc

Issue 1126253005: cc: Add LayerTreeHost::InitParams for LayerTreeHost creation. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: all Created 5 years, 7 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 (c) 2013 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2013 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/renderer/gpu/render_widget_compositor.h" 5 #include "content/renderer/gpu/render_widget_compositor.h"
6 6
7 #include <limits> 7 #include <limits>
8 #include <string> 8 #include <string>
9 9
10 #include "base/command_line.h" 10 #include "base/command_line.h"
(...skipping 461 matching lines...) Expand 10 before | Expand all | Expand 10 after
472 compositor_deps_->GetGpuMemoryBufferManager(); 472 compositor_deps_->GetGpuMemoryBufferManager();
473 cc::TaskGraphRunner* task_graph_runner = 473 cc::TaskGraphRunner* task_graph_runner =
474 compositor_deps_->GetTaskGraphRunner(); 474 compositor_deps_->GetTaskGraphRunner();
475 475
476 scoped_ptr<cc::BeginFrameSource> external_begin_frame_source; 476 scoped_ptr<cc::BeginFrameSource> external_begin_frame_source;
477 if (settings.use_external_begin_frame_source) { 477 if (settings.use_external_begin_frame_source) {
478 external_begin_frame_source = 478 external_begin_frame_source =
479 compositor_deps_->CreateExternalBeginFrameSource(widget_->routing_id()); 479 compositor_deps_->CreateExternalBeginFrameSource(widget_->routing_id());
480 } 480 }
481 481
482 cc::LayerTreeHost::InitParams params;
483 params.client = this;
484 params.shared_bitmap_manager = shared_bitmap_manager;
485 params.gpu_memory_buffer_manager = gpu_memory_buffer_manager;
486 params.task_graph_runner = task_graph_runner;
danakj 2015/05/07 00:59:56 no settings here either
sadrul 2015/05/07 04:39:26 Fixed (thankfully, lots of try failures from this
487 params.main_task_runner = main_thread_compositor_task_runner;
488 params.external_begin_frame_source = external_begin_frame_source.Pass();
489
482 if (compositor_thread_task_runner.get()) { 490 if (compositor_thread_task_runner.get()) {
483 layer_tree_host_ = cc::LayerTreeHost::CreateThreaded( 491 layer_tree_host_ = cc::LayerTreeHost::CreateThreaded(
484 this, shared_bitmap_manager, gpu_memory_buffer_manager, 492 compositor_thread_task_runner, &params);
485 task_graph_runner, settings, main_thread_compositor_task_runner,
486 compositor_thread_task_runner, external_begin_frame_source.Pass());
487 } else { 493 } else {
488 layer_tree_host_ = cc::LayerTreeHost::CreateSingleThreaded( 494 layer_tree_host_ = cc::LayerTreeHost::CreateSingleThreaded(this, &params);
489 this, this, shared_bitmap_manager, gpu_memory_buffer_manager,
490 task_graph_runner, settings, main_thread_compositor_task_runner,
491 external_begin_frame_source.Pass());
492 } 495 }
493 DCHECK(layer_tree_host_); 496 DCHECK(layer_tree_host_);
494 } 497 }
495 498
496 RenderWidgetCompositor::~RenderWidgetCompositor() {} 499 RenderWidgetCompositor::~RenderWidgetCompositor() {}
497 500
498 const base::WeakPtr<cc::InputHandler>& 501 const base::WeakPtr<cc::InputHandler>&
499 RenderWidgetCompositor::GetInputHandler() { 502 RenderWidgetCompositor::GetInputHandler() {
500 return layer_tree_host_->GetInputHandler(); 503 return layer_tree_host_->GetInputHandler();
501 } 504 }
(...skipping 438 matching lines...) Expand 10 before | Expand all | Expand 10 after
940 cc::ContextProvider* provider = 943 cc::ContextProvider* provider =
941 compositor_deps_->GetSharedMainThreadContextProvider(); 944 compositor_deps_->GetSharedMainThreadContextProvider();
942 // provider can be NULL after the GPU process crashed enough times and we 945 // provider can be NULL after the GPU process crashed enough times and we
943 // don't want to restart it any more (falling back to software). 946 // don't want to restart it any more (falling back to software).
944 if (!provider) 947 if (!provider)
945 return; 948 return;
946 provider->ContextGL()->RateLimitOffscreenContextCHROMIUM(); 949 provider->ContextGL()->RateLimitOffscreenContextCHROMIUM();
947 } 950 }
948 951
949 } // namespace content 952 } // namespace content
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698