Merge changes I7e9aea3c,I2d23aabf

* changes:
  Remove content description from QS pages
  Fix random clicks in QS

Merge "USB audio: Use 'final' where possible"

Merge "Comment out unused methods"

Merge "Fix typo in log"

DO NOT MERGE Change permissions review mode check. am: 59bd89fa29 am: 97e12736ce
am: 501c3d1005  -s ours

Change-Id: I6bc9969659e5de514c2e2fcbb6bb0de9d96a8090

DO NOT MERGE Change permissions review mode check. am: 59bd89fa29
am: 97e12736ce

Change-Id: Ib273a125444db99305bdf84eaf2c98ceb2ff5717

DO NOT MERGE Change permissions review mode check.
am: 59bd89fa29

Change-Id: Icf3fa8e171d68e6878001529a496875f52f38909

DO NOT MERGE Change permissions review mode check.

In ag/1835531 we introduced a check for permissions review mode,
that relies on an overlay resource. Since we can't use overlay
resources for the Unified Build, this CL temporarily introduces
the checking for a build property as well. This possibly won't work
for O/G, but will work for the Feldspar release of the Unified build.

BUG: 35028846
Change-Id: I4c82f242bc680028ecd872589552534a419f0152

Merge "A11y updates for new a11y stream."

Merge "Fix stale data due to stopped loader."

Merge "IP connectivity metrics: update ipconnectivity.proto"

Merge "switch to new SkPathEffect factories for Compose and Sum"

Merge "MathUtils: Remove static Random field."

Merge "CaptivePortalLoginActivity correctly gets UserAgent" am: 4903487fd9 am: 43961ee6ae
am: 4f9e934c5f

Change-Id: I234d55c758c7538717d27035fdc8864a8650d75c

Merge "CaptivePortalLoginActivity correctly gets UserAgent" am: 4903487fd9
am: 43961ee6ae

Change-Id: I8f43e7868ce43e6d1837900cfd131f556832f1a8

Merge "CaptivePortalLoginActivity correctly gets UserAgent"
am: 4903487fd9

Change-Id: Iea24e914cab4128464f8866a4b9ae47c7c5325bf

Merge "CaptivePortalLoginActivity correctly gets UserAgent"

Merge "Remove interfaceChain from IServiceManager::add()" am: 4a75e32f1d am: 7665332a7a
am: 7669e41d8a

Change-Id: Ide4a33b8532954659a9c7c997d8d5203279a270e

Merge "Remove interfaceChain from IServiceManager::add()" am: 4a75e32f1d
am: 7665332a7a

Change-Id: I9f5187d9152e151afbf3fd165e4b6f58b4ac9638

Merge "Remove interfaceChain from IServiceManager::add()"
am: 4a75e32f1d

Change-Id: I9443e22ed730c44abc6fd6f697f8b3f04b667f37

Merge "Remove interfaceChain from IServiceManager::add()"

CaptivePortalLoginActivity correctly gets UserAgent

Change I33cbcdab9809c712d998da6d925bdd4e3093f8ff added to the intent
passed from NetworkMonitor to the CaptivePortalLoginActivity an extra
argument to set the UserAgent header of probes done by the
CaptivePortalLoginActivity.

However that extra String argument is not correctly retrieved from the
Intent and instead a null pointer is returned by getParcelableExtra().
Consequently the UserAgent header field is never set.

This patch fixes this bug and replace getParcelableExtra() with
getStringExtra().

It also adds logging of the request header and response header for every
network probes sent by CaptivePortalLoginActivity.

Bug: 35937367, 32369183
Test: manually tested
Change-Id: If5c463023474ffb0401f3cda5875ccd9fc6f79f9

IP connectivity metrics: update ipconnectivity.proto

This patch adds several new fields to ipconnectivity.proto for
improving APF metrics, connect metrics, and link_layer inference.

Test: build, flashed, ran $ adb shell dumpsys connmetrics list proto
      updated metrics unit tests depending on proto schema
Bug: 34901696
Change-Id: I6f68e16f583a6b407f33f8a500133e100f3453c7

Two fixes on autofill save.

