commit db10bbab0890a233e76c259cb60e9c02016186c2
author Hangyu Kuang <hkuang@google.com> Thu May 13 20:17:34 2021 +0000
committer Hangyu Kuang <hkuang@google.com> Thu May 20 20:23:01 2021 +0000
tree 9295b5d49cecde65d7dfbc725d37b3ab03cb246a
parent f4dab5f7693b011d9b6af301861d21398b3cfa02

Revert "trancoding: lazy-start transcoding service"

This reverts commit 0621358a526d2ede2e675a1992ce304bcdcb83b8.

Reason for revert: change to conditional start.

Bug: 187271658
Test: atest MediaTranscodeManagerTest; unit tests
Change-Id: Iba169a3b50e37b72fab4e354480451cf05ab5be5

media/java/android/media/MediaServiceManager.java

diff --git a/media/java/android/media/MediaServiceManager.java b/media/java/android/media/MediaServiceManager.java
index fd89c0c..b899559 100644
--- a/media/java/android/media/MediaServiceManager.java
+++ b/media/java/android/media/MediaServiceManager.java
@@ -45,21 +45,12 @@
      */
     public static final class ServiceRegisterer {
         private final String mServiceName;
-        private final boolean mLazyStart;
-
-        /**
-         * @hide
-         */
-        public ServiceRegisterer(String serviceName, boolean lazyStart) {
-            mServiceName = serviceName;
-            mLazyStart = lazyStart;
-        }
 
         /**
          * @hide
          */
         public ServiceRegisterer(String serviceName) {
-            this(serviceName, false /*lazyStart*/);
+            mServiceName = serviceName;
         }
 
         /**
@@ -70,9 +61,6 @@
          */
         @Nullable
         public IBinder get() {
-            if (mLazyStart) {
-                return ServiceManager.waitForService(mServiceName);
-            }
             return ServiceManager.getService(mServiceName);
         }
     }
@@ -90,7 +78,7 @@
      */
     @NonNull
     public ServiceRegisterer getMediaTranscodingServiceRegisterer() {
-        return new ServiceRegisterer(MEDIA_TRANSCODING_SERVICE, true /*lazyStart*/);
+        return new ServiceRegisterer(MEDIA_TRANSCODING_SERVICE);
     }
 
     /**