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

Commit 64b8381c authored by Winson Chung's avatar Winson Chung
Browse files

Revert "1/ Add mechanism to expose shell feature directly (for Pip)" 

Revert "Add shell aidl files"

Revert "Update starting window init call for CarLauncher"

Revert submission 13608835-shell_feature_interfaces

Reason for revert: SysUI Studio aidl import issue
Reverted Changes:
I49a5a0419:2/ Add mechanism to expose other features directly...
I19425896d:Update starting window init call for CarLauncher
If048d2cd9:1/ Add mechanism to expose shell feature directly ...
Ibb8365d3c:Add shell aidl files
Ie41b0b77a:Update calls to go through the feature interface i...

Change-Id: I97e591a5e157dfd27a72a735b51a3ed80bf1bf91
parent a33a496a

libs/WindowManager/Shell/Android.bp

+1 −9
Original line number Diff line number Diff line
@@ -38,14 +38,6 @@ filegroup { 
    path: "src",

}



filegroup {

    name: "wm_shell-aidls",

    srcs: [

        "src/**/*.aidl",

    ],

    path: "src",

}



// TODO(b/168581922) protologtool do not support kotlin(*.kt)

filegroup {

    name: "wm_shell-sources-kt",
@@ -106,7 +98,7 @@ android_library { 
        ":wm_shell_protolog_src",

        // TODO(b/168581922) protologtool do not support kotlin(*.kt)

        ":wm_shell-sources-kt",

        ":wm_shell-aidls",

        "src/**/I*.aidl",

    ],

    resource_dirs: [

        "res",

libs/WindowManager/Shell/src/com/android/wm/shell/ShellCommandHandlerImpl.java

+15 −5
Original line number Diff line number Diff line
@@ -47,7 +47,21 @@ public final class ShellCommandHandlerImpl { 
    private final ShellExecutor mMainExecutor;

    private final HandlerImpl mImpl = new HandlerImpl();



    public ShellCommandHandlerImpl(

    public static ShellCommandHandler create(

            ShellTaskOrganizer shellTaskOrganizer,

            Optional<LegacySplitScreenController> legacySplitScreenOptional,

            Optional<SplitScreenController> splitScreenOptional,

            Optional<Pip> pipOptional,

            Optional<OneHandedController> oneHandedOptional,

            Optional<HideDisplayCutoutController> hideDisplayCutout,

            Optional<AppPairsController> appPairsOptional,

            ShellExecutor mainExecutor) {

        return new ShellCommandHandlerImpl(shellTaskOrganizer, legacySplitScreenOptional,

                splitScreenOptional, pipOptional, oneHandedOptional, hideDisplayCutout,

                appPairsOptional, mainExecutor).mImpl;

    }



    private ShellCommandHandlerImpl(

            ShellTaskOrganizer shellTaskOrganizer,

            Optional<LegacySplitScreenController> legacySplitScreenOptional,

            Optional<SplitScreenController> splitScreenOptional,
@@ -66,10 +80,6 @@ public final class ShellCommandHandlerImpl { 
        mMainExecutor = mainExecutor;

    }



    public ShellCommandHandler asShellCommandHandler() {

        return mImpl;

    }



    /** Dumps WM Shell internal state. */

    private void dump(PrintWriter pw) {

        mShellTaskOrganizer.dump(pw, "");

libs/WindowManager/Shell/src/com/android/wm/shell/ShellInitImpl.java

+25 −5
Original line number Diff line number Diff line
@@ -51,7 +51,31 @@ public class ShellInitImpl { 


    private final InitImpl mImpl = new InitImpl();



    public ShellInitImpl(DisplayImeController displayImeController,

    public static ShellInit create(DisplayImeController displayImeController,

            DragAndDropController dragAndDropController,

            ShellTaskOrganizer shellTaskOrganizer,

            Optional<LegacySplitScreenController> legacySplitScreenOptional,

            Optional<SplitScreenController> splitScreenOptional,

            Optional<AppPairsController> appPairsOptional,

            Optional<StartingSurface> startingSurfaceOptional,

            Optional<PipTouchHandler> pipTouchHandlerOptional,

            FullscreenTaskListener fullscreenTaskListener,

            Transitions transitions,

            ShellExecutor mainExecutor) {

        return new ShellInitImpl(displayImeController,

                dragAndDropController,

                shellTaskOrganizer,

                legacySplitScreenOptional,

                splitScreenOptional,

                appPairsOptional,

                startingSurfaceOptional,

                pipTouchHandlerOptional,

                fullscreenTaskListener,

                transitions,

                mainExecutor).mImpl;

    }



    private ShellInitImpl(DisplayImeController displayImeController,

            DragAndDropController dragAndDropController,

            ShellTaskOrganizer shellTaskOrganizer,

            Optional<LegacySplitScreenController> legacySplitScreenOptional,
@@ -75,10 +99,6 @@ public class ShellInitImpl { 
        mStartingSurfaceOptional = startingSurfaceOptional;

    }



    public ShellInit asShellInit() {

        return mImpl;

    }



    private void init() {

        // Start listening for display changes

        mDisplayImeController.startMonitorDisplays();

libs/WindowManager/Shell/src/com/android/wm/shell/common/ExecutorUtils.java

deleted100644 → 0
+0 −64
Original line number Diff line number Diff line 
/*

 * Copyright (C) 2021 The Android Open Source Project

 *

 * Licensed under the Apache License, Version 2.0 (the "License");

 * you may not use this file except in compliance with the License.

 * You may obtain a copy of the License at

 *

 *      http://www.apache.org/licenses/LICENSE-2.0

 *

 * Unless required by applicable law or agreed to in writing, software

 * distributed under the License is distributed on an "AS IS" BASIS,

 * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.

 * See the License for the specific language governing permissions and

 * limitations under the License.

 */



package com.android.wm.shell.common;



import android.Manifest;

import android.util.Slog;



import java.util.function.Consumer;



/**

 * Helpers for working with executors

 */

public class ExecutorUtils {



    /**

     * Checks that the caller has the MANAGE_ACTIVITY_TASKS permission and executes the given

     * callback.

     */

    public static <T> void executeRemoteCallWithTaskPermission(RemoteCallable<T> controllerInstance,

            String log, Consumer<T> callback) {

        executeRemoteCallWithTaskPermission(controllerInstance, log, callback,

                false /* blocking */);

    }



    /**

     * Checks that the caller has the MANAGE_ACTIVITY_TASKS permission and executes the given

     * callback.

     */

    public static <T> void executeRemoteCallWithTaskPermission(RemoteCallable<T> controllerInstance,

            String log, Consumer<T> callback, boolean blocking) {

        if (controllerInstance == null) return;



        final RemoteCallable<T> controller = controllerInstance;

        controllerInstance.getContext().enforceCallingPermission(

                Manifest.permission.MANAGE_ACTIVITY_TASKS, log);

        if (blocking) {

            try {

                controllerInstance.getRemoteCallExecutor().executeBlocking(() -> {

                    callback.accept((T) controller);

                });

            } catch (InterruptedException e) {

                Slog.e("ExecutorUtils", "Remote call failed", e);

            }

        } else {

            controllerInstance.getRemoteCallExecutor().execute(() -> {

                callback.accept((T) controller);

            });

        }

    }

}

libs/WindowManager/Shell/src/com/android/wm/shell/common/RemoteCallable.java

deleted100644 → 0
+0 −34
Original line number Diff line number Diff line 
/*

 * Copyright (C) 2021 The Android Open Source Project

 *

 * Licensed under the Apache License, Version 2.0 (the "License");

 * you may not use this file except in compliance with the License.

 * You may obtain a copy of the License at

 *

 *      http://www.apache.org/licenses/LICENSE-2.0

 *

 * Unless required by applicable law or agreed to in writing, software

 * distributed under the License is distributed on an "AS IS" BASIS,

 * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.

 * See the License for the specific language governing permissions and

 * limitations under the License.

 */



package com.android.wm.shell.common;



import android.content.Context;



/**

 * An interface for controllers that can receive remote calls.

 */

public interface RemoteCallable<T> {

    /**

     * Returns a context used for permission checking.

     */

    Context getContext();



    /**

     * Returns the executor to post the handler callback to.

     */

    ShellExecutor getRemoteCallExecutor();

}
 
 No newline at end of file