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

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

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

measure snooze UI use more precisely

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

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

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

Import translations. DO NOT MERGE am: 47749a01d7  -s ours am: 27d4c36a3d  -s ours am: 5f76e0c11d  -s ours
am: 8271e98237  -s ours

Change-Id: Ifab6a559892ef14e12e1915f540a09a00a0f2fa9

Merge "Use QS context when creating the bluetooth battery indicator" into oc-mr1-dev

Import translations. DO NOT MERGE am: 47749a01d7  -s ours am: 27d4c36a3d  -s ours
am: 5f76e0c11d  -s ours

Change-Id: I231ba5e0ac321a50cad0de5c496f84665ce8b89e

Import translations. DO NOT MERGE am: 47749a01d7  -s ours
am: 27d4c36a3d  -s ours

Change-Id: Ie5b388d1266df622b41aabc29991509e2a0dc94f

Import translations. DO NOT MERGE
am: 47749a01d7  -s ours

Change-Id: I8c9c8f1ef6b35bffc1939fd7313261aa104ffe09

NotificationPlayer: fix race conditions

This patch fixes two race conditions that affect the Looper used
  to signal the completion of a notification to abandon audio focus
  as well as the wakelock used between issuing a playback
  command and its actual start. Annotations are added to clarify
  which objects are used to synchronize which methods and variables.

Looper for notification playback completion:
  Before a notification starts playing, audio focus is requested,
  which causes the ducking of media apps. When the notification
  completes, audio focus is abandoned. If a new notification is
  to be played while one is playing, the current player is
  stopped and the Looper on which we expect the MediaPlayer
  completion callback is .quit(). But there is a race condition
  between the quitting of the current Looper whenever a sound
  is started (in startSound()) and when quit when playback
  is stopped (command STOP), and when created in
  CreationAndCompletionThread.run(). If the Looper is quit
  when another notification starts to play, the completion
  callback cannot be received, and audio focus will not be
  abandoned.
  The fix consists in synchronizing all access to mLooper
  on mCompletionHandlingLock.

Wakelock:
  Initializing and acquiring the wakelock, and releasing it
  are done in different threads (client thread vs CmdThread).
  There was no memory barrier between the initialization
  and release. The fix consists in making all wakelock
  operations synchronized on mCmdQueue.

Test: issue multiple notifications that interrupt eachother, verify focus is abandonned (in logs, check "abandonAudioFocus()")
Bug: 65866087
Bug: 64531811

Change-Id: Ie8f4091eaa96bd0bcb732e27423f6e31e76da98e

enable experiments for AOD timeout

Bug: 67009600
Test: runtest systemui
Change-Id: I5505b69051a55608b3c3803c48dd51603845e9c9

Fix launch of activities into VR Virtual display

When launching a 2D activity in VR mode by the VR app, we want to launch
it in the virtual display created by VR. But, this fails since the
virtual display owner is system and the calling UID doesn't have
ACTIVITY_EMBED permission.

We address this by making the virtual display public since it's lifetime
is only VR mode and we want to allow all apps to launch 2D activities
and route them to the vr virtual display.

Bug: 66475142
Test: Launch VrSettingsActivity by clicking on gear and check "adb shell
am stack list" to verify it launched on the virtual display

Change-Id: Idde63ea63c953c6cf3d4ea46989a963a5651ce8a
Signed-off-by: Karthik Ravi Shankar <karthikrs@google.com>

Ensure that the thumbnail path is valid even without a thumbnail.

- When there is no thumbnail snapshot, an empty sized thumbnail path is
  created since thumbnailWidth/Height are zero.  Instead, like the parent
  class, the path should default to the view width/height in such a case.

Bug: 66938643
Test: Launch PlayMovies video anew, go home, then to overview
Change-Id: I354ff0716ee44149f50ff11fa435161cf466c0cc

Reduce synchronized lock scope.

The lock was introduced earlier to prevent race conditions between
setting the visibility and configuration. However, holding the lock
for the entire method leads to an increase in the frame time.

This changelist reduces the scope of the lock to cover the core
logic where this condition applies.

Change-Id: Ia97b3680f730264c10ff5067e4f21180cfb2202e
Fixes: 67010772
Test: go/wm-smoke

AudioService: do not unmute media on headset plug in total silence mode

Fix regression introduced by fix for issue 30440012.

Bug: 66893057
Test: play a video on Chrome in DnD total silence and plug a wired
headset

Change-Id: Iee11f06edda0749a15021a4682467c14289677c3

Merge "Add API to request light navigation bar in theme" into oc-mr1-dev

Add API to request light navigation bar in theme

Such that splash screens can already have a light navigation bar,
avoiding a janky flash.

Test: CtsSystemUiTestCases
Bug: 66986286
Bug: 63630024
Change-Id: I0b89c58e9fbca08e27a6ec13db5a07f5e030c682
Merged-In: I81b34fefeb336ac2269dc5e6b98cc0b6045cea70

Merge "Import translations. DO NOT MERGE" into oc-dev am: a980312abf  -s ours
am: 9f05e68ea5  -s ours

Change-Id: Iec6173e113757c3edd2f12a7e6f36b1baae6ba89

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

Change-Id: I59f002be824830c9ab3224c25c9d9d39edca43f1

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

Merge "Frameworks: Move Log holder to be preloaded" into oc-mr1-dev

Import translations. DO NOT MERGE

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

Bug: 64712476
Change-Id: I78318e3a6213ef4ec92a409d319d076740914da0

Add multiuser support for enabling notification listener

Bug: 66504373
Bug: 65566927
Test: run CtsMediaHostTestCases
Change-Id: I7f22a3849a0dc298b8a6358327125d966068a09d

Import translations. DO NOT MERGE

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

Merge "Associate RenderNodes created for hw bitmap w/ views" into oc-mr1-dev

Merge "Handle clear all button show and hide functionality for low ram devices" into oc-mr1-dev

Merge "In-call-notif doesn't use AUDIBILITY_ENFORCED flag" into oc-mr1-dev