Add a full Blimp integration test.

blimp/client/public/blimp_client_context.h

 // Copyright 2016 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef BLIMP_CLIENT_PUBLIC_BLIMP_CLIENT_CONTEXT_H_
 #define BLIMP_CLIENT_PUBLIC_BLIMP_CLIENT_CONTEXT_H_
 
 #include <memory>
 #include <string>
 
@@ skipping 46 matching lines @@
   virtual void SetDelegate(BlimpClientContextDelegate* delegate) = 0;
 
   // Creates a new BlimpContents that will be shown in |window|.
   // TODO(mlliu): Currently we want to have a single BlimpContents. If there is
   // an existing contents, return nullptr (http://crbug.com/642558).
   virtual std::unique_ptr<BlimpContents> CreateBlimpContents(
       gfx::NativeWindow window) = 0;
 
   // Start authentication flow and connection to engine.
   virtual void Connect() = 0;
+  virtual void ConnectWithAssignment(const Assignment& assignment) = 0;

[Khushal, 2016/09/13 04:47:24]

If its never meant to be used in production code, add ForTesting to this?

[David Trainor- moved to gerrit, 2016/09/13 06:18:04]

It's actually used by other methods internally when I cleaned up the connection
flow (Connect() -> ...... -> ConnectWithAssignment()).  I could add ForTesting
here and call a valid method internally, but that might be more ugly :(.  wdyt?

[Khushal, 2016/09/13 23:47:10]

Its fine, doesn't look like something too bad to be on the public API.

 
  protected:
   BlimpClientContext() = default;
 
  private:
   DISALLOW_COPY_AND_ASSIGN(BlimpClientContext);
 };
 
 }  // namespace client
 }  // namespace blimp
 
 #endif  // BLIMP_CLIENT_PUBLIC_BLIMP_CLIENT_CONTEXT_H_