Commit Graph

4314 Commits

Author SHA1 Message Date
Kevin Chyn
0d1b06e3d5 Revert^2 "Try to ensure tests run sequentially"
c0dd229fc6

Change-Id: I1d9366f0cbe4fb41cedfed987162ec15e7845530
2021-02-19 13:45:45 -08:00
Kevin Chyn
bd18a4681d Fix NPE during fingerprint removal
1) Enroll fingerprint
2) Remove lockscreen credential
3) All fingerprints removed successfully

Bug: 180689674
Test: See steps above
Change-Id: I8ae947114ec1cc90738152ec33c13cc62c302559
2021-02-19 13:45:44 -08:00
Kevin Chyn
e0552feba5 Revert^2 "Update RemovalClient to support new biometric AIDL"
62e2c705c5

Change-Id: If7af8a38f114804fc48feba9744bd9ba936c865e
2021-02-19 13:19:54 -08:00
TreeHugger Robot
4d4817a9b3 Merge changes from topic "revert-13593269-QAPPBSWOVZ" into sc-dev
* changes:
  Revert "Update RemovalClient to support new biometric AIDL"
  Revert "Try to ensure tests run sequentially"
2021-02-19 21:07:47 +00:00
Anthony Stange
e9681f3bbb Merge "Add owners for files related to the sensors framework" am: 5927064796 am: 16d6248bc6 am: a04505c12e
Original change: https://android-review.googlesource.com/c/platform/frameworks/base/+/1587231

MUST ONLY BE SUBMITTED BY AUTOMERGER

Change-Id: I3220b76d3417cee41a6a63ffc86082a8638142b6
2021-02-19 19:19:47 +00:00
Daniel Chapin
62e2c705c5 Revert "Update RemovalClient to support new biometric AIDL"
Revert submission 13593269

Reason for revert: Bug: 180689674
Reverted Changes:
I59f6e607f:Update RemovalClient to support new biometric AIDL...
I3fdb3fe0b:Try to ensure tests run sequentially

Change-Id: I939172f2961f3f2c9f8eddd59261f75552eeb31f
2021-02-19 19:10:04 +00:00
Daniel Chapin
c0dd229fc6 Revert "Try to ensure tests run sequentially"
Revert submission 13593269

Reason for revert: Bug: 180689674
Reverted Changes:
I59f6e607f:Update RemovalClient to support new biometric AIDL...
I3fdb3fe0b:Try to ensure tests run sequentially

Change-Id: Ia9c02759db7238536e21dd05f8e247a91e9ac5aa
2021-02-19 19:10:04 +00:00
Anthony Stange
a04505c12e Merge "Add owners for files related to the sensors framework" am: 5927064796 am: 16d6248bc6
Original change: https://android-review.googlesource.com/c/platform/frameworks/base/+/1587231

MUST ONLY BE SUBMITTED BY AUTOMERGER

Change-Id: I24ab3a9bc3556fbdfe97287508dc4f539fa071f4
2021-02-19 18:58:31 +00:00
Anthony Stange
5927064796 Merge "Add owners for files related to the sensors framework" 2021-02-19 17:23:45 +00:00
Yin-Chia Yeh
73ef4bcca6 Merge "Camera: add default secure image size key" into sc-dev 2021-02-18 00:15:09 +00:00
Kevin Chyn
fe62a99c63 Merge changes I3fdb3fe0,I59f6e607 into sc-dev
* changes:
  Try to ensure tests run sequentially
  Update RemovalClient to support new biometric AIDL
2021-02-17 23:36:03 +00:00
Kevin Chyn
e5ffb5805e Merge "Retrieve authenticatorId when already targetUserId" into sc-dev 2021-02-17 22:17:07 +00:00
Kevin Chyn
e43e5b1337 Retrieve authenticatorId when already targetUserId
Previous change ag/13413165 stopped the framework from retrieving
authenticatorId when the current user is already the target user.
However, this happens immediately after enrollment, since enroll
and getAuthenticatorId are separate operations. So, add a way to
retrieve authenticatorId only when necessary.

