Merge "Improve untrusted sources UX for app side-loading" am: c23d802958

am: e3ebb33a49

* commit 'e3ebb33a49099621c08edf07f40a06a4d0429cf3':
  Improve untrusted sources UX for app side-loading

1 files changed, 17 insertions, 6 deletions

diff --git a/src/com/android/packageinstaller/PackageInstallerActivity.java b/src/com/android/packageinstaller/PackageInstallerActivity.java
index 0385ecdf..e3ad4f34 100644
--- a/src/com/android/packageinstaller/PackageInstallerActivity.java
+++ b/src/com/android/packageinstaller/PackageInstallerActivity.java
@@ -64,6 +64,8 @@ import java.io.File;
 public class PackageInstallerActivity extends Activity implements OnCancelListener, OnClickListener {
     private static final String TAG = "PackageInstaller";
 
+    private static final int REQUEST_ENABLE_UNKNOWN_SOURCES = 1;
+
     private int mSessionId = -1;
     private Uri mPackageURI;
     private Uri mOriginatingURI;
@@ -222,7 +224,7 @@ public class PackageInstallerActivity extends Activity implements OnCancelListen
                     .setPositiveButton(R.string.settings, new DialogInterface.OnClickListener() {
                         public void onClick(DialogInterface dialog, int which) {
                             Log.i(TAG, "Launching settings");
-                            launchSettingsAppAndFinish();
+                            launchSecuritySettings();
                         }
                     })
                     .setOnCancelListener(this)
@@ -327,12 +329,21 @@ public class PackageInstallerActivity extends Activity implements OnCancelListen
        return null;
    }
 
-    private void launchSettingsAppAndFinish() {
-        // Create an intent to launch SettingsTwo activity
+    private void launchSecuritySettings() {
         Intent launchSettingsIntent = new Intent(Settings.ACTION_SECURITY_SETTINGS);
-        launchSettingsIntent.setFlags(Intent.FLAG_ACTIVITY_NEW_TASK);
-        startActivity(launchSettingsIntent);
-        finish();
+        startActivityForResult(launchSettingsIntent, REQUEST_ENABLE_UNKNOWN_SOURCES);
+    }
+
+    @Override
+    public void onActivityResult(int request, int result, Intent data) {
+        // If the settings app approved the install we are good to go regardless
+        // whether the untrusted sources setting is on. This allows partners to
+        // implement a "allow untrusted source once" feature.
+        if (request == REQUEST_ENABLE_UNKNOWN_SOURCES && result == RESULT_OK) {
+            initiateInstall();
+        } else {
+            finish();
+        }
     }
 
     private boolean isInstallRequestFromUnknownSource(Intent intent) {