Use timeout consts in all ConnService tests

Fix some timeouts that were forgotten in parent CL.

Bug: b/111778226
Test: runtest --no-hidden-api-checks -x ConnectivityServiceTest.java
Change-Id: Ib848eaba8c69c07ac596aa4386a741c9a2a87fc9

Increase linger delay and test callback timeout

Tests appear to have become much slower on apct in recent runs, although
local runs do not seem to be affected. As a first measure, increase our
timeouts to eliminate flakyness.

Bug: b/111778226
Test: runtest --no-hidden-api-checks -x ConnectivityServiceTest.java
Change-Id: I7a3bfb646eb554bcd9d50d026142f2a2cff74beb

Merge "Merge generated greylists into final greylist."

Merge "Disable generating Binder#getTransactionName."

Merge "Fix order of hidl unlinkToDeath."

Merge "Fix edge effect not showing up on first scroll"

Disable generating Binder#getTransactionName.

It might have introduce a performance regression.

Bug: 111791983
Test: manual
Change-Id: I459d9e4e4d2d5d0be0531742a7910848702a1384

Merge "Turn droiddoc's metalava_merge_annotations_dir into a list."

Turn droiddoc's metalava_merge_annotations_dir into a list.

This change replaces droiddoc's metalava_merge_annotations_dir
parameter (which takes a single value) with
metalava_merge_annotations_dirs (which takes a list). This will makes
it possible to merge libcore annotations in from a separate directory
under ojluni, instead of adding them in a preprocessing step.

Test: `make metalava-api-stubs-docs` with a local change to add a second directory
Bug: 111639530
Change-Id: I8eb778038c1cb52ddb6fc86d972f555f49e836b2

Merge "Update annotation name in class2greylist."

Update annotation name in class2greylist.

Annotation was added with name @UnsupportedAppUsage after Android API
council discussion.

Bug: 110868826
Test: m class2greylist
Change-Id: I63ad61e97c5e8fdebfcead9e4ee5b060a60f97c7

Merge "Increase TEST_LINGER_DELAY_MS"

Merge "don't call setSize() when creating SurfaceControl"

Merge "Remove test dependency on ubsan runtime"

Fix order of hidl unlinkToDeath.

Calling unlinkToDeath after linking to death multiple
times will unregister the most recently registered
death recipient as expected.

Bug: 67503915
Test: hidl_test_java
Change-Id: Ia7774c3d1f8932660890716e0edb7a1180430de7

Merge changes from topic "deprecate-iis-closed"

* changes:
  Deprecate InflaterInputStream.closed field
  Correct ordering of TelephonyManager methods

Merge "Make BinderProxy @hide public class."

Merge "Add annotation for hidden members used by apps."

Merge "Move UriCodec to be near its one user"

Add annotation for hidden members used by apps.

This annotation will be added to existing hidden methods and fields that are
known to be used by apps.

These annotations will be used to replace the existing hard coded greylist
text files.

See go/hidden-api-annotations for more context.

Test: m
Bug: 110868826

Change-Id: I1b86ad8460525c2896fb84d7f1f95a521f582b9c

Merge "Break some dependencies on libcore internals"

Merge "Remove dependency on InflaterInputStream.closed field."

Remove test dependency on ubsan runtime

Requirement for this dependency was removed in aosp/696863.

Bug: b/79785496
Test: runtest frameworks-net (aosp), atest FrameworksNetTests
Change-Id: I14bb65c8f0a9b55aeae6376ff04ce18be7dc7f22

Merge "Access Identity and SignalStrength from CellInfo"

Merge "Check for /dev/memcg/system/tasks before writing to it"

Move UriCodec to be near its one user

The only user of UriCodec is android.net.Uri so it can
be moved to android.net.

This change moves the UriCodecTest to the unit tests
and there is an associated change to Uri's CTS tests
to ensure we're testing public method behavior there too.

Test: make droid && make cts
Bug: 111055375
Change-Id: Id032f849b439adf3fd7cb1a3c8b234a5eec68fac

Merge "Avoid onServiceDisconnected() for unbind Service"

Merge generated greylists into final greylist.

With annotations for greylist entries, each java target generates it own
greylist.txt file. Merge all of these into the final greylist.txt.

Bug: 110868826
Test: m
Change-Id: I6a64c65000f1fd9f49e09b5b498bdccc3560d606

