Merge changes from topic "null_callback"

* changes:
  Fixed null callback issue
  Added debug messages and watchdog

Fixed null callback issue

Fixed the vendor data/network service non-responsive
issue. The callback binder passed from frameworks
might be GC'd so that vendor data/network service skipped
calling the callback. This eventually caused data
connection state machine messed up.

Fixed by turning the weak refernce to the callback into
a strong reference. This ensure the binder alive when
vendor service needs to invoke the callback.

Test: Telephony sanity tests
Bug: 139076980
Merged-In: Ica0b7b810ffd5416ffd1b2b61f7ebc4af0dcb8ce
Change-Id: Ica0b7b810ffd5416ffd1b2b61f7ebc4af0dcb8ce
(cherry picked from commit 09ef1cb126ee59dcb4eacfc3a8afefcb0f991945)

Added debug messages and watchdog

Added debug messages and watchdog to catch
unresponsive data service issue.

Bug: 139076980
Test: Manual
Merged-In: I52dd126fddc1b1e8b18d5388a92aabaa3e2e48d5
Change-Id: I52dd126fddc1b1e8b18d5388a92aabaa3e2e48d5
(cherry picked from commit 36e2bfbaad3e1edcbb3ad0bf2be1e94643c38b07)

Merge "Add geo targeting implementation"

Merge "Added Part & Addr Constants."

Add geo targeting implementation

This CL adds support for geo targeting, including determining whether a
point is inside a simple polygon and circle.

For the judgment of the point in the polygon, the algorithm implemented
here simply considers the spherical polygon as a 2d plane polygon, which
may be not precisely. However, according to the test of real geographical
coordinates data i.e state/city/stree boundary data, this error is less than
10 meters that within 160 meters requires by the spec.

Bug: 123096618
Test: atest GeoUtils
Change-Id: I51e1fad3b48c1a58fa0510f97d9f5f9335779c30
(cherry picked from commit 7fbc74b66b58dd57deedb7336c5c1e2f0f9e2064)

Merge "Rename android.compat.IPlatformCompat"

Merge "Tweak RuntimeInit docs around MimeMap registration."

Merge "avoid generating reserved local MACs"

Merge "Move default MimeMap implementation to frameworks."

Rename android.compat.IPlatformCompat

To com.android.internal.compat.IPlatformCompat. This solves a java9
issue because libcore exported api has the same package android.compat.

Test: EXPERIMENTAL_JAVA_LANGUAGE_LEVEL_9=true make framework
Change-Id: I0918344f670669cecb04f1e9e54dbcb471b587d5

Tweak RuntimeInit docs around MimeMap registration.

Bug: 136256059
Test: Treehugger

Change-Id: I3f82a1684f47d96630badef4c13b3f507f30eaa5

Merge "Add entry to OWNERS."

Add entry to OWNERS.

Test: N/A
Change-Id: I1019015a81523e679a4a7529eb3e444283d0ba87

Move default MimeMap implementation to frameworks.

This CL topic moves the default MimeMap implementation to frameworks.
Libcore starts with a minimal implementation sufficient to pass
CtsLibcoreTestCases,  but frameworks can inject the real implementation.
Before this CL topic, the data files and logic (MimeMapImpl) were part of
core-*.jar on device; after this CL, they instead live in framework.jar.

Tests from MimeMapTest that check behavior of that default
implementation also move to a non-libcore CTS test.

Specifically, the logic and android.mime.types now live in
frameworks/base/mime. The default implementation is injected
into libcore from RuntimeInit. I chose to use a separate directory
(frameworks/base/mime/) and build java_library target ("mimemap")
in order to keep this as separate as possible from the rest of
frameworks code, to make it as easy as possible to factor this
out into a separate APEX module if we ever choose to do so.

