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

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

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

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

Merge "Show dialog if top app is killed by lmk" into oc-mr1-dev

Merge "Only construct real Throwable objects." into oc-dev am: 0868533760
am: f0278f30da

Change-Id: I3a7cbb7cad2c9663e4f7aa2e583ef5707cf6bbc2

Merge "Only construct real Throwable objects." into oc-dev
am: 0868533760

Change-Id: I260dc1d1c903f1c1c2828d68049f9f202daa0869

Handle clear all button show and hide functionality for low ram devices

Hides the clear all button at the beginning and shows it when scrolling
the list up or when swiping an app away. The clear all button will not
hide when already shown. Users can also start the scroll gesture over
the clear all button to scroll the list.

Test: manual - use low ram device and go to recents and scroll
Change-Id: Ie2d22db9bdd66308ce8ef3acf77b9f707d638ade
Fixes: 65571889

Merge "Only construct real Throwable objects." into oc-dev

Associate RenderNodes created for hw bitmap w/ views

In transition animations, in order to capture the content of a view
or a drawable in a hw bitmap, a RenderNode needs to be created. The
RenderNode was previously setup with no owning view. As a result,
in cases where RenderNode animations are triggered by the draw calls
in displaylist recording, these animations would fail for lack of a
view to animate on.

This CL ensures that when RenderNodes are created for the purpose of
populating content in a hw bitmap in transitions, there's always a
view associated with each RenderNode.

BUG: 65160121
Test: Force to repro crash by changing press state during hw bitmap
creation, which triggers a ripple animation that led to the
otherwise timing dependent and hard to repro crash.

Change-Id: I2b4ba95cad25a94d50b3904e775606f737e960e3

Don't clobber shared libs when updating app info

If we update a package without restart or if an installed
package is a target of a resource overlay we update the app
info on the applicaiton side which is then used to update
the class loader and the resources the app is loading from.
To do that we send over an app info instance for the package
which was partially populated and missing the shared lib
entries resulting in apps not finding resources and classes
from the lib.

Test: manual

bug:62685545

Change-Id: Ib40b430c85a1e5d74f270a4326497ea3bd832215

Merge "Revert "Revert "Add preopt profile logic for compiling during first boot""" into oc-mr1-dev

Frameworks: Move Log holder to be preloaded

To avoid issues with late initialization, let the holder be
initialized in the zygote.

Bug: 65927416
Test: m
Change-Id: I6f454df46d4c64d295e1f2510793d5087b74fb74

Show dialog if top app is killed by lmk

Bug: 65488249
Fixes: 65488249
Test: run an app, then kill from command line like so
      adb shell "kill \$(pidof app)"

Change-Id: I7c259542723a9659c52e4571071ec5809693816d
Merged-In: I7c259542723a9659c52e4571071ec5809693816d

Merge "Fix end() for infinite AnimatorSet" into oc-mr1-dev

Merge "Add nullptr check" into oc-mr1-dev

Merge "Minimal change to ensure that position is last saved when docking." into oc-mr1-dev

Use QS context when creating the bluetooth battery indicator

SysUI context doesn't know about the dark theme change when trying to
set the tint on the BluetoothDeviceLayerDrawable, so change the icon to
one that isn't cached so it can use the QS context and use the right
tint when in dark mode.

Test: set dark background, visual
Change-Id: If9255e2d2155ebcd355ddf26f06385e73aab68d6
Fixes: 66497162

Minimal change to ensure that position is last saved when docking.

- When a task is docked from Recents, the last position is never updated
  if the user never touches the divider.  Ensure that the default dock
  snap target is saved in this case as well.

Bug: 66903645
Test: Dock a task from Recents, go home, and hit recents
Change-Id: Ia229d2ba0eee792424fb9e9c043320cec71695dc