Merge "Quick style fixup of ISmsImplBase"

Merge "Add new "class2greylist" tool."

Avoid onServiceDisconnected() for unbind Service

onServiceDisconnected() and unbindLocked() both
are responsible to make mRemoteInstanceas null,
so ensure that the service is not unbinded already
while calling onServiceDisconnected()

Fixes:111631529
Test: Build Successful and device Running.
Change-Id: I505d97e32c4e76f06a358850e91fe189bf02654c

Deprecate InflaterInputStream.closed field

As recommended by the Android API Council this field is being
deprecated and will be removed from the API in a future release.

Bug: 111592689
Test: make checkbuild

Change-Id: I9aab7396b8b9faadb764fe9d43e80747b3eec018

Correct ordering of TelephonyManager methods

A previous change (db6a391957f51c6ed9a768f0c39c61ff7a4f3603) seems to
have added some new methods to the api/current.txt manually as they are
in the wrong order. This change corrects the ordering so that they do
not affect future changes to the api/current.txt file.

Bug: 74613795
Test: make checkbuild
Change-Id: I83f9c2ff3cd9f43add297e1c9411598f3eef094f

Break some dependencies on libcore internals

This commit swaps some internal details of libcore for some
abstractions on dedicated "internal facing" APIs. This reduces the
number of internal APIs used. There is an associated change in
libcore/

Bug: 111055375
Test: build / boot
Change-Id: Idddada1922701bd15475c840eaa76c505e545d33

Remove dependency on InflaterInputStream.closed field.

In upstream OpenJDK the InflaterInputStream.closed field is private.
During integration of OpenJDK into Android it was changed to be
protected so that it could be used by StrictJarFile, and in the process
was accidentally made part of the public API. This change removes the
StrictJarFile dependency on that field.

Bug: 111061052
Test: checkbuild, flash and run FrameworkCoreTests
Change-Id: I8aee1c444cf27e7078b848c65e0433c831ab06a0

Add new "class2greylist" tool.

This extracts signatures of methods that have the @UsedByApps annotation
for generating the greylist. It will be integrated into the build to
replace many members on greylist.txt.

Test: $ atest class2greylisttest
Bug: 110868826
Change-Id: Ifaf5859b60076c051de6be5a912ef70734330ce7

Merge "Enable the generate transaction name for system server."

Enable the generate transaction name for system server.

Size of vdex,art,oat files (flag off -> on)
- framework/boot*.vdex: +10K  (3.589M -> 3.599M)
- framework/arm64/boot*.{art,oat}: +20K  (4.378M -> 4.398M)

Test: manual
Change-Id: I4938ca5459004494a50ec81db9ef9e0f0e6e7a60

Make BinderProxy @hide public class.

To make some future refactoring easier.

Added some JavaDoc (mostly links to android.os.IBinder)
to make checkstyle happy.

Test: builds
Merged-In: If9dd6913868a34ea1e3d14fee1860a4ff368e06b
Change-Id: If9dd6913868a34ea1e3d14fee1860a4ff368e06b
(cherry picked from commit 03e6d923c579ec79939944e7a9a8ac1b36aa2bbb)

Merge changes from topic "bionic_fdsan_java"

* changes:
  ParcelFileDescriptor: support bionic's fd ownership tracking.
  Reenable fdsan in the zygote post-fork.

ParcelFileDescriptor: support bionic's fd ownership tracking.

Also, delete a bunch of deprecated functions that are package-private,
and not called anywhere in the package.

Test: booted walleye
Change-Id: I728e63e7d6f9e1ba42644f93983cfcd6a2d2634c

Reenable fdsan in the zygote post-fork.

By default, fdsan is disabled after forking to avoid breaking 'fork();
close(0..1024); exec(...)'. Reenable it in the zygote manually.

Bug: http://b/110100358
Test: booted walleye-aosp
Change-Id: I8706e4153e4e91306f50d215a62fe02d445082e1

Merge "Correct Telus gid"

Check for /dev/memcg/system/tasks before writing to it

Fixes the following errors in the logs on devices without /dev/memcg
mounted:

09-08 10:07:22.991   732   732 E Zygote  : couldn't write 1124 to
/dev/memcg/system/tasks
09-08 10:07:22.986   732   732 W main    : type=1400 audit(0.0:8):
avc: denied { create } for name="tasks" scontext=u:r:zygote:s0
tcontext=u:object_r:cgroup:s0 tclass=file permissive=0

