WorkSource: Fix NPE on set(int) / set(int, String).

mChains might be null.

Bug: 62390666
Test: WorkSourceTest
Test: run cts-dev -m CtsOsTestCases -t android.os.cts.WorkSourceTest
Change-Id: I1dbc2abc576c280ad41ad6214745b0d34c5727a0

Merge "Add tests for KeySyncUtils now that SecureBox is available"

Merge "Fix over-granting permission on low ram"

Merge "Remove the hacky HashableDimensionKey."

Merge "Add log source filtering in statsd to filter out spams."

Merge "Notify RecoverableKeyStoreLoader about successful unlock."

Merge "MacAddress: address api review comments"

Merge changes Ie1eb63dc,Ib7cc4d52

* changes:
  Renames setCallbackOnComplete method in ContextHubTransaction
  Rename ContextHubTransaction.Result IntDef

Merge "New Autofill API:AutofillManager.getAutofillServiceComponentName()"

Merge "Remove scale effect from screen off animation"

Merge "Update BatteryStatsImpl to track per-procstate cpu times."

Notify RecoverableKeyStoreLoader about successful unlock.

Use simple pattern matching to distinguish between between pin and
passwords.

Test: none
Bug: 66499222
Change-Id: I5cf7c37b22b2c90a9d6a61bd40de45d69cb1364d

Remove the hacky HashableDimensionKey.

+ Add a real HashableDimensionKey as a wrapper of the dimension.
So we can get rid of the maps that we kept.

Pay down technical debt and reduce memory usage.

Test: statsd_test & manual
Change-Id: I233280cf1e2ce93da6a8cd4e8514abb066f4016d

Add log source filtering in statsd to filter out spams.

+ Add log source whitelist in StatsdConfig
+ Some changes in UidMap API. Listener needs to be wp instead of sp.
+ Update dogfood app config to have log source
+ Increase the stats service thread pool size to 10 (9+1).

TODO: add unit tests(b/70805664). This unit test takes some time to write.

Test: statsd_test & manual

Change-Id: I129b1cc13db5114db7417580962bd7cc4438519d

Merge "Add more methods to RecoverableKeyStoreLoader."

Merge "Support new Autofill session after service returns a null FillResponse."

Merge "Settings: Add LOCAL_MAC_ADDRESS permission"

Merge "Limit collapsed SB notification icons to 4"

Merge "Add in-memory storage for listeners  used by RecoverableKeyStoreLoader"

Merge "Implement startRecoverySession"

Merge changes from topic "work_mode_api1"

* changes:
  Add access control to trySetWorkModeEnabled and make it public
  Merge setQuietModeEnabled and trySetQuietModeDisabled into one API

MacAddress: address api review comments

1) toSafeString() is renamed to toOuiString()
2) toOuiString() returns a String that only contains the first 3 bytes
   of the mac address.

Bug: 70336442
Test: runtest frameworks-net
Change-Id: I798d8fec32dbff5687a682028e5d90eb941f81c1
Merged-In: I798d8fec32dbff5687a682028e5d90eb941f81c1

(cherry pick from commit a0ecf38d302ef67aa9f767b1072369f8c417952d)

Merge "Allow hover tooltips for disabled views."

Renames setCallbackOnComplete method in ContextHubTransaction

setOnCompleteCallback seems easier to understand.

Bug: 67734082
Test: None
Change-Id: Ie1eb63dc5b90a35e847bd5d2a350f97ca277342d

Merge "Always show window divider to accessibility"

Merge "Sort "Recent Location Requests" by recency."

Merge "Fix broken MenuLayoutPortraitTest"

Settings: Add LOCAL_MAC_ADDRESS permission

This permission is needed for displaying the WIFI MAC address.

Bug: 70638295
Test: Wifi network details page shows the wifi MAC address.
Change-Id: Iaa595d520abf36ba55cfa9e888b169baae6ea1fb

Merge "Add strings for Wifi Wake onboarding notification."

New Autofill API:AutofillManager.getAutofillServiceComponentName()

Also minor fixes on isFieldClassificationEnabled() to make sure it's ignored
when not called by the service app and its calls are properly synchronized.

Test: atest CtsAutoFillServiceTestCases:LoginActivityTest#testGetAutofillServiceComponentName

Fixes: 70678279

Change-Id: I58ccb313d22c30af907f3da62f727f067240fa66