Planned work for follow-up CL:
 1. Make CTS more opinionated, with a plan to assert that all of
    the default mappings are present. How exactly the expectated
    mapping will be bundled in CTS is still TBD.
 2. Add a vendor.mime.types file (defaults to empty) where vendors
    can add additional mappings; I plan to make it such that mappings
    in that file are parsed last but never override any earlier
    mappings, as if each mime type / file extension was prefixed
    with '?'.
 3. Perhaps enforce that public APIs android.webkit.MimeTypeMap
    and java.net.URLConnection.getFileNameMap() behave consistently
    with MimeMap.getDefault().

Test: atest CtsLibcoreTestCases
Test: atest CtsMimeMapTestCases
Bug: 136256059
Change-Id: Ib955699694d24a25c33ef2445443afb7c35ed9e7

Merge "Add a platform-compat-config to the framework services."

Merge "Modify start/stop condition for mScreenDozeTimer."

Merge "TvInputManagerService patch"

TvInputManagerService patch

Fix potential null pointer access at TvInputManagerService.java

Bug: 136411969

Merge "Add agent startup-attach"

Merge "glob pattern is used for AIDL files under frameworks/base"

Merge "Start platform compat service eariler."

glob pattern is used for AIDL files under frameworks/base

This change removes the manullay curated list of AIDL files and replace
them with globs.

In addition, framework-aidl-mappings no longer sets frameworks-defaults
to its src property, but instead uses the several variables like
framework_srcs, framework_aidl_local_include_dirs, etc. to get the
same files/dirs list as the framework.

The variables will eventually be replaced with filegroups when aidl
include paths are better handled (i.e. 'path' property of all filegroups
for a module contributes to the AIDL include paths for all AIDL files in
the module).

Bug: 70046217
Test: m

Merged-In: I59728ed06d66d44bc19bcd8530042c01add5fc2b
(cherry picked from commit 1cc9566ee26ff2e4ef9d738062409e868a21e34f)
Change-Id: I59728ed06d66d44bc19bcd8530042c01add5fc2b

Merge "Do not notify SS change with invalid subId"

Do not notify SS change with invalid subId

(This is a reupload of ag/8541406.)

If a service state change is reported as a subscription is becoming
invalid (i.e. no longer connected to a phone), the current behavior can
sometimes notify this change to all PhoneStateListeners regardless of
which subId they registered to listen on (namely this happens when the
subscription was attached to the default phone).

This change removes that behavior so that no listeners are notified if
the subscription is invalid.

Test: manually verified on DSDS that when
  1. erasing eSIM when it is default and
  2. removing pSIM when it is default,
  the service state triangle for the other SIM is unaffected
  Also made sure that signal strength is rebroadcast when sub becomes
  valid, so b/138357571 did not show up 5/5 times
Bug: 133140128
Change-Id: I2ba6a971fff4e29ccd2166c381308fcd953f5bc0
Merged-In: I2ba6a971fff4e29ccd2166c381308fcd953f5bc0

Merge "BatteryService: scheduleUpdate asynchronously"

Merge changes Ie18bce98,I1c600bc2

* changes:
  Add keylayout for Xbox One USB controller
  Keylayout for xbox controllers

Add keylayout for Xbox One USB controller

We are currently missing the key layout for the Xbox One USB controller
with the following meta information:
vendor 045e, product 02ea

This layout was copied from product 02d1.

Also fix the mappings of the middle buttons to generate "select - mode -
start", similar to what has already been done for the other xbox
controllers in ag/4836521 and ag/3162575

Bug: 132451971
Bug: 133514907
Bug: 139512030
Test: tested using custom app. CTS tests will be added later.

Change-Id: Ie18bce987b512211d3e91bd1f7334afe11d83cf8
Merged-In: Ie18bce987b512211d3e91bd1f7334afe11d83cf8

Keylayout for xbox controllers

Add layout for Xbox elite controller

Test: tested with a custom app
Bug: 132451971
Bug: 139512030
Change-Id: I1c600bc2c41db9d79d7a4e184ef41abe2b5f860e
Merged-In: I1c600bc2c41db9d79d7a4e184ef41abe2b5f860e

