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

Commit e7011d2b authored by Jon Miranda's avatar Jon Miranda Committed by Android (Google) Code Review
Browse files

Revert "Add TaskbarModeSwitchRule to test both transient/persistent taskbar." 

This reverts commit d5a6b5f6.

Reason for revert: Breaks tests due to "SharedPreferences in credential encrypted storage are not available until after user is unlocked"

Bug: 258256906
Change-Id: I1de69249685f9d2e71183357cf3eda8d443c7d97
parent d5a6b5f6

quickstep/Android.bp

+0 −1
Original line number Diff line number Diff line
@@ -38,7 +38,6 @@ filegroup { 
    name: "launcher3-quickstep-oop-tests-src",

    path: "tests",

    srcs: [

        "tests/src/com/android/quickstep/TaskbarModeSwitchRule.java",

        "tests/src/com/android/quickstep/NavigationModeSwitchRule.java",

        "tests/src/com/android/quickstep/AbstractQuickStepTest.java",

        "tests/src/com/android/quickstep/TaplTestsQuickstep.java",

quickstep/src/com/android/launcher3/taskbar/TaskbarActivityContext.java

+2 −17
Original line number Diff line number Diff line
@@ -23,7 +23,6 @@ import static android.view.WindowManager.LayoutParams.TYPE_NAVIGATION_BAR_PANEL; 


import static com.android.launcher3.AbstractFloatingView.TYPE_ALL;

import static com.android.launcher3.AbstractFloatingView.TYPE_REBIND_SAFE;

import static com.android.launcher3.Utilities.IS_RUNNING_IN_TEST_HARNESS;

import static com.android.launcher3.logging.StatsLogManager.LauncherEvent.LAUNCHER_FOLDER_OPEN;

import static com.android.launcher3.taskbar.TaskbarAutohideSuspendController.FLAG_AUTOHIDE_SUSPEND_DRAGGING;

import static com.android.launcher3.taskbar.TaskbarAutohideSuspendController.FLAG_AUTOHIDE_SUSPEND_FULLSCREEN;
@@ -305,8 +304,7 @@ public class TaskbarActivityContext extends BaseTaskbarContext { 
        int windowFlags = WindowManager.LayoutParams.FLAG_NOT_FOCUSABLE

                | WindowManager.LayoutParams.FLAG_SLIPPERY

                | WindowManager.LayoutParams.FLAG_SPLIT_TOUCH;

        if (DisplayController.isTransientTaskbar(this)

                && !IS_RUNNING_IN_TEST_HARNESS) {

        if (DisplayController.isTransientTaskbar(this)) {

            windowFlags |= WindowManager.LayoutParams.FLAG_NOT_TOUCH_MODAL

                    | WindowManager.LayoutParams.FLAG_WATCH_OUTSIDE_TOUCH;

        }
@@ -859,27 +857,14 @@ public class TaskbarActivityContext extends BaseTaskbarContext { 
        mControllers.taskbarStashController.enableManualStashingDuringTests(enableManualStashing);

    }



    /**

     * Enables the auto timeout for taskbar stashing. This method should only be used for taskbar

     * testing.

     */

    @VisibleForTesting

    public void enableBlockingTimeoutDuringTests(boolean enableBlockingTimeout) {

        mControllers.taskbarStashController.enableBlockingTimeoutDuringTests(enableBlockingTimeout);

    }



    /**

     * Unstashes the Taskbar if it is stashed. This method should only be used to unstash the

     * taskbar at the end of a test.

     */

    @VisibleForTesting

    public void unstashTaskbarIfStashed() {

        if (DisplayController.isTransientTaskbar(this)) {

            mControllers.taskbarStashController.updateAndAnimateTransientTaskbar(false);

        } else {

        mControllers.taskbarStashController.onLongPressToUnstashTaskbar();

    }

    }



    protected boolean isUserSetupComplete() {

        return mIsUserSetupComplete;

quickstep/src/com/android/launcher3/taskbar/TaskbarStashController.java

+4 −16
Original line number Diff line number Diff line
@@ -37,7 +37,6 @@ import android.view.View; 
import android.view.ViewConfiguration;



import androidx.annotation.NonNull;

import androidx.annotation.VisibleForTesting;



import com.android.internal.jank.InteractionJankMonitor;

import com.android.launcher3.Alarm;
@@ -170,7 +169,6 @@ public class TaskbarStashController implements TaskbarControllers.LoggableTaskba 
    private boolean mEnableManualStashingDuringTests = false;



    private final Alarm mTimeoutAlarm = new Alarm();

    private boolean mEnableBlockingTimeoutDuringTests = false;



    // Evaluate whether the handle should be stashed

    private final StatePropertyHolder mStatePropertyHolder = new StatePropertyHolder(
@@ -269,20 +267,10 @@ public class TaskbarStashController implements TaskbarControllers.LoggableTaskba 
     * Enables support for manual stashing. This should only be used to add this functionality

     * to Launcher specific tests.

     */

    @VisibleForTesting

    public void enableManualStashingDuringTests(boolean enableManualStashing) {

        mEnableManualStashingDuringTests = enableManualStashing;

    }



    /**

     * Enables the auto timeout for taskbar stashing. This method should only be used for taskbar

     * testing.

     */

    @VisibleForTesting

    public void enableBlockingTimeoutDuringTests(boolean enableBlockingTimeout) {

        mEnableBlockingTimeoutDuringTests = enableBlockingTimeout;

    }



    /**

     * Sets the flag indicating setup UI is visible

     */
@@ -858,12 +846,12 @@ public class TaskbarStashController implements TaskbarControllers.LoggableTaskba 
     * Attempts to start timer to auto hide the taskbar based on time.

     */

    public void tryStartTaskbarTimeout() {

        if (!DisplayController.isTransientTaskbar(mActivity)

                || mIsStashed

                || mEnableBlockingTimeoutDuringTests) {

        if (!DisplayController.isTransientTaskbar(mActivity)) {

            return;

        }

        if (mIsStashed) {

            return;

        }



        cancelTimeoutIfExists();



        mTimeoutAlarm.setOnAlarmListener(this::onTaskbarTimeout);

quickstep/src/com/android/quickstep/QuickstepTestInformationHandler.java

+0 −19
Original line number Diff line number Diff line
@@ -120,18 +120,6 @@ public class QuickstepTestInformationHandler extends TestInformationHandler { 
                                .getCurrentActivityContext()

                                .getTaskbarAllAppsTopPadding());

            }



            case TestProtocol.REQUEST_ENABLE_BLOCK_TIMEOUT:

                runOnTISBinder(tisBinder -> {

                    enableBlockingTimeout(tisBinder, true);

                });

                return response;



            case TestProtocol.REQUEST_DISABLE_BLOCK_TIMEOUT:

                runOnTISBinder(tisBinder -> {

                    enableBlockingTimeout(tisBinder, false);

                });

                return response;

        }



        return super.call(method, arg, extras);
@@ -161,13 +149,6 @@ public class QuickstepTestInformationHandler extends TestInformationHandler { 
                enable);

    }



    private void enableBlockingTimeout(

            TouchInteractionService.TISBinder tisBinder, boolean enable) {

        // Allow null-pointer to catch illegal states.

        tisBinder.getTaskbarManager().getCurrentActivityContext().enableBlockingTimeoutDuringTests(

                enable);

    }



    /**

     * Runs the given command on the UI thread, after ensuring we are connected to

     * TouchInteractionService.

quickstep/tests/src/com/android/quickstep/AbstractQuickStepTest.java

+0 −1
Original line number Diff line number Diff line
@@ -39,7 +39,6 @@ public abstract class AbstractQuickStepTest extends AbstractLauncherUiTest { 
    protected TestRule getRulesInsideActivityMonitor() {

        return RuleChain.

                outerRule(new NavigationModeSwitchRule(mLauncher)).

                around(new TaskbarModeSwitchRule(mLauncher)).

                around(super.getRulesInsideActivityMonitor());

    }