Using API key specified from js

components/copresence/public/copresence_client_delegate.h

 // Copyright 2014 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 COMPONENTS_COPRESENCE_PUBLIC_COPRESENCE_CLIENT_DELEGATE_H_
 #define COMPONENTS_COPRESENCE_PUBLIC_COPRESENCE_CLIENT_DELEGATE_H_
 
 #include <string>
 #include <vector>
 
 #include "base/callback_forward.h"
 
 namespace net {
 class URLRequestContextGetter;
 }
 
 namespace copresence {
 
+class DeviceFingerprint;
 class Message;
 class WhispernetClient;
 
 enum CopresenceStatus { SUCCESS, FAIL };
 
 // Callback type to send a status.
 typedef base::Callback<void(CopresenceStatus)> StatusCallback;
 
 // A delegate interface for users of Copresence.
 class CopresenceClientDelegate {
  public:
   // This method will be called when we have subscribed messages that need to
   // be sent to their respective apps.
   virtual void HandleMessages(
       const std::string& app_id,
       const std::string& subscription_id,
       const std::vector<Message>& message) = 0;
 
   virtual net::URLRequestContextGetter* GetRequestContext() const = 0;
-
-  virtual const std::string GetPlatformVersionString() const = 0;
+  virtual const DeviceFingerprint& GetDeviceFingerprint() const = 0;

[rkc, 2014/08/13 21:45:31]

This proto can be constructed on the Copresence side without requiring an
explicit API. If it at a later point it does need any data from the consumer,
let us get that data, not the proto itself. This API should not introduce any
dependencies on internal Copresence code.

[Charlie, 2014/08/13 22:53:37]

The client already depends on our protos, because the only thing it does is to
execute a ReportRequest. In fact maybe the protos should be in the public/
directory. Anyway, it will be easier for users of the component (e.g. from
Brillo) to set this correctly if it is exposed.

[rkc, 2014/08/13 23:11:36]

Report request is is required for every report to exist. DeviceFingerprint is a
technical detail that we didn't even need to set till now. Reusing a proto
instead of creating another in-between translation API for operations makes
sense. Having a client provide a technical detail of our API isn't something we
should get into.

+  virtual const std::string& GetAPIKey() const = 0;
 
   virtual WhispernetClient* GetWhispernetClient() = 0;
 };
 
 }  // namespace copresence
 
 #endif  // COMPONENTS_COPRESENCE_PUBLIC_COPRESENCE_CLIENT_DELEGATE_H_