Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit d100067f authored by Vishnu Nair's avatar Vishnu Nair
Browse files

SurfaceControl: Add setDropInputMode api 

Introduces an API to drop input events on this SurfaceControl. This
policy will be inherited by its children. The caller must hold the
ACCESS_SURFACE_FLINGER permission.

Options include:
ALL: SurfaceControl and its children will not receive any
input regardless of whether it has a valid input channel.
OBSCURED: SurfaceControl and its children will not
receive any input if the layer is obscured, cropped by its parent or
translucent.

These policies are used to enable features that allow for a less trusted
interaction model between apps. See the bug for more details.

Test: atest libgui_test InputDispatcherDropInputFeatureTest
Bug:197364677

Change-Id: Ifcb4133306a43874e74e8fb0f42b60842daf6f25
parent 02591181

core/java/android/view/SurfaceControl.java

+15 −2
Original line number Diff line number Diff line
@@ -42,6 +42,7 @@ import android.graphics.PixelFormat; 
import android.graphics.Point;

import android.graphics.Rect;

import android.graphics.Region;

import android.gui.DropInputMode;

import android.hardware.HardwareBuffer;

import android.hardware.display.DeviceProductInfo;

import android.hardware.display.DisplayedContentSample;
@@ -151,7 +152,8 @@ public final class SurfaceControl implements Parcelable { 
            float childRelativeTop, float childRelativeRight, float childRelativeBottom);

    private static native void nativeSetTrustedOverlay(long transactionObj, long nativeObject,

            boolean isTrustedOverlay);



    private static native void nativeSetDropInputMode(

            long transactionObj, long nativeObject, int flags);

    private static native boolean nativeClearContentFrameStats(long nativeObject);

    private static native boolean nativeGetContentFrameStats(long nativeObject, WindowContentFrameStats outStats);

    private static native boolean nativeClearAnimationFrameStats();
@@ -3444,6 +3446,17 @@ public final class SurfaceControl implements Parcelable { 
            return this;

        }



        /**

         * Sets the input event drop mode on this SurfaceControl and its children. The caller must

         * hold the ACCESS_SURFACE_FLINGER permission. See {@code InputEventDropMode}.

         * @hide

         */

        public Transaction setDropInputMode(SurfaceControl sc, @DropInputMode int mode) {

            checkPreconditions(sc);

            nativeSetDropInputMode(mNativeObject, sc.mNativeObject, mode);

            return this;

        }



        /**

         * Merge the other transaction into this transaction, clearing the

         * other transaction as if it had been applied.

core/jni/android_view_SurfaceControl.cpp

+9 −0
Original line number Diff line number Diff line
@@ -870,6 +870,13 @@ static void nativeSetFixedTransformHint(JNIEnv* env, jclass clazz, jlong transac 
    transaction->setFixedTransformHint(ctrl, transformHint);

}



static void nativeSetDropInputMode(JNIEnv* env, jclass clazz, jlong transactionObj,

                                   jlong nativeObject, jint mode) {

    auto transaction = reinterpret_cast<SurfaceComposerClient::Transaction*>(transactionObj);

    SurfaceControl* const ctrl = reinterpret_cast<SurfaceControl*>(nativeObject);

    transaction->setDropInputMode(ctrl, static_cast<gui::DropInputMode>(mode));

}



static jlongArray nativeGetPhysicalDisplayIds(JNIEnv* env, jclass clazz) {

    const auto displayIds = SurfaceComposerClient::getPhysicalDisplayIds();

    jlongArray array = env->NewLongArray(displayIds.size());
@@ -2018,6 +2025,8 @@ static const JNINativeMethod sSurfaceControlMethods[] = { 
            (void*)nativeSetTrustedOverlay },

    {"nativeGetLayerId", "(J)I",

            (void*)nativeGetLayerId },

    {"nativeSetDropInputMode", "(JJI)V",

             (void*)nativeSetDropInputMode },

        // clang-format on

};

services/core/Android.bp

+0 −1
Original line number Diff line number Diff line
@@ -108,7 +108,6 @@ java_library_static { 
        ":display-device-config",

        ":display-layout-config",

        ":device-state-config",

        ":guiconstants_aidl",

        "java/com/android/server/EventLogTags.logtags",

        "java/com/android/server/am/EventLogTags.logtags",

        "java/com/android/server/wm/EventLogTags.logtags",