Start platform compat service eariler.

It's needed by ActivityManager and PackageManager.
Also use a constant in Context for the name.

Test: flashed device with ag/9025572 and ag/9204795 and the platfrom
compat was accessible.
Bug: 137769727

Change-Id: Ie1130a3f0bdd1769fe0755db0089702ea64d9db6
Merged-In: Ie1130a3f0bdd1769fe0755db0089702ea64d9db6

Add a platform-compat-config to the framework services.

Bug: 138222363
Test: m. the config builds and contains annotated changes.
Change-Id: I35ab9d028bbf23aca38494b66592779fdb4ef01e

Merge "Fix incorrect descriptions under SMS contract"

Merge "Cover framework code in hiddenapi-mappings."

BatteryService: scheduleUpdate asynchronously

IBatteryPropertiesRegistrar.scheduleUpdate calls IHealth.update()
asynchronously.

BatteryStatsImpl calls scheduleUpdate while holding a lock, which
may lead to deadlocks if the remote process calls back to the
framework.

Fixes: 139503418
Test: monitor HealthScheduleUpdate trace points
Change-Id: I8168d7c4e4a0b1d31343360d2c7f6d36c7aa692a

avoid generating reserved local MACs

Avoid generating WifiInfo.DEFAULT_MAC_ADDRESS as a randomized MAC
address since it's being used for another purpose.

Bug: 137796328
Test: atest MacAddressTest
Change-Id: Ia7beef0d0af5d7b39845e662cd343d81aef97702

Cover framework code in hiddenapi-mappings.

It was missing most of the source code outside of core/...

Test: m, m hiddenapi-mappings
Change-Id: I9559346d306499c2f9659b73a68962d640fe644d
Merged-In: I9559346d306499c2f9659b73a68962d640fe644d

Merge "API updated for ICU4J 64.2"

Merge "Update wifi interface ip state upon receiving ap disable event"

Merge "Move Half implementations to libcore to allow ART optimizations"

Merge "Move IPlatformCompat.aidl definition to framework.jar"

Update wifi interface ip state upon receiving ap disable event

When turning on/off SoftAp with high frequency, sometime calling
WifiManager#updateInterfaceIpState with IFACE_IP_MODE_TETHERED
is performed after handling WIFI_AP_STATE_DISABLED event in
WifiServiceImpl while SoftAp is being terminated. This leads to
the issue that SoftAp is unable to start as startSoftAp always
returns "false". This is because mIfaceIpModes in WifiServiceImpl
keeps the mode WifiManager.IFACE_IP_MODE_TETHERED and the CL below
rejects SoftAp start request.

e257c5c9e79e3675375ca20731cfb74df02b1064 @ frameworks/opt/net/wifi
(WifiServiceImpl: Reject startSoftAp when already tethering)

This CL updates the interface ip state upon receiving ap disable
event from Tethering class to set proper state to mIfaceIpModes.

Bug: 134806980
Test: Ran script to turn on/off soft ap frequently
Signed-off-by: Daichi Ueura <daichi.ueura@sony.com>
Change-Id: I2f89214414d93f1aa942fb8a21264a9baae3452a

Merge "Move telephony/cdma/ classes into appropriate package."

Merge "DO NOT MERGE - Merge pie-platform-release (PPRL.190801.002) into master"

Add agent startup-attach

Add ability to give 'startup-agents' that are automatically loaded
when a (debuggable) app starts. These agents are any files in the
'code_cache/startup_agents' directory. The agents are started with the
apps data_directory as an argument.