Fixes: 179737483
Bug: 178018968
Test: manual
Change-Id: I71673fc672c60733616634a2fdf3153a3b2d02e1
2021-02-17 12:29:11 -08:00
TreeHugger Robot
d271b73c5e Merge "Enforce nanoapp permissions for ContextHub APIs" into sc-dev 2021-02-17 19:13:13 +00:00
Anthony Stange
ab42e66b4c Enforce nanoapp permissions for ContextHub APIs
Adds the following behavior:
1) Ensures nanoapp messages aren't delivered to host apps unless they
hold the required permissions
2) Ensures that host apps continue to hold the required permissions set

Also pipes the package name for callback clients since their package
name can't be determined on the other side of the binder.

Bug: 166846988
Test: Disable permissions on host client and verify that nanoapp and
host lose communication after ~1 minute
Test: Re-enable permissions on host client and verify host receives
appropriate callback

Change-Id: I14af504f0b230f6ce15852f16fc8b174fdd52252
2021-02-17 16:53:37 +00:00
Anh Pham
857efd19a4 Merge changes from topic "sensor_sampling_rate_throttling-sc-dev" into sc-dev
* changes:
  Throw SecurityException regarding sampling rates.
  Add support for retrieving the Debuggable flag.
  Add HIGH_SAMPLING_RATE_SENSORS permission.
2021-02-17 07:44:54 +00:00
Kevin Chyn
d6d23e13a5 Try to ensure tests run sequentially
Asynchronous operations that are performed in close() (such as
cleanupInternalState()) should complete before close() finishes.
Otherwise, we risk subsequent tests starting and causing racey
conditions while cleanup has not finished yet.

Fixes: 180192766
Test: atest CtsBiometricsTestCases
Test: atest com.android.server.biometrics
Change-Id: I3fdb3fe0be59e826d0c14e14a2590ccdde568774
2021-02-16 21:34:34 -08:00
Kevin Chyn
b2888343b7 Update RemovalClient to support new biometric AIDL
The new AIDL interfaces accept a list of IDs to remove. Note that
the new interfaces explicitly do NOT support using "remove(0)"
to remove all enrollments.

Fixes: 179338634
Test: atest CtsBiometricsTestCases
Test: atest com.android.server.biometrics
Change-Id: I59f6e607ff876541f049ac70d06046815eba7632
2021-02-16 21:34:34 -08:00
Alex Johnston
202d4ab178 Merge "Support USB V1.3 HAL" into sc-dev 2021-02-16 20:45:55 +00:00
Anthony Stange
30780cf472 Merge "Add new ContextHubTransaction error code" into sc-dev 2021-02-16 20:41:55 +00:00
Yin-Chia Yeh
a22e0e6d74 Camera: add default secure image size key
Test: new CTS tests pass
Bug: 153663201
Change-Id: I22c9c37e7d718cad5f858f91c49a46255bf57cb8
2021-02-16 12:32:34 -08:00
TreeHugger Robot
1cc73a9adf Merge "ImageWriter: Convert Blob native format to respective public value" into sc-dev 2021-02-16 19:27:13 +00:00
Anthony Stange
2e6ebbc453 Add new ContextHubTransaction error code
To support synchronous sendMessageToNanoapp failures resulting from
invalid permissions, add a new error code that indicates the transaction
failed due to invalid permissions.

Also adds a recommendation for how apps should behave when interacting
with nanoapps.

Bug: 166846988
Test: compile
Change-Id: I189b387fbcd5c4425a12391e4413519f6befae5e
2021-02-16 17:34:59 +00:00
Albert Wang
619abd21d9 Support USB V1.3 HAL
USB V1.3 HAL include:
1. Add api to enable/disable USB data signaling
2. Add api to inquiry HAL version

Bug: 161414036
Test: build pass and functions are working normally
Signed-off-by: Albert Wang <albertccwang@google.com>
Change-Id: Idb946de553b63f1d9da565133306117a6a4a7dd8
Merged-In: Idb946de553b63f1d9da565133306117a6a4a7dd8
(cherry picked from commit bd9dfea480)
2021-02-16 15:25:29 +00:00
Anh Pham
97f66fd20c Throw SecurityException regarding sampling rates.
When apps target SDK >= S, run in debug mode, request a sampling rate greater than 200 Hz and do not have the HIGH_SAMPLING_RATE_SENSORS permission, the SystemSensorManager throws a SecurityException  when the apps try to establish a (direct or event) connection.

