Dart: Adds optional named arguments for creating bindings.

mojo/public/dart/src/application_connection.dart

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 4f0b6c8ed250056b670b9b7cddee8faeb53c7b07..05c1cab0f671deac0d42e3be2a87c3ff502635ea 100644
--- a/mojo/public/dart/src/application_connection.dart
+++ b/mojo/public/dart/src/application_connection.dart
@@ -14,10 +14,10 @@ class LocalServiceProvider implements ServiceProvider {
   ServiceProviderStub _stub;
 
   LocalServiceProvider(this.connection, this._stub) {
-    _stub.delegate = this;
+    _stub.impl = this;
   }
 
-  listen() => _stub.listen();
+  listen({Function onClosed}) => _stub.listen(onClosed: onClosed);

[sky, 2015/03/03 20:14:13]

Is there a reason not to follow the closer pattern every where like you did in
Application? Seems like everything should be closed nicely without having to
supply onClosed. The only reason to supply an onClosed is if additional
processing is necessary when something is closed unexpectedly.

[zra, 2015/03/03 21:28:54]

Done. Added here and in ApplicationConnection.

 
   void close({bool nodefer : false}) => _stub.close(nodefer: nodefer);
 
@@ -64,10 +64,11 @@ class ApplicationConnection {
   final _nameToServiceFactory = new Map<String, ServiceFactory>();
   FallbackServiceFactory _fallbackServiceFactory;
 
-  ApplicationConnection(ServiceProviderStub stub, ServiceProviderProxy proxy)
+  ApplicationConnection(
+      ServiceProviderStub stub, ServiceProviderProxy proxy)
       : remoteServiceProvider = proxy {
-    if (stub != null) _localServiceProvider =
-        new LocalServiceProvider(this, stub);
+    if (stub != null)
+      _localServiceProvider = new LocalServiceProvider(this, stub);
   }
 
   FallbackServiceFactory get fallbackServiceFactory => _fallbackServiceFactory;
@@ -92,9 +93,9 @@ class ApplicationConnection {
     _nameToServiceFactory[interfaceName] = factory;
   }
 
-  void listen() {
+  void listen({Function onClosed}) {
     assert(_localServiceProvider != null);
-    _localServiceProvider.listen();
+    _localServiceProvider.listen(onClosed: onClosed);
   }
 
   void close({bool nodefer: false}) {