commit bf13ad09b72a4a7ffd70a65ccd7a7ed722bfe517
author Martijn Coenen <maco@google.com> Thu Apr 27 09:41:13 2017 -0700
committer Martijn Coenen <maco@google.com> Fri Apr 28 14:46:29 2017 -0700
tree 2d9b9992b261dbbf52bce2d614649e40cfe62612
parent 9d3eb357a2a3a64baa4824d550b5563b3dc0b15a

Make registerPassthroughClient() synchronous.

And remove the PID argument.

Bug: 37730021
Test: lshal shows passthrough clients again
Merged-In: I0bee20ae5cde36dbc86662654d0ac9e9489bf0a2
Change-Id: I0bee20ae5cde36dbc86662654d0ac9e9489bf0a2

transport/ServiceManagement.cpp

diff --git a/transport/ServiceManagement.cpp b/transport/ServiceManagement.cpp
index 0e56d18..8121911 100644
--- a/transport/ServiceManagement.cpp
+++ b/transport/ServiceManagement.cpp
@@ -196,7 +196,7 @@
                      << ": null binderized manager.";
         return;
     }
-    auto ret = binderizedManager->registerPassthroughClient(interfaceName, instanceName, getpid());
+    auto ret = binderizedManager->registerPassthroughClient(interfaceName, instanceName);
     if (!ret.isOk()) {
         LOG(WARNING) << "Could not registerReference for "
                      << interfaceName << "/" << instanceName
@@ -341,7 +341,7 @@
         return Void();
     }
 
-    Return<void> registerPassthroughClient(const hidl_string &, const hidl_string &, int32_t) override {
+    Return<void> registerPassthroughClient(const hidl_string &, const hidl_string &) override {
         // This makes no sense.
         LOG(FATAL) << "Cannot call registerPassthroughClient on passthrough service manager. "
                    << "Call it on defaultServiceManager() instead.";