summaryrefslogtreecommitdiffstats
path: root/src/com/android/packageinstaller/permission/ui
Commit message (Collapse)AuthorAgeFilesLines
* Merge "Don't show app defined permissions in the review UI" into nyc-devSvetoslav Ganov2016-03-221-6/+17
|\
| * Don't show app defined permissions in the review UISvetoslav Ganov2016-03-211-6/+17
| | | | | | | | | | | | | | | | | | | | | | Only platform defined runtime permissions have corresponding app ops, hence there is no way to disable functionality guarded by app defined permissions. Therefore, app defined permissions should not show in the review UI. bug:27102458 Change-Id: Iedc7c4de9216007176b87cfecaeed69dbadc2068
* | Show No permissions, if there are no permissionsVinod Krishnan2016-03-182-15/+24
|/ | | | Change-Id: I285e326994268fc5b561e4a445ac2af326a64397
* Merge "Support app install from a content URI" into nyc-devSvetoslav Ganov2016-03-0613-18/+10
|\
| * Support app install from a content URISvet Ganov2016-03-0513-18/+10
| | | | | | | | | | | | bug:24079113 Change-Id: Ide1aa1667370f6b8d00ff269ef28992589656e9a
* | Update restricted icon position in RestrictedSwitchPreference.Sudheer Shanka2016-03-022-6/+14
|/ | | | | | | | | And fixed a bug where the admin disabled summary is shown even if the admin has not set a permission policy. Bug: 25603665 Bug: 27263775 Change-Id: I8cbbc4c326669a656ad5aef53896b388d556a74f
* Merge "Expose removed permissions flag as system API - installer" into nyc-devSvetoslav Ganov2016-02-232-2/+2
|\
| * Expose removed permissions flag as system API - installerSvet Ganov2016-02-222-2/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | There are some permissions that were removed from the platform and guard nothing but legacy apps may be checking them before calling APIs. Hence, these apps should get the permissions as expected despite them being a no-op. To address this the platform declares removed permissions as normal permissions that are hidden such that legacy apps can always get them. These permissions are not shown in the UI. Play needs a way to filter out these permissions like the platform as they have permissions UI too. bug:23361760 Change-Id: Ieecf69f70551d987f5fac1f128b7f7a0c242c378
* | Merge "Show permissions grant UI in requested order" into nyc-devSvetoslav Ganov2016-02-231-6/+6
|\ \
| * | Show permissions grant UI in requested orderSvet Ganov2016-02-221-6/+6
| |/ | | | | | | | | | | bug:23314383 Change-Id: I4b41b9941f147eefc5f3fc2c520aba7afabd26a0
* | No overlay when permissions shown - package installerSvet Ganov2016-02-207-46/+17
| | | | | | | | | | | | bug:26973205 Change-Id: Ibae1b971ceb1ea0d831435b9d5166482199e9184
* | Fix a NPE in GrantPermissionsActivitySvet Ganov2016-02-201-5/+6
|/ | | | | | | | | We get an NPE if an app that delcates no permissions as used requests runtime permissions. bug:21011604 Change-Id: Id0cac6dcbd78ef849a1eafa522b8a06e61b21a1b
* Use the correct userId when checking if disabled by admin.Sudheer Shanka2016-02-012-8/+9
| | | | Change-Id: I0fbe8ac8670b9fa4eb1fa35693856b47fdc974a4
* Update the summary used when permission is fixed by admin.Sudheer Shanka2016-01-262-4/+7
| | | | Change-Id: I427fd1e1c99153484944ee9955ae79ee9e2c23ef
* Merge "Add a padlock to options when disabled by admin."Sudheer Shanka2016-01-213-8/+110
|\
| * Add a padlock to options when disabled by admin.Sudheer Shanka2016-01-203-8/+110
| | | | | | | | | | | | | | | | Currently, if admin has enforced setPermissionGrantState to denied or granted, we disable the option and add a summary that the option is disabled by admin. With this change, a padlock will also be added in this case. Change-Id: I58080c914fabab045282eb3cc491901676fffaed
| * Merge "Revert "Make request permissions dialog layout robost"" am: 41d260fe98Svetoslav Ganov2016-01-153-35/+6
| |\ | | | | | | | | | | | | | | | | | | am: eccff95e32 * commit 'eccff95e32dbf09f0fe6df033daf09e8d3c4212e': Revert "Make request permissions dialog layout robost"
* | \ resolve merge conflicts of 9199155a7f to master.Svetoslav Ganov2016-01-203-250/+130
|\ \ \ | |/ / |/| | | | | Change-Id: I325541269f5a0f7c1fde7a57042543e769bed218
| * | Make the permission request dialog's layout robustSvetoslav Ganov2016-01-213-243/+151
| |/ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The old implementation was relying on a fixed window size where the content is positioned by a custom layout manager. It is possible however that subsequent permissions requests do not fit in the window as its size is computed based on the content of the first permissions request. There were also cases where the content is chopped after a rotation as the dialog size width was not re-evaluated while it should be. Further, animation from one permission request state to another was not properly done resulting in content being chopped off in some cases. The current approach is to have a dialog width for the content activity but the height is as tall as the screen allowing us to fit arbitrary large permission request content. Also we are resetting the fixed width on a configuration change so the dialog is robust to adjust size as needed. bug:24679384 bug:25755378 Change-Id: I4d23f81d8e59ce23bf9a27155ebb5ec6e2e6752c (cherry picked from commit c6dc4bb52b07886346b02b326c5c32a8299ed73e)
| * Merge "Revert "Make request permissions dialog layout robost""Svetoslav Ganov2016-01-153-35/+6
| |\ | | | | | | | | | | | | | | | | | | am: 41d260fe98 * commit '41d260fe9854819ca04a3b5908e6ab889bf3ffc4': Revert "Make request permissions dialog layout robost"
| | * Revert "Make request permissions dialog layout robost"Brian Carlstrom2016-01-143-35/+6
| | | | | | | | | | | | This reverts commit ecaeae17f52d6562d23dfec91e44bc3c0b4a6d13.
* | | Merge "Make request permissions dialog layout robost" am: 816baf3566Svetoslav Ganov2016-01-153-6/+35
|\| | | | | | | | | | | | | | | | | | | | am: b0a98d2e47 * commit 'b0a98d2e4778e39b62b3731cbc27cf6fdb541e24': Make request permissions dialog layout robost
| * | Merge "Make request permissions dialog layout robost"Svetoslav Ganov2016-01-153-6/+35
| |\| | | | | | | | | | | | | | | | | | | am: 816baf3566 * commit '816baf35660c3c46ffcb9be7ec72d343fb0e1400': Make request permissions dialog layout robost
| | * Make request permissions dialog layout robostSvet Ganov2016-01-133-6/+35
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | If the min width of a dialog changes in portrait vs landscape it was possible that a part of the request permissions dialog is chopped off. The custom content layout manager was using a fixed width and changing orientation may lead to a dialog width lesser than the fixed width of the layout manager. Another problem was that if the above occurs and the window width changes then the window may not be tall enough to fit the content. To address this we have to do a gross move and re-add the window to the window manager, so it can be resized. Another issue was that if the "Don't ask again" checkbox is shown not for the first but say the second permission request (in the case of multi-permission request in one API call) the content was chopped off as the height measurement for how much the content needs to be was restricted by the parent measure spec. Now we measure with no restrictions to accommodate the whole content. The way reqeust permissions dialog is implemented is problematic as it is a dialog styled activity which means we may need to resize its window. It is better to implement it as a fullscreen activity that has a custom content layout mangar that makes the content look like a dialog. Since this is risky at this point we do targeted fixes to address the above issues. bug:24679384 Change-Id: If51a360ba17dfb71b66dcf841ea47c17606eba27
* | | Legacy apps shouldn't be able to request permissionsSvetoslav2015-12-041-0/+11
| | | | | | | | | | | | | | | | | | bug:26032074 Change-Id: If6566411e08b3a27eecc2ca559c2b902dc8c1e65
* | | Add some things missing in the previous CL.Svet Ganov2015-12-012-0/+2
| | | | | | | | | | | | Change-Id: I17ae37d84c0d78bcfcaa848efb17f46c7f7c915a
* | | Add optional permission review for legacy apps - installerSvet Ganov2015-12-015-65/+632
| | | | | | | | | | | | Change-Id: Ifc88b2fa259d2d22bea6b5500cded2714ad4da85
* | | Fix build am: 0937012335Svetoslav2015-11-215-11/+11
|\| | | | | | | | | | | | | | | | | | | | am: 2a6f5d0fc6 * commit '2a6f5d0fc6b3fc65120ba7ed13f81af2d59eb11f': Fix build
| * | Fix buildSvetoslav2015-11-205-11/+11
| | | | | | | | | | | | Change-Id: I2a3e235bb13f1920c14f6776ee3a1ef7285ea548
* | | resolve merge conflicts of c10abb25f3 to cw-e-dev. am: c3d14f58d8 am: f4aa668c22Svetoslav2015-11-203-274/+3
|\| | | | | | | | | | | | | | | | | | | | am: b44b5380b4 * commit 'b44b5380b41ea7a0337709494f6e63eea60c522e': Clean up package installer styling
| * | resolve merge conflicts of c10abb25f3 to cw-e-dev.Svetoslav2015-11-203-274/+3
| |\ \ | | | | | | | | | | | | Change-Id: I965fb579e2730fa1cdb182bc18e134f262929328
| | * \ Clean up package installer stylingSvetoslav2015-11-2020-29/+1774
| | |\ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | am: 2cf17ddcef * commit '2cf17ddcef3c8dd260bd3d174123842c81a7d025': Clean up package installer styling
| | | * | Clean up package installer stylingSvetoslav2015-11-1820-29/+1774
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Package installer should be using exclusively the device defaut themes on handhelds (phones and tables) to allow OEMs customize its look and feel. Also for handhelds the installer should use the same preference framework as the settings app to ensure it looks like the settings app on an OEMs device. This change needs to be picked up by a GMS build to deliver to partners ensuring the installer UI is consistent with the device UI. bug:24286616 Change-Id: I92e39fd1488e76b0b23b7f1efa13e04ed5bbc7ba
* | | | | resolve merge conflicts of 56ac33bf1e to master.Svetoslav2015-11-2020-25/+2041
|\| | | | | | | | | | | | | | | | | | | Change-Id: I1439964d551cdc6b829fc99edd8b2c138dc1abf5
| * | | | resolve merge conflicts of c10abb25f3 to cw-e-dev.Svetoslav2015-11-2020-27/+2041
| | | | | | | | | | | | | | | | | | | | Change-Id: I3fe38a9ac62466b38efec834dceb712d2782c518
* | | | | Remove use of allow_stacked_button_bar.Filip Gruszczynski2015-11-201-3/+0
|/ / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | We are trying to remove as many size thresholds as possible for the multi window support. This value will be now calculated in code during the creation of the view. Bug: 25394588 Change-Id: I7dfaa23c491fbab9a320c52877deff8a26ddb91e
* | | | Button bar animating when it isn't supposed toAnthony Hugh2015-11-021-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | There was an incorrect calculation for determining when the content was occluded by the button bar or not. BUG: 25164689 Change-Id: I121f6e0232da2d1536e0fbbf697e00a4f9c1841f
* | | | Fix out of bounds exceptionAnthony Hugh2015-10-282-11/+21
| |_|/ |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | It looks like the cached index had become invalid at some point. It's unclear why, but the logs suggest that a bunch of Home services crashed. Rather than cache the index within the Activity, I have changed the code to send the index along with the dialog activity launch. This way when the dialog returns, the index will be saved within the callback. The index should be enough because the permission set should not change for the app between screens and the permission set is alphabetized. BUG: 25334674 Change-Id: I8f9189960aa7fc9b09cc25f594397523550ac626
* | | Fix Permissions Dialog animationsAnthony Hugh2015-10-211-4/+36
| | | | | | | | | | | | | | | | | | | | | | | | | | | Make handler run on UI thread to avoid race conditions. Update some logic to prevent animation from running when it shouldn't be. BUG: 24954323 Change-Id: Ia1635cbb3c9128080a8e4a9c3ce6ee935c4d957e
* | | Reduce the top padding when message isn't at top.Danny Epstein2015-10-201-1/+2
| | | | | | | | | | | | | | | | | | | | | | | | If either the page number or an icon is shown above the message, reduce the top padding on round screens. Bug: 24961379 Change-Id: Iec9193e3bbdd8ac607743cd0010f2331edbebc81
* | | Add top padding to grant permission content.Danny Epstein2015-10-191-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | Add top padding to the content inside the scroll view for the grant permission dialog so that you can read the beginning of the text on round watches. Bug: 24961379 Change-Id: I9311fbaabd72bd8f8aee4f71845c1ae11bfeab8c
* | | am 6986f29b: Auto grant permissions in an already granted group.Svetoslav Ganov2015-10-151-2/+18
|\| | | | | | | | | | | | | | * commit '6986f29bb93a6aaaeaaddcab081d0e0afd7082e8': Auto grant permissions in an already granted group.
| * | Auto grant permissions in an already granted group.Svetoslav Ganov2015-10-151-2/+18
| | | | | | | | | | | | | | | | | | bug:24074183 Change-Id: Iad6f9b7d5b83d0b0121796891bcd069658995a47
* | | am 86f1ee8c: Revert "Auto grant permissions in an already granted group."Ian Pedowitz2015-10-151-18/+2
|\| | | | | | | | | | | | | | * commit '86f1ee8c2db04d37d48f25a60086e5a27a812fb9': Revert "Auto grant permissions in an already granted group."
| * | Revert "Auto grant permissions in an already granted group."Ian Pedowitz2015-10-151-18/+2
| | | | | | | | | | | | | | | | | | This reverts commit c7e595a22c11247b3e6b0fbfa324542321ab9d14. Change-Id: I0a319ef590e9c0f830af470155213677bed684f4
* | | am c7e595a2: Auto grant permissions in an already granted group.Svetoslav2015-10-151-2/+18
|\| | | | | | | | | | | | | | * commit 'c7e595a22c11247b3e6b0fbfa324542321ab9d14': Auto grant permissions in an already granted group.
| * | Auto grant permissions in an already granted group.Svetoslav2015-10-151-2/+18
| |/ | | | | | | | | | | | | bug:24074183 Change-Id: I8eb4e4c76ed27721b04e09ae009ddd2daec1e0b1 (cherry picked from commit ecc46fe0ce772338786cb9e23cb872189e6a285c)
* | Modifies wear Permission title height to avoid clippingAlex Hills2015-10-131-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | We were explicitly setting the height of the permission screen title, which wasn't 100% guaranteed to work in all languages. This change sets minHeight (to maintain UI consistency) instead, and changes the height to wrap_content to enable it to expand further if necessary. Bug:24774285 Bug:24391658 Change-Id: Icf25ad6a176537b0f34a30f53220c73fccec2d24
* | Permissions redline fixesAnthony Hugh2015-10-052-50/+99
| | | | | | | | | | | | | | | | | | | | | | | | This fixes the button bar animation to work on round devices and also fixes the max height animation to make sure the vertical button layout animates to max height when there are only two buttons. Also adjust some redlines like margins on the buttons and updated some files that were missing our licensing header. Change-Id: Ifad4d0f92605be04db2d3c991b4e8a26eadb4b14
* | Merge "Add enabled/disabled icons for permissions on Wear" into cw-e-devAnthony Hugh2015-09-251-2/+48
|\ \