Merge "Fix issue with deferred start of the background loader." into oc-dev

Merge "Fix a race condition in the A2dpProfile constructor" into oc-dev

Merge "VolumeShaper: Enable xOffset from Java" into oc-dev

Merge "Unbreak APK 26 apps running on "O" preview/beta builds." into oc-dev

Merge "Unbreak even more APK 26 apps running on "O" preview/beta builds." into oc-dev

Merge "Enable starting window logging" into oc-dev

Enable starting window logging

Also extend it to make sure we catch the bug!

Test: Open a couple of cold/hot apps, make sure logs look accurate
Bug: 37888853
Change-Id: Ied394969748d4d2d40359af15e0b491a2dc2b078

Merge "Fix refactoring typo" into oc-dev

Merge "Captive portals: reinspect http result after fallback probe" into oc-dev

Captive portals: reinspect http result after fallback probe

When DNS queries that precede the http and https probe take a long time
to complete, it is possible that the fallback probe fires earlier than
either the http and https probes.

This causes the detection logic to ignore the result of the http probe,
which may cause the system to miss captive portals advertising
themselves on the url used by the http probe but not on the url used by
the fallback probe.

This patch fixes that issue by forcing the detection to wait on the
result of the http probe if the fallback probe does not find a portal.

Bug: 38259299
Bug: 36532213
Test: manually tested with nearby captive portal networks
Merged-In: I07ea23a2b5f694c5ada9633169af98409efedff1
Change-Id: Ie016cad52719d07167acf026f738d31000c78cd2

(cherry picked from commit fc47442b31761e64e188c51c3dd2a782ecbd9c62)

Merge "Optimize latency when unlocking phone" into oc-dev

Merge "WifiManager: add unspecified ip mode" into oc-dev

Merge "Don't report displays that are going to be removed" into oc-dev

VolumeShaper: Enable xOffset from Java

Previously only accessible from native.
Also improve documentation and comments.

Test: CTS VolumeShaperTest, Ducking
Bug: 38353147
Change-Id: I27bb34f0a5d28f80d138111bbeeb92653b5195c5

Fix for Bluetooth not restarting on User Switch

The flag that keeps track of the Bluetooth adapter status(mEnable) is
not guarded correctly and could go out of sync with the adapter status.
This causes User Switch handling to not function correctly since it
checks for the status of the mEnable flag before restarting Bluetooth.
This fix queries the Bluetooth Adapter for the status instead of using
the mEnable flag.

Bug: b/37648701
Test: Tested switching users multiple times in both phone and carkit.
Also tested pairing and connection for sanity.

Change-Id: I17facecf8e1de90df11c11d5b0a184f48a0356e4
(cherry picked from commit 7d1203a2f4fdbafe26d33421c3cdd83f07a5fba8)

Merge "Fix GraphicBuffer leaks in system_server" into oc-dev

Merge "Reset hidden state of AppToken that was cleared due to starting window" into oc-dev

Fix refactoring typo

Seems like when we renamed/rearranged the methods we didn't ignore
mPolicyVisibility..and that's the most important part!

Fixes a bug where the dock divider isn't animating when unlocking
the phone.

Bug: 36200726
Change-Id: I4c5e9ca7f1b6e7bf21f793e1cf39d92fe77489e8

Unbreak even more APK 26 apps running on "O" preview/beta builds.

Bug: 38391358
Test: builds
Change-Id: Ic48f9cb53da3ed8241e5d40d3096304ee4dfebd2

Optimize latency when unlocking phone

Latency when unlocking the phone regressed a bit for two reasons:
- For lockscreen -> app we now have to create a full starting
window containing the snapshot, while previously this was just
showing a surface.
- For lockscreen -> home, we can't use the saved surface anymore
because currently we don't support snapshotting translucent
activities. However, in the long term, we want home screen to be
more involved into transitions anyways, so we'll have to wait for
the first frame draw anyways.

However, crystal ball trainee developer Jorim added some
artificial latency in this transition 3 years ago, because he knew
that it is going to be an issue at some point so we have some
headroom to improve! Genius! On a more serious note, it was because
he didn't understand how to read systraces with binders involved (to
be fair, there was also no binder tracing).

