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

Unified Diff: services/ui/ws/window_server.cc

Issue 2797453002: Rename DisplayCompositor to MojoFrameSinkManager. (Closed)
Patch Set: Rebase. Created 3 years, 8 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: services/ui/ws/window_server.cc
diff --git a/services/ui/ws/window_server.cc b/services/ui/ws/window_server.cc
index 364c06b29a6ef2c01c673fb5b5e7d627a16e9083..0e1e3a40d8133dc476fff69f41599ba88e6fffa4 100644
--- a/services/ui/ws/window_server.cc
+++ b/services/ui/ws/window_server.cc
@@ -45,8 +45,6 @@ struct WindowServer::CurrentDragLoopState {
WindowTree* initiator;
};
-// TODO(fsamuel): DisplayCompositor should be a mojo interface dispensed by
-// GpuHost.
WindowServer::WindowServer(WindowServerDelegate* delegate)
: delegate_(delegate),
next_client_id_(1),
@@ -56,12 +54,12 @@ WindowServer::WindowServer(WindowServerDelegate* delegate)
next_wm_change_id_(0),
gpu_host_(new GpuHost(this)),
window_manager_window_tree_factory_set_(this, &user_id_tracker_),
- display_compositor_client_binding_(this) {
+ frame_sink_manager_client_binding_(this) {
user_id_tracker_.AddObserver(this);
OnUserIdAdded(user_id_tracker_.active_id());
- gpu_host_->CreateDisplayCompositor(
- mojo::MakeRequest(&display_compositor_),
- display_compositor_client_binding_.CreateInterfacePtrAndBind());
+ gpu_host_->CreateFrameSinkManager(
+ mojo::MakeRequest(&frame_sink_manager_),
+ frame_sink_manager_client_binding_.CreateInterfacePtrAndBind());
}
WindowServer::~WindowServer() {
@@ -535,8 +533,8 @@ WindowManagerState* WindowServer::GetWindowManagerStateForUser(
user_id);
}
-cc::mojom::DisplayCompositor* WindowServer::GetDisplayCompositor() {
- return display_compositor_.get();
+cc::mojom::FrameSinkManager* WindowServer::GetFrameSinkManager() {
+ return frame_sink_manager_.get();
}
bool WindowServer::GetFrameDecorationsForUser(
@@ -638,7 +636,7 @@ void WindowServer::HandleTemporaryReferenceForNewSurface(
current->GetOrCreateCompositorFrameSinkManager()->ClaimTemporaryReference(
surface_id);
} else {
- display_compositor_->DropTemporaryReference(surface_id);
+ frame_sink_manager_->DropTemporaryReference(surface_id);
}
}
@@ -675,11 +673,11 @@ void WindowServer::OnWindowHierarchyChanged(ServerWindow* window,
ProcessWindowHierarchyChanged(window, new_parent, old_parent);
if (old_parent) {
- display_compositor_->UnregisterFrameSinkHierarchy(
+ frame_sink_manager_->UnregisterFrameSinkHierarchy(
old_parent->frame_sink_id(), window->frame_sink_id());
}
if (new_parent) {
- display_compositor_->RegisterFrameSinkHierarchy(new_parent->frame_sink_id(),
+ frame_sink_manager_->RegisterFrameSinkHierarchy(new_parent->frame_sink_id(),
window->frame_sink_id());
}
@@ -818,7 +816,7 @@ void WindowServer::OnSurfaceCreated(const cc::SurfaceInfo& surface_info) {
// If the window doesn't exist then we have nothing to propagate.
if (!window) {
- display_compositor_->DropTemporaryReference(surface_info.id());
+ frame_sink_manager_->DropTemporaryReference(surface_info.id());
return;
}
« no previous file with comments | « services/ui/ws/window_server.h ('k') | third_party/WebKit/Source/platform/graphics/CanvasSurfaceLayerBridge.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698