Sort "Recent Location Requests" by recency.

The list of apps with recent location requests is currently displayed in
alphabetical order on device (Settings -> Security & Location ->
Location). Sort this list by recency: (1) apps that are currently
requesting location are displayed  on the top. (2) For apps that are not currently
requesting location, the ones with most recent location request finish
time come first.

Bug: 70350519
Test: Robolectric
Test: manual
Change-Id: I5c757defcd7645d254c9c47e3c83f7e323247a71

Fix over-granting permission on low ram

Bug: 69258381
Bug: 69587247
Test: cts-tradefed run gts-dev -m GtsPermissionTestCases
Change-Id: Id51723dfaa711c18962a9ba1caa9cfc35e6cc53d

Implement startRecoverySession

Test: adb shell am instrument -w -e package com.android.server.locksettings.recoverablekeystore com.android.frameworks.servicestests/android.support.test.runner.AndroidJUnitRunner
Change-Id: I25e99f6014ef5e831420367040de7e1a80f134f0

Merge "Add subtype for priority"

Merge "Improve nine patch scaling"

Merge "Use RecoverableKeyStoreDb in RecoverableKeyGenerator"

Merge "Unhide HINT_HIDDEN from slices api and rename to HINT_SHORTCUT"

Merge "Back PlatformKeyManager with database"

Merge "Communicate relevantEvents=0 for packages excluded from whitelist"

Remove scale effect from screen off animation

Not resizing anymore per UX guidelines.

Test: visual, press power button.
Change-Id: Ic11134cd89d35b5421d269e52ab7715318d0e180
Fixes: 36899914
Bug: 64155983

Merge "Improve Fragment+Loader deprecation Javadoc"

Add more methods to RecoverableKeyStoreLoader.

1) Methods to get key status.
2) Register pending intent to get notification about new recovery
snapshots.

Test: none
Bug: 66499222
Change-Id: I4d5f8c1a6581b5e08f4589e19961d93c499689e1

Merge "Don't include mockito in frameworks-base-testutils" am: f924642729 am: cf7aa7c09e
am: 30d2c14006  -s ours

Change-Id: Id53b8f79a485f48d942d592d4ab6192f8545e356

Merge "Don't include mockito in frameworks-base-testutils" am: f924642729
am: cf7aa7c09e

Change-Id: I6b2486c69af1a05f8282a0d648403522adf2b471

Improve nine patch scaling

Apply separate coefficient for X and Y, when scaling nine patch
lattice dividers. This is reducing rounding error and fixing
a nine patch issue in Clock app alarm background.
There are other issues in nine patch scaling (see ag/3378768).

Test: Ran clock app
Bug: 70353853
Change-Id: Ibbbfddc47767fb3314cdb88820e520b3f472e727

Merge "Don't include mockito in frameworks-base-testutils"
am: f924642729

Change-Id: I4470f63baac7bc33dff0dac94da3a0d93c322dc7

Merge "BluetoothHidDevice: rename onIntrData to onInterruptData (1/5)"

Merge "Make accessibility anchor id a long"

Merge "[ipsec-doze] Add fchown to IpSecService to support doze" am: 4edb4c8ec4 am: 5949912523
am: 53a0c20517

Change-Id: Id6a5ff71ead4566b9a91b9e9b6c78d81f1ac0f54

Merge "Don't include mockito in frameworks-base-testutils"

Merge "Don't include mockito in frameworks-base-testutils"

Merge "Fix wrong wallpaper AoD state"

Add subtype for priority

Intended to use with SliceItem type int to note the priority of an item
(such as an action) within a template.

Test: none
Bug: 68378561
Change-Id: I109adf4ea77638b7d27f3aaa7f8eb1c325287f0c

Merge "[ipsec-doze] Add fchown to IpSecService to support doze" am: 4edb4c8ec4
am: 5949912523

Change-Id: I9036fc2768dd9664ab7407b359f3c689c7ce700e

Communicate relevantEvents=0 for packages excluded from whitelist

Together with checking isObservableEventType this will result in a11y events
not being generated for packages that are excluded form a11y-service(s)
package whitelist

Test: cts-tradefed run singleCommand cts -d --module CtsAccessibilityServiceTestCases
Change-Id: Id65607aaccc7af7d870d009d609917ff3c6d0712

Merge "[ipsec-doze] Add fchown to IpSecService to support doze"
am: 4edb4c8ec4

