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 "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

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

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

Add nullptr check

Bug: 65717856
Test: builds
Change-Id: I188d3366e956f087c6317ae0b56d5fb19a20708d

In-call-notif doesn't use AUDIBILITY_ENFORCED flag

Audibilty enforced flag should only be used to make sure system
sounds are audible when this is mandated by a given country regulation.
Here, it was used incorrectly.  Also, adjusted in-call notification
sound to be less loud.

Fixes: 66808711
Test: manual

Change-Id: I119a718fb5fc605b6f6b3e387e5e9f956cfd5fff

Fix crash due to TextView.getText() returning null.

This CL ensures that a null text is not passed to
TextClassificationHelper.reset(...)

This CL does not handle the issue of TextView.getText() being null.

Bug: 65682674
Test: none
Merged-In: Id8bdc062ecf21cc14810586580fef9052672cfd8
Change-Id: Id8bdc062ecf21cc14810586580fef9052672cfd8

Declare support for Ethernet if the service is running.

On some devices, support for TYPE_ETHERNET is not specified in
the networkAttributes config resource, even though the device is
capable of supporting Ethernet (e.g., via USB host adapters).
This leads to Ethernet working but various connectivity APIs
behaving as if it was not - for example, no CONNECTIVITY_ACTION
broadcasts will be issues when it connects or disconnects.

Ensure that ConnectivityService always treats Ethernet as
available if the service is running. Currently the service is
started if the device supports FEATURE_ETHERNET or
FEATURE_USB_HOST.

(cherry picked from commit 7bbe3eee52c08ee92a81b7bed395ca5499554cc4)

Bug: 37359230
Test: bullhead builds, boots
Test: ConnectivityServiceTest passes
Test: Ethernet is available even if removed from networkAttributes resource
Test: ConnectivityManagerTest CTS test passes
Change-Id: I9b6db4edeaf966ee6715011dd92770b9d25dd938
Merged-In: I9b6db4edeaf966ee6715011dd92770b9d25dd938

Revert "Revert "Add preopt profile logic for compiling during first boot""

Disabled the logic for first boot profile compilation for non-extracted APKs.
This prevents going from speed-profile->speed-profile if things are preopted.

Test: build, flash, verify no speed/speed-profile compilation during first boot

Bug: 66903791

This reverts commit 4a299d56e613ab85c0e732020e0041e566b21e84.

Change-Id: Iaa526e605cf57b2f24ba72d6786b35d5347cd3c5

Merge "Do not throw on call to isTetheringSupported w/o permission" into oc-mr1-dev

Merge "AudioService: Don't honour SCO ON request if SCO is not connected" into oc-mr1-dev

Merge "Fix spelling mistake in dumpsys output" into oc-mr1-dev

Merge "Remove obsolete & broken handling of precompiled sys apps" into oc-mr1-dev

Fix end() for infinite AnimatorSet

BUG: 66880433
Test: repo steps in the bug above

Change-Id: If34b7386b7ff5cd28144e99c37957abddb068e9c

Merge "Added missing quote (") on anchor tag." into oc-mr1-dev

Merge "Adjusted chooser padding so 2nd line isn't cutoff." into oc-mr1-dev

Merge "Don't convert all sounds to vibrations in vibrate mode" into oc-mr1-dev

Merge "Shorter notification text on AoD" into oc-mr1-dev

Merge "Fix crash when creating a HW Bitmap on a detached view" into oc-mr1-dev

Remove obsolete & broken handling of precompiled sys apps

During the times we used verify-profile for OTAs we had to ensure that
prebuilds do not regress by verifying at runtime. For that we added a
hackish check in PMS.

Now, we no longer use verify-profile and we have no configuration which
will leave the prebuilds unverified (assume-verify does not count here).
So, we can remove this check which blocks system apps to be compiled with
speed-profile.

(Also worth mentioning that
!Environment.getReferenceProfile(pkg.packageName).exists() was broken...
most likely because of an SElinux policy update which we did not handled
:/ )

Bug: 66903791
Test: build & boot

(cherry picked from commit efc022768eba2f33d3fadbda7eaa6adf1730d3fc)

Merged-In: I981e463d61c42ab8455d489ff86cc300632a2a90
Change-Id: I981e463d61c42ab8455d489ff86cc300632a2a90

Fix spelling mistake in dumpsys output

Bug: 65843255
Test: build
Change-Id: Icc748dd14b52641e37b149f51d61a4086e55ed41

Merge "Update the bt background color." into oc-mr1-dev

