AAPT2: Add supported manifest elements

Adds some elements present in res/values/attrs_manifest.xml to manifest
parsing.

<additional-certificate> in <manifest><application><uses-static-library>
<meta-data> in <manifest><permission>
<package-verifier> in <manifest>
<restrict-update> in <manifest>

Bug: 79755007
Test: manual
Change-Id: I50520a99ce5ced9908001430b2108d7a5c71f8a7
Merged-In: I50520a99ce5ced9908001430b2108d7a5c71f8a7
(cherry picked from commit 66f6cfb8a561a653db084b97adfa8c4d48979000)

Merge "Migrate all offline/online sdk docs generation to use Metalava"

Merge "Leave spaces between Operator name and Clock in StatusBar"

Merge changes from topic "remove-hidl-boot-jars"

* changes:
  pm: Add hidl libraries to old class paths.
  preloaded-classes: remove core HIDL libs

Leave spaces between Operator name and Clock in StatusBar

In StatusBar, there are no spaces between Operator name and Clock,
so leaving spaces to improve design.

Bug: 114133664
Test: Manual, enable operator name in status row
config_showOperatorNameInStatusBar, insert sim card with operator name,
watch status row.

Change-Id: I1a2e33436ed50d429ae4b1769d0a72d501233a22

Merge "Initial rename to IpServer"

Initial rename to IpServer

Rename TetherInterfaceStateMachine to IpServer. IControlsTethering
is folded into IpServer.Callback and some of the dependencies in
TetheringDependencies are moved into IpServer.Dependencies.

Several things still need fixing, including:
    - convert message passing into method calls
    - the calls that enable forwarding should be moved up out of
      IpServer into the Tethering layer above it

Test: as follows
    - built, flashed, booted
    - runtest frameworks-net passes
Change-Id: I015f800ed23c8aa5c8c81a74d7b508abfcaab659

Merge "Remove the last of the ResolvUtil uses"

Merge "Add new telecom metrics for emergency dialer"

Merge "Bandwidth-related commands porting"

Merge "Simplify logic for connect in HfpClientProfile"

Merge "Simplify logic for connect in A2dpSinkProfile"

Merge "Add New TelephonyManager APIs"

Migrate all offline/online sdk docs generation to use Metalava

These targets all depend on libcore sources which are blocker for us to
enable java9 feature, so convert them to Metalava.

Also enable API level annotations for api-stubs-docs, offline-sdk-docs,
and online-sdk-docs.

Test: m -j docs
Bug: b/78245848
Change-Id: I354d699a79cc5e6580b50e0613e7602c77b9c0b5

Merge "OMS: fix typos in permission check message"

Merge "ByteBucketArray.h: fix typo"

Merge "Update TelephonyManager document"

Merge "Restrict IMM.notifyUserAction() with targetSdkVersion <= P"

Merge "Use UID as requestID"

Merge "For metalava target, use droidstubs instead of droiddoc"

Merge "Remove IInputMethodManager.getEnabledInputMethodList() from greylist"

Update TelephonyManager document

Bug: 111453847
Test: build
Merged-In: Ief746e771dc9435782dec75cf2259d5b9f040c27
Change-Id: Ief746e771dc9435782dec75cf2259d5b9f040c27

Merge "Use multiple patterns and emails in per-file syntax."

pm: Add hidl libraries to old class paths.

These two libraries:
android.hidl.base-V1.0-java
android.hidl.manager-V1.0-java

are being removed from BOOT_JARS. This change facilitates linking to them
for libraries or prebuilts in or before P.

Test: atest android.content.pm.AndroidHidlUpdaterTest
Bug: 77307025

Change-Id: Ic0db24cc68d66f5dbfab126ce7e304eec0bfc969

preloaded-classes: remove core HIDL libs

These are being removed from boot jars.

Test: (sanity) boot device, no longer see 'Class not found for preloading'
Bug: 77307025
Change-Id: I854d3397ff362495ff392c2c4ea8594d1b2449bd

Merge "Fix typo in android.net.Uri documentation."

Simplify logic for connect in HfpClientProfile

* connect: remove connected device check logic.
  HeadsetClientStateMachine will check it.
