| Commit message (Collapse) | Author | Age | Files | Lines |
|\
| |
| |
| |
| |
| |
| |
| | |
6dfb1f6553
am: 18817ebb89
Change-Id: I5f2254900247fda33ca2b48b38428f01ef11b914
|
| |\
| | |
| | |
| | |
| | |
| | | |
am: 6dfb1f6553
Change-Id: I78ed7c841680ed3d82835bca2394384311e6d8b1
|
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
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)
|
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
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)
|
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
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
|
| |
| |
| |
| |
| | |
Fixes: 29260288
Change-Id: Iaa309f362ae6a296f7ba365e418d4a6f55b23f0c
|
|\|
| |
| |
| |
| |
| | |
am: 9c177f8d28
Change-Id: Ieac52df39c5481ea8c964be61d3a5a5c9bc024e3
|
| |\ |
|
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
... at least unless we want to show a progress dialog.
Fixes: 29094372
Change-Id: I45e0bbf32355a93635c773c526151b849bc1e8a3
|
|\| |
| | |
| | |
| | |
| | |
| | | |
am: 869786e3e4
Change-Id: Ic1d8d6221194731a6c967f736a1352a905bba8b0
|
| |/
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
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
|
|\|
| |
| |
| |
| |
| |
| |
| |
| | |
am: 6622140841
* commit '662214084178a687a32823decd4f42ba82313def':
Show installer UI for user to confirm side-loading
Change-Id: I017268c1d12c5fdfaea099e9428c3def1b728bdd
|
| |
| |
| |
| |
| |
| |
| |
| |
| | |
This change fixes a regression where we stopped showing
a confirmation UI for an app side-loading.
bug:28551937
Change-Id: Ia2114d273473d6e00696f9de8c523190a1b444ad
|
|\|
| |
| |
| |
| |
| |
| |
| |
| | |
am: e071d11592
* commit 'e071d1159277a00e0484dbf43899bc90ea27176c':
Format the template using HTML tags.
Change-Id: I1858bc3c1e6d6fb5f4b37a2ef4a287de635045ae
|
| |
| |
| |
| |
| |
| |
| | |
Manual formatting is to complicated to do.
Fixes: 28852429
Change-Id: Ibab723b82a2b14ad94a3038dffccc96f86ba8ee4
|
|\|
| |
| |
| |
| |
| |
| |
| |
| | |
am: 6fdd93b339
* commit '6fdd93b33927675a6a062f0b8fd00ef3866ac2f7':
Finish package installer activity on a cancel result
Change-Id: If34ab52b7c25cdbf69a06db5b4cf7e87b62e12bd
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
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
|
|\|
| |
| |
| |
| |
| |
| |
| |
| | |
am: 622c5b563e
* commit '622c5b563e1f2e7a78e64129f350387613161153':
Add missing else statement
Change-Id: I0e6e13bf0bbd368375bd924c496d1b8e011cbf19
|
| |\ |
|
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
This change fixes a regression where we endup installing apps
even if unknown sources is disabled - missing else statement.
bug:28551937
Change-Id: I05dd28a28eb9ffd9eda8a6ee2fdccd501d2e08d1
|
|\| |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
nyc-dev
am: 1be5a77aaa
* commit '1be5a77aaa3bfba81063cbba38422dfe30c4ce40':
Take advantage of new MotionEvent flag to prevent tapjacking.
Change-Id: I416339e7bb12371480432699499871cf3c43bcab
|
| |\ \
| | | |
| | | |
| | | | |
nyc-dev
|
| | | |
| | | |
| | | |
| | | |
| | | | |
Bug: 26677796
Change-Id: I563541f0a42564b854af0f8037c1d4741c79a2ac
|
|\| | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
am: 55dbb25ae3
* commit '55dbb25ae341353ef10312d89a248d4e658be9ee':
Ensure app label properly bolded
Change-Id: If437d0944f8dc45107d67252ccf1f17e8c748ca3
|
| |\ \ \ |
|
| | | |/
| | |/|
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
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
|
|\| | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
am: 988f3bcb21
* commit '988f3bcb21b7026cfab89d22ef912f98c9c9f8a2':
Use badged icons for permissions.
Change-Id: I922e3df467d31be00c80128c91330b381853ce70
|
| |\ \ \ |
|
| | |/ /
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
Please note that the labels stay unbadged. I.e. The label will always be
"Contacts" instead of "Work Contacts".
Fixes: 28403737
Change-Id: Ib994db946b07098a8c0f95eab5787fb30532fe62
|
|\| | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
am: f682d8970c
* commit 'f682d8970cc10e9943232cde1888d214637e610e':
Finish whole installation on cancel dialog.
Change-Id: I785f39748871178c428330c70428d2a8b4979f78
|
| |\ \ \ |
|