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

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

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

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

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

Fixing session abandon logging bug

Previously, making two selections quickly one after the other could lead
to the first session missing the terminal event, and the second one getting
terminated prematurely - getting marked incorrectly as abandoned.

Bug: 64914512
Test: Manually tested that logs are correct.
Change-Id: Icd75dcabe707b591f30629b9b9b42c5459ed7dda

Fixed bug in handling of OTHER as terminal

This bug resulted in a tap on a non-default action mode item to get logged
as OTHER, followed by ABANDON.

Bug: 64914512
Test: Manually tested that logs are correct.
Change-Id: I3cdba7ccff896939e22c567843f335c54d4a97f3

Merge "Select correct system audio route in updateAudioRoutes" into oc-mr1-dev

Select correct system audio route in updateAudioRoutes

While update audio routes, MediaRouter had an access to the variable
of MediaRouterService.mGlobalBluetoothA2dpOn, which was updated in
the callback of AudioRoutesObserver.dispatchAudioRoutesChanged().
However, since updateAudioRoutes() was also called by the same
callback, mGlobalBluetoothA2dpOn could be used in updateAudioRoutes()
before its value was updated.

Bug: 65629167
Test: passed MediaRouterTest
Test: Manually confirmed that the issue is fixed with this CL
Change-Id: Id0034996a51a6d8d1e8bd9d2c6ca386aabbb8baf
Merged-In: Id0034996a51a6d8d1e8bd9d2c6ca386aabbb8baf

Merge "Logs how long it took to show the Autofill UI after a field was focused." into oc-mr1-dev

Hide autofill popup if anchor removed

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

bug:64601372

Change-Id: Ic55dabb99cd5015d318ab3be1231b6aa5c67294e

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

Logs how long it took to show the Autofill UI after a field was focused.

Test: adb shell logcat -b events | grep sysui_
Test: adb dumpsys autofill
Test: cts-tradefed run commandAndExit cts-dev -m CtsAutoFillServiceTestCases

Bug: 65376559
Bug: 66508098

Merged-In: Ia17f8b30d6eef0141b020085985b9ebacff7a455
Change-Id: Ia17f8b30d6eef0141b020085985b9ebacff7a455

Merge "Speculative fix for missing target crash" into oc-mr1-dev

Merge "Add metrics (and moar logging) for when a service disables itself." into oc-mr1-dev

Merge "Remove isHighEndGfx check" into oc-mr1-dev

Add null check in a11y callback

Bug: 65475473
Test: Verified it still builds
Change-Id: I9ad1249c44cb73153066f67164c90333779d284e

LockSettingsStrongAuth shouldn't get fingerprintservice on unsupported devices

Fixes: 65838275

Test: tested on ryu and walleye, no logs when system starts and everything
works fine

Change-Id: I7d53214a021e9094618162109f3c38c0e029bda3

Merge "Add systrace for VectorDrawable inflation and draw" into oc-mr1-dev

Add systrace for VectorDrawable inflation and draw

We didn't trace the draw from cache.
Here we add trace for draw into bitmap, which is normally heavy.

fix: 65060698
Bug: 65060698
Test: run test app and get systrace and check

Change-Id: Ia81127c4aa285b3277e9c9edbdf356d85cb28b5e
(cherry picked from commit cf0c41dbc221c2619212c7e25e6d90a9c4d05b05)

Merge "hwuitest: added hwui test for TV application" into oc-mr1-dev

Add preopt profile logic for compiling during first boot

Allows pushing profiles on device that are used to optimize apps
during first boot.

Added logic to copy profiles to reference location for compressed APKs.

Bug: 38032017
Bug: 64503246
Test: adb shell rm -rf /data/app/*
Test: adb shell rm /data/system/package*
Test: adb shell stop && adb shell start
Test: look at apps in /data/app to make sure they are profile compiled

Change-Id: I30452098a19a65a331d098107d37498e5b10c10f