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

Unified Diff: blimp/client/app/compositor/browser_compositor.cc

Issue 2317753002: cc: Abstract the LayerTreeHost. (Closed)
Patch Set: Rebase Created 4 years, 3 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 side-by-side diff with in-line comments
Download patch
« no previous file with comments | « blimp/client/app/compositor/browser_compositor.h ('k') | blimp/client/core/compositor/blimp_compositor.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: blimp/client/app/compositor/browser_compositor.cc
diff --git a/blimp/client/app/compositor/browser_compositor.cc b/blimp/client/app/compositor/browser_compositor.cc
index a76bd23ac245529cb6210db8a928f455c76357d7..81e278eb340cf6a64b55f072c62ca3d3497b5538 100644
--- a/blimp/client/app/compositor/browser_compositor.cc
+++ b/blimp/client/app/compositor/browser_compositor.cc
@@ -91,7 +91,7 @@ BrowserCompositor::BrowserCompositor(
root_layer_->SetBackgroundColor(SK_ColorWHITE);
host_->GetLayerTree()->SetRootLayer(root_layer_);
- host_->set_surface_client_id(surface_id_allocator_->client_id());
+ host_->SetSurfaceClientId(surface_id_allocator_->client_id());
}
BrowserCompositor::~BrowserCompositor() {
@@ -122,7 +122,7 @@ void BrowserCompositor::SetAcceleratedWidget(gfx::AcceleratedWidget widget) {
// Kill all references to the old widget.
if (widget_ != gfx::kNullAcceleratedWidget) {
// We are always visible if we have a widget.
- DCHECK(host_->visible());
+ DCHECK(host_->IsVisible());
host_->SetVisible(false);
host_->ReleaseOutputSurface();
display_.reset();
@@ -164,7 +164,7 @@ void BrowserCompositor::HandlePendingOutputSurfaceRequest() {
DCHECK(output_surface_request_pending_);
// Can't handle the request right now since we don't have a widget.
- if (!host_->visible())
+ if (!host_->IsVisible())
return;
DCHECK_NE(gfx::kNullAcceleratedWidget, widget_);
@@ -188,7 +188,7 @@ void BrowserCompositor::HandlePendingOutputSurfaceRequest() {
display_ = base::MakeUnique<cc::Display>(
nullptr /*shared_bitmap_manager*/, gpu_memory_buffer_manager,
- host_->settings().renderer_settings, std::move(begin_frame_source),
+ host_->GetSettings().renderer_settings, std::move(begin_frame_source),
std::move(display_output_surface), std::move(scheduler),
base::MakeUnique<cc::TextureMailboxDeleter>(task_runner));
display_->SetVisible(true);
« no previous file with comments | « blimp/client/app/compositor/browser_compositor.h ('k') | blimp/client/core/compositor/blimp_compositor.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698