commit 7ae4e2fa40fbf6035ab15952a3412031aa477798
author Chong Zhang <chz@google.com> Fri Apr 17 15:24:34 2020 -0700
committer Chong Zhang <chz@google.com> Wed Apr 29 12:03:09 2020 -0700
tree 8a7acb8da489b094941e57ac408c809d3209132b
parent 4187c8951b6ec3611720c2f0c2f2ce8da14d9087

transcoding: use uid instead of pid for scheduling

This change replaces pid usage with uid, there is no functional
change other than that.

bug: 154734285
bug: 145233472
test: unit tests

Change-Id: I2000a84486a561a393211cb0b098e456c2173227

media/libmediatranscoding/TranscodingClientManager.cpp

diff --git a/media/libmediatranscoding/TranscodingClientManager.cpp b/media/libmediatranscoding/TranscodingClientManager.cpp
index b05e3a6..71ffcba 100644
--- a/media/libmediatranscoding/TranscodingClientManager.cpp
+++ b/media/libmediatranscoding/TranscodingClientManager.cpp
@@ -59,8 +59,8 @@
     // Pointer to the client manager for this client
     TranscodingClientManager* mOwner;
 
-    ClientImpl(const std::shared_ptr<ITranscodingClientCallback>& callback, pid_t pid,
-            uid_t uid, const std::string& clientName, const std::string& opPackageName,
+    ClientImpl(const std::shared_ptr<ITranscodingClientCallback>& callback, pid_t pid, uid_t uid,
+               const std::string& clientName, const std::string& opPackageName,
                TranscodingClientManager* owner);
 
     Status submitRequest(const TranscodingRequestParcel& /*in_request*/,
@@ -99,7 +99,7 @@
     int32_t jobId = mNextJobId.fetch_add(1);
 
     *_aidl_return =
-            mOwner->mJobScheduler->submit(mClientId, jobId, mClientPid, in_request,
+            mOwner->mJobScheduler->submit(mClientId, jobId, mClientUid, in_request,
                                           ITranscodingClientCallback::fromBinder(mClientCallback));
 
     if (*_aidl_return) {