diff --git a/core/api/system-current.txt b/core/api/system-current.txt index 37f3cbd84eef4..635d3f03b9b85 100644 --- a/core/api/system-current.txt +++ b/core/api/system-current.txt @@ -6946,7 +6946,7 @@ package android.media.tv.tuner { } public class Lnb implements java.lang.AutoCloseable { - method public void addCallback(@NonNull android.media.tv.tuner.LnbCallback, @NonNull java.util.concurrent.Executor); + method public void addCallback(@NonNull java.util.concurrent.Executor, @NonNull android.media.tv.tuner.LnbCallback); method public void close(); method public boolean removeCallback(@NonNull android.media.tv.tuner.LnbCallback); method public int sendDiseqcMessage(@NonNull byte[]); diff --git a/media/java/android/media/tv/tuner/Lnb.java b/media/java/android/media/tv/tuner/Lnb.java index 50a208344c3a1..3b70890302555 100644 --- a/media/java/android/media/tv/tuner/Lnb.java +++ b/media/java/android/media/tv/tuner/Lnb.java @@ -167,10 +167,10 @@ public class Lnb implements AutoCloseable { private Lnb() {} - void setCallbackAndOwner(Executor executor, @Nullable LnbCallback callback, Tuner tuner) { + void setCallbackAndOwner(Tuner tuner, Executor executor, @Nullable LnbCallback callback) { synchronized (mCallbackLock) { if (callback != null && executor != null) { - addCallback(callback, executor); + addCallback(executor, callback); } } setOwner(tuner); @@ -179,12 +179,12 @@ public class Lnb implements AutoCloseable { /** * Adds LnbCallback * - * @param callback the callback to receive notifications from LNB. * @param executor the executor on which callback will be invoked. Cannot be null. + * @param callback the callback to receive notifications from LNB. */ - public void addCallback(@NonNull LnbCallback callback, @NonNull Executor executor) { - Objects.requireNonNull(callback, "callback must not be null"); + public void addCallback(@NonNull Executor executor, @NonNull LnbCallback callback) { Objects.requireNonNull(executor, "executor must not be null"); + Objects.requireNonNull(callback, "callback must not be null"); synchronized (mCallbackLock) { mCallbackMap.put(callback, executor); } diff --git a/media/java/android/media/tv/tuner/Tuner.java b/media/java/android/media/tv/tuner/Tuner.java index 1e32cadc27480..ef0270b5414c1 100644 --- a/media/java/android/media/tv/tuner/Tuner.java +++ b/media/java/android/media/tv/tuner/Tuner.java @@ -62,7 +62,9 @@ import android.os.Looper; import android.os.Message; import android.os.Process; import android.util.Log; + import com.android.internal.util.FrameworkStatsLog; + import java.lang.annotation.Retention; import java.lang.annotation.RetentionPolicy; import java.lang.ref.WeakReference; @@ -2147,12 +2149,12 @@ public class Tuner implements AutoCloseable { Objects.requireNonNull(executor, "executor must not be null"); Objects.requireNonNull(cb, "LnbCallback must not be null"); if (mLnb != null) { - mLnb.setCallbackAndOwner(executor, cb, this); + mLnb.setCallbackAndOwner(this, executor, cb); return mLnb; } if (checkResource(TunerResourceManager.TUNER_RESOURCE_TYPE_LNB, mLnbLock) && mLnb != null) { - mLnb.setCallbackAndOwner(executor, cb, this); + mLnb.setCallbackAndOwner(this, executor, cb); setLnb(mLnb); return mLnb; } @@ -2186,7 +2188,7 @@ public class Tuner implements AutoCloseable { mLnbHandle = null; } mLnb = newLnb; - mLnb.setCallbackAndOwner(executor, cb, this); + mLnb.setCallbackAndOwner(this, executor, cb); setLnb(mLnb); } return mLnb;