Refactor OAuth2TokenService to have profile- and device-based implementations.

chrome/browser/chromeos/settings/device_oauth2_token_service.cc

Index: chrome/browser/chromeos/settings/device_oauth2_token_service.cc
diff --git a/chrome/browser/chromeos/settings/device_oauth2_token_service.cc b/chrome/browser/chromeos/settings/device_oauth2_token_service.cc
new file mode 100644
index 0000000000000000000000000000000000000000..0605dad87812775c3a04d6964d980b18866ab732
--- /dev/null
+++ b/chrome/browser/chromeos/settings/device_oauth2_token_service.cc
@@ -0,0 +1,49 @@
+// Copyright 2013 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.
+
+#include "chrome/browser/chromeos/settings/device_oauth2_token_service.h"
+
+#include "base/prefs/pref_registry_simple.h"
+#include "base/prefs/pref_service.h"
+#include "chrome/browser/browser_process.h"
+#include "chrome/browser/chromeos/cros/cert_library.h"
+#include "chrome/browser/chromeos/cros/cros_library.h"
+#include "chrome/common/pref_names.h"
+
+namespace chromeos {
+
+DeviceOAuth2TokenService::DeviceOAuth2TokenService(
+    net::URLRequestContextGetter* getter)
+    : OAuth2TokenService(NULL) {
+}
+
+DeviceOAuth2TokenService::~DeviceOAuth2TokenService() {
+}
+
+// static
+void DeviceOAuth2TokenService::RegisterPrefs(PrefRegistrySimple* registry) {
+  registry->RegisterStringPref(prefs::kDeviceRobotAnyApiRefreshToken, "");
+}
+
+void DeviceOAuth2TokenService::SetAndSaveRefreshToken(
+    const std::string& refresh_token) {
+  std::string encrypted_refresh_token =
+      CrosLibrary::Get()->GetCertLibrary()->EncryptWithSystemSalt(
+          refresh_token);
+
+  g_browser_process->local_state()->SetString(
+      prefs::kDeviceRobotAnyApiRefreshToken,
+      encrypted_refresh_token);
+}
+
+std::string DeviceOAuth2TokenService::GetRefreshToken() {
+  std::string encrypted_refresh_token =
+      g_browser_process->local_state()->GetString(

[Mattias Nissler (ping if slow), 2013/03/26 12:04:32]

Seems like the local state pointer could be passed in as well here now that
we're hosted in g_browser_process. Less globals, better testability.

[David Roche, 2013/03/27 05:09:51]

Done.

+          prefs::kDeviceRobotAnyApiRefreshToken);
+
+  return CrosLibrary::Get()->GetCertLibrary()->DecryptWithSystemSalt(
+      encrypted_refresh_token);
+}
+
+}  // namespace chromeos