Merge "Clearing up invalid entries when SyncStorageEngine starts" into nyc-dev am: fc9e382229 am: 7d9e844124 am: 25ec24740b am: 4f31a88d4b am: bf1b7e9030  -s ours am: 84aa3f4c81  -s ours
am: a8e0d5f73a  -s ours

Change-Id: I34f6294cfa3d9e23bfe9b3f0edfb03da4fd5dfd2

Merge "Clearing up invalid entries when SyncStorageEngine starts" into nyc-dev am: fc9e382229 am: 7d9e844124 am: 25ec24740b am: 4f31a88d4b am: bf1b7e9030  -s ours
am: 84aa3f4c81  -s ours

Change-Id: I701fa204e66a8dd239e3ed3913f734e8ea503049

Merge "Clearing up invalid entries when SyncStorageEngine starts" into nyc-dev am: fc9e382229 am: 7d9e844124 am: 25ec24740b am: 4f31a88d4b
am: bf1b7e9030  -s ours

Change-Id: Ifcb9a5a08af1722bc5df28c470d4ca356b865acb

Merge "Clearing up invalid entries when SyncStorageEngine starts" into nyc-dev am: fc9e382229 am: 7d9e844124 am: 25ec24740b
am: 4f31a88d4b

Change-Id: Ia97ee26a24e49809ad7484f850d256bc733ee239

Merge "Clearing up invalid entries when SyncStorageEngine starts" into nyc-dev am: fc9e382229 am: 7d9e844124
am: 25ec24740b

Change-Id: I858f75717157bd9269bcd43d4198935302132d5a

Merge "Clearing up invalid entries when SyncStorageEngine starts" into nyc-dev am: fc9e382229
am: 7d9e844124

Change-Id: Ifc197fbb16e1bb6773b2f08c7c62ca92f82238f7

Merge "Clearing up invalid entries when SyncStorageEngine starts" into nyc-dev
am: fc9e382229

Change-Id: I80e814ed0266b3868bd7039508bbc50b2e2a51aa

Merge "Clearing up invalid entries when SyncStorageEngine starts" into nyc-dev

Merge "Revert "Remove a compat hack SurfaceView#setWindowType()"" into oc-mr1-dev

Merge "Documented use of AutofillManager.notifyViewVisibilityChanged()" into oc-mr1-dev

Revert "Remove a compat hack SurfaceView#setWindowType()"

This CL reverts my previous CL [1] that aimed to get rid of
a nasty compatibility hack that was introduced for Bug 36345857.

For those who are interested in, what happenned are:

 1. @hide method SurfaceView#setWindowType() was removed [2].
 2. It broke some app (Bug 36345857).  We had to work around
    it by re-introducing SurfaceView#setWindowType()
    temporarily [3].
 3. Some app switched to the correct implementation when
    running on Android O devices.
 4. We removed that compatibility hack [1] (Bug 62054282).
 5. Android O MR1 is set to be "REL" [4].
 6. It broke some app, probably because of some unfortunate
    mistake in the version check logic in that app.
 7. We end up introducing the same hack again for O MR1.

 [1]: Icee198c554de558cfa4ffe0b264064969839654e
      7a1ad6d97ca1984789446211a986d5ad1b5c81e2
 [2]: Ie56b6f7ab16f32d7fc459b8eba26594337ad55de
      d5c7dd6da810a6b89151b337bea79fd817e6b72a
 [3]: I5217f6417a73690ae8a978754218b7b089070fdd
      3b5011afc9e17963607269bfb6665d04e3ab4ca1
 [4]: I054e3ecff49803e61e7741753fe6764a567d72c4
      62a835d0ef89e51f4a97fecf8576224551b545a5

Bug: 36345857
Bug: 62054282
Fixes: 65508814
Test: Manually verified that Bug 65508814 is not reproducible
Change-Id: If8a3f726789daa22f73e1962e938f071d3c09414