Added missing quote (") on anchor tag.

Test: 'm -j doc-comment-check-docs' then verify resulting HTML
Fixes: 66958057

Change-Id: Ie5914a5af191866404b6fbf244c78e145c59a8b1

Merge "Re-add callback to onCancel() when screen is touched outside Save UI." into oc-mr1-dev

Merge "Add null check in a11y callback" into oc-mr1-dev

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

Change-Id: I2dab29638f0813d6e52c9b4ec7b279b7ddcb06a0

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

Change-Id: I3820c0c5decca7f1a839a40a65e9021fa6e999d1

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

Shorter notification text on AoD

Change-Id: I961c7348d6ce2389c9ac0fba980237548eaf26c9
Fixes: 66242485
Test: Send notification with big text, look at AoD

Import translations. DO NOT MERGE am: 1df9bbaafe  -s ours am: 6cc2a9cdc2  -s ours am: 5cf5642cbd  -s ours
am: 7b3e838ec1  -s ours

Change-Id: I4706a62f6bc875294a66ae70f157949b36df60a8

Import translations. DO NOT MERGE am: 1df9bbaafe  -s ours am: 6cc2a9cdc2  -s ours
am: 5cf5642cbd  -s ours

Change-Id: Ie5c5a0ddabdc8dc0bfd081c48c3c279425cc2682

Import translations. DO NOT MERGE am: 1df9bbaafe  -s ours
am: 6cc2a9cdc2  -s ours

Change-Id: I4d3d9da8dcdc0c898c49856cae530c14f6aa915e

Import translations. DO NOT MERGE
am: 1df9bbaafe  -s ours

Change-Id: I2bf2ceaea1279df64cf7409c57c75039caaa7879

Merge "Quick fix: Make incorrect animator usage a warning" into oc-mr1-dev

Re-add callback to onCancel() when screen is touched outside Save UI.

That callback was accidentally removed because it should not trigger the
negative listener, but the right approach was to pass a null listener.

Test: cts-tradefed run commandAndExit cts-dev -m CtsAutoFillServiceTestCases

Bug: 66235266
Fixes: 66944844

Change-Id: I5b71d3d03335d952e3b47078d1373f9783366437

Merge "Fix wifi tile animating full-dark when slashing/unslashing" into oc-mr1-dev

Import translations. DO NOT MERGE

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

Bug: 64712476
Change-Id: I348ab7ada9d98e745fe4716beadb6857d39705ec

Do not throw on call to isTetheringSupported w/o permission

...just return false instead. This will change in P.

Test: Made an app to test this. Made sure it doesn't have
Test: the required permission. Checked it crashes with
Test: SecurityException without this change. Checked it
Test: doesn't with it.

Bug: 65404184

Change-Id: Id20d3c240ec5d70d085e0366b92ab3a514f3e7c8

Import translations. DO NOT MERGE

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

Fix crash when creating a HW Bitmap on a detached view

BUG: 65160121
Test: Unable to repro the crash before or after the fix

Change-Id: I84fa28557c67a6672b8d82443d4da7be4f28a50d

Merge "Restrict when a window receives new Configurations." into oc-mr1-dev

Quick fix: Make incorrect animator usage a warning

Change-Id: I6d59033433a63d02be746bb8523b312d7c8fe9fa
Bug: 65160121
Test: Repro from bug doesn't repro anymore

Merge "LockSettingsStrongAuth shouldn't get fingerprintservice on unsupported devices" into oc-mr1-dev

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

Revert "Add preopt profile logic for compiling during first boot"

Test: flash build and forrest
Bug: 66903791

This reverts commit 20ad4b92a7528f24adf6e4f6d23d9d3a02a8792a.

Change-Id: I2c0f22f2a7c04221ed7b1562e516be98d27c7a50

Merge "Hide autofill popup if anchor removed" into oc-mr1-dev

Restrict when a window receives new Configurations.

It is possible that a window will relayout while it's being hidden,
but after the Configuration has changed. In this case, the window can
receive an updated Configuration incompatible with its settings.
Additionally, an Activity's window may relayout and receive a
Configuration during resume after its visibility has changed but
before the Configuration has been updated based on the current
AppWindowTokens.

This changelist addresses these issues by first only sending an
updated Configuration to the client if the associated AppWindowToken
is not requested to be hidden. In this case, the last reported
Configuration is returned instead. For the resume issue, we
address the race condition by making setting the visibility and
updating the configuration synchronized.

Fixes: 64916689
Test: go/wm-smoke
Test: place clock widget on launcher, launch landscape only activity,
      return to launcher, verify layout.

Change-Id: Ie07068be64120c2fdbe380d58af330372df6a1ab

Merge "FRP: Add config flag for disabling credential FRP" into oc-mr1-dev

Merge "Fixed that telephony registry service reference incorrectly assigned" into oc-mr1-dev

Merge "Fixing session abandon logging bug" into oc-mr1-dev

Merge "Fixed bug in handling of OTHER as terminal" into oc-mr1-dev

FRP: Add config flag for disabling credential FRP

Bug: 65258606
Test: Verify FRP still works when not disabled
Change-Id: I78e308b6eaba73c794615b1eebdbfacb334ae72f