Test: Install debuggable apk (here com.antonioleiva.bandhookkotlin)
  walleye:/ $ run-as com.antonioleiva.bandhookkotlin sh
  walleye:/data/data/com.antonioleiva.bandhookkotlin $ mkdir code_cache/startup_agents
  walleye:/data/data/com.antonioleiva.bandhookkotlin $ cp /data/local/tmp/libtifasts32.so code_cache
  walleye:/data/data/com.antonioleiva.bandhookkotlin $ cp /data/local/tmp/libtifasts64.so code_cache
  walleye:/data/data/com.antonioleiva.bandhookkotlin $ cp /data/local/tmp/libchainagentss32.so code_cache/startup_agents/
  walleye:/data/data/com.antonioleiva.bandhookkotlin $ cp /data/local/tmp/libchainagentss64.so code_cache/startup_agents/
  walleye:/data/data/com.antonioleiva.bandhookkotlin $ echo $PWD/code_cache/libtifasts32.so=log,ClassLoad > chain_agents.txt
  walleye:/data/data/com.antonioleiva.bandhookkotlin $ echo $PWD/code_cache/libtifasts64.so=log,ClassLoad >> chain_agents.txt
  Start bandhookkotlin
  Examine logcat

Bug: 135627501

Change-Id: Ib82b27df90c7964a995288d8b2b3d348a11cdd80
Merged-In: Ib82b27df90c7964a995288d8b2b3d348a11cdd80

Move IPlatformCompat.aidl definition to framework.jar