* Remove member variable V and related checks.
* wrap if/else statement in curly brackets.
* add robotest for HfpClientProfile.

Bug: 111812003
Test: make -j42 RunSettingsLibRoboTests ROBOTEST_FILTER=HfpClientProfileTest
Change-Id: Ie36efd6d0a9f8e6501a3690b026ea4bd39744b5c
Merged-In: Ie36efd6d0a9f8e6501a3690b026ea4bd39744b5c

Simplify logic for connect in A2dpSinkProfile

* connect: remove connected device check logic. A2dpSinkService
  will check it.
* Remove member variable V and related checks.
* wrap if/else statement in curly brackets.
* add robotest for A2dpSinkProfile.

Bug: 111812003
Test: make -j42 RunSettingsLibRoboTests ROBOTEST_FILTER=A2dpSinkProfileTest
Change-Id: Ia0a118d73e5bb35ed3b2b75e67aeb98ae936981f
Merged-In: Ia0a118d73e5bb35ed3b2b75e67aeb98ae936981f

Remove the last of the ResolvUtil uses

This is the completion of cleanup work started in aosp/674269.
Consequently, also remove ResolvUtil.

Test: as follows
    - built, flashed, booted
    - runtest frameworks-net passes
    - presubmit
Bug: 64133961
Bug: 72345192
Bug: 73872000
Bug: 78548486
Change-Id: I980f8a9b67f3f06f2d97a60f8e58e92c36316d40

Merge "Fix ConcurrentModificationException crash due to race in IpClient"

Fix ConcurrentModificationException crash due to race in IpClient

Bug: https://issuetracker.google.com/issues/114838819

Test: manual
Change-Id: Ifd1522ccaca10c0416fc2d00641c4effee37fc6c

Merge "Make SubscriptionManager.isValidSubscriptionId public"

Merge "Avoiding potential NPE when dumping to proto."

For metalava target, use droidstubs instead of droiddoc

droidstubs is used to generate stubs files only.

Test: m -j api-stubs-docs
Bug: b/8d0763849
Change-Id: Ia7c3d19ed2ec7b0f5b96008e5780dbcd5a32402b

Add New TelephonyManager APIs

Bug:111453847
Test: build test
Merged-In: I83a5f4149f92ce495cd066f7158e7ff37a66894f
Change-Id: I83a5f4149f92ce495cd066f7158e7ff37a66894f

Merge "DO NOT MERGE base: getDataUsageInfo get cycle validation"

Merge changes from topic "ownerz"

* changes:
  Tweaks to AOSP upload hook logic.
  Upload hook that guides people towards AOSP.

Remove IInputMethodManager.getEnabledInputMethodList() from greylist

App developers must never need to call this IPC method directly via
reflection.  The same functionality has been exposed as a public API
InputMethodManager#getEnabledInputMethodList() since its beggining.

Just for the record, this is what
InputMethodManager#getEnabledInputMethodList() is doing.

  public List<InputMethodInfo> getEnabledInputMethodList() {
      try {
          return mService.getEnabledInputMethodList();
      } catch (RemoteException e) {
          throw e.rethrowFromSystemServer();
      }
  }

Bug: 112722706
Test: compile
Merged-In: Idf9fa866f73dbd151bb67f60dbd57041d9055a23
Change-Id: Idf9fa866f73dbd151bb67f60dbd57041d9055a23

DO NOT MERGE base: getDataUsageInfo get cycle validation

When the DataUsageController calculate the DataUsageInfo
based on the cycle from NetworkPolicyManager, that cycle
should be not null. Otherwise, it couldn't be used to do
operation like toInstant() directly.

The error could be got from logcat is as:
E AndroidRuntime: Caused by: java.lang.NullPointerException:
Attempt to invoke virtual method 'java.time.Instant
java.time.ZonedDateTime.toInstant()' on a null object reference

E AndroidRuntime:          at com.android.settingslib.net.
DataUsageController.getDataUsageInfo
(DataUsageController.java:140)

Inside of the core/java/android/net/NetworkPolicyManager.java:
public static Iterator<Pair<ZonedDateTime, ZonedDateTime>>
 cycleIterator(NetworkPolicy policy) {
... ...
return Pair.create(null, null);
... ...
}
That should be avoided.

