Merge "Sets AutofillClient on attachBaseContext()" into oc-mr1-dev

Merge "Adds a filtering bitmask to DownloadStateCallback" into oc-mr1-dev

Merge "Improved javadocs for Autofill save workflow." into oc-mr1-dev

Merge "Remove verification for DownloadReceiver when failure" into oc-mr1-dev

Merge "Select only preinstalled Spell Checker Services" into oc-mr1-dev

Sets AutofillClient on attachBaseContext()

When an activity overrides attachBaseContext(), the attached context must share
the same AutofillCient, otherwise it will break autofill in the activity.

Test: cts-tradefed run commandAndExit cts-dev -m CtsAutoFillServiceTestCases -t android.autofillservice.cts.AttachedContextActivityTest
Test: cts-tradefed run commandAndExit cts-dev -m CtsAutoFillServiceTestCases

Fixes: 67350639

Change-Id: I6b0c4c2fab5fa68eb6ee0714e107ca0816efb2cb

Merge "Decouple Content Name Locale from Content Language Locale" into oc-mr1-dev

Merge "AudioService: log ducking and incall muting of audio players" into oc-mr1-dev

Adds a filtering bitmask to DownloadStateCallback

Adds the ability for an EMBMS app to specify a filtering
bitmask to DownloadStateCallback, which is used to filter
out callbacks that the EMBMS app doesn't need to listen
to.

Test: Testapps
Bug: 67112896
Merged-In: Iab96a457f45f759267b9a5a41df533c9988ede94
Change-Id: Id7f26ba3c3606c644b0ca3376df1107f5834facb

Select only preinstalled Spell Checker Services

When we are setting a new spell checker as the default one in
Secure.Settings, TSMS#findAvailSpellCheckerLocked can pick up
any available spell checker service. This violates the principle
that user should be warned whenever we are setting an untrusted
spell checker service as the default service, since the warning
dialog is never shown.

Bug: 64764051
Fixes: 67410643
Test: Manually as follows:
0. Make sure AOSP keyboard is pre-installed.
1. adb shell settings put --user 0 secure selected_spell_checker com.android.inputmethod.latin/.spellcheck.AndroidSpellCheckerService
2. tapas SampleSpellCheckerService
3. make -j
4. adb install --user 0  -r out/target/product/generic/system/app/SampleSpellCheckerService/SampleSpellCheckerService.apk
5. adb shell pm disable com.android.inputmethod.latin
6. adb shell settings get --user 0 secure selected_spell_checker
    -> com.android.inputmethod.latin/.spellcheck.AndroidSpellCheckerService
7. adb reboot
8. adb shell settings get --user 0 secure selected_spell_checker
    -> com.android.inputmethod.latin/.spellcheck.AndroidSpellCheckerService
Change-Id: I298ffbcfa5e32f43753f54fbebc40a414a5c0f9e
Merged-In: I298ffbcfa5e32f43753f54fbebc40a414a5c0f9e
(manual cherrypick of 0f0a37ba7def8db2c3e0ff1d7d8a30360e889f3f)

Merge "Fix Cell broadcast sound in total silence" into oc-mr1-dev

Merge "Fix NPE in ActivityThread" into oc-mr1-dev

Merge "Update NPMS to inform NMS of the changes in fw rules synchronously." into oc-mr1-dev

Merge "Fixes rounded corners for clear all button on low ram devices" into oc-mr1-dev

Merge "Fixed URL used on sample." into oc-mr1-dev

Update NPMS to inform NMS of the changes in fw rules synchronously.

Currently, NPMS informs NMS synchronously at some events and
asynchronously at some others. If these sync and async calls get
interleaved, then NMS will end up in an inconsistent state.

In N-MR1, NPMS was updated to inform NMS asynchronously to avoid some
lock contentions during screen unlock. This shouldn't be a problem any more
since NPMS no longer updates its internal state on ui thread and also
netd calls have been optimized from ~50ms to ~7ms since then.

Bug: 66015813
Test: cts-tradefed run singleCommand cts-dev -m CtsHostsideNetworkTests -t \
      com.android.cts.net.HostsideRestrictBackgroundNetworkTests

Change-Id: I24ccb5819430f19014630e7b90cba1f0b993e430

Fixes rounded corners for clear all button on low ram devices

Fixed rounded corners for clear all button by using a themed
background color on a shape with rounded corners instead of
programmatically changing the background color.

Change-Id: Iaaf4f0d94d417b0064c7e6bf4895b49d9fa6561f
Fixes: 67368134
Test: manual - go to recents and scroll up to show clear all button

