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

Unified Diff: blimp/client/support/compositor/blimp_embedder_compositor.cc

Issue 2382873002: Replace usage of SurfaceId's client_id with FrameSinkId (Closed)
Patch Set: Rebased Created 4 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 side-by-side diff with in-line comments
Download patch
Index: blimp/client/support/compositor/blimp_embedder_compositor.cc
diff --git a/blimp/client/support/compositor/blimp_embedder_compositor.cc b/blimp/client/support/compositor/blimp_embedder_compositor.cc
index c0a95d4a49944a6a60238c9de22d0ada13ab486d..62eb7415d0551e34a00f55994975fe082018298f 100644
--- a/blimp/client/support/compositor/blimp_embedder_compositor.cc
+++ b/blimp/client/support/compositor/blimp_embedder_compositor.cc
@@ -86,11 +86,11 @@ BlimpEmbedderCompositor::BlimpEmbedderCompositor(
CompositorDependencies* compositor_dependencies)
: compositor_dependencies_(compositor_dependencies),
surface_id_allocator_(base::MakeUnique<cc::SurfaceIdAllocator>(
- compositor_dependencies->AllocateSurfaceClientId())),
+ compositor_dependencies->AllocateFrameSinkId())),
compositor_frame_sink_request_pending_(false),
root_layer_(cc::Layer::Create()) {
- compositor_dependencies_->GetSurfaceManager()->RegisterSurfaceClientId(
- surface_id_allocator_->client_id());
+ compositor_dependencies_->GetSurfaceManager()->RegisterFrameSinkId(
+ surface_id_allocator_->frame_sink_id());
cc::LayerTreeHostInProcess::InitParams params;
params.client = this;
@@ -105,14 +105,14 @@ BlimpEmbedderCompositor::BlimpEmbedderCompositor(
root_layer_->SetBackgroundColor(SK_ColorWHITE);
host_->GetLayerTree()->SetRootLayer(root_layer_);
- host_->SetSurfaceClientId(surface_id_allocator_->client_id());
+ host_->SetFrameSinkId(surface_id_allocator_->frame_sink_id());
host_->SetVisible(true);
}
BlimpEmbedderCompositor::~BlimpEmbedderCompositor() {
SetContextProvider(nullptr);
- compositor_dependencies_->GetSurfaceManager()->InvalidateSurfaceClientId(
- surface_id_allocator_->client_id());
+ compositor_dependencies_->GetSurfaceManager()->InvalidateFrameSinkId(
+ surface_id_allocator_->frame_sink_id());
}
void BlimpEmbedderCompositor::SetContentLayer(

Powered by Google App Engine
This is Rietveld 408576698