Now, we can completely fix the introduces latencies above by
removing this latency of 100ms, and we are 30-70ms better than
before! However, this requires a lot of discipline in SystemUI.
Currently, the callback to dismiss Keyguard takes around 30ms. By
moving all non-essential binder calls of the main thread or to the
next frame, we bring this down to 5ms, such that window animation
and Keyguard animation starts about at the same time.

Test: Take systrace, unlock phone...profit!

Fixes: 38294347
Change-Id: I481fe4ecf358ed09f7142dd9e7ecd290b53c500c
Merged-In: I3ea672bc2eca47221bc6c9f3d7c56b6899df207d

Unbreak APK 26 apps running on "O" preview/beta builds.

Bug: 38391358
Test: runtest systemui-notification
Change-Id: Ib78dd3e2a3dcb16faae33a0d5a07475187fb3463

Fix a race condition in the A2dpProfile constructor

When constructing A2dpProfile objects, we were creating an
A2dpServiceListener before initializing the mWrapperFactory instance
variable that it depended on. If the onServiceConnected callback happens
immediately after the listener is created, we'd get a null pointer
exception. The fix is to move initialization of mWrapperFactory up before
the construction of the A2dpServiceListener.

Bug: 38391718
Test: run boot tests
Change-Id: Ib0be947099862b19da7737b54b99b944241b605f

Merge "AAPT2: Respect format attr in <array> resource" into oc-dev

Merge "AudioFocusRequest: longer code sample, javadoc fixes" into oc-dev

Fix GraphicBuffer leaks in system_server

IWindowId is being kept alive by binder in certain cases, which
will leak WindowState, which can leak a whole lot including
the snapshot GraphicBuffer.

- Fix a leak in WindowContainer.mTmpChain1/2
- When the window gets removed, make sure to release the reference
to the window state so even if Binder keeps IWindowId alive, we
don't keep WindowState alive
- Make sure that even if we leak TaskSnapshotSurface we don't leak
the GraphicBuffer.

Test: Boot, some basic sanity testing.
Bug: 36279079
Change-Id: I4a0a784b32bc2df47934b1bed1d62b0682beb2dd

Merge "Match attributes to actions for pip a11y" into oc-dev

Merge "Add config option to enable/disable pause workaround." into oc-dev

Merge "We notify VR Core when a 2D intent is running in VR in persistent mode." into oc-dev

Merge "AAPT2: Update locale script filtering logic" into oc-dev

Merge "Fix call to get the current user id from a secondary SystemUI process." into oc-dev

Merge "Add a "Use high quality audio" option to Bluetooth A2DP device settings" into oc-dev

Merge "Purge Nonexistent User Jobs on Boot" into oc-dev

Merge "Stop system process crash in TouchExplorer" into oc-dev

Merge "Optimize App -> Recents" into oc-dev

Merge "WifiP2pWfdInfo: Only pass the wfd info to WifiNative" into oc-dev

Merge "Fix TimePicker crash on rotation." into oc-dev

We notify VR Core when a 2D intent is running in VR in persistent mode.

One of the paths that sends these notifications incorrectly indicated
that a VR app was not running. Fix the callsite to include the current
VR component instead of null.

Bug: 38004624
Test: Launch VR app, verify null component is not sent to VR core
while it's running.

Change-Id: I0589ee51952a503bfddec214999ed5bce48715ba

Fix call to get the current user id from a secondary SystemUI process.

- The secondary SystemUI process's KeyguardUpdateMonitor is not updated
  so we can't rely on it to get the current user id.

Bug: 38372598
Test: Switch users, launch some tasks, and switch back. Ensure that original
      tasks still exist

Change-Id: I26a7c70b5f7032a106dc1342c8b366e4797e8fa0

Merge "Fix broken LoaderLifecycleTest#loaderIdentityTest" into oc-dev

Stop system process crash in TouchExplorer

We need to straighten out this state machine so we can
fix bugs in it without a high risk of creating others
(b/38246304). In the meantime, catching this exception at
least allows the device to keep operating.

Bug: 37338581
Test: Reproduced the conditions of the crash. Verified that
we now print the log message.