Bug: 67860826
Test: boot Taimen, no more error/denial in the logs.
Change-Id: I37f481fa4e9e9116688cca4f090aa26f96f602c9
Merged-In: I37f481fa4e9e9116688cca4f090aa26f96f602c9
(cherry picked from commit 6bdc3a28e75bdb936e7c172bc8d305b2c30c9f06)

Merge "Use StrictJarManifest.ATTRIBUTE_NAME_NAME instead of Attributes.Name.NAME."

Merge "Fix an incorrect null check"

Correct Telus gid

Bug: 34036790
Test: static code review
Change-Id: I3f81c98919425c35471d0bfcf88d77eb88895d12

Quick style fixup of ISmsImplBase

Test: Compiles
Change-Id: Ic27db060e402d8a3239fe588a229c246c3228184

Access Identity and SignalStrength from CellInfo

Accessing common CellIdentity and CellSignalStrength
information should not require RTTI. Thus, add
overloaded methods that allow the access the common
versions of these functions from a top level CellInfo.

Bug: 10151416
Test: compilation

Change-Id: I18f31c5cec075dcd68cc493e24c2d551138dd73c

Merge "Ensure dropbox could copy tombstone files to special folder."

Merge "Frameworks: Let GuardedBy accept multiple locks"

Ensure dropbox could copy tombstone files to special folder.

Process of tombstoned have changed the generation flow for
tombstone, and dropbox could not be notified when generating new
tombstone file any more, so dropbox could not copy and compress
tombstone file to /data/system/dropbox. We need to modify
observer events from CLOSE_WRITE to CREATE, and it could
work normally.

Bug: http://b/111608961
Test: 1 After tombstone is triggered, we could see the tombstone
      file in the data/system/dropbox directory.

Signed-off-by: Haoran Li <lihaoran5@huawei.com>
Change-Id: I9d6a31773e4a58658ffab214b1e337f27e9f3ae6

Merge "Use Base64 from android.util in LocalTransport"

Frameworks: Let GuardedBy accept multiple locks

Widen the definition to take advantage of errorprone support.

Bug: 72666911
Test: m
Test: m RUN_ERROR_PRONE=true javac-check
Change-Id: Id792ee70b41b786da717f916e143786fe6308937

Fix an incorrect null check

Found an incorrect null check when debugging.

Test: Manual
Change-Id: I7ea3e2cf88117ccef48f994719986e371e45d777

Increase TEST_LINGER_DELAY_MS

The current timeout is short enough that lingered networks are torn down
before assertions can be run, causing flakyness in testMultipleLingering.
This CL sets the timeout to 300 in that particular test.

Verified that 300ms is enough with 500 successful runs on Sailfish.

An alternative would be to mock firing of the lingering timer, but
keeping the WakeupMessage logic realistic has already proven useful in
detecting races.

Bug: b/110673602
Test: runtest --no-hidden-api-checks frameworks-net
Change-Id: I6f9a656190bcff0b071ae2584d0f5fa74158472b

Use Base64 from android.util in LocalTransport

Stop using bouncycastle as requested in the bug.

