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

Unified Diff: mojo/public/dart/src/application_connection.dart

Issue 954653002: Make Sky embedder.connection deal with null handles (Closed) Base URL: https://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 | « no previous file | sky/framework/embedder.dart » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: mojo/public/dart/src/application_connection.dart
diff --git a/mojo/public/dart/src/application_connection.dart b/mojo/public/dart/src/application_connection.dart
index 4f314686a51bb58796a30c91be753758a08f5d8d..c44944e2a21e02bc9693367cae3359d7e67465e1 100644
--- a/mojo/public/dart/src/application_connection.dart
+++ b/mojo/public/dart/src/application_connection.dart
@@ -74,11 +74,9 @@ class ApplicationConnection {
assert(!proxy.isBound &&
(remoteServiceProvider != null) &&
remoteServiceProvider.isBound);
- var applicationPipe = new core.MojoMessagePipe();
- var proxyEndpoint = applicationPipe.endpoints[0];
- var applicationEndpoint = applicationPipe.endpoints[1];
- proxy.bind(proxyEndpoint);
- remoteServiceProvider.connectToService(proxy.name, applicationEndpoint);
+ var pipe = new core.MojoMessagePipe();
+ proxy.bind(pipe.endpoints[0]);
+ remoteServiceProvider.connectToService(proxy.name, pipe.endpoints[1]);
return proxy;
}
« no previous file with comments | « no previous file | sky/framework/embedder.dart » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698