Change-Id: I1f4f84b5529b2bf638e225d474808e3d42484e78

Reset hidden state of AppToken that was cleared due to starting window

If a starting window is tranferred to a token from another token that is
visible, we also make the receiving token visible, however we don't
reset the start when the starting window is tranferred out again when no
other windows were added to the token. We now do.

Fixes: 38317227
Test: Launch Audible app music player and verify the LibraryActivity
token isn't in the visible state.

Change-Id: I0186c79aa24d64362f742229d60c75eb0c1a3f97

Add config option to enable/disable pause workaround.

Test: Manual/unit
Bug: 35304446
Change-Id: I9a0c73ce62812a87e30d9759a0d205ffa282caf2

Merge changes I858017b8,Ife32b600 into oc-dev

* changes:
  Improved the media color extraction algorithm
  Fixed broken coretests

Merge "PackageManager: Change post-install GC request" into oc-dev

Fix TimePicker crash on rotation.

We were using two different types of layouts in landscape and portrait.

I unintentionally changed it from RelativeLayout to LinearLayout originally,
without even removing relative layout params.

Bug: 37650833
Test: Manual check that TimePicker no longer crashes on device rotation
Change-Id: If951e83c32377c0ed52d553ceb80683c4fba8b85

Merge "Fingerprint: call onRemovalSucceeded with # of remaining templates." into oc-dev

Optimize App -> Recents

- Use same strategy to draw thumbnail bitmap by moving
startActivity off from the main thread and start caching the future
when AM is busy executing startActivity.
- Move some binder calls off the main thread.
- Add some trace points for better analysis in the future.
- Make toggleRecentApps asynchronous so we don't have to wait on
the next app-vsync - it's totally not needed.

Test: com.android.apptransition.tests.LatencyTests

Fixes: 32668632
Change-Id: Id6483e26e9d6c1e319bceaa8268da976cedfbca3
Merged-In: I15db41f2e821779972ab3b430033c70aa1dfd907

WifiManager: add unspecified ip mode

Add an unspecified IP mode that allows WifiServiceImpl to clear state
in the interface-based IP mode tracking.

Bug: 31466854
Test: frameworks/base/wifi/tests/runtests.sh
Change-Id: I15485cdab95ed4308cf5c2877fba40eac82bf162

Merge "Revert "Fix recent apps in system UI for Skia pipeline"" into oc-dev

Merge "Set notification channel to MTP notification." into oc-dev

Merge "Add more JSDoc about ErrnoException" into oc-dev

Set notification channel to MTP notification.

Bug: 38252594
Test: Manually connect MTP device to Android
Change-Id: Ib7ab12b2401d36dbb4cca2fd9c18cbd3505ed9ef

Merge "AudioEffect: return DEAD_OBJECT for FAILED_TRANSACTION" into oc-dev

Merge "Fix issue #38244875: Bring back testChargingConstraintFails" into oc-dev

Purge Nonexistent User Jobs on Boot

In the case that a user has been removed but their jobs still exist on
disk, the JobSchedulerService will remove all jobs not associated with
current users on boot.
Exposed UserManagerService#getUserIds() via UserManagerInternal for
quick user id retrieval.

Fixes: 38261977
Test: manual

Change-Id: Id4b3c0a4142b4818fcd875eef18ea03f3c45ca40
Signed-off-by: Michael Wachenschwanz <mwachens@google.com>

Revert "Fix recent apps in system UI for Skia pipeline"

This reverts commit 625dd56a45bfe95c5f1baa1891529503ff3374a9.

Reason for revert: Caused a memory leak, b/38330767
Bug: 38136140
Bug: 38330767
Test: manual, verified memory isn't leaking doing the steps in b/38330767

Change-Id: I998bea04788d58ba6bad71c1691d5a3b33190c1b
Merged-In: I98b2dfd750be57a15785808e2d5723616e2ce20a

Fix issue with deferred start of the background loader.

- Previously we were (incorrectly) assuming that the creation of the
  BackgroundTaskLoader would be followed synchronously by a call to start()
  the background loader itself.  However, now that we post the start of
  the loader, the run() call can be made before the start() call.