1.Fixed race condition on save request - if the service was unbound, it would
  mark the RemoteFillService as completed and the retry would be dropped.
2.Removed session when save request is canceled.

Change-Id: Ib71514cdcfecc38c07dd7656174d0db68ed758d1
Fixes: 36001206
Test: CtsAutoFillServiceTestCases pass

Merge "Hook up screen brightness knob to DisplayPowerController."

Merge "Add OverlayManagerService#setEnableExclusive"

Merge "Avoid cross lock acquisition leading to deadlocks"

Merge "Add version of @UiThreadTest that works at class level"

Merge "CEC: Add a method to control standby mode of the device"

Merge "Make Bitmap.get/setPixel[s] color space aware"

Merge "Audio focus: framework enforces ducking"

Merge "Fix API checking for some"

Avoid cross lock acquisition leading to deadlocks

Test: manual

bug:36007738

Change-Id: I3db09af07857435682754916849433a809547974

Merge "Updated javadoc for ACTION_MANAGE_EXTERNAL_SOURCES"

Merge "Shell can now fake battery presence"

Merge "fallout from getting rid of IGraphicBufferAlloc"

Merge "Provided an AutofillCallback API."

Make Bitmap.get/setPixel[s] color space aware

Bug: 32984164
Test: cts-tradefed run singleCommand cts-dev --module CtsGraphicsTestCases --test android.graphics.cts.BitmapColorSpaceTest
Change-Id: Iee8c4abb264f0d2a44bae59788fbf4dcb2d0fdf7

Merge "VolumeShaper: Update Builder methods and Object overrides"

Merge "Fix bug in error checking in GnssLocationProvider"

Merge "Throw at getPackageSizeInfoAsUser() callers."

Fix API checking for some

Using AIDL constants in the definition of public API static ints seems
to confuse build tools sometimes.  Switch to just duplicating the
constants until that's worked out.

Test: Builds, at least for me!
Change-Id: I0fbe95bc0492eb8b39687f25a6c7e073fbb65dc4

Fix stale data due to stopped loader.

Bug 33185424

When stopping an CursorLoader while data is being loaded,
the load task will be canceled. This CL marks the data as
changed if the cancel is called while the loader is stopped.

Test: I63b48210a25be72d13a2a6182eb1757cbe6a1949
Change-Id: Ibf9c5facdcc5160f6ed146c5fdd063549ac2a7a8

Merge "Optimize ViewStructure for autofill by removing irrelevant nodes."

Merge "Use LinedHashMap in AccountManagerService to return accounts in the order of database addition"

fallout from getting rid of IGraphicBufferAlloc

Test: compiled & run
Bug: cleanup
Change-Id: I1590105d3abef985c2ae7c1a03cdf5fd2ec4bef9

Merge "Move lagging users over to new storage API."

Throw at getPackageSizeInfoAsUser() callers.

But only when targeting O; if they're old apps then log and pass
a soft failure message to their callback.

Test: builds, boots
Bug: 35749102
Change-Id: I6c6b5a37c23669e7b1ed3c30207bca610821eb1f

Merge "update uiautomator doclava build to use new location for templates as per: https://googleplex-android-review.git.corp.google.com/#/c/1524290/"

Merge "Fixed a bug where the scrim was distracting when pulling on HUN"

switch to new SkPathEffect factories for Compose and Sum

Test: existing Paint tests exercise these factories

Change-Id: I38abd91203b06b1141dcb4fd96478f9e0a1d18ba

Merge "Disable versioning for pathInterpolator when no-version-vectors is on."

Optimize ViewStructure for autofill by removing irrelevant nodes.

Test: CtsAutoFillServiceTestCases (with new tests) pass
Fixes: 35840787

Change-Id: Iaa2c1907c8383b4a820fd7204e67fa2d276ad2b8

Merge "Revert "Throw at getPackageSizeInfoAsUser() callers.""

Hook up screen brightness knob to DisplayPowerController.

This knob is designed to effectively tune the battery saver.

Bug: 34693888
Test: FrameworksServicesTests
Change-Id: I9fd7896f59fabd9baf649cdb12f01b1baaf4c34b

Merge "AC3/EAC3 MaxInputChannelCount support"