Fixed URL used on sample.

Test: nope!
Bug: 66963249

Change-Id: Ieb46c00d97c8657a4e7498a8136c6da46760fbdd

Merge "AAPT2: Fix R.java styleable + indices ordering" into oc-mr1-dev

Merge "AAPT2: Preserve line numbers when deserializing from binary XML" into oc-mr1-dev

Improved javadocs for Autofill save workflow.

Test: ran 'm -j doc-comment-check-docs' and checked resulting HTML

Bug: 67367585

Change-Id: I25e0f2f6667a4fad6e039f9518c973bac0bc8c7c

Remove verification for DownloadReceiver when failure

When the ACTION_DOWNLOAD_RESULT_INTERNAL intent is received
with a failure result, don't require that extras exist
in the receiver.

Test: Manual, Testapps
Bug: 67112896
Merged-In: I9ce32afc0c6e99b0a61e05a2d2b65855f054abd9
Change-Id: I9ac43a33aff88755cbc804890ce230743195f44d

Decouple Content Name Locale from Content Language Locale

There was an assumption that content language Locale was
a 1:1 mapping with content locale name, which is not the
case. This change separates the two and provides a new
API for retreiving the set of Locales for content name.

Test: Manual
Bug: 67112896
Merged-In: If87094f15e3a1dfe36d37745ed3d2cf79cc441f1
Change-Id: I44cb527ceb77fe321500f9d5fc00f6880ee52cb7

Merge "Foreground service notis must be correctly formed." into oc-mr1-dev

Merge "measure snooze UI use more precisely" into oc-mr1-dev

Merge "Fix a cache calculation regression." into oc-mr1-dev

Fix a cache calculation regression.

Cache size was being double-counted for apps in their drilldown.

Change-Id: I44ec03470050dfbce392348c3336dd5e9bb54c36
Fixes: 67028136
Test: SettingsLib integ tests

AudioService: log ducking and incall muting of audio players

Event logger: add support for logging the text of the event in
  the logcat at the same time to avoid having the duplicate the
  logcat entry and the logger event. See printLog() method.
AudioPlaybackMonitor: log ducking and incall muting.

Test: play media and notification, verify logs appear in logcat and dumpsys audio
Bug:65363602
Change-Id: I2af0c2bbd2fde428f07761ca45e7950a16843604

measure snooze UI use more precisely

Bug: 67003813
Test: runtest systemui
Change-Id: Id7ba68c8308e911cd506943f84c034ade21d1514

Fix NPE in ActivityThread

Calling LoadedApk.getResources() instead of accessing its
mResources directly accounts for the case where this might
not have been initialized.

Bug: 67327250
Test: none
Change-Id: Iaa392a90eefb60ad479cedf8c882976dbe390387

Merge "AudioService: do not unmute media on headset plug in total silence mode" into oc-mr1-dev

Merge "Enable experimentation on notification snooze options" into oc-mr1-dev

AAPT2: Preserve line numbers when deserializing from binary XML

Bug: 65827146
Test: make aapt2_tests
Change-Id: Idbfea640eb8727fe209ebc6eb072296845d9f282

Merge "Reduce lock interactions in backup transport management" into oc-mr1-dev

AAPT2: Fix R.java styleable + indices ordering

Make sure that Styleables are directly followed by their indices.
If not, Robolectric breaks. This is not strictly incorrect to have
an arbitrary ordering in R.java, but its easier to just support
Robolectric in this case.

Bug: 65837293
Merged-In: Ia59ba58427ade386d075ca9fc9eb5b53e35beca0
Test: make aapt2_tests

(cherry picked from commit af85c4deb667843a227d62275fe6992005f4c38d)

Change-Id: Ia59ba58427ade386d075ca9fc9eb5b53e35beca0

Enable experimentation on notification snooze options

Bug: 67003813
Test: runtest systemui

Change-Id: Id46ef3938e2edc633f64cec7b66e10d80c9a0bff

Merge "ImageFormat: Document units/coordinate system for DEPTH_POINT_CLOUD" into oc-mr1-dev

Merge "Lower watchdog timeout for ChooserTargetService responses" into oc-mr1-dev

Merge "Fix Transition test failure." into oc-mr1-dev

Reduce lock interactions in backup transport management

1. process package update broadcasts on our background thread rather
   than on the main looper thread

2. don't synchronize unnecessarily around access to simple
   transport metadata

