| Commit message (Collapse) | Author | Age | Files | Lines |
|
|
|
|
|
|
|
|
|
| |
For display purposes, we should always be using safe labels. This
prevents excessively long labels from preventing app uninstallation.
Bug: 62537081
Test: Manual. Install APK from bug and see that it can be uninstalled
Change-Id: Ic0e0d47f5ff76e4a73b30988d0a024fb3ee651ae
(cherry picked from commit 25e686d3cf555b99993e8a5c7edb2375171e5392)
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
In cases where an app had a long list of permissions the Install button
was not being enabled.
Test: Manually clicked on an apk that required a long list of
permissions. Checked that the install button is enabled. Switch out and
back to the activity to force a pause/resume. The button gets disabled
and enabled as expected. Also the existing test passes as before:
gts-tradefed run gts-dev -m PackageInstallerTapjacking
Bug: 35056974
Change-Id: I2ef7d5ce4e137969c70d2929271620c2bcb723f2
(cherry picked from commit 4c6afe94942c33e293d456be70ffa933d7d3e374)
|
|
|
|
|
|
| |
This reverts commit 4b0c302eae8d772d09346718559cc42548485f75.
Change-Id: Ic69486cc7d1f374a8aba1f4a566e1d96722975fc
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Replacing app opps fix with the flag that is used
elsewhere.
Bug: 37442941
Test: Verified that toast and system overlay still goes
away on permission and a11y service capability screens.
Merged-In: I27d357b2aa61193380b7ef1dcc88bbac9553644e
Change-Id: Ib7a7251109ed0f125db2bd49f5fc0baa67207b5d
(cherry picked from commit bb1948fe1a3fa2ca93f508af2e97f5ec21e71ade)
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
If mOkCanInstall is false when onResume() is called, the button gets
disabled and never re-enabled. This fix is for branches that have
change I42c15d05dffc1f6db69c9f67148f7ceb6e8bd8ae ("Split layout for
confirm step.") which causes the button to be disabled and enabled
for different reasons. Here the button is constantly enabled and its
action changes depending on the value of mOkCanInstall. The attribute
filterTouchesWhenObscured should prevent tapjacking when overlays are
present, so revert this commit.
This reverts commit cec631f26e2cbd7c51fd7c28e05d379d6d0f86db.
REGRESSION-1262
Change-Id: I819aab95506672d291e07193b71f074292862754
|
|
|
|
|
|
|
|
|
|
|
| |
The install button can be disabled for other reasons, hence we have to
store the enabled state independent of the resumed/paused state.
Test: gts-tradefed run gts-dev -m PackageInstallerTapjacking
Bug: 35056974
Change-Id: I2effa0f5afacfaed217a030550a778e32912cfbb
(cherry picked from commit 172eddd2ff07d9849496de586b3a785d8abc4441)
|
|\
| |
| |
| |
| |
| | |
am: f44d80b62c
Change-Id: I11cdcc5060319b4f613be187a742a0a79f57f080
|
| |
| |
| |
| |
| |
| |
| |
| | |
Otherwise, there will be a screen with background color splashes if the
user has any permission revoked in a group as user fixed.
Bug: 34722442
Change-Id: I38a3793a366c8132f5b7c5294c478b4876cd4b9e
|
|\|
| |
| |
| |
| |
| |
| |
| | |
is changed.
am: 275897a5a5
Change-Id: Id4ddf57e660e0ea339f8bafa1aa02dcce0c0c894
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
is changed.
This only applies to individually controlled permission group.
Test: Manually test on a local build.
Bug: 34561693
Change-Id: I24f83a56d4011253ed2aa78dd0613021dfe56136
|
|\|
| |
| |
| |
| |
| | |
am: b8e4193e57
Change-Id: I06ddf41de29c70df865434685ee7cb652cfddbbd
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
Meanwhile, aligned with handheld logic that revoking individually
controlled permission is marked as user fixed.
The original commit id I1d6cbc478c789e7680f41b92b9f3fa2504576ecd.
Test: Manual test on a local build.
Bug: 34440353
Change-Id: Ie3bb75225246eefef954926f94295181f4daebd0
|
| |\
| | |
| | |
| | | |
App." into cw-f-dev
|
|\ \ \
| | |/
| |/|
| | |
| | |
| | | |
am: f7d0ef9f74
Change-Id: I7cc4dcd66edcd68366f40fd67e812d76fb2dcb3b
|
| | |
| | |
| | |
| | |
| | |
| | | |
Test: Manually tested on a permission test apk.
Bug: 34157400
Change-Id: Ia2ce3ac6c567f869535ec92a896c6d0fd9ec6318
|
|\ \ \
| | |/
| |/|
| | |
| | |
| | | |
am: 8bb2553798
Change-Id: Ibd2704c76e63a57c9f4aee6085e909cf984df68f
|
| |/
| |
| |
| |
| |
| | |
Test: Manually test with a testing apk.
Bug: 34157556
Change-Id: I78ffbfa2fbbff14fb05242d7979cd226b1725ba7
|
|\|
| |
| |
| |
| |
| | |
am: 1019d73896
Change-Id: I1e9d82744c4294786ff7cce16ef2eca945c5de9c
|
| |
| |
| |
| |
| |
| |
| |
| |
| | |
Test: Manually tested on sawshark_sw where WRITE_CONTACTS and
READ_CONTACTS are shown separately for Android Wear App. Also
tested on sawshark where only one "Contacts" group shown on
behalf of WRITE_CONTACTS and READ_CONTACTS.
Bug: 33856934
Change-Id: I299c54e38236d039204a01d8113f427b1fc71c43
|
| |
| |
| |
| |
| |
| |
| |
| | |
am: e931327ee0
Fixes: 33578153
Change-Id: I9b7f116b8c05d5e3da71d31266bf94ac38ef987d
(cherry picked from commit b2ebcceb8d04dcf85e7d49af1630de60af2bf6f1)
|
|\|
| |
| |
| |
| |
| | |
am: 528811c980
Change-Id: If422232abd3b6b5059b32c2d1297ccd751f1167e
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
If permission review is enabled toggling bluetoth on or off
results in a user prompt to collect consent. This applies
only to legacy apps, i.e. ones that don't support runtime
permissions as they target SDK 22.
Also added a configuration resource which controls whether
permission review mode is enabled. By default it is not and
an OEM can change this via an overlay. For now we also keep
the old mechanism to toggle review mode via a build property
which is still used and will be removed when clients have
transitioned.
bug:28715749
Change-Id: I3783fa99f3fca881a4b60ea1e7c0f68098302ae4
|
|\ \
| |/
|/| |
|
| |
| |
| |
| |
| |
| |
| |
| | |
b/32703627
Test: Navigate to Apps > App permissions > Location > Show system apps
and toggle Google Play Services on or off
Change-Id: Iff80776811a222f835863a958a19c57b613e0168
|
|\ \
| |/
|/|
| |
| |
| |
| |
| | |
6dfb1f6553 am: 18817ebb89
am: d49c29ae9a
Change-Id: I6507b6707dccaaa2c8cad30d9b1b044ca426c02c
|
| |\
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
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)
|
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
BUG=32427299
Change-Id: I04c229919c24c09131244fc8acb2c64d8902bc80
|
|\| | |
| | | |
| | | |
| | | |
| | | |
| | | | |
am: df52c788da
Change-Id: I2444b6612f988b5ac8ff6de351ef53f3b9516255
|
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
Test: Simulated OK result from settings and triggered run-once
workflow
Fixes: 32105068
Change-Id: I9f6c3b56b09d520d2bc9c6eeb6c75f151e3f1aff
|
| | | |
| | | |
| | | |
| | | |
| | | | |
Bug: 31779188
Change-Id: I655d8a38a478e0fe863b6e52611f9f43d77240bf
|
| | | |
| | | |
| | | |
| | | |
| | | | |
Bug: 31779188
Change-Id: Ifb42b6505fcd8a1928d31c6a20620b24450e420f
|
| | | |
| | | |
| | | |
| | | |
| | | | |
Bug: 31779188
Change-Id: I41aecafefaeaaa15f70f43334df3ff367b3a312c
|
|\ \ \ \ |
|
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
Bug: 29250733
Bug: 31779188
Change-Id: I814db5f94bdb840b04c5e8d9e12b565cfcd5fbc1
|
|\ \ \ \ \
| |/ / / /
|/| / / /
| |/ / /
| | | |
| | | | |
am: b4090d16e4
Change-Id: I605d0e80afe0057dc10fa022b68f04e6d6cc2460
|
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
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)
|
|\| | |
| | | |
| | | |
| | | |
| | | |
| | | | |
am: 435a65b477
Change-Id: Ib82f212f4ae8a8c3a5258c072ffb9ed13d28ff62
|
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
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
|
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
Handles IllegalArgumentException from the deletePackage call.
It's unclear right now what the root cause of this crash is,
but we will follow up with that in a future release.
BUG: 31425541
Change-Id: I24eba7c06f090dd6dcd2f75268cd6be1d1f2f500
|
|\| | |
| | | |
| | | |
| | | |
| | | |
| | | | |
am: 9137038766
Change-Id: I728b7ff5bc46436dfced714a8ff5dd422e2e70a9
|
| | | |
| | | |
| | | |
| | | |
| | | | |
Bug: 30056625
Change-Id: I1ffc7ba88ea5f99591b66c08e72ea07d60ab82d5
|
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
Bug: 31301246
Change-Id: I2d93ee0e8913b8bd696730bb9d24535c6f9ac817
|
|\| | |
| | | |
| | | |
| | | |
| | | |
| | | | |
am: d7d3b73bd6
Change-Id: I4a72e89f7a05c72ab7e47c01fc76ff5099e34a94
|
| | | |
| | | |
| | | |
| | | |
| | | | |
Bug: 30056625
Change-Id: Ie8e46fc7a2c340fb04b64feb331af0d882fe9672
|
|\| | |
| | | |
| | | |
| | | |
| | | |
| | | | |
am: 5fe91f8d06
Change-Id: I49fa46b1b8a43d74a1fa29f551f173da12c37960
|
| | | |
| | | |
| | | |
| | | |
| | | | |
Bug: 30056625
Change-Id: Ie8e46fc7a2c340fb04b64feb331af0d882fe9672
|
| |\ \ \ |
|