Merge "Add null check for IInputMethodPrivilegedOperations" into sc-dev

This commit is contained in:
Wilson Wu
2021-04-08 02:03:29 +00:00
committed by Android (Google) Code Review

View File

@@ -18,6 +18,7 @@ package com.android.internal.inputmethod;
import android.annotation.AnyThread;
import android.annotation.DrawableRes;
import android.annotation.NonNull;
import android.annotation.Nullable;
import android.net.Uri;
import android.os.IBinder;
@@ -28,6 +29,8 @@ import android.view.inputmethod.InputMethodSubtype;
import com.android.internal.annotations.GuardedBy;
import java.util.Objects;
/**
* A utility class to take care of boilerplate code around IPCs.
*/
@@ -47,7 +50,7 @@ public final class InputMethodPrivilegedOperations {
* @param privOps Binder interface to be set
*/
@AnyThread
public synchronized void set(IInputMethodPrivilegedOperations privOps) {
public synchronized void set(@NonNull IInputMethodPrivilegedOperations privOps) {
if (mPrivOps != null) {
throw new IllegalStateException(
"IInputMethodPrivilegedOperations must be set at most once."
@@ -90,7 +93,8 @@ public final class InputMethodPrivilegedOperations {
* @param privOps Binder interface to be set
*/
@AnyThread
public void set(IInputMethodPrivilegedOperations privOps) {
public void set(@NonNull IInputMethodPrivilegedOperations privOps) {
Objects.requireNonNull(privOps, "privOps must not be null");
mOps.set(privOps);
}