Change-Id: I75a3cb9e9d9d7d3b6c0ede86074a00ddcd86028e

Merge "[ipsec-doze] Add fchown to IpSecService to support doze"

Improve Fragment+Loader deprecation Javadoc

Make it clear that the framework versions of
Fragments and Loaders should be replaced with their
identically named versions in the Support Library.

Reference the recent work on Lifecycle as an example
of a Support Library only change that should motivate
developers to switch to the Support Library
equivalents.

Test: Reviewed new Javadoc
BUG: 70336450
Change-Id: I93151fa0a316bba824c2e55f80f7de9d30bed3c6

Merge "Cherrypick from from oc-mr1-dev doc: Fixing Javadoc formatting. Original-Change-Id: I5ee04bb96234441c5740665540b5e803fcf9710f"

Merge changes I0face862,If2d091e5,I4e18e63f

* changes:
  AAPT2: Sort artifacts based on the Play Store rules.
  AAPT2: Allow empty group definitions
  AAPT2: Get list of multi-APK artifacts without APK file

Merge "Add new Wifi settings notification drawable."

Merge "Offload switchUser calls to worker threads"

Merge "Implement Call handover initiation side APIs." am: e2d080481a am: 9053309827
am: 875fd19c48  -s ours

Change-Id: I2176b3a20ed52d5bae63cf30853385b33ed9f96a

Merge "1/ index app name. 2/ fix a bug in app update."

Merge "Implement Call handover initiation side APIs." am: e2d080481a
am: 9053309827

Change-Id: I98828fd1ef99d42ffbfb8650731c1d2b848bc062

Merge "Log d when transport updates its attributes"

Merge "Implement Call handover initiation side APIs."
am: e2d080481a

Change-Id: Ib9840c4ce4abf64f36e21cbc3256050e4216314b

Merge "Update setLineSpacing docs to mention units"

Rename ContextHubTransaction.Result IntDef

Use RESULT_ instead of TRANSACTION_ to be consistent with the name.

Bug: 67734082
Test: make
Change-Id: Ib7cc4d527cc160ec6788449231cdffa5c3e8fc0d

Merge "Implement Call handover initiation side APIs."

Merge "Add MessagingStyle#isGroupConversation()."

Merge "WorkSource: Add support for chaining attribution."

Merge "Update GLUtils to support F16"

Fix broken MenuLayoutPortraitTest

Change-Id: Ic24bfbdc5d9aeb286e1f1ba589896331bb8aa2ba
Test: None

Merge "Display Cutout: Make API public"

Back PlatformKeyManager with database

It was previously using SharedPreferences

Test: adb shell am instrument -w -e package com.android.server.locksettings.recoverablekeystore com.android.frameworks.servicestests/android.support.test.runner.AndroidJUnitRunner
Change-Id: I76a75edb93a3feca645e49162c0d8b3e9485385b

Add tests for KeySyncUtils now that SecureBox is available

SecureBox is to be merged before this.

Test: adb shell am instrument -w -e package com.android.server.locksettings.recoverablekeystore com.android.frameworks.servicestests/android.support.test.runner.AndroidJUnitRunner
Change-Id: I40f783c165faa595c8913ff073915726ae71524d

Log d when transport updates its attributes

Bug: 17140907
Test: Triggered an update and checked logs
Change-Id: I21926086ec0078872f3840737f77f79417178f6c

Merge "Merge "Add support for selecting alternate JDWP implementations" am: cbeea23861 am: f5ec7f2e26 am: b24c17efa3"

Merge "Add support for selecting alternate JDWP implementations" am: cbeea23861 am: f5ec7f2e26
am: b24c17efa3

Change-Id: I200e5430fd9696affa37f6330999379173837f2b

Merge "AAPT2: Add flag to compile command for outputting symbols"

Merge "Add support for selecting alternate JDWP implementations" am: cbeea23861
am: f5ec7f2e26

Change-Id: I95d3064791d497986d0ece010be05749904d1ed3

Merge "AAPT2: treat manifest validation errors as warnings when asked"

Merge "Add support for selecting alternate JDWP implementations"
am: cbeea23861

Change-Id: Ifa4ffb6610152f1e919bc10fc7c8697c4bb99a0d

WorkSource: Add support for chaining attribution.

WorkSource objects now hold references to zero or more WorkChain
objects, each of which represent a "chain" of attribution.