Considering it could be happened in the NetworkPolicyManager side,
and it is reasonable if it did it, validating operation is better
to be considered by any caller such as the DataUsageController.

Bug: 113952713

Test: specify the hasNext to be false but the DataUsageController
don't crash.

Change-Id: I2469c129a160e30615ea6011adf18688f951eac8
Signed-off-by: John Zhao <yuankuiz@qualcomm.corp-partner.google.com>

Restrict IMM.notifyUserAction() with targetSdkVersion <= P

With my change [1] in Android N, applications no longer need to
manually call this hidden API even when directly implementing
InputConnection interface without extending BaseInputConnection [2].

So far applications that we have confirmed to be using this hidden API
is Chromium and browsers that are supposed to be built on top of
Chromium source code [3].  Actually my change [1] was per request from
Chromium team and Chromium does not use this hidden API on Android N
and later devices [4].

Moving forward, with this CL applications that target API 29 and later
are no longer allowed to use this hidden API since we see no actual
use case of this hidden API on Android N and later devices so far.

If you have any reason to continue relying on this hidden API, please
leave a comment in Bug 114740982.

 [1]: I571d6cc9c6e461d8994aa7496e7e18be13766411
      159dd47db34151314e2449347251ad32b30e5ea1
 [2]: This scenario had not been suppored until Android N.
      See also other CLs that are tagged with Bug 24688781.
 [3]: https://cs.chromium.org/chromium/src/content/public/android/java/src/org/chromium/content/browser/input/InputMethodManagerWrapperImpl.java?rcl=525c72b0f4694d6b3f93bcc7797d0411c2fa683c&l=113-128
 [4]: crbug.com/551193

Bug: 114159783
Bug: 114740982
Test: compile
Merged-In: I41d6419438cef92cce81cc33d9db00dd2ee89d33
Change-Id: Iac81f49413605271fccc0b8aec3e6ed95925f061

Merge "Convert AudioAttribute to Stream correctly for TTS"

Avoiding potential NPE when dumping to proto.

Inspired by aosp/736453. Cherry-picked from ag/4934453

Bug: 113087610
Test: Android builds
Change-Id: I9f0dae9cd5a839947c7ed150980eb30b928beef7

Make SubscriptionManager.isValidSubscriptionId public

And rename subId to subscriptionId.

Test: Treehugger
Bug: 113072551
Change-Id: I4bdf705d7f81bad423ed867972fd1ed7fe5717d0

Merge "Add API to listen to opportunistic subscriptions change."

Tweaks to AOSP upload hook logic.

Lists all matching files, and slightly more robust path handling.

Bug: 113136846
Test: manual
Change-Id: Ic1971a3df97b8b137993b5856b47d0d706a996cd

Upload hook that guides people towards AOSP.

The canonical location of certain parts of the source tree live in
AOSP, and should not be changed internally to avoid merge conflicts.

This initially starts with "OWNERS" files, but the hook can easily
be repeated to match any files based on file path regexes.

Bug: 113136846
Test: manual
Change-Id: I113e63b6133f20389d84ec0d6d8b81cdbdf35d38
Merged-In: I113e63b6133f20389d84ec0d6d8b81cdbdf35d38

Merge "Framework: Remove a nonsensical substring(0)"

Merge "Reset launch start times when removing a process"

Merge "Revert "Check in P dark greylist, use it for hidden API list generation""

OMS: fix typos in permission check message

Calls to enforceChangeOverlayPackagesPermission and handleIncomingUser
include a string parameter to be added to the exception thrown if the
caller doesn't hold the expected permission. Fix a couple of typos in
these strings.

Test: builds, boots
Change-Id: I70256c41d61f8af3735697c6e63644e5ac46fd84

ByteBucketArray.h: fix typo

Bug: 78815803
Test: builds, boots
Change-Id: I93d7ce5255911797d6afdb05caf0cf7dbfdc22b9

Convert AudioAttribute to Stream correctly for TTS

Add FLAG_BEACON for STREAM_TTS then use it for
conversion from AudioAttribute to stream type.

Bug: 113303490
Test: Use AudioAttributes.toLegacyStreamType