Test: atest CtsSensorTestCases CtsSensorRatePermissionTestCases
Bug: 136069189
Change-Id: I33893c93949f0f055ce5ded1a5ed9d188de709ed
2021-02-16 09:25:21 +01:00
Treehugger Robot
440006cb4d Merge "Add owners for contexthub framework code" am: b6ce95e889 am: c9ba08c6c4 am: 42a1125261
Original change: https://android-review.googlesource.com/c/platform/frameworks/base/+/1587271

MUST ONLY BE SUBMITTED BY AUTOMERGER

Change-Id: I2b3f28ec38ac5f712556c3495ac80b7cc9e94eec
2021-02-16 02:24:56 +00:00
Treehugger Robot
42a1125261 Merge "Add owners for contexthub framework code" am: b6ce95e889 am: c9ba08c6c4
Original change: https://android-review.googlesource.com/c/platform/frameworks/base/+/1587271

MUST ONLY BE SUBMITTED BY AUTOMERGER

Change-Id: Ib51770039f161702612c5116752dc776bdae034c
2021-02-15 22:47:38 +00:00
Curtis Belmonte
53928f8bed Merge "Split acquire messages for face auth and enrollment" into sc-dev 2021-02-12 21:59:06 +00:00
TreeHugger Robot
a5a543f82c Merge "Camera: Switch the reader callback when the sequence ends" into sc-dev 2021-02-12 18:28:17 +00:00
Anthony Stange
8b97ec7957 Add owners for contexthub framework code
Bug: 166846988
Change-Id: Id2b82d4df8083d4b70be2e27d31e32ead7ce7c75
Test: presubmits
2021-02-12 17:43:07 +00:00
Anthony Stange
101500307f Add owners for files related to the sensors framework
Bug: 136069189
Change-Id: I61b5e77c5bf07b22be725714f743f8ef84a96482
Test: presubmits
2021-02-12 17:27:21 +00:00
Emilian Peev
725bd108a7 Camera: Switch the reader callback when the sequence ends
Depending on timing the first captured buffer from an internal
repeating request can be incorrectly propagated to the client
preview surface. This can happen, if the first 'onCaptureStarted'
callback gets delayed and executes after the first buffer from the
internal request gets queued in the camera preview output surface.
Proactively switch the camera preview output image reader callback
to loopback after the current capture sequence ends.

Bug: 179450326
Test: Camera CTS
Change-Id: I8232ea5171a0205da5705b6af5fdd3cdab297335
2021-02-11 15:50:40 -08:00
Philip P. Moltmann
8d6a0cb6af Merge "Remove me from OWNERS files" am: 25ee5a7fb7 am: bf8de269cb am: 39198ddd84
Original change: https://android-review.googlesource.com/c/platform/frameworks/base/+/1573324

MUST ONLY BE SUBMITTED BY AUTOMERGER

Change-Id: I429557a8537fb6417fcb9539978964ade75c8dee
2021-02-11 23:15:42 +00:00
Philip P. Moltmann
39198ddd84 Merge "Remove me from OWNERS files" am: 25ee5a7fb7 am: bf8de269cb
Original change: https://android-review.googlesource.com/c/platform/frameworks/base/+/1573324

MUST ONLY BE SUBMITTED BY AUTOMERGER

Change-Id: Ibe2c5fd92070b1aebdc46755c8667b3ed22348ab
2021-02-11 22:57:48 +00:00
Philip P. Moltmann
a8eb3d3e69 Remove me from OWNERS files
Test: n/a
Change-Id: I6113011e6ab0997285d5b7a86303fc60a3b98a2b
Exempt-From-Owner-Approval: Previously approved
2021-02-11 22:28:43 +00:00
Darryl Johnson
f131f0bfd2 Merge "Add MAX/MIN_DEVICE_STATE constants to restrict possible states to range." into sc-dev 2021-02-11 21:32:59 +00:00
Curtis Belmonte
4f54f79586 Split acquire messages for face auth and enrollment
Updates FaceManager to distinguish between face enrollment and
authentication when producing help messages, returning more detailed
messages for the former and less noisy ones for the latter.

Test: Manually tested face enrollment and authentication.

Bug: 179044580

Change-Id: I406b371804de94383e65a077380b26bfb9ecf5e7
2021-02-11 13:08:19 -08:00
TreeHugger Robot
ba4fa60b35 Merge "Introduce SystemVibratorManager" into sc-dev 2021-02-11 13:12:18 +00:00
Lais Andrade
55a0aebe7f Introduce SystemVibratorManager
Add default system implementation of vibrator manager service that uses
IVibratorManagerService to access the device vibrators.

