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
diff --git a/services/mediatranscoding/MediaTranscodingService.cpp b/services/mediatranscoding/MediaTranscodingService.cpp
index 0289613..a13bec0 100644
--- a/services/mediatranscoding/MediaTranscodingService.cpp
+++ b/services/mediatranscoding/MediaTranscodingService.cpp
@@ -48,7 +48,7 @@
}
}
-// DummyTranscoder and DummyProcessInfo are currently used to instantiate
+// DummyTranscoder and DummyUidPolicy are currently used to instantiate
// MediaTranscodingService on service side for testing, so that we could
// actually test the IPC calls of MediaTranscodingService to expose some
// issues that's observable only over IPC.
@@ -67,21 +67,21 @@
}
};
-class DummyProcessInfo : public ProcessInfoInterface {
- bool isProcessOnTop(int32_t pid) override {
- (void)pid;
+class DummyUidPolicy : public UidPolicyInterface {
+ bool isUidOnTop(uid_t uid) override {
+ (void)uid;
return true;
}
};
MediaTranscodingService::MediaTranscodingService()
: MediaTranscodingService(std::make_shared<DummyTranscoder>(),
- std::make_shared<DummyProcessInfo>()) {}
+ std::make_shared<DummyUidPolicy>()) {}
MediaTranscodingService::MediaTranscodingService(
const std::shared_ptr<TranscoderInterface>& transcoder,
- const std::shared_ptr<ProcessInfoInterface>& procInfo)
- : mJobScheduler(new TranscodingJobScheduler(transcoder, procInfo)),
+ const std::shared_ptr<UidPolicyInterface>& uidPolicy)
+ : mJobScheduler(new TranscodingJobScheduler(transcoder, uidPolicy)),
mClientManager(new TranscodingClientManager(mJobScheduler)) {
ALOGV("MediaTranscodingService is created");
}