summaryrefslogtreecommitdiffstats
path: root/src
Commit message (Collapse)AuthorAgeFilesLines
* DO NOT MERGE Reworked additional permissions fragment for each app am: ↵Keyvan Amiri2016-11-241-30/+45
|\ | | | | | | | | | | | | | | 6dfb1f6553 am: 18817ebb89 Change-Id: I5f2254900247fda33ca2b48b38428f01ef11b914
| * DO NOT MERGE Reworked additional permissions fragment for each appKeyvan Amiri2016-11-241-30/+45
| |\ | | | | | | | | | | | | | | | am: 6dfb1f6553 Change-Id: I78ed7c841680ed3d82835bca2394384311e6d8b1
| | * DO NOT MERGE Reworked additional permissions fragment for each appKeyvan Amiri2016-11-231-30/+45
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fixed packageinstaller crash when selecting additional permissions for an app e.g. "Live Channels". setHeader should not be called in onCreate since it will update title UI which is initialized in onCreateView in SettingsWithHeader.java. Also, updated the UI for AdditionalPermissionsFragment for each individual app to include the app banner and title in the heading. Change-Id: Idfc12a28b38b84fcc5d97de4f85ada968e122d40 Fixes: 32338150 Test: Visual inspection of the permission UI (cherry picked from commit c6d15a867dbbf9cb730e409a7b7426baeb64ac9f)
| | * Hide toast windows when permissions UI shownSvetoslav Ganov2016-07-201-0/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | When the permissions UI is showing we are hiding the system alrert windows that can overlay it in an attempt to redress the UI. It is however, also possible to use a toast window to overlay the permissions UI. This change makes sure we hide the toasts in addition to system alert windows when the permissions UI is showing. bug:30087538 Change-Id: Ifb9ecf4593ca24b628212f0737d574a537f0cccd (cherry picked from commit cc5e0323e83d6124f45ca75f55bc58793b550035)
* | | Support install once feature.Philip P. Moltmann2016-10-131-8/+18
| | | | | | | | | | | | | | | | | | | | | Test: Simulated OK result from settings and triggered run-once workflow Fixes: 32105068 Change-Id: I9f6c3b56b09d520d2bc9c6eeb6c75f151e3f1aff
* | | If the permission dialog would stick out, move it.Philip P. Moltmann2016-09-301-1/+6
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Also scroll the whole dialog if the screen is too small to for the dialog. Test: Executed steps in the bug. Dialog looks fine. Also toyed around with display scaling and font scaling to make sure dialog always looks ok (but might scroll) on very small displays. Fixes: 31442798 Change-Id: I74328afde38ec2170a14aaa8e77675ad2164460e (cherry picked from commit 3a5c306e5b6bf2285980aa34cb8eb00fbcf5ea2c)
* | | Parse package before enabling unknown sourcesPhilip P. Moltmann2016-09-271-12/+38
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | So that when we return from the settings app we do not have distinguish between file and content-uri based install anymore Test: Manually went through the following workflows: - Install from file - Unknown sources is already enabled - Unknown sources is not enabled yet - Return 0 from settings - unknown sources was enabled - unknown sources was not enabled - Return -1 from settings - unknown sources was enabled - unknown sources was not enabled - Install from content URI - Unknown sources is already enabled - Unknown sources is not enabled yet - Return 0 from settings - unknown sources was enabled - unknown sources was not enabled - Return -1 from settings - unknown sources was enabled - unknown sources was not enabled Fixes: 31328285 Change-Id: I1d23f2ba9a835f7bffc59108565b286aeb675411
* | | Calculate and apply 'installed size'Ryuki Nakamura2016-09-121-0/+9
| | | | | | | | | | | | | | | Bug: 30056625 Change-Id: I1ffc7ba88ea5f99591b66c08e72ea07d60ab82d5
* | | Resolve merge conflict from Ia3d8f54624801696eaec4f9216d7bbefb0b105a4.Philip P. Moltmann2016-09-011-1/+1
| | | | | | | | | | | | | | | Bug: 30056625 Change-Id: Ie8e46fc7a2c340fb04b64feb331af0d882fe9672
* | | Read and apply 'packageName'Ryuki Nakamura2016-08-311-1/+4
| | | | | | | | | | | | | | | Bug: 30056625 Change-Id: Ie8e46fc7a2c340fb04b64feb331af0d882fe9672
* | | Merge "Fix the visibility for disabled apps in settings" into nyc-mr1-devLei Yu2016-08-301-0/+4
|\ \ \
| * | | Fix the visibility for disabled apps in settingsjackqdyulei2016-08-261-0/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Bug: 30770664 Test: Manual - 1. Disable one app 2. Go to Settings -> Apps -> menu -> App permissions 3. to check the visibility of that app. Don't show disabled app in the PermissionAppsFragment. Change-Id: I0d97f7011c13c9a78e988901285f10ca9e5b2cd8 (cherry picked from commit 92c2b41b91bffcc5ef1dbbe37ef66720753180ad)
* | | | Fix the sorting bugs for system appsjackqdyulei2016-08-261-0/+2
|/ / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Bug: 30770664 Test: Manual - 1. Go to settings -> Apps -> Menu -> App permissions 2. Click one permissions folder. 3. Observe the sequence of apps. 4. Click the "show system" menu. 5. Observe the sequence of apps. Now the system apps and user apps are merged and sorted by alphabetic order. Change-Id: I45c933f9be30f138085f46f9515c38bf998bfb9a (cherry picked from commit b7ea973e7ad8e8c0a5dad420e0fc705efc92ec0b)
* / / Fix an exception in PackageInstallerSvetoslav Ganov2016-08-231-4/+6
|/ / | | | | | | | | | | | | | | | | | | | | | | It is apparently possible for an activity to get a configuration change callback when its view is no longer attached to the window manager. Check if a view is attached before removing it from the window manager. bug:30389560 Change-Id: I384401742eca61d140c7590eb5c67150a15da83b
* | Requested unused permissions reported as granted for app using none.Svet Ganov2016-08-111-0/+2
| | | | | | | | | | | | | | | | | | | | | | If the app uses no permissions and requested a runtime permission, the latter is reported granted while it is correctly not granted. Now we initialize everything as denied in case we exit eraly as the app does not use any permissions. bug:30747277 Change-Id: I08dbf086e7c3d5f66b6831c54910fbb4f0943388
* | Prevent "dont't ask again" being inpropoerly propagatedSvet Ganov2016-08-041-1/+1
| | | | | | | | | | | | | | | | | | | | | | If the first permission of a several permission request is denied with prejudice the rest of the denied permissions in the sequence are also incorrectly denied with prejudice even if the user didn't select "don't ask again". bug:30653785 Change-Id: Ib942dd3c692d1c3a09dc1a64f2947ded0f193923
* | Remove lefover secure buttonSvetoslav Ganov2016-07-181-56/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | An incorrect fix for a security bug where apps with overlay access can redress the permission message added a secure button implementation that refuses interaction if the window is covered. While this was fixed in M branches we did not merge the change in NYC because we now hide all overlays. However, the app op which controls overlays allows privileged apps to bypass the app op. It is fine to have priv apps overlay stuff on top of the permissions UI except that the allow button show be clickable. This change removes the offending code. bug:29879858 Change-Id: I9d0b5e62b5ef029069e37f34dbcd90d4909d966b
* | Merge "Trigger transparency dialog only if restriction is set by admin." ↵TreeHugger Robot2016-07-141-5/+11
|\ \ | | | | | | | | | into nyc-mr1-dev
| * | Trigger transparency dialog only if restriction is set by admin.Sudheer Shanka2016-07-121-5/+11
| | | | | | | | | | | | | | | Bug: 30104199 Change-Id: I97a16938c36b4dadc9167652f6297acf829366e9
* | | Hide toast windows when permissions UI shownSvetoslav Ganov2016-07-131-0/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | When the permissions UI is showing we are hiding the system alrert windows that can overlay it in an attempt to redress the UI. It is however, also possible to use a toast window to overlay the permissions UI. This change makes sure we hide the toasts in addition to system alert windows when the permissions UI is showing. bug:30087538 Change-Id: Ifb9ecf4593ca24b628212f0737d574a537f0cccd
* | | Read and apply 'installLocation'Philip P. Moltmann2016-07-111-0/+8
|/ / | | | | | | | | | | | | | | | | | | ... when installing app via packageInstaller. This allows the apk to have some limited control over where it will be installed. Change-Id: Id5140e7d2a2e3b8dd59815a0febf5c05cee08fbf Fixes: 30056625
* | Do not allow downgrades of embedded microappsVinod Krishnan2016-07-062-4/+20
| | | | | | | | | | | | | | | | - flag is passed in from https://critique.corp.google.com/#review/126666577 Bug: 29641183 Change-Id: I1becbb446d824331430eb37d99bdbc7a36dd13db
* | Don't offer menu items it they are no-opSvet Ganov2016-06-292-13/+37
| | | | | | | | | | | | | | | | | | | | | | In the permission apps screen we have a menu item to show/hide system apps. It is possible that no system app uses the permission making the menu item do nothing. We don't add the menu item if no system app uses the permission. bug:29395895 Change-Id: I1756981b2f711fc7a886bd4227196fef1a3cea03
* | Only log (not crash) when activity is not found.Philip P. Moltmann2016-06-171-1/+6
|\| | | | | | | | | | | am: 87ec609c14 Change-Id: I07c7e57630269d9d37d8e10dfa49e59e88cec022
| * Only log (not crash) when activity is not found.Philip P. Moltmann2016-06-161-1/+6
| | | | | | | | | | Fixes: 29260288 Change-Id: Iaa309f362ae6a296f7ba365e418d4a6f55b23f0c
* | Merge \"Hide Status and Navigation Bar when uninstalling\" into nyc-devPhilip P. Moltmann2016-06-101-0/+8
|\| | | | | | | | | | | am: 9c177f8d28 Change-Id: Ieac52df39c5481ea8c964be61d3a5a5c9bc024e3
| * Merge "Hide Status and Navigation Bar when uninstalling" into nyc-devPhilip P. Moltmann2016-06-101-0/+8
| |\
| | * Hide Status and Navigation Bar when uninstallingPhilip P. Moltmann2016-06-021-0/+8
| | | | | | | | | | | | | | | | | | | | | ... at least unless we want to show a progress dialog. Fixes: 29094372 Change-Id: I45e0bbf32355a93635c773c526151b849bc1e8a3
* | | Fix a crash in package installerSvet Ganov2016-06-092-2/+1
|\| | | | | | | | | | | | | | | | | am: 869786e3e4 Change-Id: Ic1d8d6221194731a6c967f736a1352a905bba8b0
| * | Fix a crash in package installerSvet Ganov2016-06-082-2/+1
| |/ | | | | | | | | | | | | | | | | | | | | | | The first bug was that we were starting the install before having the package info for the installed app. The second was that we were calling the permission review UI despite we were there before the user selected to install. i.e. the user already reviewed the permissions. bug:29197887 Change-Id: I2fac4d014dde7f7744a6b831c2b0ff510e29e8e8
* | Merge "Show installer UI for user to confirm side-loading" into nyc-devSvetoslav Ganov2016-06-011-7/+1
|\| | | | | | | | | | | | | | | | | am: 6622140841 * commit '662214084178a687a32823decd4f42ba82313def': Show installer UI for user to confirm side-loading Change-Id: I017268c1d12c5fdfaea099e9428c3def1b728bdd
| * Show installer UI for user to confirm side-loadingSvetoslav Ganov2016-06-011-7/+1
| | | | | | | | | | | | | | | | | | This change fixes a regression where we stopped showing a confirmation UI for an app side-loading. bug:28551937 Change-Id: Ia2114d273473d6e00696f9de8c523190a1b444ad
* | Merge "Format the template using HTML tags." into nyc-devPhilip P. Moltmann2016-05-251-11/+4
|\| | | | | | | | | | | | | | | | | am: e071d11592 * commit 'e071d1159277a00e0484dbf43899bc90ea27176c': Format the template using HTML tags. Change-Id: I1858bc3c1e6d6fb5f4b37a2ef4a287de635045ae
| * Format the template using HTML tags.Philip P. Moltmann2016-05-231-11/+4
| | | | | | | | | | | | | | Manual formatting is to complicated to do. Fixes: 28852429 Change-Id: Ibab723b82a2b14ad94a3038dffccc96f86ba8ee4
* | Finish package installer activity on a cancel resultSvetoslav Ganov2016-05-181-0/+3
|\| | | | | | | | | | | | | | | | | am: 6fdd93b339 * commit '6fdd93b33927675a6a062f0b8fd00ef3866ac2f7': Finish package installer activity on a cancel result Change-Id: If34ab52b7c25cdbf69a06db5b4cf7e87b62e12bd
| * Finish package installer activity on a cancel resultSvetoslav Ganov2016-05-181-0/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | We made a change to the PackageInstaller to start settings for a result to allow partners to return OK to continue the install flow, otherwise requiring the user to start install again (the desired behavior on Nexus). This change finishes the package installer activity if settings does not return success to continue the install flow. bug:28838075 Change-Id: I4ff1b1a713d46a8584baf66f379ab070226c125f
* | Merge "Add missing else statement" into nyc-devSvet Ganov2016-05-181-2/+2
|\| | | | | | | | | | | | | | | | | am: 622c5b563e * commit '622c5b563e1f2e7a78e64129f350387613161153': Add missing else statement Change-Id: I0e6e13bf0bbd368375bd924c496d1b8e011cbf19
| * Merge "Add missing else statement" into nyc-devSvetoslav Ganov2016-05-181-2/+2
| |\
| | * Add missing else statementSvet Ganov2016-05-181-2/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | This change fixes a regression where we endup installing apps even if unknown sources is disabled - missing else statement. bug:28551937 Change-Id: I05dd28a28eb9ffd9eda8a6ee2fdccd501d2e08d1
* | | Merge "Take advantage of new MotionEvent flag to prevent tapjacking." into ↵Michael Wright2016-05-161-0/+56
|\| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | nyc-dev am: 1be5a77aaa * commit '1be5a77aaa3bfba81063cbba38422dfe30c4ce40': Take advantage of new MotionEvent flag to prevent tapjacking. Change-Id: I416339e7bb12371480432699499871cf3c43bcab
| * | Merge "Take advantage of new MotionEvent flag to prevent tapjacking." into ↵TreeHugger Robot2016-05-161-0/+56
| |\ \ | | | | | | | | | | | | nyc-dev
| | * | Take advantage of new MotionEvent flag to prevent tapjacking.Michael Wright2016-03-311-0/+56
| | | | | | | | | | | | | | | | | | | | Bug: 26677796 Change-Id: I563541f0a42564b854af0f8037c1d4741c79a2ac
* | | | Merge "Ensure app label properly bolded" into nyc-devSvetoslav Ganov2016-05-141-2/+2
|\| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | am: 55dbb25ae3 * commit '55dbb25ae341353ef10312d89a248d4e658be9ee': Ensure app label properly bolded Change-Id: If437d0944f8dc45107d67252ccf1f17e8c748ca3
| * | | Merge "Ensure app label properly bolded" into nyc-devSvetoslav Ganov2016-05-141-2/+2
| |\ \ \
| | * | | Ensure app label properly boldedSvetoslav Ganov2016-05-131-2/+2
| | | |/ | | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This change ensures the app name is properly bolded in the premissions request UI even if it is a substring of "Allow" :) bug:28719607 Change-Id: I5a313f67c19a2d882732b1c97f0dbee5782b22f5
* | | | Merge "Use badged icons for permissions." into nyc-devPhilip P. Moltmann2016-05-131-10/+12
|\| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | am: 988f3bcb21 * commit '988f3bcb21b7026cfab89d22ef912f98c9c9f8a2': Use badged icons for permissions. Change-Id: I922e3df467d31be00c80128c91330b381853ce70
| * | | Merge "Use badged icons for permissions." into nyc-devPhilip P. Moltmann2016-05-131-10/+12
| |\ \ \
| | * | | Use badged icons for permissions.Philip P. Moltmann2016-05-111-10/+12
| | |/ / | | | | | | | | | | | | | | | | | | | | | | | | | | | | Please note that the labels stay unbadged. I.e. The label will always be "Contacts" instead of "Work Contacts". Fixes: 28403737 Change-Id: Ib994db946b07098a8c0f95eab5787fb30532fe62
* | | | Merge "Finish whole installation on cancel dialog." into nyc-devPhilip P. Moltmann2016-05-131-1/+1
|\| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | am: f682d8970c * commit 'f682d8970cc10e9943232cde1888d214637e610e': Finish whole installation on cancel dialog. Change-Id: I785f39748871178c428330c70428d2a8b4979f78
| * | | Merge "Finish whole installation on cancel dialog." into nyc-devPhilip P. Moltmann2016-05-131-1/+1
| |\ \ \