Change implementation of SystemVibrator to use VibratorManagerService
and delete local service VibratorService.

Introduce missing public APIs for VibratorManager and Vibrator.getId.

Bug: 167946816
Test: VibratorManagerServiceTest, VibratorTest, VibrationEffectTest
Change-Id: I55cdeb72c7ca39ccf0c7b2fda60b16de1031801e
2021-02-11 09:26:21 +00:00
Darryl L Johnson
836dc0dae8 Add MAX/MIN_DEVICE_STATE constants to restrict possible states to range.
This adds a MAXIMUM_DEVICE_STATE and MINIMUM_DEVICE_STATE constant to
DeviceStateManager that contains the max allowed device state indentifier
which reserves any larger values for future defined constants in the
platform. The constant will be used by CTS to validate that the set
of supported states is in the range
[MINIMUM_DEVICE_STATE, MAXIMUM_DEVICE_STATE].

The values match the max allowed value returned by the default provider
impl which is defined in the XSD schema.

Bug: 159401801
Bug: 177235528
Test: atest DeviceStateTest
Change-Id: Ia5a8e2841bac19d36e9d6cd00950d09628a7f719
2021-02-10 22:29:06 -08:00
Kevin Chyn
0ee5f13d6b Merge "Add logging to biometric test path" into sc-dev 2021-02-11 01:21:09 +00:00
Kevin Chyn
630229f0dc Add logging to biometric test path
Bug: 179856730
Test: presubmit
Change-Id: I8178ff5ba18106f6e00102a85f9f41fbf9928074
2021-02-10 11:30:58 -08:00
Curtis Belmonte
c229696530 Merge "Pass face AIDL frame data through to FaceManager" into sc-dev 2021-02-09 23:28:25 +00:00
Fiona Campbell
50997aeb8c Merge "Enable per display brightness setting" into sc-dev 2021-02-09 07:24:17 +00:00
Curtis Belmonte
18ccd6c925 Pass face AIDL frame data through to FaceManager
Adds plumbing to send acquired frame info from the onAuthenticationFrame
and onEnrollmentFrame AIDL callbacks to FaceManager. Currently,
FaceManager drops most of this data and only passes the acquireInfo and
vendorCode fields to appropriate authentication/enrollment callbacks.

Test: atest CtsBiometricsTestCases

Bug: 178414967
Change-Id: If997ddd13ce1d9bda2341df7c1f8d6347571f4e0
2021-02-08 15:14:39 -08:00
TreeHugger Robot
2fa12f8ec3 Merge "Add vibrator state listener support for input device vibrator" into sc-dev 2021-02-08 19:05:34 +00:00
Kevin Chyn
aae04715d1 Add fingerprint enroll reason
1) Existing metrics logging-ness can be derived from it
2) EnrollReason is clearer and can be used by other parts of the
   system, such as SysUI
3) Do not show progressbar in SysUI if it's "find sensor"

Bug: 179447737
Test: Builds
Change-Id: I84f82b5c81a668e11c1a380e6fa4e3080db53dc9
2021-02-06 15:51:43 -08:00
Kevin Chyn
b92e7e8af6 Merge "Make FingerprintManager show BiometricPrompt for UDFPS" into sc-dev 2021-02-06 23:39:55 +00:00
Ilya Matyukhin
26e3ecaaed Merge changes from topic "fix-isUdfps" into sc-dev
* changes:
  Provide default UDFPS location for fake UDFPS
  Remove retry in Fingerprint21 getService
2021-02-06 22:20:04 +00:00
Curtis Belmonte
45807b0b4a Make FingerprintManager show BiometricPrompt for UDFPS
For apps that are still using FingerprintManager, have the system show
BiometricPrompt when authenticating with a UDFPS sensor. This will
ensure that the user is directed to authenticate on the correct part of
the screen, rather than requiring the app to draw an authentication UI.

Test: FingerprintManager test app, Keyguard, and Settings.

Bug: 174490952
Change-Id: I60ba2dce983812996f6baa44557533575051e476
Merged-In: I60ba2dce983812996f6baa44557533575051e476
2021-02-06 20:16:15 +00:00