Merge "Replace AutoFillType by view constants."

Merge "Surrounding some field names with quotes to avoid breaking CSV assumptions."

Move lagging users over to new storage API.

Over the last month we've been moving everyone over to the new
StorageStatsManager public APIs, but we missed these users.

The ApplicationsState changes are straightforward, but we had to
completely rewrite StorageMeasurement to use the new fast-path
quota APIs.

Test: builds, boots, UI using StorageMeasurement works.
Bug: 36056120
Change-Id: If02177c95bf8c96ae4eceac4d631a168f99bef84

Merge "log fs_stat from fs_mgr and mount_all time"

Merge "Adds an explicit id for visibleToInstantApps attr"

Merge "Do not disable Open Network Notifications setting on profile change."

Merge "Updating moveTaskToBack to handle PiP activities"

Merge "Return generic Binder on onBind" am: c6613f7206 am: 2a71bd5150
am: 2fca3afd4b

Change-Id: Ic668d9bb43f58f28c096083f89c6bec30cf9640a

Merge "Return generic Binder on onBind" am: c6613f7206
am: 2a71bd5150

Change-Id: Ie447b8094ca9c483aac3886c795240301b5f2ace

Merge "Add BatterySaverPolicy for power save mode"

Merge "Return generic Binder on onBind"
am: c6613f7206

Change-Id: Ibc8cce3c63b93820f6bdd842e6a751233240874d

Merge "Add metric for openProxyFileDescriptor."

Merge "Return generic Binder on onBind"

Merge "TextClassifier: Introduce LocaleList in the API."

Merge "Add new constants to track successful contact queries Bug: 35803126,35873712 Test: No need to add any tests for this"

Merge "Bluetooth 5 Advertising API" am: 1bb08c25b8 am: fc3c9d0e9e
am: c162879c6a

Change-Id: I61382d466f28311b8ba5dd4ab120747eb0708501

Merge "Bluetooth 5 PHY selection API" am: 104eeedb42 am: 4f6034160c
am: 3730cd9608  -s ours

Change-Id: I5d0dfea7068f123e544686d0522d06a3a1bbf095

Merge "Bluetooth 5 periodc scan API (1/2)" am: f1d76a71c8 am: 8acd7391bd
am: 600a67a8c8  -s ours

Change-Id: Ibc1fc04aa569bb641a006b490ebf8450480b00ed

Merge "Bluetooth 5 Advertising API" am: 1bb08c25b8
am: fc3c9d0e9e

Change-Id: Ib8e53386754ccfec27314bc1d29aaabde0cb7976

log fs_stat from fs_mgr and mount_all time

- fs_mgr reports fs_stat int value during bootup when mounting
  an ext4 partition.
- This includes information like:
  if previous shutdown was clean
  if e2fsck has fixed any fs issue
  if any mounting or quota related operations has failed.
- Each information is set as flag. Flag definitions are avaiable from
  FsStatFlags in system/core/fs_mgr/fs_mgr.cpp
- log ro.boottime.init.mount_all.[early|default|late] which represent
  time duration for each mount stage. Depending on device's config, some of
  these stages may not exist. So do not log if value is 0, which means not existing.

bug: 32246772
bug: 35949600
bug: 35329915

Test: manual, check log after boot-up.

Change-Id: I4ceca88776119d88d5352d6793be122e82688279

Merge "Bluetooth 5 PHY selection API" am: 104eeedb42
am: 4f6034160c

Change-Id: I3183ead1802d166aae903c180be46f19ea8291ee

Merge "Bluetooth 5 enhanced scanning API" am: bed6fb3f5a am: 71760b71fd
am: f64a341eb7

Change-Id: I309e654e65001d9b97c457a272601761e54f9a73

Merge "Bluetooth 5 periodc scan API (1/2)" am: f1d76a71c8
am: 8acd7391bd

Change-Id: If28e751c19883914f3a44eb9fdec03e08a6a1901

Merge "Bluetooth 5 feature check API (1/2)" am: a06a0dfdb7 am: e9bd9be3cb
am: 0798f31c34

Change-Id: I871c18d414ce5a7517eba911c7e4ee441377f14c

