Tests for the Copresence API.

components/copresence/public/copresence_client.h

Index: components/copresence/public/copresence_client.h
diff --git a/components/copresence/public/copresence_client.h b/components/copresence/public/copresence_client.h
index 40b8bf97bfe4720e83416fae7eabd5e89574a568..6b7c6b3d1de08c545cc22959cf4a4b18032ae1d6 100644
--- a/components/copresence/public/copresence_client.h
+++ b/components/copresence/public/copresence_client.h
@@ -41,18 +41,20 @@ struct PendingRequest {
 // client.
 class CopresenceClient : public base::SupportsWeakPtr<CopresenceClient> {
  public:
-  // The delegate must outlive us.
-  explicit CopresenceClient(CopresenceClientDelegate* delegate);
+  CopresenceClient();
   virtual ~CopresenceClient();
 
+  // The delegate must outlive us.
+  virtual void Initialize(CopresenceClientDelegate* delegate);

[not at google - send to devlin, 2014/08/06 21:45:27]

I don't really like having separate Initialize functions, it adds complexity to
object construction. If an object isn't valid after construction then you should
have a static Create method which constructs + initializes, then make the
constructor+Initialize private. then make sure that the correct Create
implementation is linked in.

[Charlie, 2014/08/06 21:54:41]

I'm not quite following what you're suggesting. But would you be ok with a
protected constructor that the MockCopresenceClient uses to inject all
dependencies? Then we could revert to the old public constructor.

On 2014/08/06 21:45:27, kalman wrote:

[not at google - send to devlin, 2014/08/06 22:02:00]

I don't think my suggestion is actually implementable because chances are the
mock and the real copresence clients can be linked together in the same binary,
somehow.

but I really hope it's possible to implement this mock stuff without exposing an
Initialize method or a protected constructor. like, it makes sense for the
Delegate to always be passed in on construction. odd that one code path (to
construct the real one) ends up calling Initialize, and the other one (for
testing) doesn't.

[Charlie, 2014/08/07 22:24:15]

Switching to a Create() method per the CL-level thread. In order for subclassing
to work at all, it seems like we need at least a protected constructor.

+
   // This method will execute a report request. Each report request can have
   // multiple (un)publishes, (un)subscribes. This will ensure that once the
   // client is initialized, it sends all request to the server and handles
   // the response. If an error is encountered, the status callback is used
   // to relay it to the requester.
-  void ExecuteReportRequest(copresence::ReportRequest request,
-                            const std::string& app_id,
-                            const StatusCallback& callback);
+  virtual void ExecuteReportRequest(copresence::ReportRequest request,
+                                    const std::string& app_id,
+                                    const StatusCallback& callback);
 
   // Called before the API (and thus the Client) is destructed.
   void Shutdown();