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

Unified Diff: services/view_manager/view_manager_service_unittest.cc

Issue 893183003: De-client the ViewManager interfaces (Closed) Base URL: git@github.com:domokit/mojo.git@master
Patch Set: Created 5 years, 10 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 | « services/view_manager/view_manager_service_impl.cc ('k') | services/window_manager/window_manager_app.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: services/view_manager/view_manager_service_unittest.cc
diff --git a/services/view_manager/view_manager_service_unittest.cc b/services/view_manager/view_manager_service_unittest.cc
index b492a1430a99e78f7b56e4e2c52dde2ca5e7f148..b684fa5f260780b3242c4e62da94fde9b616a9ab 100644
--- a/services/view_manager/view_manager_service_unittest.cc
+++ b/services/view_manager/view_manager_service_unittest.cc
@@ -51,6 +51,7 @@ class TestViewManagerClient : public mojo::ViewManagerClient {
void OnEmbed(uint16_t connection_id,
const String& embedder_url,
ViewDataPtr root,
+ mojo::ViewManagerServicePtr view_manager_service,
InterfaceRequest<ServiceProvider> services,
ServiceProviderPtr exposed_services,
mojo::ScopedMessagePipeHandle window_manager_pipe) override {
@@ -110,9 +111,7 @@ class TestViewManagerClient : public mojo::ViewManagerClient {
class TestClientConnection : public ClientConnection {
public:
explicit TestClientConnection(scoped_ptr<ViewManagerServiceImpl> service_impl)
- : ClientConnection(service_impl.Pass()) {
- set_client(&client_);
- }
+ : ClientConnection(service_impl.Pass(), &client_) {}
~TestClientConnection() override {}
TestViewManagerClient* client() { return &client_; }
@@ -143,6 +142,7 @@ class TestConnectionManagerDelegate : public ConnectionManagerDelegate {
ClientConnection* CreateClientConnectionForEmbedAtView(
ConnectionManager* connection_manager,
+ mojo::InterfaceRequest<mojo::ViewManagerService> service_request,
mojo::ConnectionSpecificId creator_id,
const std::string& creator_url,
const std::string& url,
@@ -193,6 +193,7 @@ class TestWindowManagerInternal : public mojo::WindowManagerInternal {
void CreateWindowManagerForViewManagerClient(
uint16_t connection_id,
mojo::ScopedMessagePipeHandle window_manager_pipe) override {}
+ void SetViewManagerClient(mojo::ScopedMessagePipeHandle) override {}
private:
DISALLOW_COPY_AND_ASSIGN(TestWindowManagerInternal);
« no previous file with comments | « services/view_manager/view_manager_service_impl.cc ('k') | services/window_manager/window_manager_app.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698