update uiautomator doclava build to use new location for templates
as per:
https://googleplex-android-review.git.corp.google.com/#/c/1524290/

Change-Id: Ib8588f01005a691474622e6ca81f63234f796f82

Merge "Bluetooth 5 Advertising API"
am: 1bb08c25b8

Change-Id: Ifd95aacf025929d3c9f5a1762f31ed20c26fabfd

Merge "Bluetooth 5 enhanced scanning API" am: bed6fb3f5a
am: 71760b71fd

Change-Id: I6a5de71a10fb1e326e57f3a9bd665229535a6571

Surrounding some field names with quotes to avoid breaking CSV assumptions.

Some names can include commas, and if they're not surrounded by quotes,
then CSV parsing breaks, as was the case in
https://github.com/google/battery-historian/issues/94#issuecomment-282790498
and several other circumstances.

Test: flashed local device and ran batterystats output through Historian

Change-Id: I070c5d873d9f79996a581bf0a5f664136c0af454

Merge "Bluetooth 5 PHY selection API"
am: 104eeedb42

Change-Id: I0ee2daaf0771b405c8a14598a70459879d31c953

Merge "Bluetooth 5 feature check API (1/2)" am: a06a0dfdb7
am: e9bd9be3cb

Change-Id: I462d459638f2b41f8806b877a6b307e21e6e30d3

Merge "Bluetooth 5 periodc scan API (1/2)"
am: f1d76a71c8

Change-Id: I7b68746e3dc02a12d13380a90bcf26cebfd31f16

Merge "Bluetooth 5 enhanced scanning API"
am: bed6fb3f5a

Change-Id: I27d61ec7446c36143dcd5a94158451e1be9a5411

Merge "Bluetooth 5 feature check API (1/2)"
am: a06a0dfdb7

Change-Id: I59cf0e8f496e6a1201a3a6ec832c33cd1c8972df

Merge "Bluetooth 5 Advertising API"

Updating moveTaskToBack to handle PiP activities

- When a PiP activity calls moveTaskToBack(), we should fall through to
  the same behaviour as dismissing the PiP if there are no other tasks
  to show.

Bug: 36034065
Test: android.server.cts.ActivityManagerPinnedStackTests
Test: #testMovePipToBackWithNoFullscreenStack
Test: #testMovePipToBackWithVisibleFullscreenStack
Test: #testMovePipToBackWithHiddenFullscreenStack

Change-Id: I83c3d27dabc031546f9f1cf20b808bb8f13d4646
Signed-off-by: Winson Chung <winsonc@google.com>

Bluetooth 5 Advertising API

Test: manual
Bug: 30622771
Change-Id: Id6856e6110872ec50ff1af54ddc75c0104a6459c
(cherry picked from commit 5324a14cf490656269ef862d7f8f6b139a21c0e6)

Merge "Bluetooth 5 PHY selection API"

Revert "Throw at getPackageSizeInfoAsUser() callers."

This reverts commit 597c93454699eaed99d6b417080417d75777e72d.

Change-Id: I7d966ef973050ef791403062774fd7f576a65388

Merge changes I6823a613,I7763b473

* changes:
  Fixed a bug where the low-priority header wasn't cleared
  Fixes an issue where the notification header could be invisible

Bluetooth 5 PHY selection API

Bug: 30622771
Test: manual
Change-Id: I50262a56a70466439f9700549c3c0e7bd49e2e8d
(cherry picked from commit db5a87d50db9d4b3d642603f6c329b83ee5851f6)

Merge "Bluetooth 5 periodc scan API (1/2)"

Bluetooth 5 periodc scan API (1/2)

Bug: 30622771
Test: manual
Change-Id: I61853bc71f6013e9406d1d151bb51ea4484bb92c
(cherry picked from commit a48e03745becc96181c676dc3d194d0572f11c10)

Merge "Bluetooth 5 enhanced scanning API"

Bluetooth 5 enhanced scanning API

Bug: 30622771
Test: manual
Change-Id: I2c8065fbcedf48777ce18c7d8fe621e568b3fd75
(cherry picked from commit 9de522c6e48497028d36a1f8ad8f8adf4b7b1ae6)