Change-Id: I7fa2b3f095437f8077545d1fff795bb0d8fe6b89

Revert "Check in P dark greylist, use it for hidden API list generation"

This reverts commit 3cc74c71ef907a76547424aea6597f7b2907e9fb.
We saw app crashes due stricter classification of non-SDK APIs.

Bug: 115284625
Bug: 113881436
Test: phone boots

Change-Id: I689d0fbf66fabeffa5032e13f2f1f314c50b6cc5

Fix typo in android.net.Uri documentation.

This typo has been present since before the "auto import"
of cupcake sources in 2009.

Bug: 114704047
Test: Treehugger
Test: Checked through source inspection that this really returns
      the encoded form: its implementation is ssp.getEncoded(),
      whose implementation in turn is:
      return hasEncoded ? encoded : (encoded = encode(decoded));
Change-Id: I0e4fed702e6b3012429778ccbd18a495a5d69e9f

Merge "Remove services.net JAR dependency"

Remove services.net JAR dependency

Test: as follows
    - built, flashed, booted
    - runtest frameworks-net passes
    - captive portal interaction works as expected in Private DNS
      strict mode
Bug: 64133961
Bug: 72345192
Bug: 73872000
Bug: 78548486

Merged-In: Ia2ab2fb15e7898d99d30da20699e0bbd7a5a0964
Change-Id: I290811305fd89a4fb5633b47173aaec28e9ed4b5

Merge "Try to set the hop limit to something suitable for 64share"

Merge "Simplify logic for disconnect and getConnectionStatus in MapProfile"

Bandwidth-related commands porting

remove bandwidth enable command
netd will handle it

Test: runtest frameworks-net passes
Test: manual testing of bandwidth works
Change-Id: I2cccbc8e682e64368640bdea01f03574b3948c2a

Merge "Stop invoke initAppOps in Camera default constructor."

Add API to listen to opportunistic subscriptions change.

Add a new set of APIs so that caller can listen to
opportunistic subscriptions changes.

Bug: 92796390
Test: build
Change-Id: Iaa741141bf1bd04c4b1618d5bfeac733a0690ed8
Merged-In: Iaa741141bf1bd04c4b1618d5bfeac733a0690ed8

Reset launch start times when removing a process

When removing an exiting process that is running foreground activities, clear the launch times for
the current windowing mode. When an app process is removed, activities from the process may be
relaunched. In the case of forceStopPackageLocked the activities are finished before any window
is drawn, and the launch time is not cleared. This will be incorrectly used to calculate launch time
for the next launched activity launched in the same windowing mode.

Bug: 80084651
Test: adb shell am start -W com.amazon.mShop.android.shopping/com.amazon.mShop.home.HomeActivity  && sleep 1 && adb shell am force-stop com.amazon.mShop.android.shopping
Merged-In: I2c4f0716c922baa1ad209b4ea1fa7ce366e0e108
Change-Id: I2c4f0716c922baa1ad209b4ea1fa7ce366e0e108

Merge "Framework: Be more verbose in A/B OTA"

Merge "Framework: Ensure important fdsan function is compiled"

Framework: Ensure important fdsan function is compiled

Compile ForwardingOs fdsan-using functions. Follow-up to commit
5625660c650a0923d4c7f020f2eb7eef7d5962d4.

The function is relatively hot now, and importantly part of the
'abort' stack for fdsan issues.

Test: m
Change-Id: Iad9883b8fd85fc362bb3252739e3e3acc45373af

Framework: Be more verbose in A/B OTA

Bug: 114091593
Test: m
Change-Id: Ib0895188455a8340cefb7d0fd609012440b6473b

Use multiple patterns and emails in per-file syntax.

Test: build/make/tools/checkowners.py -c -v OWNERS
Change-Id: I46a4788c0cda69d0a3aadcf1f3dc539aa3c62fc3

Merge "Netd unconditionally enable bandwidth control"

Merge "Include dark greylist entries from annotations."

Simplify logic for disconnect and getConnectionStatus in MapProfile

* disconnect: remove connected device check logic. BluetoothMapService
  will check it.
* getConnectionStatus: remove connected device check logic.
  BluetoothMapService will check it.
* Rewrite the annotation about this class.
* Remove member variable V and related checks.
* wrap if/else statement in curly brackets.