Merge "Don't log TYPE_DATASET_SELECTED on dataset authentication." into oc-mr1-dev

Merge "Destroy Save UI when it's hidden without a pending restoration." into oc-mr1-dev

Merge "Added more logging to diagnose a racy runtime restart." into oc-mr1-dev

Destroy Save UI when it's hidden without a pending restoration.

This scenario typically happens when the BACK button is tapped after
the SaveUI is restored.

Test: cts-tradefed run commandAndExit cts-dev -m CtsAutoFillServiceTestCases -t android.autofillservice.cts.SimpleSaveActivityTest#testTapLink_launchTrampolineActivityThenTapBackAndStartNewSession
Test: cts-tradefed run commandAndExit cts-dev -m CtsAutoFillServiceTestCases -t android.autofillservice.cts.PreSimpleSaveActivityTest#testTapLink_launchTrampolineActivityThenTapBackAndStartNewSession
Test: cts-tradefed run commandAndExit cts-dev -m CtsAutoFillServiceTestCases
Fixes: 65459113

Merged-In: I07c6d9b6d219889c50caeaff99b883841b87d9cc
Change-Id: I07c6d9b6d219889c50caeaff99b883841b87d9cc

Merge "Follow-up on bug #65298216: DeviceIdleController needs to..." into oc-mr1-dev

Merge "Sealed sessions need to be destroyed or validated" into oc-mr1-dev

Merge "Add logging const for smart setting suggestion enabled/disabled state" into oc-mr1-dev

Merge "Fixed a flakey test" into oc-mr1-dev

Merge "Do not process set WindowAnimator param when screen is already on." into oc-mr1-dev

Merge "Fix NativeCallbackThread race condition." into oc-mr1-dev

Sealed sessions need to be destroyed or validated

- Always destroy session on validate failure.
- Always validate session when restoring a sealed session.
- Never persist destroyed sessions

Fixes: 65456964
Test: cts-tradefed run singleCommand cts-dev -m CtsContentTestCases --test=android.content.pm.cts.InstallSessionTransferTest
      Installed apps via Google Play Store
      Installed apps via Package installer
Change-Id: I38940608eca6d32a9a454937d273706f3ce57d79

Merge "Don't call disconnected AccessibilityServices" into oc-mr1-dev

Merge "Handle configuration splits when creating the class loader context" into oc-mr1-dev

Merge "Javadoc: Added alternative to Package verification." into oc-dev am: 092349600d
am: 33d476c170

Change-Id: I59c6ddc48fad9178b03c7a1da7957238d4045ea7

Add logging const for smart setting suggestion enabled/disabled state

Test: builds
Bug: 64121058

Change-Id: I4136d9bfab8c828e85b36ee25421189389a5dddc
Merged-In: I9d1e27082d535da90a5a30f6c2fc9b717c07bf1b

Merge "Camera: fix race in onRepeatingRequestError callback" into oc-mr1-dev

Merge "Cap user-visible cache size at quota." into oc-mr1-dev

Merge "Javadoc: Added alternative to Package verification." into oc-dev
am: 092349600d

Change-Id: Ic765df95fc660fb0df8a0f027d739ea999a36702

Merge changes from topics "embms-0728-adjustments", "embms-0721-adjustments", "fix-streaming-api", "mock-unhide-embms-streaming" into oc-mr1-dev

* changes:
  Embms adjustments for 7/28
  Embms API adjustments for 7/21
  Fix lint errors in the streaming API
  Mock-un-hide the MBMS streaming APIs

Merge "Javadoc: Added alternative to Package verification." into oc-dev

Merge "Fixed the icon clipping of the notification shelf" into oc-mr1-dev

Merge "Add pseudo-filtering on authenticated views." into oc-mr1-dev

Merge "Make sure top activity is stopped on sleep if paused." into oc-mr1-dev