Bug: 38310660
Test: This is an intermittent bug, hard to repro.
Change-Id: I4dfd18b8c3c127429c790177e98ca41ec70f493d

Merge "Updating Overview to work with PiP" into oc-dev

AudioFocusRequest: longer code sample, javadoc fixes

Test: make -j offline-sdk-docs
Bug: 38124458
Change-Id: I18e83175f6919fd6afeae9475c00eded829cb96a

Merge "Discover primary service by UUID for PTS tests (4/4)" into oc-dev

Merge "Use ArrayList as children" into oc-dev

Merge "Don't specify screen orientation for ReceiverActivity." into oc-dev

Merge "Specify behavior around DocumentsProvider#openDocument()." into oc-dev

AAPT2: Update locale script filtering logic

AAPT got an update to its locale script filtering logic that
dealt with enabling the removal of specific locales like
b+sr+Latn. Update AAPT2 to do the same.

Bug: 28055669
Bug: 29412034
Test: aapt2_tests
Change-Id: I3315c0814796cff05dfa1032d66d0306fe2b12bc

Merge "ViewRootImpl: Guard against null in performMeasure" into oc-dev

Merge "Fixed leak of app token in exiting list." into oc-dev

Merge "Fix interaction of magnification and Final-crop." into oc-dev

Merge "Restrict access to instant app data in usage stats" into oc-dev

Merge "SurfaceView: Reset window stopped state when attaching to window." into oc-dev

Don't report displays that are going to be removed

When display is removed from system, it is immediately removed from
AM, but removal can be deferred in WM if there is an active animation
on that display. If AM then requests display ids in focus order from
WM, it might get a display that was already deleted in AM.
This CL skips displays marked for deferred removal.

Bug: 38166277
Test: DisplayContentTests#testDontReportDeferredRemoval
Change-Id: I49544963f476fc58a609094781ed46541af8a238

Discover primary service by UUID for PTS tests (4/4)

Bug: 38123054
Test: manual
Change-Id: I89e088e60f4325f1ece59d22efda0c907a3b716a
(cherry picked from commit fe2bf16a2b287c3c748cd6fa7c14026becfe83ff)

Merge "Always add the instant app installer" into oc-dev

Merge "Fix broken retaildemo tests" into oc-dev

Fix broken LoaderLifecycleTest#loaderIdentityTest

Bug 37994415

We have a preemptive check for getLoaderManager() to ensure that
the host is set before returning a value. In this test, the host
was not set because the fragment was restored and on the back stack.

The solution is to pop the back stack to ensure that it is added
so that it comes up to the proper state before getting the loader
manager.

Test: ran the broken test
Change-Id: Ic91cc5ca9468da22bdfef28cda1d4488b2f737f0

Fix issue #38244875: Bring back testChargingConstraintFails

- Add new "get-job-state" shell command that the tests can use
  to find out what is going on with a job.
- Fix an issue in the battery controller where it would not
  immediately stop with a "battery not low" constraint when
  the battery becomes low.
- Also an optimization to rescheduling jobs, for every scheduled
  job that is not ready don't look to see if it is active.
  Instead, go through the active jobs and see if any are no
  longer ready.

Test: bit -t CtsJobSchedulerTestCases:*

Change-Id: I5611886653258ca337eee97c5ee1b9b3dfdb6d85

Specify behavior around DocumentsProvider#openDocument().

Add supported modes and contracts on receiving unsupported modes.
Test: Builds.
Bug: 11279339

Change-Id: I8c403f6168baadb354813d4c1e630bb8662ddb34

Fixed leak of app token in exiting list.

- Remove app token from exiting list when it's last window is removed.
- Remove app token from exiting list when it's removal isn't delayed.

Test: Launch a few apps and press back and make sure the exit app to
list dump is empty.
Fixes: 38337456

Change-Id: Ia2f22c6f1add5f867e25583ec38ffccc71217ee7

Use ArrayList as children

Such that traversal is O(n) instead of O(n²)

Test: WindowContainerTests
Bug: 32668632
Change-Id: Ie52026a76a0f65c71ff6bd35d13b3d0c5d44bb3f

Merge "Check for mAttachInfo nullness when getting the autofillable drawable." into oc-dev

Merge "Optimize hot launching recents" into oc-dev