Bug: 111812003
Test: manual - disconnect MAP from Settings UI
Change-Id: I2469821f0ae2e3f384844326b30c7d3c4c88ab63

Merge "Framework: Ensure important fdsan function is compiled"

Framework: Ensure important fdsan function is compiled

Compile IoUtils fdsan-using functions.

Test: m
Change-Id: If1f3d7fc57890309c15794cee5cd0f338cbd2bd8

Include dark greylist entries from annotations.

@UnsupportedAppUsage annotations support setting a maxTargetSdk property
on the which results in the member being placed on the dark grey list,
or in future on a P or Q blacklist.

Bug: 114361293
Test: m
Change-Id: I33be28f8afb034a030bdf5cf29d43ba72f50bfcd

Merge "Fix usage of internal mockito API"

Merge "Move ApplicationInfo.usesNonSdkApi to private flags"

Merge changes from topic "hiddenapi-dark-grey"

* changes:
  Ignore comment lines in hidden api lists
  Mark hidden API lists .KATI_RESTAT
  Check in P dark greylist, use it for hidden API list generation

Netd unconditionally enable bandwidth control

Remove code which is used to enable bandwidth controller.
Netd default enable it.

Test: The bindwith control start correctly on new devices.
Change-Id: I9498d2175104b31a6d37ea3ba6937c91c560d0a2

Merge "Always enable bandwidth control in Q"

Merge "Build android.test.* with java_sdk_library"

Fix usage of internal mockito API

The StubberImpl now takes an argument. Set it to default.

Test: m -j TtsTests
Change-Id: Id1175bd40489c7dee2c8c0993a65836d86e38cb3

Merge changes from topic "b113052379-cp-from-pi-dev-p2"

* changes:
  Camera: Documentation update to remove physicalIds
  Camera: Documentation updates for calibration and distortion correction

Merge "Make SubscriptionManager.getSubId public"

Merge "Fix compilation when targeting OpenJDK 9."

Fix compilation when targeting OpenJDK 9.

CL [1] added a new compilation target that uses internal API
that is now in the jdk.compiler module and is not exported
(it is subject to removal without notice in future versions of
OpenJDK). The fact that these packages are not exported suggests
that a replacement API is already available, but I don't know the
details.

For now, this CL uses javac command-line flags to export these
packages, which fixes compilation. Since the commit message from
CL [1] implies that the entire tool can be removed once bug 77284273
is fixed, a longer-term solution should not be required.

[1] https://android-review.googlesource.com/c/platform/frameworks/base/+/745655
    commit bad89e5e5b171a71e42d7c738ec97a39747e4318

Test: The following breaks before but not after this CL:
  EXPERIMENTAL_USE_OPENJDK9=true make unsupportedappusage-annotation-processor
Bug: 113853502

Change-Id: I24b77af9262891428a01e2eccc18fa3a5c3b3bc9

Move ApplicationInfo.usesNonSdkApi to private flags

No need to create an extra field for a boolean flag. Move the recently
added ApplicationInfo.usesNonSdkApi to one bit in privateFlags.

This also solves an issue where the field was not propagated during
copying of the data structure.

Bug: 113315999
Test: phone boots
Merged-In: I09f8f39454c013a84893ac304904a4412fc542bf
Change-Id: I09f8f39454c013a84893ac304904a4412fc542bf
(cherry picked from commit a5b4df2a8937fed8cb07fa5b63252d013c808fe7)

Ignore comment lines in hidden api lists

It is sometimes useful to add comments at the top of a source file. This
patch changes hidden API list generation to ignore lines beginning with
a hash.

Note that due to the sorting constraints on hidden API lists, comments
can be only at the top of the files.

Test: m appcompat
Change-Id: I1bc6fd44d1b1f10a5adc45093d7f7ed5a0c5a54f

Mark hidden API lists .KATI_RESTAT

Change hidden API list generation build rule so that it checks if
outputs have changed and only commit them when changes have been made.
.KATI_RESTAT instructs ninja to restat the outputs and remove reverse
dependencies when rebuilding dependencies is not needed.

Bug: 113278235
Test: m appcompat
Change-Id: Ic3ec82534a6d79b6a85f865a6e9eb5f0b3f12424