Merge "Update QS if the content gets stale" into oc-mr1-dev

Follow-up on bug #65298216: DeviceIdleController needs to...

...watch interactivity, not display state

Fix locking.

Bug: 65298216
Test: manual
Change-Id: If761ce73c82c23295720aea73c3a7fdda40ceedf

Fixed a flakey test

Since the test could be off the UI thread, cancelling
would not be synchronized properly.

Test: runtest -x packages/SystemUI/tests/src/com/android/systemui/statusbar/notification/NotificationInflaterTest.java
Change-Id: Icfb02b637b76cdfb853c88130eabddf75203909b
Fixes: 65031224

Merge "Make settings themed apps have light nav DO NOT MERGE" into oc-mr1-dev

Merge "Define priority for video threads" into oc-mr1-dev

Merge "Fixes recents entrance animation when launching before onStop happens" into oc-mr1-dev

Merge "OemLockService: remove canUserAllowOemUnlock()" into oc-mr1-dev

Merge "Fixed CharSequenceTransformation to use find() instead of matches()." into oc-mr1-dev

Camera: fix race in onRepeatingRequestError callback

Test: CTS
Bug: 64180296
Change-Id: I72b76695d580d2cfddb02a6da9d7ab04cdcfb98a

Don't log TYPE_DATASET_SELECTED on dataset authentication.

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

Bug: 65007529
Fixes: 65403592

Merged-In: I98f1198cbe1836c8154b73a1227be4f6947abc19
Change-Id: I98f1198cbe1836c8154b73a1227be4f6947abc19

Make settings themed apps have light nav DO NOT MERGE

This is needed to help with differential aging.

The nav needs to be white with a 1dp divider on it, so add support
for the divider and add hidden attribute to set the nav buttons
inverted.

Change-Id: I4a5329f7486a6774ca4de8362caebbe8ba421aad
Test: Open settings
Bug: 63630024

Merge "Always treat password as sufficient if no password policy is set" into oc-mr1-dev

Merge "Cat rescue." into oc-mr1-dev

Fix NativeCallbackThread race condition.

Bug: b/65286959
Test: instrumentation
Change-Id: I0086a362b9b916d87118ed478aa8222874df19b1

OemLockService: remove canUserAllowOemUnlock()

No longer used by Settings as the logic was wrong.

Test: gts-tradefed run gts -m GtsOemLockServiceTestCases
Bug: 65124732
Change-Id: I44e5f697aabd2b5eefecf64060502b5c9ef5f911
(cherry picked from commit d37fe2d3e1cbf03161df9f4e16f791a99445cdc1)

Update QS if the content gets stale

For system tiles this is more often, but for 3P tiles its pretty rare
except for on boot.

Test: runtest systemui
Fixes: 34804413

Change-Id: I166e2ab6a609acd63b9b4f19579bab12beabba27

Do not process set WindowAnimator param when screen is already on.

Doing so can lead to the param be processed during onPause,
potentially causing the device erroneously waking up.

Change-Id: Id0f0bc74873e6e41bd2b10dc423a435ca050947f
Fixes: 65412431
Bug: 65385210
Test: cts/hostsidetests/services/activityandwindowmanager/util/run-test CtsServicesHostTestCases android.server.cts.ActivityManagerActivityVisibilityTests#testTurnScreenOnActivity_withRelayout

Merge "skip filtered entries instead of returning" into oc-mr1-dev

Always treat password as sufficient if no password policy is set

Handle the special case when work profile is freshly created on a FDE device

Test: cts-tradefed run cts-dev -m CtsDevicePolicyManagerTestCases
      -t com.android.cts.devicepolicy.MixedManagedProfileOwnerTest#testPasswordSufficientInitially
Bug: 63887564
Change-Id: Ie8e430d4ff63be74fb2d4fc3ad3a8735f1de48b0

Merge "Add method to update tile's remoteview using summaryUri" into oc-mr1-dev