We mustn't block the main looper thread for anything that might wind
up interlocked with calls to the transport, because those might take
arbitrary amounts of time.  We were previously entering such an
implicitly interlocked code path during package-changed broadcast
handling, and in pathological cases were causing the watchdog to
restart the system.  This situation is addressed in a couple of ways:
first, by no longer performing package-update work on the main looper
thread at all; and second, by eliminating lock reliance entirely from
data-access paths that don't actually need it.

Bug: 65438129
Bug: 64133971
Test: manual + CTS
Change-Id: I361ad4a0729f319db7339bd341a6d33aa3b64fed
Merged-In: I361ad4a0729f319db7339bd341a6d33aa3b64fed

Merge "FIxed typo." into oc-mr1-dev

ImageFormat: Document units/coordinate system for DEPTH_POINT_CLOUD

For camera devices, DEPTH_POINT_CLOUD coordinates are in meters and
are in the camera's local coordinate system.

Test: Built docs, manually verified output.
Bug: 67308479
Change-Id: I028e13a2fb79688a7ce440f905fa82826fc38a03

Merge "Fix launch of activities into VR Virtual display" into oc-mr1-dev

Merge "Import translations. DO NOT MERGE" into oc-dev am: 995627c3ee  -s ours
am: 899f5cf309  -s ours

Change-Id: I79586b1902cfed6baa9db8d615e94b4fadab4740

Merge "Import translations. DO NOT MERGE" into oc-dev
am: 995627c3ee  -s ours

Change-Id: I1acc544f6cab8fdde4d7b66b110baf46114a3730

Merge "Don't crash onServiceDisconnected..." into oc-mr1-dev

Merge "Import translations. DO NOT MERGE" into oc-mr1-dev

Lower watchdog timeout for ChooserTargetService responses

Services providing direct share targets were given up to 5 seconds
to respond to a query, and for UI stability we don't show any until
all services have responded. 5 seconds is a long time to wait and
leads to user complaints when there is even one app that doesn't
respond in time.

Lower the timeout to 2 seconds as a more reasonable maximum.

Bug: 67306843
Test: manual, existing CTS
Change-Id: If55f0485240bc3ccbf8a32aa0e15903ea574ec2b

Merge "Import translations. DO NOT MERGE" into oc-dev

Import translations. DO NOT MERGE am: 45de16dde5  -s ours am: a8c4c7aa94  -s ours am: a77aa847f8  -s ours
am: 595adddf1c  -s ours

Change-Id: Iacfa5b853c7c7c091a06e4c06e20bfbe24c8adf7

Import translations. DO NOT MERGE am: 45de16dde5  -s ours am: a8c4c7aa94  -s ours
am: a77aa847f8  -s ours

Change-Id: Iad25d5ad2bdda26aba5a49bf7a118d86d8eb0b2d

Import translations. DO NOT MERGE am: 45de16dde5  -s ours
am: a8c4c7aa94  -s ours

Change-Id: I6485896b6b814d124c5a7a7b69e0e0a5910a383b

Import translations. DO NOT MERGE
am: 45de16dde5  -s ours

Change-Id: Idaa0738ef0b990ef955a0f6c55c7a0905739439a

Merge "AudioService: more logs for audio mode changes" into oc-mr1-dev

Import translations. DO NOT MERGE

Auto-generated-cl: translation import
Exempt-From-Owner-Approval: translation import

Bug: 64712476
Change-Id: I35ffe572de6f4cf7b2d871bae713da183300b353

Merge "DO NOT MERGE Ignore DUN in describeImmutableDifferences" into oc-mr1-dev

Import translations. DO NOT MERGE

Auto-generated-cl: translation import
Exempt-From-Owner-Approval: translation import

Bug: 64712476
Change-Id: I78f2a4cfa2cd99a1ca3f8fb25bea13e286b6275e

DO NOT MERGE Ignore DUN in describeImmutableDifferences

This patch changes describeImmutableDifferences in NetworkCapabilities
to ignore differences in NET_CAPABILITY_DUN, so that updateCapabilities
in ConnectivityService to not report wtf errors when a NetworkAgent
degrades its NetworkCapabilities object by removing NET_CAPABILITY_DUN.

Bug: 65257223
Test: runtest frameworks-net
Change-Id: I115ed1b366da01a3f8c3c6e97e0db8ce995fd377

Import translations. DO NOT MERGE

Change-Id: I9ca153d375ff59721c164c9841f7c873aaa562fc
Auto-generated-cl: translation import
Exempt-From-Owner-Approval: translation import

Merge "revamp app compat test harness" into oc-mr1-dev

revamp app compat test harness