This change also changes most WorkSource APIs to be able to
deal with WorkChains. Several APIs do not necessarily make sense with
WorkChains and have been left unmodified :

- boolean diff(WorkSource): Does not account for WorkChains for now.
  This is a public API so we shouln't change it unless we decide to
  make WorkChains public.
- setReturningDiffs(WorkSource): Used internally only, will be removed
  in a future change.
- addReturningNewbs(WorkSource): Used for testing only, fill be removed
  in a future change.

In addition, two new (hidden) APIs have been added to add and query
WorkChains. These APIs have only been added to facilitate testing. They
will most likely change when we add non-test users.

Co-Authored-By: Yang Lu <yanglu@google.com>
Test: WorkSourceTest
Bug: 62390666
Change-Id: Iff361eb98e079c7b2146c092dc27a3618a813c94

Merge changes from topic "fix-atm-visibility-issues"

* changes:
  Add android.test.mock.stubs-system
  Fix visibility issues with android.test.mock.stubs

Merge "Add support for selecting alternate JDWP implementations"

Add access control to trySetWorkModeEnabled and make it public

Test: Quick Settings -> Toggle work mode
Test: Settings -> Work profile settings -> Toggle work mode
Test: Turn off work mode -> Settings -> Turn on work mode in the suggestion
Test: Turn on work mode through tapping on work app

Test: cts-tradefed run cts-dev --module DevicePolicyManager --test com.android.cts.devicepolicy.QuietModeHostsideTest

BUG: 70212757

Change-Id: I335c3ab2306a3eb794cd35a3a7b0d184dc31f85e

Use RecoverableKeyStoreDb in RecoverableKeyGenerator

This removes the layer of abstraction provided by RecoverableKeyStorage,
as it doesn't seem particularly useful, given how easy it is to just use
the real classes in tests. This also hooks up actually saving to the
database. I've modified the class so that you can have a single instance
for generating keys and just pass the 'uid' through, rather than having
to create a new instance per uid. I think this will simplify its use.
Also it no longer returns the key handle. As you can just get it out of
the AndroidKeyStore yourself, it doesn't seem useful to have the method
also do that.

Test: adb shell am instrument -w -e package com.android.server.locksettings.recoverablekeystore com.android.frameworks.servicestests/android.support.test.runner.AndroidJUnitRunner
Change-Id: Ice144a84690150648cbb5da64c8a4cecc2576349

AAPT2: treat manifest validation errors as warnings when asked

Bug: 65670329
Test: updated
Change-Id: Ic554cc20134fce66aa9ddf8d16ddffe0131c50e9

Merge "WorkSource: Remove unused WorkSource.stripNames."

AAPT2: Add flag to compile command for outputting symbols

Only XML files can define resources inside of them, so the fragment
R.txt will only be created for XML files. The fragment R.txt will
contain files defined inside the XML files and the file itself.
For example for res/layout/my_layout.xml that defines "@+id/myView" the
fragment R.txt will contain "default int id myView" and "default int
layout my_layout".
Resources defined with the "public" keyword will have the word "public"
in the partial R.txt, resources defined with the "java-symbol" keyword
will have the word "private, and all other resources will have the word
"default".
If a string is declared in values/strings.xml as:
'<string name="foo">text</string>'
then the partial R.txt will contain "default int string foo". If the
same string is also marked as public in the values/public.xml as:
'<public type="string" name="foo" id="0x7f000001"/>
then the partial R.txt for that file will cointain:
"public int string foo".
Also, the resource IDs will be skipped as this is only for compilation,
proper IDs will be generated at linking phase.

Test: manual
Change-Id: I37d07d5ee4a9f2e5a60a54e48579eba86ae7dd60

Merge "Deprecate StaticLayout and DynamicLayout constructors"

Merge "Make getGmtOffsetText public."

Merge "Add storage for platform key IDs to SQLite db"

Update setLineSpacing docs to mention units

Bug: 64896499
Test: none, docs update
Change-Id: I1b94005643764f6484dca2b02845adde7135a37e

Merge "DPM: Implement installing certificates for generated keys"

Merge "Implement the SecureBox crypto functions"

Merge changes I7cd58841,I800bde6f

* changes:
  Implements getContextHubs
  Merges ContextHubService notion of hub ID/handle

Merge "OutputConfiguration: Fix missing mIsShared in parcel read"