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

Unified Diff: components/view_manager/view_manager_app.cc

Issue 1281663002: Mandoline: Allow submitting CompositorFrames directly to mojo::Views (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Rebased Created 5 years, 4 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 | « components/view_manager/view_manager_app.h ('k') | components/view_manager/view_tree_impl.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: components/view_manager/view_manager_app.cc
diff --git a/components/view_manager/view_manager_app.cc b/components/view_manager/view_manager_app.cc
index 4cac58c8e8baaf232675c5e2dfaac93e1af8c61c..69b3288d357e14fb4ffd4d5ac1df81667038c856 100644
--- a/components/view_manager/view_manager_app.cc
+++ b/components/view_manager/view_manager_app.cc
@@ -10,7 +10,6 @@
#include "components/view_manager/connection_manager.h"
#include "components/view_manager/gles2/gpu_impl.h"
#include "components/view_manager/public/cpp/args.h"
-#include "components/view_manager/surfaces/surfaces_impl.h"
#include "components/view_manager/surfaces/surfaces_scheduler.h"
#include "components/view_manager/view_tree_host_connection.h"
#include "components/view_manager/view_tree_host_impl.h"
@@ -41,10 +40,6 @@ ViewManagerApp::ViewManagerApp()
ViewManagerApp::~ViewManagerApp() {
if (gpu_state_)
gpu_state_->StopControlThread();
-
- auto surfaces = surfaces_;
- for (auto& surface : surfaces)
- surface->CloseConnection();
}
void ViewManagerApp::Initialize(ApplicationImpl* app) {
@@ -66,16 +61,13 @@ void ViewManagerApp::Initialize(ApplicationImpl* app) {
if (!gpu_state_.get())
gpu_state_ = new gles2::GpuState;
- connection_manager_.reset(new ConnectionManager(this));
+ connection_manager_.reset(new ConnectionManager(this, surfaces_state_));
}
bool ViewManagerApp::ConfigureIncomingConnection(
ApplicationConnection* connection) {
// ViewManager
connection->AddService<ViewTreeHostFactory>(this);
- // Surfaces
- // TODO(fsamuel): This should go away soon.
- connection->AddService<mojo::Surface>(this);
// GPU
connection->AddService<Gpu>(this);
return true;
@@ -126,18 +118,6 @@ void ViewManagerApp::Create(
new gles2::GpuImpl(request.Pass(), gpu_state_);
}
-void ViewManagerApp::Create(
- mojo::ApplicationConnection* connection,
- mojo::InterfaceRequest<mojo::Surface> request) {
- surfaces_.insert(
- new surfaces::SurfacesImpl(this, surfaces_state_, request.Pass()));
-}
-
-void ViewManagerApp::OnSurfaceConnectionClosed(
- surfaces::SurfacesImpl* surface) {
- surfaces_.erase(surface);
-}
-
void ViewManagerApp::CreateViewTreeHost(
mojo::InterfaceRequest<mojo::ViewTreeHost> host,
mojo::ViewTreeHostClientPtr host_client,
« no previous file with comments | « components/view_manager/view_manager_app.h ('k') | components/view_manager/view_tree_impl.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698