diff options
author | Gabriele M <moto.falcon.git@gmail.com> | 2017-10-13 22:27:54 +0200 |
---|---|---|
committer | Gabriele M <moto.falcon.git@gmail.com> | 2017-10-15 19:53:41 +0200 |
commit | 4b0c302eae8d772d09346718559cc42548485f75 (patch) | |
tree | 266c0d2a056f1e91dbaa8144c717517405b8ebc3 /src | |
parent | cec631f26e2cbd7c51fd7c28e05d379d6d0f86db (diff) | |
download | android_packages_apps_PackageInstaller-4b0c302eae8d772d09346718559cc42548485f75.tar.gz android_packages_apps_PackageInstaller-4b0c302eae8d772d09346718559cc42548485f75.tar.bz2 android_packages_apps_PackageInstaller-4b0c302eae8d772d09346718559cc42548485f75.zip |
Revert "DO NOT MERGE Disable overlays while installer is resumed"
If mOkCanInstall is false when onResume() is called, the button gets
disabled and never re-enabled. This fix is for branches that have
change I42c15d05dffc1f6db69c9f67148f7ceb6e8bd8ae ("Split layout for
confirm step.") which causes the button to be disabled and enabled
for different reasons. Here the button is constantly enabled and its
action changes depending on the value of mOkCanInstall. The attribute
filterTouchesWhenObscured should prevent tapjacking when overlays are
present, so revert this commit.
This reverts commit cec631f26e2cbd7c51fd7c28e05d379d6d0f86db.
REGRESSION-1262
Change-Id: I819aab95506672d291e07193b71f074292862754
Diffstat (limited to 'src')
-rw-r--r-- | src/com/android/packageinstaller/PackageInstallerActivity.java | 25 |
1 files changed, 2 insertions, 23 deletions
diff --git a/src/com/android/packageinstaller/PackageInstallerActivity.java b/src/com/android/packageinstaller/PackageInstallerActivity.java index dab8863e..de685825 100644 --- a/src/com/android/packageinstaller/PackageInstallerActivity.java +++ b/src/com/android/packageinstaller/PackageInstallerActivity.java @@ -16,6 +16,7 @@ */ package com.android.packageinstaller; +import android.app.Activity; import android.app.ActivityManagerNative; import android.app.AlertDialog; import android.app.Dialog; @@ -52,8 +53,6 @@ import android.widget.TabHost; import android.widget.TextView; import com.android.packageinstaller.permission.utils.Utils; -import com.android.packageinstaller.permission.ui.OverlayTouchActivity; - import java.io.File; import java.io.FileOutputStream; import java.io.IOException; @@ -70,8 +69,7 @@ import java.io.OutputStream; * Based on the user response the package is then installed by launching InstallAppConfirm * sub activity. All state transitions are handled in this activity */ -public class PackageInstallerActivity extends OverlayTouchActivity implements OnCancelListener, - OnClickListener { +public class PackageInstallerActivity extends Activity implements OnCancelListener, OnClickListener { private static final String TAG = "PackageInstaller"; private static final int REQUEST_ENABLE_UNKNOWN_SOURCES = 1; @@ -220,25 +218,6 @@ public class PackageInstallerActivity extends OverlayTouchActivity implements On } } - @Override - protected void onResume() { - super.onResume(); - - if (mOk != null) { - mOk.setEnabled(mOkCanInstall); - } - } - - @Override - protected void onPause() { - super.onPause(); - - if (mOk != null) { - // Don't allow the install button to be clicked as there might be overlays - mOk.setEnabled(false); - } - } - private void showDialogInner(int id) { // TODO better fix for this? Remove dialog so that it gets created again removeDialog(id); |