Remoting daemon process to support unofficial client ID

remoting/host/signaling_connector.h

 // Copyright (c) 2012 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 REMOTING_HOST_SIGNALING_CONNECTOR_H
 #define REMOTING_HOST_SIGNALING_CONNECTOR_H
 
 #include "base/basictypes.h"
 #include "base/memory/weak_ptr.h"
 #include "base/threading/non_thread_safe.h"
@@ skipping 15 matching lines @@
 class SignalingConnector
     : public base::SupportsWeakPtr<SignalingConnector>,
       public base::NonThreadSafe,
       public SignalStrategy::Listener,
       public net::NetworkChangeNotifier::OnlineStateObserver,
       public net::NetworkChangeNotifier::IPAddressObserver,
       public GaiaOAuthClient::Delegate {
  public:
   struct OAuthCredentials {
     OAuthCredentials(const std::string& login_value,
-                     const std::string& refresh_token_value);
+                     const std::string& refresh_token_value,
+                     const OAuthClientInfo& client_info);
     std::string login;
     std::string refresh_token;
+    OAuthClientInfo client_info;

[Wez, 2012/04/19 00:13:04]

I realise it's not your fault this structure has no documentation, but I think a
comment explaining the fields is required.

[Alpha Left Google, 2012/04/20 22:13:50]

I'll add comments, please see if they match. *I think* this is what the fields
are for, Sergey probably has the most context.

   };
 
   // OAuth token is updated refreshed when |oauth_credentials| is
   // not NULL.
   SignalingConnector(XmppSignalStrategy* signal_strategy);
   virtual ~SignalingConnector();
 
   void EnableOAuth(scoped_ptr<OAuthCredentials> oauth_credentials,
                    const base::Closure& oauth_failed_callback,
                    URLRequestContextGetter* url_context);
@@ skipping 37 matching lines @@
   base::Time auth_token_expiry_time_;
 
   base::OneShotTimer<SignalingConnector> timer_;
 
   DISALLOW_COPY_AND_ASSIGN(SignalingConnector);
 };
 
 }  // namespace remoting
 
 #endif  // REMOTING_HOST_SIGNALING_CONNECTOR_H