- migrate away from deprecated JUnit3 paradigm
- scan dropbox entries for app errors, instead of probing
  task list
- use IActivityController to suppress crash dialogs and record
  detected app errors
- use combined dropbox and activity controller detected errors
  to determine app errors

Bug: 67002148
Test: run harness against apps known to crash
Change-Id: If108cfdc7474a13e24f0d8350a7cbf99e3b51c46

Fix ClipboardService device lock check for cross profile am: 6d79675e0c am: 095eda9307 am: a4f6ca3e77 am: 85439e9114 am: 17735f183a am: d93f91a137 am: 9865205a4e am: 7c7945c959 am: 36153a9689  -s ours am: 9b3559aee9
am: eee38701b5

Change-Id: Id0ce1ee8cebf5c5968188e918da57849f2ffd3c6

Fix ClipboardService device lock check for cross profile am: 6d79675e0c am: 095eda9307 am: a4f6ca3e77 am: 85439e9114 am: 17735f183a am: d93f91a137 am: 9865205a4e am: 7c7945c959 am: 36153a9689  -s ours
am: 9b3559aee9

Change-Id: I85c08a27dc190d7c6710fd2b928d8e018d08ede8

Fix ClipboardService device lock check for cross profile am: 6d79675e0c am: 095eda9307 am: a4f6ca3e77 am: 85439e9114 am: 17735f183a am: d93f91a137 am: 9865205a4e am: 7c7945c959
am: 36153a9689  -s ours

Change-Id: I341930f4f2195705ec0ddcef29e807602f93f98b

Fix ClipboardService device lock check for cross profile am: 6d79675e0c am: 095eda9307 am: a4f6ca3e77 am: 85439e9114 am: 17735f183a am: d93f91a137 am: 9865205a4e
am: 7c7945c959

Change-Id: I0d4f109c2e73ee7088bb8b77eba563d60ee0113f

Fix ClipboardService device lock check for cross profile am: 6d79675e0c am: 095eda9307 am: a4f6ca3e77 am: 85439e9114 am: 17735f183a am: d93f91a137
am: 9865205a4e

Change-Id: I7c34d6b58d5aacadd6efa07ce12f9b7e9898768a

Fix ClipboardService device lock check for cross profile am: 6d79675e0c am: 095eda9307 am: a4f6ca3e77 am: 85439e9114 am: 17735f183a
am: d93f91a137

Change-Id: If4bb2848e0df71257a1f7e162524ebd08a5dad4d

Fix ClipboardService device lock check for cross profile am: 6d79675e0c am: 095eda9307 am: a4f6ca3e77 am: 85439e9114
am: 17735f183a

Change-Id: I71795f8f2f454956797ffaa78d22ca3573e5d626

Fix ClipboardService device lock check for cross profile am: 6d79675e0c am: 095eda9307 am: a4f6ca3e77
am: 85439e9114

Change-Id: Ic616bde55a3a60eec10d9330907e6d8164b76c86

Fix ClipboardService device lock check for cross profile am: 6d79675e0c am: 095eda9307
am: a4f6ca3e77

Change-Id: If3b713963f393b7a16352d03c28dced60e2df7db

Fix ClipboardService device lock check for cross profile am: 6d79675e0c
am: 095eda9307

Change-Id: I4ff17f12c20f95efe68115cc3d6b7d5e9b47228c

Fix ClipboardService device lock check for cross profile
am: 6d79675e0c

Change-Id: Icea573859a6d441e18665d9da855370582f2992a

Merge "Fix ClipboardService device lock check for cross profile" into oc-dev am: 8155446b77
am: 8c8e5ce966

Change-Id: Ia6e606bbe766ae439c8285e2e573be98f512bca0

Merge "Fix ClipboardService device lock check for cross profile" into oc-dev
am: 8155446b77

Change-Id: I21712b3913ab073d1e0265bc8eb73d9fa33c032c

Merge "Fix ClipboardService device lock check for cross profile" into oc-dev

Merge "Increase the duration allowed for a TC result." into oc-mr1-dev

Merge "Fixing Launcher package name for privilege permission" into oc-mr1-dev

Merge "Disable long-press home Orb animation." into oc-mr1-dev

Fix Transition test failure.

Bug: 67049319

TransitionUtils was returning null when the View wasn't attached,
but Visibility transitions can do that intentionally. This CL
temporarily adds detached views to the view hierarchy as part of
an overlay while creating the hardware bitmap representation.

Test: ran transition CTS tests
Change-Id: Ie335619953653dce0224514f0d5c9c8eb00ee1a9

