[Blimp Net] Makes BlimpTransport.Connect async-only

BUG=558643

Review URL: https://codereview.chromium.org/1459923004

Cr-Commit-Position: refs/heads/master@{#360919}

1 files changed, 6 insertions, 8 deletions

diff --git a/blimp/net/blimp_transport.h b/blimp/net/blimp_transport.h
index 56d5e34..dcc68c6 100644
--- a/blimp/net/blimp_transport.h
+++ b/blimp/net/blimp_transport.h
@@ -22,14 +22,12 @@ class BlimpTransport {
 
   // Initiate or listen for a connection.
   //
-  // Returns net::OK if a connection is established synchronously.
-  // Returns an error code if a synchronous error occurred.
-  // Returns net::ERR_IO_PENDING if the connection is being established
-  // asynchronously. |callback| is later invoked with the connection outcome.
-  //
-  // If the connection is successful, the BlimpConnection can be taken by
-  // calling TakeConnectedSocket().
-  virtual int Connect(const net::CompletionCallback& callback) = 0;
+  // |callback| will be invoked with the connection outcome:
+  //   * net::OK if a connection is established successful, the BlimpConnection
+  //     can be taken by calling TakeConnection().
+  //   * net::ERR_IO_PENDING will never be the outcome
+  //   * Other error code indicates no connection was established.
+  virtual void Connect(const net::CompletionCallback& callback) = 0;
 
   // Returns the connection object after a successful Connect().
   virtual scoped_ptr<BlimpConnection> TakeConnection() = 0;