Added unit tests for getCredentialsFromAuthCode.

remoting/host/setup/mock_oauth_client.h

+// Copyright 2015 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_SETUP_MOCK_OAUTH_CLIENT

[Lambros, 2015/04/24 18:15:50]

nit: REMOTING_HOST_SETUP_MOCK_OAUTH_CLIENT_H_

[John Williams, 2015/04/24 21:05:13]

Done.

+#define REMOTING_HOST_SETUP_MOCK_OAUTH_CLIENT
+
+#include "remoting/host/setup/oauth_client.h"
+
+namespace remoting {
+
+class MockOAuthClient : public OAuthClient {
+ public:
+  MockOAuthClient(const std::string& user_email,
+                  const std::string& refresh_token) :
+      user_email_(user_email), refresh_token_(refresh_token) {}

[Lambros, 2015/04/24 18:15:50]

Style nit - move ctor to .cc file.
See "Stop inlining constructors and destructors." at
http://www.chromium.org/developers/coding-style/cpp-dos-and-donts
for background.

[John Williams, 2015/04/24 21:05:13]

Done.

+
+  ~MockOAuthClient() override;

[Lambros, 2015/04/24 18:15:50]

nit: add blank line after dtor.

[John Williams, 2015/04/24 21:05:13]

Done.

+  void GetCredentialsFromAuthCode(
+      const gaia::OAuthClientInfo& oauth_client_info,
+      const std::string& auth_code,
+      CompletionCallback on_done) override;
+
+ private:
+  std::string user_email_;
+  std::string refresh_token_;
+};
+
+}  // namespace remoting
+
+#endif  // REMOTING_HOST_SETUP_MOCK_OAUTH_CLIENT