Merge "Alternative fix for preventing PiP from screenshot sharing." into oc-mr1-dev

Merge "Add spacing in QS footer and user switcher." into oc-mr1-dev

Merge "Add username beside user switch." into oc-mr1-dev

Merge "Make ASSIST launch VIS and add a new keycode for TV (ALL_APPS)" into oc-mr1-dev

Merge "ONA: Make ONA related broadcasts protected." into oc-mr1-dev

Merge "Always enable nav and status bar opaque/translucency" into oc-mr1-dev

Merge "Javadoc update for permission requirements" into oc-mr1-dev

Merge "Fixes seascape recents task centering for low ram devices" into oc-mr1-dev

Define priority for video threads

Video threads have soft realtime requirements that
must be met in order to maintain reliable frame delivery
even when the system is under high load. A new priority
has been defined that can be used by video threads to
enable them to be scheduled appropriately relative
to other system threads. This CL updates the media codec
looper to use the new priority.

Tests: GTS media tests, Manual verification with
Play Movies, Netflix, com.google.vr.sdk.samples.videoplayer

Change-Id: Ibfee09bb6a5359ab96ec2184775c486e4bd77871
related-to-bug:63898080

Merge "Updates the background of the clear all button for low ram devices" into oc-mr1-dev

Merge "Changed the accessibility experience for notifications" into oc-mr1-dev

Fixed the icon clipping of the notification shelf

Before the icon wasn't clipped to it's notification container
and could go out of bounds. We're now clipping it to it's
surrounding notification.

Change-Id: I1b9a3650eb3e3f0bcd7d7cef087afb7cc41ccf62
Fixes: 63477094
Test: add notifications, observe icon animations, better clipping

ONA: Make ONA related broadcasts protected.

Bug: 65257065
Bug: 37357441

Test: build with ONA, check that broadcasts do not log WTF errors.
Change-Id: I623da59f5cc7ca3feff6cc156664c05fbeb34107

Merge "Improve NewDatabasePerformanceTests" into oc-mr1-dev

Alternative fix for preventing PiP from screenshot sharing.

- Revert 942a85c6d608dd6a4a31937fe0b57ae43105d5b2
- Use a more specific fix for the triggering of PiP from the SysUI
  screenshot share intent.

Bug: 63984385
Bug: 63581685
Test: android.server.cts.ActivityManagerPinnedStackTests
Test: Click share on the screenshot action, ensure activity below does
      not enter PiP
Change-Id: Id8041d50dd429030c20940dca19e5c62bbc0c6a4

Merge "Allow using SQLiteGlobal for testing" into oc-mr1-dev

Javadoc update for permission requirements

Bluetooth scanning requires holding these permissions for results
to be delivered.

Bug: 65013767
Test: N/A
Change-Id: I0b5fa9efa7fc8d5cff25319fbd7719cedee6a4aa

Always enable nav and status bar opaque/translucency

Removed the logic to enable or disable translucency and have it fully
enable all the time. Having it on has very or no impact on RAM.

Test: boot device and look at nav and status bar
Change-Id: Id9e613abc689ab8c1b53424aa92e864f75f8ed02
Fixes: 65286386

Make ASSIST launch VIS and add a new keycode for TV (ALL_APPS)

    - KEYCODE_ASSIST: Looks for default VIS in system partition
      and launches it.
    - Ensure foreground activity assist context extras are collected
      and sent to VIS.
    - KEYCODE_ALL_APPS: Sends Intent.ACTION_ALL_APPS. For TV,
      this targets default launcher package in system partition.

Test: Manual: ASSIST: adb shell input keyevent 219
Test: Manual: ALL_APPS: adb shell input keyevent 284
Test: also tested with --longpress for these keys
Test: Manual: Checked with dummy app populating overridden
      onProvideAssistData()
Test: Keypresses after overriding ALL_APPS and ASSIST
      on Nexus Remote