Bug: 111440841
Test: 1. Without changes
         a. adb shell bmgr transport android/com.android.internal.backup.LocalTransport
         b. adb shell bmgr backupnow com.android.providers.settings
         c. adb shell ls /cache/backup/1/_delta/<kv_package> #=> Base64 encoded keys
      2. Build and flash this CL
         a. adb shell bmgr restore 1 com.android.providers.settings #=> verify stuff restored
         b. adb shell rm /cache/backup/1/_delta/com.android.providers.settings/* /data/backup/com.android.internal.backup.LocalTransport/com.android.providers.settings
 c. adb shell bmgr backupnow com.android.providers.settings
 d. adb shell ls
 /cache/backup/1/_delta/com.android.providers.settings # Verify same keys as 1c

Merged-In: I305bbae0e0af3639c1d45def19872e6da84624df
Change-Id: I305bbae0e0af3639c1d45def19872e6da84624df
(cherry picked from commit 7a6e03271995fa20754663a66cd334aebf077e84)

Merge "API updated for ICU4J 62.1"

Merge "[DO NOT MERGE] Do not dismiss volume dialog onAnimationEnd while dialog is showing"

API updated for ICU4J 62.1

- ICU 62 upgrades to Unicode 11. Add new APIs for Unicode 11.0

Bug: 109930825
Bug: 109837373
Test: make
Change-Id: I3f23bbe624a10511a613cb07fd35e8272d1461a2

Merge "[DO NOT MERGE] Update P light greylist in AOSP"

[DO NOT MERGE] Update P light greylist in AOSP

Publish the final light greylist of P for external developers.

Test: N/A
Change-Id: I5513ee2ee555bf1dced7f52b36de3df7ac85747a

[DO NOT MERGE] Do not dismiss volume dialog onAnimationEnd while dialog is showing

Bug: 80362864
Bug: 111478851

Change-Id: Iec6e27285bf35238d378e6561896f1d0b303fa24

Merge "Ensure Notification re-post with setOnlyAlertOnce(true) silences sound."

Merge "UsbDescriptor: Add finite timeout to control transfer"

Ensure Notification re-post with setOnlyAlertOnce(true) silences sound.

Where a notification channel has an associated looping sound, it should
be possible to re-post the notification with setOnlyAlertOnce(true) to
cause the notification channel's sound to silence.

When re-posting the notification, this CL fixes an issue where
NotificationManagerService wouldn't call clearSoundLocked to stop playing
the sound, resulting in the notification continuing to playing the sound
associated with the notification channel.

Test: Manual testing, updated unit test
Bug: 110348674
Merged-In: I3ec26f0804bcf59f7356a2329b73ba8ed8f7ea51
Change-Id: I3ec26f0804bcf59f7356a2329b73ba8ed8f7ea51

Merge "Remove unneeded JNI method"

Merge "Track OkHttp changes"

Track OkHttp changes

As part of creating a full facade over OkHttp code there
are some changes to OkHttp-related classes that must be tracked.
See the related external/okhttp commit for details.

Some non-functional lint / WS changes have been made as part of this
change.

Test: Ran CtsLibcoreOkHttpTestCases entirely
Test: Ran CtsLibcoreTestCases entirely
Test: run cts-dev -m CtsNetTestCases -t android.net.http.cts.HttpResponseCacheTest
Bug: 111055375
Change-Id: I0b0ec21fa376ea33e2fa5486e203e92d3c3337cf

Merge changes If285d71a,Iffe5be7c

* changes:
  Revert^2: Build dark greylist from public + light grey API
  [DO NOT MERGE] Revert "Build dark greylist from public + light grey API"

Merge "Remove SimChangeListener from Tethering"

Remove unneeded JNI method

Test: ran PFD CTS tests
Change-Id: I9a40319595c757835f7e19a50dc0311077c9e627
Merged-In: I9a40319595c757835f7e19a50dc0311077c9e627
(cherry picked from commit d7bd9190759e3a6658455ada987453928caec8a8)

Merge "old paths contains all entries when updating"

Merge "Revert "Revert "Update SubscriptionInfo o use strings for mccmnc"""

old paths contains all entries when updating

In certains circumstances, only the base and split APKs were included in
the "old paths" list when updating the application info. Instead, this
list should contain _all_ elements, including any additional libraries
that may be added to the overall classpath.

Bug: 77342775
Bug: 80337129

Test: Manual. Install a package. Install a split with --dont_kill. See that the path doesn't contain duplicate entries
Test: Uninstall GMS updates and reinstall. Verify that GMS Dex files aren't extrated out of the APK.

(cherry picked from commit b9656a93aac435cefaa4ca0e1b06495d690ba019)

Merged-In: Id9739cce215ab07bff1b17966583c0cf51a0b34a
Change-Id: I3c61b1b61dc1ab8a1a51a5be68f19ae9af586692

Remove SimChangeListener from Tethering

SimChangeListener is obsolete. It is replace by
ACTION_CARRIER_CONFIG_CHANGED.

Test: as follows
    - built, flashed, booted
    - runtest frameworks-net passes (with --no-hidden-api-checks)
    - tethering/hotspot on/off
Bug: 111490073

Change-Id: I82644ea136f43869a953a1f7b72dc489fe90f380

Merge "Avoid use of DateUtilsBridge.icuTimeZone()"

Merge "Added documentation for network registration state"

Revert "Revert "Update SubscriptionInfo o use strings for mccmnc""

This reverts commit 0082b161ca42b41925fd5973f1779fc4c46f6034.

Reason for revert: build breaks fixed

Change-Id: I75ea63ceaae6e970ce51a971c9a4635d0ce3f9e4
Merged-In: I45c0bfefb7ffe98e3eab8e68d0e1170881ae9f4c

Merge "Fix unnecessary copy initialization warnings"

Added documentation for network registration state

Also renamed denial reason to reject cause to match
the 3GPP spec.

Bug: 73659459
Test: Build
Change-Id: Ia67ebf94771c7ff5f5d90f6cdd303cb2716f9186
(cherry picked from commit b4094993f742fbe9d07bd5783030aa82a3045f79)

Fix unnecessary copy initialization warnings

Bug: 30413862
Test: build with WITH_TIDY=1
Change-Id: I333d3a6b3394f50712148eeabf2253f1e01b21f3

Merge "Re-Number RIL Message IDs to Match RIL.h"

Avoid use of DateUtilsBridge.icuTimeZone()

DateUtilsBridge is only used in one place in framework.
The usage is avoidable by sticking to java.util. This
decreases the number of non-public libcore classes in
use by framework.

Test: build
Bug: 111055375
Change-Id: Ifd6576b5b6df1e154f16ec201a4575d0b95de66d

Merge "Update InCallService API docs to explain how to show incoming call UI."

Merge "Make Connection#onSilence a public API."

Merge "Revert "Update SubscriptionInfo o use strings for mccmnc""

Use StrictJarManifest.ATTRIBUTE_NAME_NAME instead of Attributes.Name.NAME.

Introduce a constant StrictJarManifest.ATTRIBUTE_NAME_NAME so that
callers (StrictJarManifest{,Reader}) don't need to rely on
Attributes.Name.NAME which was added via an Android patch.

Bug: 35910877
Test: Treehugger

Change-Id: I292fbefc6c5561609dcd8281d1e2a931a1ffa5b2

Revert^2: Build dark greylist from public + light grey API

The dark greylist is meant to contain non-SDK APIs which are "related"
to APIs accessible to apps without restrictions. The list used to be
built as all non-SDK APIs in the same package as those on the light
greylist. That is not sufficient and should include the public SDK API
packages too.

This patch modifies the makefile to generate a list of public API DEX
signatures with Doclava and then use that as a basis for the dark
greylist generation.

Bug: 80461842
Bug: 64382372
Test: make out/target/common/obj/PACKAGING/hiddenapi-blacklist.txt
Change-Id: If285d71ab82c3046fca94c325d664bd9073c1a69

[DO NOT MERGE] Revert "Build dark greylist from public + light grey API"

This reverts commit 6f6a126b9c64e49abcabcb6ea7b563535d22b04e.

Reason for revert: Bad merge into internal

Change-Id: Iffe5be7c10e8438bd46f71dc22e0867c0d316d40

Revert "Update SubscriptionInfo o use strings for mccmnc"

This reverts commit 0c149bd2d84d6e7c21dd4d7fff1e4dcfd51318f3.

Reason for revert:  reverting this topic as it breaks several branches.

Change-Id: Ia01984242e54b5db5d853135b322ebb1284a4d43
Merged-In: I45c0bfefb7ffe98e3eab8e68d0e1170881ae9f4c

Merge "Update SubscriptionInfo o use strings for mccmnc"

Re-Number RIL Message IDs to Match RIL.h

-Renumber RIL requests to match the message IDs
 for NATT Keepalive messages.
-Update comments to indicate RIL+HAL and HAL-only
 messages.
-Add new message offsets to explicitly support
 HAL-only messages in the list.

Bug: 110181475
Test: runtest frameworks-telephony
Change-Id: I58544f9291f426b5c195ece81113e15ed423d2df

Merge "Audio assets: add NFC sounds"

Merge "Revert "Added documentation for network registration state""

Merge "Remove Duplicate Override for Operator Alphas"

Revert "Added documentation for network registration state"

This reverts commit b4094993f742fbe9d07bd5783030aa82a3045f79.

Reason for revert: Caused merge conflict. Need to come up with a better solution.

Change-Id: Id7b7f35c25775a7a095b77a90724cf3a0f8daf7e

Merge "WRITE_APN_SETTINGS is also required to read"

Merge "Move XML object factory logic to libcore"

Merge "Added documentation for network registration state"