Check in P dark greylist, use it for hidden API list generation

We want to preserve the same semantics as P wrt dark greylist. This will
eventually become the "blacklist for target SDK >= P".

Bug: 113881436
Test: phone boots
Change-Id: I3bb72102e1d054eb0f2cae089ab633a0bc2bb276

Merge "Tighten up neighbor timers if IpReachabilityMonitor is in use"

Add new telecom metrics for emergency dialer

These includes,
1. The source where user initiated the call.
2. The duration between START_CONNECTION and REQUEST_DISCONNECT.

Bug: 111818008
Test: manual and check the result of
'adb shell dumpsys telecom' for metrics data
Change-Id: Iddc7f256362abe9fe252bd12cead05a7d77d329e

Tighten up neighbor timers if IpReachabilityMonitor is in use

Currently the kernel sends 3 probes, 1000ms apart when a neighbor is
in NUD_PROBE state (either naturally or artificially moved to that
state when, for example, confirming network configuration following
an AP association/roam).

With this change the device sends 5 probes at 750ms apart.  This
gives the network infrastructure a chance to respond to any of
5 probes instead of any of only 3, at a cost of increasing the total
probe time from 3s to 3.75s.

Future changes may read these values from settings (or from somewhere
else) so that experiments may be conducted or values adjusted based on
other information.

Test: as follows
    - built, flashed, booted
    - runtest frameworks-net passes
    - examinaton of /proc settings on wifi shows updated values
    - connected to a test network, disconnected the backhaul, ran
      "adb shell dumpsys wifi ipclient confirm" while watching
      "adb shell tcpdump -n -i wlan0 -l -vv arp", and observed
      5 ARP who-has ("solicits") ~750ms apart.
Bug: 62476366
Bug: 62845299
Change-Id: Ib805531b08624805bb63703a5b338b9ff48171d4

Framework: Remove a nonsensical substring(0)

String.substring(0) will return the same String instance. There is
also no reason to make a copy.

Found by errorprone.

Bug: 114129741
Test: m
Change-Id: Ic089f04efc3c73f70aa15a132d62ac665c8b8f4c

Merge "Add process-lifecycle-owner-initializer to tests."

Merge "Convert AudioAttribute to Stream correctly"

Merge changes I7f1ef2e8,I498bc261

* changes:
  Convert sysui to bp
  Cleanup trying to move to bp

Add process-lifecycle-owner-initializer to tests.

Bug: 112587252
Bug: 114073652
Test: This is to fix the tests.
Change-Id: I083db4e2eb4efd8d4a914f33ac259aa317e568de
Merged-In: I083db4e2eb4efd8d4a914f33ac259aa317e568de
(cherry picked from commit 5f5ad2e6274f28772d70de0a97301fd17ab7f4d3)
(cherry picked from commit 0757deaad67bfa7d09e210566fdc7638fd8b8e3f)

Convert sysui to bp

Test: tests gonna test, builds gonna build
Change-Id: I7f1ef2e82ce2c74253c57de8d73079dcfe2d2264
Merged-In: I7f1ef2e82ce2c74253c57de8d73079dcfe2d2264
(cherry picked from commit a2f2d82f2698ca4d9fcfad2a94b5b683646c085d)
(cherry picked from commit 1a8ee9d2f9bda161301469488d9a27baf5398233)
(cherry picked from commit 83ce30d807c3621f47f753e75f41d2b887e3ccba)

Cleanup trying to move to bp

 - Remove some resources that don't appear to do anything
 - Don't assume resource ids are final, because they aren't when
   compiled as a library

Test: build
Change-Id: I498bc2612d3f47df5f567c18c29e1666431e971c
Merged-In: I498bc2612d3f47df5f567c18c29e1666431e971c
(cherry picked from commit 05dd5671720985c03f35ec476591fb0d8399a018)
(cherry picked from commit a9b2fcad7a1ba9b5a459f46cc40504c7ea4fe590)
(cherry picked from commit 4c9d66e56cc8c324c2dc87f75e6338b7464d6eaf)

Merge "statsd: fix double close."

Merge "Processor for @UnsupportedAppUsage annotations."