Convert chromeos::CryptohomeClient::TpmCanAttemptOwnership to asynchronous

chrome/browser/chromeos/cros/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 "chrome/browser/chromeos/cros/cryptohome_library.h"
 
 #include <map>
 
+#include "base/bind.h"
 #include "base/memory/weak_ptr.h"
 #include "base/string_number_conversions.h"
 #include "base/string_util.h"
 #include "chromeos/dbus/cryptohome_client.h"
 #include "chromeos/dbus/dbus_thread_manager.h"
 
+namespace chromeos {
+
 namespace {
 
 const char kStubSystemSalt[] = "stub_system_salt";
 
-}
+// Does nothing.  Used as a Cryptohome::VoidMethodCallback.
+void DoNothing(DBusMethodCallStatus call_status) {}

[stevenjb, 2012/07/26 17:19:52]

Should we consider adding this where DBusMethodCallStatus is defined? Haven't we
defined this same function elsewhere?

[hashimoto, 2012/07/27 04:37:18]

We're having the exactly same function in cros_network_functions.cc.
Will do in a separate patch.

[stevenjb, 2012/07/30 22:24:38]

Sounds good.

 
-namespace chromeos {
+}  // namespace
 
 // This class handles the interaction with the ChromeOS cryptohome library APIs.
 class CryptohomeLibraryImpl : public CryptohomeLibrary {
  public:
   CryptohomeLibraryImpl() : weak_ptr_factory_(this) {
   }
 
   virtual ~CryptohomeLibraryImpl() {
   }
 
@@ skipping 16 matching lines @@
     return result;
   }
 
   virtual bool TpmIsBeingOwned() OVERRIDE {
     bool result = false;
     DBusThreadManager::Get()->GetCryptohomeClient()->TpmIsBeingOwned(&result);
     return result;
   }
 
   virtual void TpmCanAttemptOwnership() OVERRIDE {
-    DBusThreadManager::Get()->GetCryptohomeClient()->TpmCanAttemptOwnership();
+    DBusThreadManager::Get()->GetCryptohomeClient()->TpmCanAttemptOwnership(
+        base::Bind(&DoNothing));
   }
 
   virtual void TpmClearStoredPassword() OVERRIDE {
     DBusThreadManager::Get()->GetCryptohomeClient()->TpmClearStoredPassword();
   }
 
   virtual bool InstallAttributesGet(
       const std::string& name, std::string* value) OVERRIDE {
     std::vector<uint8> buf;
     bool success = false;
@@ skipping 143 matching lines @@
 CryptohomeLibrary* CryptohomeLibrary::GetImpl(bool stub) {
   CryptohomeLibrary* impl;
   if (stub)
     impl = new CryptohomeLibraryStubImpl();
   else
     impl = new CryptohomeLibraryImpl();
   return impl;
 }
 
 } // namespace chromeos