Bug: 37950079
Bug: 38496261
Change-Id: I96d9de67d4a442a8c6cb7feec28fc9aeb0dcbc18

Merge "Bluetooth: add metrics for pairing with devices without names" into oc-mr1-dev

Changed the accessibility experience for notifications

The expand button is now not labeling the notification
anymore since it was too wordy.

Change-Id: I14374b1a97389b8441fdf44eb4993954e765e89f
Fixes: 65123834
Test: use talkback and navigate to expand button, verify not speaking notification anymore

Fixed CharSequenceTransformation to use find() instead of matches().

Test: cts-tradefed run commandAndExit cts-dev -m CtsAutoFillServiceTestCases -t android.autofillservice.cts.CharSequenceTransformationTest#testReplaceAllByOne
Test: cts-tradefed run commandAndExit cts-dev -m CtsAutoFillServiceTestCases

Fixes: 65459117

Change-Id: Ib5ff31b6b12e5583e736a0593e941e1556ae7384

Fixes seascape recents task centering for low ram devices

Added missing system insets left to the calculation.

Test: manual - go to recents and rotate to seascape
Change-Id: I25cd2f35b81e350e2934859423985257c4d6daea
Fixes: 65420616

Improve NewDatabasePerformanceTests

They didn't properly report timings. It included setup time.
Also cleaned up code and added CREATE TABLE performance test.

Cherry-pick from 66c1852cced1c09f2dc7d536efd875c44d9a1cfe

Test: run_newdb_perf_test.sh
Bug: 63942513
Bug: 65302651
Merged-In: I15fcb5fa43f81e8ea39c2e29252527a8f3714398
Change-Id: I15fcb5fa43f81e8ea39c2e29252527a8f3714398

skip filtered entries instead of returning

When retrieving all packages / apps on the system, we should just
skip over any filtered entries rather than returning 'null'.

Change-Id: If874be3904b6d5a1103e099eb2ddff839091121e
Fixes: 65389431
Test: Manual:
Test: bit FrameworksCoreTests:android.content.pm.PackageManagerTests\#testGetInstalledPackages ; passes
Test: adb install --instantapp somepackage.apk
Test: bit FrameworksCoreTests:android.content.pm.PackageManagerTests\#testGetInstalledPackages ; passes

Make sure top activity is stopped on sleep if paused.

Currently we have logic in place to handle moving a top resumed
activity to paused and eventually stopped if entering sleep. However,
we do not handle the case when the top activity is already paused.

This changelist addresses this issue by identifying this condition
and moving the activity to the stopped state.

Change-Id: I8d6f9282ff863cf8017f6d64ab60911fb1c376c2
Fixes: 65457314
Test: cts/hostsidetests/services/activityandwindowmanager/util/run-test CtsServicesHostTestCases android.server.cts.ActivityManagerDisplayTests#testExternalDisplayActivityTurnPrimaryOff

Merge "Fixes clear all button disappearing after rotate on low ram device" into oc-mr1-dev

Merge "Finishes the session when the service returns a null response on authentication." into oc-mr1-dev

Bluetooth: add metrics for pairing with devices without names

Bug: 34685932
Test: make, unit test
Change-Id: I486c210790fe72a594d6144dfd17b01a7a4a9003
Merged-In: I486c210790fe72a594d6144dfd17b01a7a4a9003

Merge "Fix crash in presubmit tests" into oc-mr1-dev

Fix crash in presubmit tests

Test: runtest systemui
Bug: 65289824
Change-Id: Ibbce459e5e87ea667e91d546b7568f9addbb1c66

Merge "Bluetooth: Thread-safe binder invocation" into oc-mr1-dev

Merge "Introduced AppRestrictionsLock" into oc-mr1-dev

Merge "Always add local subnet routes to the interface's routing table" into oc-mr1-dev

Added more logging to diagnose a racy runtime restart.

