Update FWMP in TPM

chromeos/dbus/cryptohome_client.cc

Index: chromeos/dbus/cryptohome_client.cc
diff --git a/chromeos/dbus/cryptohome_client.cc b/chromeos/dbus/cryptohome_client.cc
index 86fe5b7e65ab7ff67ad86c151812aee85abde460..89ab099b008c819b0893900bd7b313f1a48abf2c 100644
--- a/chromeos/dbus/cryptohome_client.cc
+++ b/chromeos/dbus/cryptohome_client.cc
@@ -920,6 +920,20 @@ class CryptohomeClientImpl : public CryptohomeClient {
     dbus::MessageWriter writer(&method_call);
     writer.AppendProtoAsArrayOfBytes(request);
 
+    proxy_->CallMethod(&method_call, kTpmDBusTimeoutMs,
+                       base::Bind(&CryptohomeClientImpl::OnBaseReplyMethod,
+                                  weak_ptr_factory_.GetWeakPtr(), callback));
+  }
+
+  void AsyncTpmUpdateFirmwareManagementParameters(
+      const std::string& method_name,
+      const google::protobuf::MessageLite& request,
+      const ProtobufMethodCallback& callback) override {
+    dbus::MethodCall method_call(cryptohome::kCryptohomeInterface, method_name);

[Daniel Erat, 2017/03/06 21:18:26]

it looks like a bunch of these methods are all copy-and-pasted versions of each
other with different method names. would you mind moving this to an internal
helper method that takes a method name, MessageLite, and callback so that there
isn't so much duplication?

[igorcov, 2017/03/09 12:22:57]

Done, thanks for suggestion.

+
+    dbus::MessageWriter writer(&method_call);
+    writer.AppendProtoAsArrayOfBytes(request);
+
     proxy_->CallMethod(&method_call,
                        kTpmDBusTimeoutMs ,
                        base::Bind(&CryptohomeClientImpl::OnBaseReplyMethod,