There are non-app process usecases in framework code that need to have
access to this API.
Created a new package android.compat in frameworks/base/core following
previous definition of android.compat.Compatibility for app processes
(http://cs/android/libcore/luni/src/main/java/android/compat/Compatibility.java).

Bug: 137769727
Test: m
Change-Id: Ifc1b97ad40c2baf65a86169e101acfa72e3aae5f
Merged-In: Ifc1b97ad40c2baf65a86169e101acfa72e3aae5f

Merge "API: Explicitly track incompatibilities with previous API versions"

Move Half implementations to libcore to allow ART optimizations

Move Half FP16 implementations to libcore, to allow ART compiler to
optimize these methods with intrinsic implementations.

For example, on ARM64 with ARMv8.2 FP16 half registers and instructions:
- Half toFloat/toHalf can be implemented with FCVT;
- Half floor/ceil/round can be implmented with FRINT(pna);
- Half max/min can be implmented with FMIN/FMAX.

Such fast Half FP16 intrinsics can help accelerate ColorLong ARGB
encoding/decoding in Android framework.

Change-Id: I6225ebf8aa825b0394ce8f13e12db317f5c6e3fd

Merge "Get resource based on subId for multi-SIM"

Move telephony/cdma/ classes into appropriate package.

Not changing the java packages, but rather moving the source location. This allows running metalava on them.

Test: m
Change-Id: I3b46644361239d85818dd885f461dfc94de49dca

Get resource based on subId for multi-SIM

The resource loading is done based on the last SIM to come up
which is not a deterministic design. Thus, update the way to get
the resource based on the subId.

Test: atest FrameworksNetTests
Test: manually test with avoid bad wifi feature supported sim
Bug: 138956509
Change-Id: Ib5b085d97103889600773d269e03b939c29ca47d

DO NOT MERGE - Merge pie-platform-release (PPRL.190801.002) into master

Bug: 139369544
Change-Id: I835905a51836e7218f0be8c25335b9548ae6b2d3

Merge "Modify RCS UCE APIs to use AIDL Interfaces."

API updated for ICU4J 64.2

Expose new stable APIs from ICU 64

Bug: 125283072
Test: m api-stubs-docs-update-current-api droid
Change-Id: Id689aab0d4387725929ee41d80f3e8a26c0810fa

Merge "Prevent Infinite Recursion in Telecom Sessions"

Snap for 5674421 from ae0c2eeba3f60c62f63ec83b63d9c0593a0430fe to pi-platform-release

Change-Id: I5b1dc16c0696ce8452f8798e68fe417d7533c5df

Merge "Don't allow upstream if entitlement don't know any downstream"

Don't allow upstream if entitlement don't know any downstream

In case tethering donwstream is setup but don't ask EntitlementManager
to run provisioning. Cellular upstream should not be allowed if
EntilementManager don't have any donwstream record.

Bug: 134649258
Test: -build, flash, boot
      -manual test with carrier SIM
Change-Id: I609722c79f0a45a1c2164d39721ef5883e436feb

Prevent Infinite Recursion in Telecom Sessions

There looks to be a case where infinite recursion
is occuring when printing information about
Telecom Sessions. Set a reasonable limit of
iterations and throw a warning when this condition
occurs to better track down a root cause.

Bug: 139045282
Test: atest TelecomUnitTests:SessionTest
Merged-In: Iae69c5db070c5e58ae5d37c12d1c4df027138611
Change-Id: Iae69c5db070c5e58ae5d37c12d1c4df027138611

Merge "Add documentation for Locked Messages"

Merge "SurfaceControl: Fix captureLayers JNI"

API: Explicitly track incompatibilities with previous API versions

Bug: 139128921
Test: make checkapi
Change-Id: I1f17b57705e2f7350b55ab278e06fc4e988771ff

Merge "Fix reference leak on TvInputHardwareManager"

Merge "Fix a lock issue in AudioPortEventHandler"

SurfaceControl: Fix captureLayers JNI

Missed a call-point in last refactoring.

Bug: 135965618
Bug: 130211665
Bug: 133054270
Test: Manual
Merged-In: I43fa93eea7d79f5b773497e3bb65750866eca2cb
Change-Id: I43fa93eea7d79f5b773497e3bb65750866eca2cb
(cherry picked from commit bf9298f1c9b848ff0db4ae0be3913ad531bafa9e)

Merge "Add klhyun@ to media OWNERS"

Fix a lock issue in AudioPortEventHandler

It used "this" as a lock object to guard mListeners. but "this" is
pointing to mHandler in handler thread.
This causes the IndexOutOfBoundsException.

bug: 138951761
Test: manual
Change-Id: I826c7c68ecff4fc0d39addd3d56bc989e00848bc

Merge "Fix statsd ASAN failure"

Fix statsd ASAN failure

If tagId is not found in kAllPullAtomInfo map, StatsPuller will read
outside of the object. Add a condition check to gracefully handle
unrecognised tagId and fix the ASAN failure.

Bug: 139037732
Test: boot aosp_x86-eng
Change-Id: Ic62b89866dbec82f05e6f8e6afca9b312f44a801
Merged-In: I5db561c4223efc4ef6b1441490d319cc6ec8afc7
Exempt-From-Owner-Approval: trivial change, owner OOO.

Merge "Use String8 for UTF16 to UTF8 conversion."

Fix incorrect descriptions under SMS contract

Fixes: 117203793
Test: N/A, only modifies documentation
Change-Id: I5c4bc23e41bc0d1c97bccf7e288c4c41d8e1a438

Use String8 for UTF16 to UTF8 conversion.

This is the only actual user of libcutils' jstring.h.

Test: treehugger
Change-Id: I0444d42e5a3ca57de3bffca48aa265f267c3d4dc

Merge "Stop showing disclosure UI when isDisclosureEnabled is false."

Merge "Fix interfaceVersion in DhcpServerCallbacks"

Merge "Notify the ART runtime when boot is complete."

Merge "Upload platform compat config generated from framework target."

Fix interfaceVersion in DhcpServerCallbacks

The getInterfaceVersion method was missed when freezing the AIDL
interface version.
This causes Q devices to return 0 instead of 3 on this interface.

Bug: 139110717
Test: toggle tethering, dumpsys network_stack version
Change-Id: If587cc9ead4191b9c2ad2ae65473bd89e7c37b5c

Notify the ART runtime when boot is complete.

Needed for jit-zygote performance improvements.

Test: device boots
Bug: 119800099
Change-Id: I1e46f49d94440384473430d8afebe29fbee9c68e

Add klhyun@ to media OWNERS

Test: N/A
Change-Id: I4328217199581fcb181f74a5c3baabd3a9148136

Merge "Added disallowed APN types configuration support"

Stop showing disclosure UI when isDisclosureEnabled is false.

Bug: 137395050
Change-Id: Ic7768f7d3306f5d9d5c5c988afda4e8710c4ccd0
Test: Manual tests

Modify RCS UCE APIs to use AIDL Interfaces.

Integrate AIDL backend into RCS UCE APIs.

Test: Telephony Unit Tests
Change-Id: I7a1af154676797ce5adf7b037c883827f4085d52

Merge "Fix StorageManagerService linkToDeath more than one time when vold/storaged died."

Merge "Remove obsolete methods from DumpstateListener implementations"

Merge "API updated for ICU4J 64.2"

Merge "Update API signature file to reflect the correct logp nullability"

Remove obsolete methods from DumpstateListener implementations

Note that with the new Bugreporting API, SystemServer is the only
expected DumpstateListener implementation. Once we fully migrate Shell
app, we can remove the implementation in BugreportService as well.

BUG: 128980174
Test: bugreport from power menu, observe progress bar

Change-Id: I40d654a70bd9ceb3a29f8a0113b85616100f4ee9
Merged-In: I40d654a70bd9ceb3a29f8a0113b85616100f4ee9

Update API signature file to reflect the correct logp nullability

A matching patch to the fix in java.util.logging.Logging nullability.
See https://android-review.googlesource.com/c/platform/libcore/+/1096198/

Test: make -j update-api
Change-Id: I0b653de474b6162764cfa711eb967aada4c0ee50

Merge "Extract test utilities for ConnectivityService"

Merge "Add carrier config for data roaming notification"

Add carrier config for data roaming notification

This config, when enabled, shows the user a notification when the device
switches to data roaming and the user has data roaming on.

Bug: 133358875
Test: manual
Change-Id: Ia091575ff0fa7e02f0c6a98bae0e66ef73beff81
Merged-In: Ia091575ff0fa7e02f0c6a98bae0e66ef73beff81

Merge "Add a documentation in EmergencyNumber#getNumber"

Merge "Adds better logging for ImsFeature features"

Add a documentation in EmergencyNumber#getNumber

If the number starts with a carrier prefix, the
 carrier prefix is configured in
 {@link CarrierConfigManager#KEY_EMERGENCY_NUMBER_PREFIX_STRING_ARRAY}.

This is important because it provides a solid reference for Apps to
strip the prefix, which they may not prefer to display.

Bug: 137225076
Test: Treehugger
Change-Id: I1f6bd74f1d6eb86fe3326246e0ddb08b9cf7ac11
Merged-In: I1f6bd74f1d6eb86fe3326246e0ddb08b9cf7ac11
(cherry picked from commit 552a58ae51e4f8ef1a0bcefe9a475b4511454c17)

Merge "Added the min match system config for loose phone numbers comparison"

Added the min match system config for loose phone numbers comparison

Specific min match is required for each country.

Test: atest PhoneNumberUtilsTest

Change-Id: I7f9e9ad824d44f2ebf8367beb6ec337e3eb3a8ad
Merged-In: I7f9e9ad824d44f2ebf8367beb6ec337e3eb3a8ad
Signed-off-by: Taesu Lee <taesu82.lee@samsung.com>
Bug:134246556

Merge "Fix dependency on APK used by test"

Merge "Add documentation in TelephonyManager#getNetworkCountryIso"

Merge "Clean up duplicated uses-permission"

Merge "[view compiler] Remove dex_builder_test.cc"

Upload platform compat config generated from framework target.

Add compat-changeid-annotation-processor and upload the generated compat
config using the platform_compat_config rule.
Make both libcore and framework compat configs required for framework to
make sure they get onto the device. Libcore will be removed once the
config is uploaded to APEX.

Test: flashed device locally, config files found and read by
com.android.server.compat.CompatConfig.
Bug: 138222363

Change-Id: Ie04fde8b4b18e7e34d95257f5ef8ca78d5076b4f