Also fixed a possible NPE on saveLocked() and improved locking.

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

Merged-Id: I4b8368a9d19b4b4da76533dadb013ff2e2922955
Change-Id: I4b8368a9d19b4b4da76533dadb013ff2e2922955

Merge "Fix presubmit tests" into oc-mr1-dev

Fix presubmit tests

Test: runtest systemui
Bug: 65289824
Change-Id: Ibf2b61670d09a76eb908bd9d322ae36e0f7af3e0

Merge "Correctly block system work profile notifications." into oc-mr1-dev

Merge "Ensure *# are included in TTS spans generated with PhoneNumberUtils." into oc-mr1-dev

Add method to update tile's remoteview using summaryUri

- Calling com.android.settingslib.drawer.TileUtils#updateTileData no
longer sets the summary in RemoteView.
- To update summary and title of remoteview,
com.android.settingslib.drawer.TileUtils#updateTileUsingSummaryUri
should be called

bug: 62713030
Test: Added robolectric tests, also manually tested settings
Change-Id: I8c63601a901d5b140e36baec424c226724c43bc4

Merge "Ensures zen rule update locale retains enabled val" into oc-mr1-dev

Always add local subnet routes to the interface's routing table

For some networks such as mobile data connections, its LinkProperties
does not contain routes for the local subnet so no such route is added
to the interface's routing table. This can be problematic especially
if the device is in VPN lockdown mode where there exists high-priority
PROHIBIT routing rule which in turn blocks the network's default gateway
route from being added (next hop address hitting the prohibit rule).

We fix this by patching LinkProperties to always include direct connected routes
when they are received by ConnectivityService. This has the added advantage that
when apps get LinkProperties, they see the directly connected routes as well.

Bug: 63662962
Test: runtest frameworks-core -c android.net.LinkPropertiesTest
Test: runtest frameworks-services -c com.android.server.ConnectivityServiceTest
Test: Start with device with mobile data, set up ics-OpenVPN in always-on
      lockdown mode. Turn off mobile data then turn it back on, observe
      mobile data connectivity is restored and VPN successfully reconnects.

(cherry picked from commit 1bb5c0818f0c4fb426e13b65a3ba3db7f36c3d88)

Change-Id: Ia14f88bcf49d37286519c26dff6b7180303e2cbe

Merge "Fix disabled views with nearest touch frame" into oc-mr1-dev

Merge "Support NAT keepalives" into oc-mr1-dev

Support NAT keepalives

Test: as follows
    - built
    - flashed
    - booted
    - "runtest frameworks-net" passes
    - turned on debugging and walked around watching IpReachabilityMonitor
      force neighbors into NUD_PROBE state without error
    - observed NAT callbacks happening without any reported errors
    - watched:
        adb shell cat /proc/net/nf_conntrack | egrep '192[.]168[.]43[.]' | sort -n -k5
      for correct timeout updates

Bug: 29337859
Bug: 32163131

Change-Id: Icb23da64cfaa3a19f7bc75fba426a52b0994fb0f

Bluetooth: Thread-safe binder invocation

* Binder object may become null between null check and actual invocation
  if using a instance private variable assignable by service connection
  callbacks
* The solution to this problem without locking is to assign existing
  binder variable to a local final variable before the null check
* Any further invocation to a disconnected binder object will result in
  RemoteException that is caught by the try-catch block
* Read and write to volatile variable is always atomic and hence thread-safe
* Removed unnecessary synchronization in BluetoothAdapter constructor
* Private mConnection objects should be final
* Simplfied several return statements where booleans can be returned
  directly
* Removed unnecessary catches for NPE since there won't be any

Bug: 64724692
Test: make, pair and use devices, no functional change
Change-Id: Ifc9d6337c0d451a01484b61243230725d5314f8e
(cherry picked from commit 16eeac356c68f3674677169b618774718ec982b2)