pkg/isolate: library renaming, removed unused method, fix creation of TimeoutException

lib/ports.dart

 // Copyright (c) 2015, the Dart project authors.  Please see the AUTHORS file
 // for details. All rights reserved. Use of this source code is governed by a
 // BSD-style license that can be found in the LICENSE file.
 
 /// Utility functions for setting up ports and sending data.
 ///
 /// This library contains a number of functions that handle the
 /// boiler-plate of setting up a receive port and receiving a
 /// single message on the port.
 ///
 /// There are different functions that offer different ways to
 /// handle the incoming message.
 ///
 /// The simplest function, [singleCallbackPort], takes a callback
 /// and returns a port, and then calls the callback for the first
 /// message sent on the port.
 ///
 /// Other functions intercept the returned value and either
 /// does something with it, or puts it into a [Future] or [Completer].
-library dart.pkg.isolate.ports;
+library isolate.ports;
 
+import "dart:async";
 import "dart:isolate";
-import "dart:async";
+
 import "src/lists.dart";
 
 /// Create a [SendPort] that accepts only one message.
 ///
 /// The [callback] function is called once, with the first message
 /// received by the receive port.
 /// All futher messages are ignored.
 ///
 /// If [timeout] is supplied, it is used as a limit on how
 /// long it can take before the message is received. If a
@@ skipping 258 matching lines @@
         _completer = new Completer.sync(),
         _callback = callback,
         _zone = Zone.current {
     _receivePort.handler = _handleResponse;
     if (timeout != null) {
       _timer = new Timer(timeout, () {
         // Executed as a timer event.
         _receivePort.close();
         if (!_completer.isCompleted) {
           if (throwOnTimeout) {
-            _completer.completeError(new TimeoutException(timeout));
+            _completer.completeError(
+                new TimeoutException('Timeout waiting for response, timeout'));

[Lasse Reichstein Nielsen, 2015/03/24 10:35:38]

I think "timeout" should be the second argument, not part of the string.

[kevmoo, 2015/03/24 13:21:19]

Done.

           } else if (onTimeout != null) {
             _completer.complete(new Future.sync(onTimeout));
           } else {
             _completer.complete(timeoutValue);
           }
         }
       });
     }
   }
 
@@ skipping 37 matching lines @@
       // what the user expects, and because it may create an error that needs
       // to be propagated to the original completer. If that completer was
       // created in a different error zone, an error from the root zone
       // would become uncaught.
       _zone.run(() {
         _completer.complete(new Future.sync(() => _callback(v)));
       });
     }
   }
 }