Merge "Add Open Network Available metrics to wifi.proto" into oc-mr1-dev

Increase the duration allowed for a TC result.

Increase the duration allowed for a TextClassifier (TC) result when
it is being initialized for the first time.
Initialization typically takes more time that other calls to the TC.
A timeout during initialization means the user doesn't see the
effect of running the TC the first time. Subsequent calls typically
work fine as they're less likely to timeout.
We increase this value only slightly because we don't want the user
waiting too long for a slow TC's result. Finding the right balance
is an art and most likely depends on the TC being used.

Note that although a TextView's TC can change for each individual
call to the TC, such a situation does not benefit from the bump in
the timeout. Again, we really want to discourage larger timeouts.

Fixes: 66217281
Test: bit FrameworksCoreTests:android.widget.TextViewActivityTest
Test: bit CtsWidgetTestCases:android.widget.cts.TextViewTest
Test: Manual test:
  - Delayed TextClassifier response by about 150ms (allowing only 50ms for initialization)
  - Before cl: Sometimes the result is not shown to the user the
    first time the textclassifier runs
  - After cl: Results consistently show for even for first
    textclassifier call
Merged-In: Iabc8279fa57ef6a9a2db0c7e9c7e3ab8c131bf02
Change-Id: Iabc8279fa57ef6a9a2db0c7e9c7e3ab8c131bf02

Fixing Launcher package name for privilege permission

Test: None
Bug: 67017745
Change-Id: I37b903c0c771e54a68bf4dac42a0ce39641b7933

Foreground service notis must be correctly formed.

If an app targets api 26, it needs a notification channel, even for
foreground service notifications.

Change-Id: Id3131d4bb2748e9d23f66dd4f5570842bc50a44f
Fixes: 66905243
Test: manual

Merge "Update configuration whenever the visible window is laid out." into oc-mr1-dev

Merge "Fixed warning on getWebDomain()" into oc-mr1-dev

Merge "enable experiments for AOD timeout" into oc-mr1-dev

Merge "Don't show no sim when a sim has been detected." into oc-mr1-dev

Fixed warning on getWebDomain()

Test: ran 'm -j doc-comment-check-docs' and checked resulting HTML.
Bug: 66963249

Change-Id: If063961f41f297da1ed67e73616972f455edc117

Update configuration whenever the visible window is laid out.

The configuration update during layout is currently restricted to
whether the client is hidden. This does not cover all the conditions
where the window can be laid out, such as if it is the starting
window.

This changelist makes sure the same condition is used in both cases.

Change-Id: I64a2f70718b949f3803e8cafcdabd2be5b5885e6
Fixes: 67027059
Test: Repeatedly open Gmail and ensure no jump-cut occurs.

Don't show no sim when a sim has been detected.

Instead indicate a lack of signal until we have a subscription to
tell us what to show.

Test: updated existing tests
Change-Id: I170f3782597584e5e761b5c9f9fce3cdbdc26a4f
Fixes: 65744290

Merge "Reduce synchronized lock scope." into oc-mr1-dev

Merge "Make RoutInfo.select() called in app's main thread" into oc-mr1-dev

Merge "NotificationPlayer: fix race conditions" into oc-mr1-dev

Make RoutInfo.select() called in app's main thread

Bug: 66995486
Test: passed MediaRouter test
Test: Checked theat IllegalStateException doesn't happen
Change-Id: Idb75998cd7910a430e68ea928d28c9908260d5fc

Merge "Prevent system crash when adjusting volume with illegal direction" into oc-mr1-dev

Merge "Add multiuser support for enabling notification listener" into oc-mr1-dev

Fix Cell broadcast sound in total silence

PlayerBase:
 Update player volume when the audio attributes are updated in case
we need to unmute
PlaybackActivityMonitor:
 Unmute alarm stream if needed when an alarm with flag
 FLAG_BYPASS_INTERRUPTION_POLICY starts from an app with
 privileged permission MODIFY_PHONE_STATE.

Bug: 63617557
Test: check sound with cell broadcast in total silence
Change-Id: Ifacb1d96a2d8d44047d9f9642d1e672fcf756cda

Merge "Don't clobber shared libs when updating app info" into oc-mr1-dev

AudioService: more logs for audio mode changes

Added more logs to audio mode changes to track both the requested mode
and the actually selected mode.

Bug: 65513486
Test: Verify logs after various voice and VoIP scenarii
Change-Id: I486f091803ef7776ff325db908e8e462852ea3c1

Merge "Ensure that the thumbnail path is valid even without a thumbnail." into oc-mr1-dev