Remove TpmIsBeingOwned APIs.

we are obsoleting the unnecessary `TpmIsBeingOwned()`.

BUG=b:172775884
BUG=b:172748724
TEST=build ok.

Change-Id: Ie1dc4044060caadb24a9470dbb98d51bcfc383e1
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2532157
Commit-Queue: Leo Lai <cylai@google.com>
Reviewed-by: Maksim Ivanov <emaxx@chromium.org>
Cr-Commit-Position: refs/heads/master@{#836064}

chromeos/dbus/cryptohome/cryptohome_client.cc

@@ -298,21 +298,6 @@ class CryptohomeClientImpl : public CryptohomeClient {
     return CallBoolMethodAndBlock(&method_call, owned);
   }
 
-  // CryptohomeClient override.
-  void TpmIsBeingOwned(DBusMethodCallback<bool> callback) override {
-    dbus::MethodCall method_call(cryptohome::kCryptohomeInterface,
-                                 cryptohome::kCryptohomeTpmIsBeingOwned);
-    CallBoolMethod(&method_call, std::move(callback));
-  }
-
-  // CryptohomeClient override.
-  // TODO(hashimoto): Remove this method. crbug.com/141011
-  bool CallTpmIsBeingOwnedAndBlock(bool* owning) override {
-    dbus::MethodCall method_call(cryptohome::kCryptohomeInterface,
-                                 cryptohome::kCryptohomeTpmIsBeingOwned);
-    return CallBoolMethodAndBlock(&method_call, owning);
-  }
-
   // CryptohomeClient override.
   void TpmCanAttemptOwnership(VoidDBusMethodCallback callback) override {
     dbus::MethodCall method_call(cryptohome::kCryptohomeInterface,

chromeos/dbus/cryptohome/cryptohome_client.h

@@ -231,14 +231,6 @@ class COMPONENT_EXPORT(CRYPTOHOME_CLIENT) CryptohomeClient {
   // TODO(hashimoto): Remove this method. crbug.com/141012
   virtual bool CallTpmIsOwnedAndBlock(bool* owned) = 0;
 
-  // Calls TpmIsBeingOwned method.
-  virtual void TpmIsBeingOwned(DBusMethodCallback<bool> callback) = 0;
-
-  // Calls TpmIsBeingOwned method and returns true when the call succeeds.
-  // This method blocks until the call returns.
-  // TODO(hashimoto): Remove this method. crbug.com/141011
-  virtual bool CallTpmIsBeingOwnedAndBlock(bool* owning) = 0;
-
   // Calls TpmCanAttemptOwnership method.
   // This method tells the service that it is OK to attempt ownership.
   virtual void TpmCanAttemptOwnership(VoidDBusMethodCallback callback) = 0;

chromeos/dbus/cryptohome/fake_cryptohome_client.cc

@@ -192,16 +192,6 @@ bool FakeCryptohomeClient::CallTpmIsOwnedAndBlock(bool* owned) {
   return true;
 }
 
-void FakeCryptohomeClient::TpmIsBeingOwned(DBusMethodCallback<bool> callback) {
-  base::ThreadTaskRunnerHandle::Get()->PostTask(
-      FROM_HERE, base::BindOnce(std::move(callback), true));
-}
-
-bool FakeCryptohomeClient::CallTpmIsBeingOwnedAndBlock(bool* owning) {
-  *owning = true;
-  return true;
-}
-
 void FakeCryptohomeClient::TpmCanAttemptOwnership(
     VoidDBusMethodCallback callback) {
   base::ThreadTaskRunnerHandle::Get()->PostTask(

chromeos/dbus/cryptohome/fake_cryptohome_client.h

@@ -75,8 +75,6 @@ class COMPONENT_EXPORT(CRYPTOHOME_CLIENT) FakeCryptohomeClient
   bool CallTpmIsEnabledAndBlock(bool* enabled) override;
   void TpmIsOwned(DBusMethodCallback<bool> callback) override;
   bool CallTpmIsOwnedAndBlock(bool* owned) override;
-  void TpmIsBeingOwned(DBusMethodCallback<bool> callback) override;
-  bool CallTpmIsBeingOwnedAndBlock(bool* owning) override;
   void TpmCanAttemptOwnership(VoidDBusMethodCallback callback) override;
   void Pkcs11IsTpmTokenReady(DBusMethodCallback<bool> callback) override;
   void Pkcs11GetTpmTokenInfo(

chromeos/dbus/cryptohome/tpm_util.cc

@@ -26,12 +26,6 @@ bool TpmIsOwned() {
   return result;
 }
 
-bool TpmIsBeingOwned() {
-  bool result = false;
-  CryptohomeClient::Get()->CallTpmIsBeingOwnedAndBlock(&result);
-  return result;
-}
-
 bool InstallAttributesGet(const std::string& name, std::string* value) {
   std::vector<uint8_t> buf;
   bool success = false;

chromeos/dbus/cryptohome/tpm_util.h

@@ -24,10 +24,6 @@ COMPONENT_EXPORT(CRYPTOHOME_CLIENT) bool TpmIsEnabled();
 // Blocking call. Returns whether the device has already been owned.
 COMPONENT_EXPORT(CRYPTOHOME_CLIENT) bool TpmIsOwned();
 
-// Blocking call. Returns whether the device is being owned (Tpm password is
-// generating).
-COMPONENT_EXPORT(CRYPTOHOME_CLIENT) bool TpmIsBeingOwned();
-
 // Blocking calls to CryptohomeClient methods.
 COMPONENT_EXPORT(CRYPTOHOME_CLIENT)
 bool InstallAttributesGet(const std::string& name, std::string* value);