| Commit message (Collapse) | Author | Age | Files | Lines |
|\
| |
| |
| |
| |
| |
| | |
Conflicts:
src/com/android/packageinstaller/permission/ui/handheld/GrantPermissionsViewHandlerImpl.java
Change-Id: Ia57d17750406754f1e19266f51f0115e3cf774ba
|
| |
| |
| |
| |
| |
| |
| | |
Test: Simulated OK result from settings and triggered run-once
workflow
Fixes: 32105068
Change-Id: I9f6c3b56b09d520d2bc9c6eeb6c75f151e3f1aff
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
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)
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
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
|
| |
| |
| |
| |
| | |
Bug: 30056625
Change-Id: I1ffc7ba88ea5f99591b66c08e72ea07d60ab82d5
|
| |
| |
| |
| |
| | |
Bug: 30056625
Change-Id: Ie8e46fc7a2c340fb04b64feb331af0d882fe9672
|
| |
| |
| |
| |
| | |
Bug: 30056625
Change-Id: Ie8e46fc7a2c340fb04b64feb331af0d882fe9672
|
| |\ |
|
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
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)
|
| |/
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
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)
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
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
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
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
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
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
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
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
|
| |\
| | |
| | |
| | | |
into nyc-mr1-dev
|
| | |
| | |
| | |
| | |
| | | |
Bug: 30104199
Change-Id: I97a16938c36b4dadc9167652f6297acf829366e9
|
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
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
|
| |/
| |
| |
| |
| |
| |
| |
| |
| |
| | |
... 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
|
| |
| |
| |
| |
| |
| |
| |
| | |
- flag is passed in from https://critique.corp.google.com/#review/126666577
Bug: 29641183
Change-Id: I1becbb446d824331430eb37d99bdbc7a36dd13db
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
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
|
| |\
| | |
| | |
| | |
| | |
| | | |
am: 87ec609c14
Change-Id: I07c7e57630269d9d37d8e10dfa49e59e88cec022
|
| |\ \
| | | |
| | | |
| | | |
| | | |
| | | | |
am: 9c177f8d28
Change-Id: Ieac52df39c5481ea8c964be61d3a5a5c9bc024e3
|
| |\ \ \
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
am: 869786e3e4
Change-Id: Ic1d8d6221194731a6c967f736a1352a905bba8b0
|
| |\ \ \ \
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | | |
am: 6622140841
* commit '662214084178a687a32823decd4f42ba82313def':
Show installer UI for user to confirm side-loading
Change-Id: I017268c1d12c5fdfaea099e9428c3def1b728bdd
|
| |\ \ \ \ \
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | | |
am: e071d11592
* commit 'e071d1159277a00e0484dbf43899bc90ea27176c':
Format the template using HTML tags.
Change-Id: I1858bc3c1e6d6fb5f4b37a2ef4a287de635045ae
|
| |\ \ \ \ \ \
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | | |
am: 6fdd93b339
* commit '6fdd93b33927675a6a062f0b8fd00ef3866ac2f7':
Finish package installer activity on a cancel result
Change-Id: If34ab52b7c25cdbf69a06db5b4cf7e87b62e12bd
|
| |\ \ \ \ \ \ \
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | | |
am: 622c5b563e
* commit '622c5b563e1f2e7a78e64129f350387613161153':
Add missing else statement
Change-Id: I0e6e13bf0bbd368375bd924c496d1b8e011cbf19
|
| |\ \ \ \ \ \ \ \
| | | | | | | | | |
| | | | | | | | | |
| | | | | | | | | |
| | | | | | | | | |
| | | | | | | | | |
| | | | | | | | | |
| | | | | | | | | |
| | | | | | | | | |
| | | | | | | | | |
| | | | | | | | | | |
nyc-dev
am: 1be5a77aaa
* commit '1be5a77aaa3bfba81063cbba38422dfe30c4ce40':
Take advantage of new MotionEvent flag to prevent tapjacking.
Change-Id: I416339e7bb12371480432699499871cf3c43bcab
|
| |\ \ \ \ \ \ \ \ \
| | | | | | | | | | |
| | | | | | | | | | |
| | | | | | | | | | |
| | | | | | | | | | |
| | | | | | | | | | |
| | | | | | | | | | |
| | | | | | | | | | |
| | | | | | | | | | | |
am: 55dbb25ae3
* commit '55dbb25ae341353ef10312d89a248d4e658be9ee':
Ensure app label properly bolded
Change-Id: If437d0944f8dc45107d67252ccf1f17e8c748ca3
|
| |\ \ \ \ \ \ \ \ \ \
| | | | | | | | | | | |
| | | | | | | | | | | |
| | | | | | | | | | | |
| | | | | | | | | | | |
| | | | | | | | | | | |
| | | | | | | | | | | |
| | | | | | | | | | | |
| | | | | | | | | | | | |
am: 988f3bcb21
* commit '988f3bcb21b7026cfab89d22ef912f98c9c9f8a2':
Use badged icons for permissions.
Change-Id: I922e3df467d31be00c80128c91330b381853ce70
|
| |\ \ \ \ \ \ \ \ \ \ \
| | | | | | | | | | | | |
| | | | | | | | | | | | |
| | | | | | | | | | | | |
| | | | | | | | | | | | |
| | | | | | | | | | | | |
| | | | | | | | | | | | |
| | | | | | | | | | | | |
| | | | | | | | | | | | | |
am: f682d8970c
* commit 'f682d8970cc10e9943232cde1888d214637e610e':
Finish whole installation on cancel dialog.
Change-Id: I785f39748871178c428330c70428d2a8b4979f78
|
| |\ \ \ \ \ \ \ \ \ \ \ \
| | | | | | | | | | | | | |
| | | | | | | | | | | | | |
| | | | | | | | | | | | | |
| | | | | | | | | | | | | |
| | | | | | | | | | | | | |
| | | | | | | | | | | | | |
| | | | | | | | | | | | | |
| | | | | | | | | | | | | | |
am: 51b7ac8dc1
* commit '51b7ac8dc1b2e6ccb2d3af2f1c1d84b529977e3a':
Fine tune UI of package installer.
Change-Id: I0b2407e42d74ac3c15b3926f202b327de5561c9a
|
| |\ \ \ \ \ \ \ \ \ \ \ \ \
| | | | | | | | | | | | | | |
| | | | | | | | | | | | | | |
| | | | | | | | | | | | | | |
| | | | | | | | | | | | | | |
| | | | | | | | | | | | | | |
| | | | | | | | | | | | | | |
| | | | | | | | | | | | | | |
| | | | | | | | | | | | | | | |
am: f63739bb3f
* commit 'f63739bb3f5de9927171b84630ec5872ba85fa6c':
Fix app install flow
Change-Id: I37bdfd9f7e005d57eb90e46f87f4cabcfaa6519c
|
| | | | | | | | | | | | | | |
| | | | | | | | | | | | | | |
| | | | | | | | | | | | | | |
| | | | | | | | | | | | | | |
| | | | | | | | | | | | | | |
| | | | | | | | | | | | | | |
| | | | | | | | | | | | | | |
| | | | | | | | | | | | | | |
| | | | | | | | | | | | | | |
| | | | | | | | | | | | | | |
| | | | | | | | | | | | | | |
| | | | | | | | | | | | | | |
| | | | | | | | | | | | | | |
| | | | | | | | | | | | | | |
| | | | | | | | | | | | | | |
| | | | | | | | | | | | | | |
| | | | | | | | | | | | | | |
| | | | | | | | | | | | | | |
| | | | | | | | | | | | | | |
| | | | | | | | | | | | | | |
| | | | | | | | | | | | | | |
| | | | | | | | | | | | | | |
| | | | | | | | | | | | | | |
| | | | | | | | | | | | | | | |
The default directory for PackageInstaller moved from
/data/user/foo to /data/user_de/foo. Not sure on the exact reason
why, but it has something to do with the file based encryption
changes in N. This was causing the temp APK file that we were
passing to PackageManager for installs to be in an invalid location.
This resulted in failed installations.
This is OK though because we never really wanted to use a temporary
file. We were using it due to limitations in the PackageInstaller
APIs. Since implementation, new APIs have been added to allow us
to pass a stream through instead.
This CL updates the install logic to use the stream API instead.
Because this code path is being deprecated and replaced, I've opted
to do the bare minimum needed to keep this code operational. I've
also borrowed installation code from Wearsky/Finsky codebase to
make it easier and more reliable.
BUG: 28021618
Change-Id: Ia245abf56649e350e99cf48d2b503906d3cd855c
|
| | | | | | | | | | | | | | |
| | | | | | | | | | | | | | |
| | | | | | | | | | | | | | |
| | | | | | | | | | | | | | |
| | | | | | | | | | | | | | |
| | | | | | | | | | | | | | |
| | | | | | | | | | | | | | |
| | | | | | | | | | | | | | | |
The permission of dial out will be auto enabled if only disable it under call
group permission. This change disable auto enable permission in this case.
Change-Id: Ibe8c9064fcd4fdfd33da69d68c145a5108a484d2
CRs-Fixed:1079155
|
|\ \ \ \ \ \ \ \ \ \ \ \ \ \ \ |
|
| | | | | | | | | | | | | | | |
| | | | | | | | | | | | | | | |
| | | | | | | | | | | | | | | |
| | | | | | | | | | | | | | | |
| | | | | | | | | | | | | | | |
| | | | | | | | | | | | | | | |
| | | | | | | | | | | | | | | |
| | | | | | | | | | | | | | | | |
Update application permission setting,the storage permission
only show one configration.
Change-Id: Ic53e6a165e759a401bd8e1e3037ce795a478c3b3
CRs-Fixed: 1066060
|
| | | | | | | | | | | | | | | |
| | | | | | | | | | | | | | | |
| | | | | | | | | | | | | | | |
| | | | | | | | | | | | | | | |
| | | | | | | | | | | | | | | |
| | | | | | | | | | | | | | | |
| | | | | | | | | | | | | | | |
| | | | | | | | | | | | | | | | |
In grant process, when require permission, will grant
all group permissions instead of single permission.
CRs-Fixed: 1066060
Change-Id: Iafe1d308a6ecb301da1c00450838a87344fc7f5f
|
|/ / / / / / / / / / / / / / /
| | | | | | | | | | | | | | |
| | | | | | | | | | | | | | |
| | | | | | | | | | | | | | |
| | | | | | | | | | | | | | |
| | | | | | | | | | | | | | |
| | | | | | | | | | | | | | | |
When revoke one permission in the settings, add a
warning dialog to users.
Change-Id: I157a4b0aab0e8d4f9719b4ff4d68f7eb11922ec3
|
| | | | | | | | | | | | | | |
| | | | | | | | | | | | | | |
| | | | | | | | | | | | | | |
| | | | | | | | | | | | | | |
| | | | | | | | | | | | | | |
| | | | | | | | | | | | | | |
| | | | | | | | | | | | | | |
| | | | | | | | | | | | | | |
| | | | | | | | | | | | | | | |
Package installer stopped when tap email permissions, because email
defines an additional permission, but package installer can not find
its group,so it throws null point exception, so show every single
permission only for platform permission group.
Change-Id: Ib56b4b4b6fef2ec112321787d34c35c211652a60
|
| | | | | | | | | | | | | | |
| | | | | | | | | | | | | | |
| | | | | | | | | | | | | | |
| | | | | | | | | | | | | | |
| | | | | | | | | | | | | | |
| | | | | | | | | | | | | | |
| | | | | | | | | | | | | | |
| | | | | | | | | | | | | | | |
UpdateEveryPermissionPreference func is only for CTA test.
Add persist.sys.strict_op_enable for CTA code, otherwise
will cause nullPointerException.
Change-Id: I3ad2019e245e8971e22657edaa76c7348d485e34
|
| |_|_|_|_|_|_|_|_|_|_|_|_|/
|/| | | | | | | | | | | | |
| | | | | | | | | | | | | |
| | | | | | | | | | | | | |
| | | | | | | | | | | | | |
| | | | | | | | | | | | | | |
Enable permission control for each permission.
Change-Id: Id045e432536233f0924cc7497a5dd528066da701
|
| |_|_|_|_|_|_|_|_|_|_|_|/
|/| | | | | | | | | | | |
| | | | | | | | | | | | |
| | | | | | | | | | | | |
| | | | | | | | | | | | | |
Fixes: 29260288
Change-Id: Iaa309f362ae6a296f7ba365e418d4a6f55b23f0c
|
|\ \ \ \ \ \ \ \ \ \ \ \ \
| |_|_|_|_|_|_|_|_|_|_|_|/
|/| | | | | | | | | | | | |
|
| | |_|_|_|_|_|_|_|_|_|/
| |/| | | | | | | | | |
| | | | | | | | | | | |
| | | | | | | | | | | |
| | | | | | | | | | | |
| | | | | | | | | | | |
| | | | | | | | | | | | |
... at least unless we want to show a progress dialog.
Fixes: 29094372
Change-Id: I45e0bbf32355a93635c773c526151b849bc1e8a3
|
|/ / / / / / / / / / /
| | | | | | | | | | |
| | | | | | | | | | |
| | | | | | | | | | |
| | | | | | | | | | |
| | | | | | | | | | |
| | | | | | | | | | |
| | | | | | | | | | |
| | | | | | | | | | |
| | | | | | | | | | |
| | | | | | | | | | |
| | | | | | | | | | | |
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
|
| |_|_|_|_|_|_|_|_|/
|/| | | | | | | | |
| | | | | | | | | |
| | | | | | | | | |
| | | | | | | | | |
| | | | | | | | | |
| | | | | | | | | |
| | | | | | | | | |
| | | | | | | | | | |
This change fixes a regression where we stopped showing
a confirmation UI for an app side-loading.
bug:28551937
Change-Id: Ia2114d273473d6e00696f9de8c523190a1b444ad
|
| |_|_|_|_|_|_|_|/
|/| | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | | |
Manual formatting is to complicated to do.
Fixes: 28852429
Change-Id: Ibab723b82a2b14ad94a3038dffccc96f86ba8ee4
|
| |_|_|_|_|_|_|/
|/| | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | | |
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
|
|\ \ \ \ \ \ \ \
| |_|_|_|_|_|_|/
|/| | | | | | | |
|