cryptohome: Move stateless wrapper functions out of CryptohomeLibrary

chromeos/cryptohome/cryptohome_library.cc

 // 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.
 
 #include "chromeos/cryptohome/cryptohome_library.h"
 
 #include <map>
 
 #include "base/bind.h"
 #include "base/memory/weak_ptr.h"
@@ skipping 19 matching lines @@
 
 // This class handles the interaction with the ChromeOS cryptohome library APIs.
 class CryptohomeLibraryImpl : public CryptohomeLibrary {
  public:
   CryptohomeLibraryImpl() {
   }
 
   virtual ~CryptohomeLibraryImpl() {
   }
 
-  virtual bool TpmIsEnabled() OVERRIDE {
-    bool result = false;
-    DBusThreadManager::Get()->GetCryptohomeClient()->CallTpmIsEnabledAndBlock(
-        &result);
-    return result;
-  }
-
-  virtual bool TpmIsOwned() OVERRIDE {
-    bool result = false;
-    DBusThreadManager::Get()->GetCryptohomeClient()->CallTpmIsOwnedAndBlock(
-        &result);
-    return result;
-  }
-
-  virtual bool TpmIsBeingOwned() OVERRIDE {
-    bool result = false;
-    DBusThreadManager::Get()->GetCryptohomeClient()->
-        CallTpmIsBeingOwnedAndBlock(&result);
-    return result;
-  }
-
-  virtual bool InstallAttributesGet(
-      const std::string& name, std::string* value) OVERRIDE {
-    std::vector<uint8> buf;
-    bool success = false;
-    DBusThreadManager::Get()->GetCryptohomeClient()->
-        InstallAttributesGet(name, &buf, &success);
-    if (success) {
-      // Cryptohome returns 'buf' with a terminating '\0' character.
-      DCHECK(!buf.empty());
-      DCHECK_EQ(buf.back(), 0);
-      value->assign(reinterpret_cast<char*>(buf.data()), buf.size() - 1);
-    }
-    return success;
-  }
-
-  virtual bool InstallAttributesSet(
-      const std::string& name, const std::string& value) OVERRIDE {
-    std::vector<uint8> buf(value.c_str(), value.c_str() + value.size() + 1);
-    bool success = false;
-    DBusThreadManager::Get()->GetCryptohomeClient()->
-        InstallAttributesSet(name, buf, &success);
-    return success;
-  }
-
-  virtual bool InstallAttributesFinalize() OVERRIDE {
-    bool success = false;
-    DBusThreadManager::Get()->GetCryptohomeClient()->
-        InstallAttributesFinalize(&success);
-    return success;
-  }
-
-  virtual bool InstallAttributesIsInvalid() OVERRIDE {
-    bool result = false;
-    DBusThreadManager::Get()->GetCryptohomeClient()->
-        InstallAttributesIsInvalid(&result);
-    return result;
-  }
-
-  virtual bool InstallAttributesIsFirstInstall() OVERRIDE {
-    bool result = false;
-    DBusThreadManager::Get()->GetCryptohomeClient()->
-        InstallAttributesIsFirstInstall(&result);
-    return result;
-  }
-
   virtual std::string GetSystemSalt() OVERRIDE {
     LoadSystemSalt();  // no-op if it's already loaded.
     return system_salt_;
   }
 
   virtual std::string EncryptWithSystemSalt(const std::string& token) OVERRIDE {
     // Don't care about token encryption while debugging.
     if (!base::SysInfo::IsRunningOnChromeOS())
       return token;
 
@@ skipping 110 matching lines @@
 
   DISALLOW_COPY_AND_ASSIGN(CryptohomeLibraryImpl);
 };
 
 class CryptohomeLibraryStubImpl : public CryptohomeLibrary {
  public:
   CryptohomeLibraryStubImpl()
     : locked_(false) {}
   virtual ~CryptohomeLibraryStubImpl() {}
 
-  virtual bool TpmIsEnabled() OVERRIDE {
-    return true;
-  }
-
-  virtual bool TpmIsOwned() OVERRIDE {
-    return true;
-  }
-
-  virtual bool TpmIsBeingOwned() OVERRIDE {
-    return true;
-  }
-
-  virtual bool InstallAttributesGet(
-      const std::string& name, std::string* value) OVERRIDE {
-    if (install_attrs_.find(name) != install_attrs_.end()) {
-      *value = install_attrs_[name];
-      return true;
-    }
-    return false;
-  }
-
-  virtual bool InstallAttributesSet(
-      const std::string& name, const std::string& value) OVERRIDE {
-    install_attrs_[name] = value;
-    return true;
-  }
-
-  virtual bool InstallAttributesFinalize() OVERRIDE {
-    locked_ = true;
-    return true;
-  }
-
-  virtual bool InstallAttributesIsInvalid() OVERRIDE {
-    return false;
-  }
-
-  virtual bool InstallAttributesIsFirstInstall() OVERRIDE {
-    return !locked_;
-  }
-
   virtual std::string GetSystemSalt() OVERRIDE {
     return kStubSystemSalt;
   }
 
   virtual std::string EncryptWithSystemSalt(const std::string& token) OVERRIDE {
     return token;
   }
 
   virtual std::string DecryptWithSystemSalt(
       const std::string& encrypted_token_hex) OVERRIDE {
@@ skipping 46 matching lines @@
 void CryptohomeLibrary::SetForTest(CryptohomeLibrary* impl) {
   CHECK(!g_test_cryptohome_library || !impl);
   g_test_cryptohome_library = impl;
 }
 
 // static
 CryptohomeLibrary* CryptohomeLibrary::GetTestImpl() {
   return new CryptohomeLibraryStubImpl();
 }
 
-} // namespace chromeos
+namespace cryptohome_util {
+
+bool TpmIsEnabled() {
+  bool result = false;
+  DBusThreadManager::Get()->GetCryptohomeClient()->CallTpmIsEnabledAndBlock(
+      &result);
+  return result;
+}
+
+bool TpmIsOwned() {
+  bool result = false;
+  DBusThreadManager::Get()->GetCryptohomeClient()->CallTpmIsOwnedAndBlock(
+      &result);
+  return result;
+}
+
+bool TpmIsBeingOwned() {
+  bool result = false;
+  DBusThreadManager::Get()->GetCryptohomeClient()->
+      CallTpmIsBeingOwnedAndBlock(&result);
+  return result;
+}
+
+bool InstallAttributesGet(
+    const std::string& name, std::string* value) {
+  std::vector<uint8> buf;
+  bool success = false;
+  DBusThreadManager::Get()->GetCryptohomeClient()->
+      InstallAttributesGet(name, &buf, &success);
+  if (success) {
+    // Cryptohome returns 'buf' with a terminating '\0' character.
+    DCHECK(!buf.empty());
+    DCHECK_EQ(buf.back(), 0);
+    value->assign(reinterpret_cast<char*>(buf.data()), buf.size() - 1);
+  }
+  return success;
+}
+
+bool InstallAttributesSet(
+    const std::string& name, const std::string& value) {
+  std::vector<uint8> buf(value.c_str(), value.c_str() + value.size() + 1);
+  bool success = false;
+  DBusThreadManager::Get()->GetCryptohomeClient()->
+      InstallAttributesSet(name, buf, &success);
+  return success;
+}
+
+bool InstallAttributesFinalize() {
+  bool success = false;
+  DBusThreadManager::Get()->GetCryptohomeClient()->
+      InstallAttributesFinalize(&success);
+  return success;
+}
+
+bool InstallAttributesIsInvalid() {
+  bool result = false;
+  DBusThreadManager::Get()->GetCryptohomeClient()->
+      InstallAttributesIsInvalid(&result);
+  return result;
+}
+
+bool InstallAttributesIsFirstInstall() {
+  bool result = false;
+  DBusThreadManager::Get()->GetCryptohomeClient()->
+      InstallAttributesIsFirstInstall(&result);
+  return result;
+}
+
+}  // namespace cryptohome_util
+}  // namespace chromeos