AudioEffect: return DEAD_OBJECT for FAILED_TRANSACTION

Test: CTS AudioSecurityTest
Bug: 38346284
Change-Id: I96b4995c2d11b4a33c4b2fa6987922e26c91d336

Restrict access to instant app data in usage stats

- Events are obfuscated based on whether the app was instant or not at
the time each event was logged.

- UsageStats are obfuscated based on whether each app is instant or
not at the moment.

Bug 38202133
Test: Manual test using UsageStatsTest and instant apps

Change-Id: I3c74309196b88d010d317cb0dd6749bf4624e876

AAPT2: Respect format attr in <array> resource

Bug: 38152130
Test: make aapt2_tests
Change-Id: I84f352afb1a8fd2f329354f789aaa36c5ef88e47

Always add the instant app installer

Even if we filter out an instant app because the caller doesn't
have access to it, always make sure to add the installer when
we have the possibility of an instant app.

NOTE: This does not have CTS tests since it's impossible to test
without involving the resolver. CTS tests will be added in MR1+
when we can make changes to the APIs and add the necessary methods
to mock out the resolver service.

Change-Id: I92c88eb04520c0c5e0c58db6d668ed0a0e552a35
Fixes: 38268052
Test: cts-tradefed run commandAndExit cts-dev -m CtsAppSecurityHostTestCases -t android.appsecurity.cts.EphemeralTest

Merge "Remove deprecated recommendation request code." into oc-dev

Merge "catch USER_ALL in badging check" into oc-dev

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

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

Fix broken retaildemo tests

Bug: 38162695
Test: runtest frameworks-services -c
com.android.server.retaildemo.RetailDemoModeServiceTest &&
runtest frameworks-services -c
com.android.server.retaildemo.PreloadAppsInstallerTest

Change-Id: Iac60202414753b3e04e89cb29271cbd150a3acdd

Merge "ignore IllegalArgumentException during uninstall" into oc-dev

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

Add a "Use high quality audio" option to Bluetooth A2DP device settings

This change adds a checkbox reading "Use high quality audio: <codec
name>" to the details dialog for A2DP audio bluetooth devices that
support codecs other than the mandatory SBC (eg AAC, LDAC, aptX, etc.).

Bug: 37441685
Test: manually

Change-Id: I3eed50b9cced6f9f7f4efd70955de1bf9c309346

Check for mAttachInfo nullness when getting the autofillable drawable.

Test: manual verification on Caviar (automated test will be added later)
Test: CtsAutoFillServiceTestCases pass

Bug: 38341498
Fixes:  38323841

Change-Id: I15cc792de87987cc19a229c2ab2dfc317877f7ec

catch USER_ALL in badging check

Bug: 38264652
Test: runtest systemui-notification
Change-Id: Id482ed4984eef83ad0e9876acda5addb65aca5f9

PackageManager: Change post-install GC request

Only run a blocking GC when updating. Otherwise, only request a
concurrent GC to clean up.

Bug: 38259387
Test: m
Test: Device boots
Test: Install works
Change-Id: Id1a0e956f26e166d927f833075bc4807f32b94f7

Merge "Invoke BLKSECDISCARD to securely delete sensitive data" into oc-dev

WifiP2pWfdInfo: Only pass the wfd info to WifiNative

WifiP2pWfdInfo.getDeviceInfoHex() is used to set the WFD info in
wpa_supplicant. The length of this WFD info will be calculated and
prepended by the HIDL interface itself, so there is no need for this
length to prepended in the framework. So, get rid of it.

Bug: 38222615
Test: Compiles
Change-Id: If8ff6e558dad974a053265e0a8c8222943065823

Merge "Also add starting window when activity is not alive" into oc-dev

ignore IllegalArgumentException during uninstall

It's expected and normal to throw an IllegalArgumentException when
trying to uninstall a package that doesn't exist. Ignore it if it
gets thrown.

Bug: 38197362
Test: bit FrameworksCoreTests:android.content.pm.PackageManagerTests#testFlagFManifestI
Change-Id: Ie0d90010678b09c91747f6e5c4655691951ad3a9

Merge "Remove internal listener when cloning" into oc-dev