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

Unified Diff: mojo/public/cpp/bindings/lib/interface_impl_internal.h

Issue 289063015: Mojo: Remove SetClient from generated interfaces (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: rebase Created 6 years, 7 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 | « mojo/public/cpp/bindings/interface_ptr.h ('k') | mojo/public/cpp/bindings/lib/interface_ptr_internal.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: mojo/public/cpp/bindings/lib/interface_impl_internal.h
diff --git a/mojo/public/cpp/bindings/lib/interface_impl_internal.h b/mojo/public/cpp/bindings/lib/interface_impl_internal.h
index 77652909a5fc90c61f03c44ec55cb31f0be51c57..bf770c182c40f0dddf4040569c47d420133ddd19 100644
--- a/mojo/public/cpp/bindings/lib/interface_impl_internal.h
+++ b/mojo/public/cpp/bindings/lib/interface_impl_internal.h
@@ -29,7 +29,6 @@ class InterfaceImplState : public ErrorHandler {
explicit InterfaceImplState(InterfaceImplBase<Interface>* instance)
: router_(NULL),
- client_(NULL),
proxy_(NULL) {
assert(instance);
stub_.set_sink(instance);
@@ -66,14 +65,11 @@ class InterfaceImplState : public ErrorHandler {
proxy_ = new typename Client::Proxy_(router_);
- instance()->SetClient(proxy_);
instance()->OnConnectionEstablished();
}
Router* router() { return router_; }
-
- void set_client(Client* client) { client_ = client; }
- Client* client() { return client_; }
+ Client* client() { return proxy_; }
private:
InterfaceImplBase<Interface>* instance() {
@@ -85,7 +81,6 @@ class InterfaceImplState : public ErrorHandler {
}
Router* router_;
- Client* client_;
typename Client::Proxy_* proxy_;
typename Interface::Stub_ stub_;
« no previous file with comments | « mojo/public/cpp/bindings/interface_ptr.h ('k') | mojo/public/cpp/bindings/lib/interface_ptr_internal.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698