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

Commit ae775a45 authored by TreeHugger Robot's avatar TreeHugger Robot Committed by Automerger Merge Worker
Browse files

Merge "Prevents scroll capture from targetting secure windows" into sc-dev am: 7ee0307a 

Original change: https://googleplex-android-review.googlesource.com/c/platform/frameworks/base/+/15374948

Change-Id: I1885cd79b8ffd54110919f719a05eaeab26b9492
parents bb8e49b9 7ee0307a

services/core/java/com/android/server/wm/DisplayContent.java

+4 −1
Original line number Diff line number Diff line
@@ -61,6 +61,7 @@ import static android.view.WindowManager.LayoutParams.FIRST_APPLICATION_WINDOW; 
import static android.view.WindowManager.LayoutParams.FLAG_NOT_FOCUSABLE;

import static android.view.WindowManager.LayoutParams.FLAG_NOT_TOUCHABLE;

import static android.view.WindowManager.LayoutParams.FLAG_NOT_TOUCH_MODAL;

import static android.view.WindowManager.LayoutParams.FLAG_SECURE;

import static android.view.WindowManager.LayoutParams.FLAG_SPLIT_TOUCH;

import static android.view.WindowManager.LayoutParams.LAST_APPLICATION_WINDOW;

import static android.view.WindowManager.LayoutParams.SOFT_INPUT_STATE_ALWAYS_HIDDEN;
@@ -5829,7 +5830,9 @@ class DisplayContent extends RootDisplayArea implements WindowManagerPolicy.Disp 
                        return false; /* continue */

                    }

                }



                if (nextWindow.isSecureLocked()) {

                    return false; /* continue */

                }

                return true; /* stop, match found */

            }

        });

services/tests/wmtests/src/com/android/server/wm/DisplayContentTests.java

+26 −0
Original line number Diff line number Diff line
@@ -42,6 +42,7 @@ import static android.view.WindowInsetsController.BEHAVIOR_SHOW_TRANSIENT_BARS_B 
import static android.view.WindowManager.LayoutParams.FIRST_SUB_WINDOW;

import static android.view.WindowManager.LayoutParams.FLAG_LAYOUT_INSET_DECOR;

import static android.view.WindowManager.LayoutParams.FLAG_LAYOUT_IN_SCREEN;

import static android.view.WindowManager.LayoutParams.FLAG_SECURE;

import static android.view.WindowManager.LayoutParams.LAYOUT_IN_DISPLAY_CUTOUT_MODE_SHORT_EDGES;

import static android.view.WindowManager.LayoutParams.PRIVATE_FLAG_NO_MOVE_ANIMATION;

import static android.view.WindowManager.LayoutParams.TYPE_APPLICATION;
@@ -1745,6 +1746,31 @@ public class DisplayContentTests extends WindowTestsBase { 
        assertEquals(activityWindow, result);

    }



    @Test

    public void testFindScrollCaptureTargetWindow_secure() {

        DisplayContent display = createNewDisplay();

        Task rootTask = createTask(display);

        Task task = createTaskInRootTask(rootTask, 0 /* userId */);

        WindowState secureWindow = createWindow(null, TYPE_APPLICATION, "Secure Window");

        secureWindow.mAttrs.flags |= FLAG_SECURE;



        WindowState result = display.findScrollCaptureTargetWindow(null,

                ActivityTaskManager.INVALID_TASK_ID);

        assertNull(result);

    }



    @Test

    public void testFindScrollCaptureTargetWindow_secureTaskId() {

        DisplayContent display = createNewDisplay();

        Task rootTask = createTask(display);

        Task task = createTaskInRootTask(rootTask, 0 /* userId */);

        WindowState secureWindow = createWindow(null, TYPE_APPLICATION, "Secure Window");

        secureWindow.mAttrs.flags |= FLAG_SECURE;



        WindowState result = display.findScrollCaptureTargetWindow(null,  task.mTaskId);

        assertNull(result);

    }



    @Test

    public void testFindScrollCaptureTargetWindow_taskId() {

        DisplayContent display = createNewDisplay();