Merge "Add support for send Message with messaging options"
am: 1400db6748

Change-Id: I29ffbb64c35214761b7f6921212920cf8ffc8599

Merge "Add support for send Message with messaging options"

Merge "EMBMS - Add TestApi and provide service override"
am: 1fab41e771

Change-Id: I5c44393e98c432b3c7a28c7b5b1906aa80f62372

Merge "EMBMS - Add TestApi and provide service override"

Merge "Prevent INTERACT_ACROSS_USERS exception in DefaultDialerManager."
am: 6d0f1eff8b

Change-Id: I55e7f4684f8160f70548e662cd030fce565b16a9

Merge "Enable HID Device Profile Service (2/2)"
am: 7e3fbd59d1

Change-Id: I993b08e37f0ebcc7bca0b2b536348c41dd548861

Merge "Handle alternate status values."
am: b728e56959

Change-Id: I1c82dd103b03eecba025a7638e06906699429c74

Merge "Prevent INTERACT_ACROSS_USERS exception in DefaultDialerManager."

Merge "Enable HID Device Profile Service (2/2)"

Merge "Handle alternate status values."

Prevent INTERACT_ACROSS_USERS exception in DefaultDialerManager.

When TelecomManager methods perform a read phonestate check from a work
profile user, it is possible to get an INTERACT_ACROSS_USERS exception.
This is due to the fact that the filterByIntent method was not calling
queryIntentActivitiesAsUser.

Test: Manual
Bug: 31304557
Merged-In: I6bc7419ba260050281d83e33f7f328ec5ccb3cd8
Change-Id: I2a1869162e5a52aed986a99cc378a6c630af4a70

Handle alternate status values.

Some carriers use sendrecv to indicate a call in a conference is active,
and sendonly to indicate that the remote party has held the call.
Adding support for these status values.

This ensures that remotely held calls for these carriers are set to held.

Test: Manual
Bug: 33272103
Merged-In: Ie386bdb4c77df1305d80f604f48174794c3ee0c1
Change-Id: I4a950c4e143d5ac396b30569dbda0b1cf4e92a3e

Merge "Ensure CallId is not null when connection creation is complete."
am: 47b23176bd

Change-Id: Ifddd99f22edfa787a3447cbaa66dd4e70a430ba3

Merge "Ensure CallId is not null when connection creation is complete."

Merge "Fix: AccountManagerService crashed by accessing closed db"
am: 6ac8f42d9e

Change-Id: I35f289e43a7406087a13258b5e6cbf5e179dc529

Merge "Fix: AccountManagerService crashed by accessing closed db"

Ensure CallId is not null when connection creation is complete.

When calling "get" on the mConnectoinById map, a null callId will result
in an NPE.

In findConnectionForAction, defaulting to returning the "null" connection
which is the same behavior as if it isn't found.
In notifyCreateConnectionComplete, specifically checking if the callId
is null and skipping the onCreateConnectionComplete callback.
This scenario is possible if the connection is remove from the
ConnectionService before the connection complete callback comes back from
Telecom.

Test: Manual
Fixes: 62588734
Merged-In: I1de7a0a4cc9c630019e38b8154bfc1d470f56ae3
Change-Id: Ie610c51155ed417e0f916000fe20e4484bdb6603

Merge "Fixed NullPointerException in IpSecService-Netd binder layer"
am: bf74a93eb2

Change-Id: Ia73c6bda7cb265f91648139e15f4bd2668e6c531

Merge "Add the operator name and MCC/MNC in string format into the CellIdentity."
am: 1565803cd2

Change-Id: Ie0a1305eb73256f358a853d87778f755d3b9e4d0

Merge "Fixed NullPointerException in IpSecService-Netd binder layer"

Merge "Don't allocate GlobalRefs for BinderProxy"
am: 1455a1d0c0

Change-Id: Id971af4742bb26790449c78f9520b9a7744df39a

Merge "Add the operator name and MCC/MNC in string format into the CellIdentity."

Merge "Don't allocate GlobalRefs for BinderProxy"

Enable HID Device Profile Service (2/2)

Add Profile Proxy for HID Device Profile.

* Add a helper method doUnbind() to deal with unbinding to Bluetooth HID
Service.

* Fix docstrings.

Bug: 63384609
Test: SL4A Bluetooth HID test
Change-Id: I168c21c3b010baac9889c556635b914c0ba3f267

Merge "Remove unused method on VersionedBroadcastListener"
am: 536d241fb2

Change-Id: I63c8ad050330dd4e938388d03fa9cf270b108e56

Merge "Remove unused method on VersionedBroadcastListener"

Merge changes from topic "tz_catchup1"
am: 4e0a02ce1f

Change-Id: I629930379afd2e3037bbd460ebc83726a8bb0008

Be more explicit about users
am: b755291e65

Change-Id: I0dd17273997adc207b5e389624b820e80ccaaa89

Merge changes from topic "tz_catchup1"

* changes:
  Add newly available version information to dumpsys
  Be more explicit about users

Remove unused method on VersionedBroadcastListener

Test: as follows
    - built
    - flashed
    - booted
    - runtest frameworks-net passes
    - manually enabling USB tethering toward Wi-Fi worked
Bug: 32163131

Change-Id: I38e4b77a7d59584ebeb450ebe5a9dcbcb36f4fa1

Merge "Convert Binder & BinderProxy to NativeAllocationRegistry"
am: bd6d3c5743

Change-Id: Iae1e438cc9595efe1dd68c579d42ae129ac53e5e

Add support for send Message with messaging options

Add support for following messaging options.

1.Add support for message priority in 3gpp2.

2.Add link control support while sending multiple messages.

3.Add validity period support in 3gpp.

Change-Id: I19f4fbc5a3b6922884196b301d30280acd9e501e

MSIM(Telephony): Add changes to SMS APIs as per new SmsManager design

Change SMS Msim APIs as per new SmsManager design

Change-Id: I14ad7685e23b14203a76dfe3cc5a95a337f7d0e3

Merge "Convert Binder & BinderProxy to NativeAllocationRegistry"

Fix: AccountManagerService crashed by accessing closed db

Symptom:
When a current user was switched, AccountManagerService crashed with
IllegalStateException.

Root cause:
accountsDb was closed when user was stopped. AccountManagerService
tried to insert a log record to the closed db and it failed.

Solution:
It catches the exception if it's failed to insert a log record.

Bug: 68233592
Change-Id: I97b63e4777bf7f8a1d38d96e494f6a21847d374b

Merge "IMMS should preserve enabled/selected IMEs upon boot"
am: 171c82acb4

Change-Id: I13b12269eb1cd29bb28bd6a1dadac59a2d539380

Merge "IMMS should preserve enabled/selected IMEs upon boot"

EMBMS - Add TestApi and provide service override

Add @TestApi annotations to methods that CTS needs
Add a manifest meta-data key that allows client apps to specify the
exact component name of the MBMS service to bind to.

Bug: 68049452
Test: CTS
Change-Id: I50654c41da38696a25fad93d0a5e0b0c11fa0b42

IMMS should preserve enabled/selected IMEs upon boot

This is a follow up CL to my refactoring CL [1], which unintentionally
changed the condition to reset default enabled IMEs when the device
boots up.

Previously, InputMethodManagerService (IMMS) resets default enabled
IMEs upon device boot only for the first boot scenario, by checking
whether Settings.Secure.DEFAULT_INPUT_METHOD is already set or not.

My refactoring CL accidentally replaced that check with unconditional
"true", which means now IMMS always resets default enabled IMEs every
time the device boots up.  This behavior change is of course
unintentional but has no effect after the user unlocks the device
because IMMS also discard all the settings change made while
UserManager.isUserUnlocked() returns false [2].  Hence the above
behavior change is completely hidden on direct-boot disabled devices,
where the system behaves as if the user unlocked the device
immediately after the boot is completed.  The behavior change is
observable only on direct-boot devices.

Anyway, IMMS should try to do its best to keep the last used IME and
user selected IMEs even in user locked state as long as those IMEs
are compatible with direct-boot.  This CL revives the previous
behavior by adding the same condition check again.

 [1]: I5b37c450db4b25b3e635b6d634293a34eec8b9d4
      7924782c000733b2d7a180701b74988f0154adee
 [2]: Ifa2225070bf8223f8964cf063c86889e312c5e9a
      ed4952ad0f76a70549777472cd9cefcbc8705917

Fixes: 67093433
Fixes: 67491290
Test: Manually verified as follows
      1. Check out AOSP master
      2. Build an OS image for a direct-boot aware device then
         flash it.
      3. Open
            development/samples/SoftKeyboard/AndroidManifest.xml
         then add
            android:directBootAware="true"
         to the IME service.
      4. Open
            development/samples/SoftKeyboard/res/xml/method.xml
         then add
            android:isAsciiCapable="true"
         to "en_US" IME subtype.
      5. tapas SoftKeyboard
      6. make -j
      7. adb install -r $OUT/system/app/SoftKeyboard/SoftKeyboard.apk
      8. Open system settings:
           System -> Languages & input -> Virtual keyboard
     -> Manage keyboards
      9. Enable Sample Soft keyboard
     10. Disable Android Keyboard (AOSP)
     11. Open system settings:
           Security -> Screen lock -> Password
         then set up a device password "aaaa".
     12. Reboot the device.
     13. Make sure that
          * Sample Soft keyboard is shown in the unlock screen.
          * AOSP Keyboard is not enabled in the unlock screen.
Change-Id: Id624d577d941245cca944498dc6935eb364865cf

Don't allocate GlobalRefs for BinderProxy

This removes all GlobalRef allocation as part of building BinderProxys.
Previously these were used to map IBinders to the corresponding
Java object, so the Java objects could be reused. We now keep
that mapping at the Java level.

This means we often need to call into Java to look up or allocate
a BinderProxy. But this replaces a prior call to Java to dereference
a WeakReference. The Java custom Java map-to-WeakReference data
structure is probably not terribly efficient, but the original
attachement mechanism did not seem to be either. And this
avoids potentially even more catastrophic issues when the number
of GlobalRefs approaches its limit.

We decrease GC triggering frequency from 200 to 1000 allocated
references. This now only applies to other kinds of JNI References
allocated by Binder.

I saw a maximum bucket size of 16 for the ProxyMap data structure
while briefly exercising a freshly booted device. That occurred
in system_server.

Bug: 65760710

Test: Built and booted master with some debugging output. Looks sane.
Change-Id: I322c4d8e9c8e198586d591580c2cdbb094906677

Fixed NullPointerException in IpSecService-Netd binder layer

Replaced null with empty byte array in createTransportModeTransform
for null auth.getKey, crypt.getKey() and authCrypt.getKey()

Test: Ran CTS test
Bug: 68052730
Change-Id: I5110b1297b4bfbb5766e8ecfd3d64f8110b52945

Merge changes I59b6e04f,I55694d89
am: 9009eabbfe

Change-Id: I217e32a7ab25fd73f3361015efae9bbbd1fde0f4

Merge changes I59b6e04f,I55694d89

* changes:
  Remove Parcelable interface from DefaultNetworkEvent
  Extract logging of default network events

Merge "HwBinder: Clean up code, add logging"
am: 28ed91161d

Change-Id: Ie7d511812c6dc419ca793c28a4cee1150a90bd66

Merge "HwBinder: Clean up code, add logging"

Merge "Convert framework-protos to Android.bp"
am: d4609af986

Change-Id: I6ebf88a0931b7f9f49ac8044deb46b100129bb9f

Merge "Convert framework-protos to Android.bp"

Convert Binder & BinderProxy to NativeAllocationRegistry

Change the Java BinderProxy to only contain a single native pointer,
so that we can get by with a single NativeAllocationRegistry
registration. This adds some indirections and a new allocation. But it
marginally reduces the number of (expensive) JNI field lookups from
native code, and the extra allocation involves significantly less
overhead than registering each object twice. This also cleans up the
code a little by avoiding some explicit reference count adjustments.

Change BinderProxy Binder to use NativeAllocationRegistry instead of
finalize().

Change the mObject field in Binder to hold a non-reference-counted
but owning pointer to JavaBBinderHolder. Have JavaBBinderHolder no
longer inherit from RefBase.

Make it clear that neither Binder.mObject, not BinderProxy.mNativeData
can be null. Remove null checks.

Avoid checking for null returns from C++ new. It would throw
anyway, which would cause the process to abort.

Test: Booted master.

Bug: 65760710

Change-Id: I323d4bdc7e25f8c27b847b6fe2c073eac3f2efe5

Merge "Fix issue #64224738: Document return value of IBinder.transact()"
am: ceb336fc8a

Change-Id: Ibcd3386a93a3bb5de8fb84a178678dab67beb464

Merge "Fix issue #64224738: Document return value of IBinder.transact()"

Merge "android.os.HwBinder: fix native static method signature"
am: 65944e79b2

Change-Id: I09295da4a6021bd02c5c5e89d1d4ce3a8be82c96

Merge "android.os.HwBinder: fix native static method signature"

HwBinder: Clean up code, add logging

Used ScopedUtfChars to handle lifetime, and correctly throw NPE. It
is illegal to call JNI String functions with a null argument.

Output the service name when starting a thread pool to aid debugging.

Bug: 68217725
Test: m
Test: Device boots
Change-Id: Ie40d3e77a63918330ef1a8a9e0c294f2c451a52a

Fix issue #64224738: Document return value of IBinder.transact()

Also add appropriate @NonNull and @Nullable annotations.

Test: built

Change-Id: I22de48105ef685baf594cfc004dd3e27e2ba09e9
Merged-In: I22de48105ef685baf594cfc004dd3e27e2ba09e9
(cherry picked from commit 4cd650c0085e6dd20d3f46c5b668e54537f887cf)

Merge "Fix dial emergency call redial issue by China Unicom."
am: 3ab81b9694

Change-Id: I3895caab83c75a8f0d0251da6154d5a93fe09694

Merge "Fix dial emergency call redial issue by China Unicom."

android.os.HwBinder: fix native static method signature

Test: pass
Change-Id: I54c384abaad7c7aafab31836f34edee5c7c75aac

Merge "Make it possible to dial an operator specific code"
am: f4f4ba3d42

Change-Id: I9af399f6ab0d5f95e76b0fd3cc769acc6a75a5b5

Merge "Make it possible to dial an operator specific code"

Merge "Improve NetworkCapabilities docs."
am: 14a51058b1

Change-Id: Ib3c6dd78c7ed41526a734fbc91a81cb4c7fae6d7

Merge "Improve NetworkCapabilities docs."

Add the operator name and MCC/MNC in string format into the CellIdentity.

Bug: 63984327
Test: Basic telephony sanity
Change-Id: Ia5e7f5b980efbde647f95f3e83a62d56132b90da

Merge "Switch to listening for CarrierConfig changes for provisioning rechecks"
am: c21effd526

Change-Id: I58dfd65873b65a59b832afea88f8d246b088aa6b

Merge "Switch to listening for CarrierConfig changes for provisioning rechecks"

Add newly available version information to dumpsys

The tzlookup.xml file now contains a rules version
so we can output that from dumpsys timezone now
too. That is used in xTS tests to confirm that
the correct tzlookup.xml file is being used
on device.

Bug: 65575839
Test: PTS: run pts -m PtsTimeZoneTestCases
Change-Id: Ib8cd3a41c625c6b442c0b9775108e4015ed020e2
Merged-In: Ib8cd3a41c625c6b442c0b9775108e4015ed020e2
(cherry picked from commit c90361404a1560d60756e4199af04d25ef688e2c)

Be more explicit about users

Be more explicit about users executing processes in the time
zone updates code.The code was already running everything as the
system user but now that's more explicit / by design.

To service unit tests:
make -j30 FrameworksServicesTests
adb install -r -g "${ANDROID_PRODUCT_OUT}/data/app/FrameworksServicesTests/FrameworksServicesTests.apk"
adb shell am instrument -e package com.android.server.timezone -w com.android.frameworks.servicestests \
  "com.android.frameworks.servicestests/android.support.test.runner.AndroidJUnitRunner"

Bug: 64111659
Test: Unit tests: See above
Test: Manual testing installing updates as secondary device user
Test: PTS: run pts -m PtsTimeZoneTestCases
Change-Id: Idb754f3e1aa3830ba1ada8ef5740f9f7340f03d5
Merged-In: Idb754f3e1aa3830ba1ada8ef5740f9f7340f03d5
(cherry picked from commit 1281f39f86a8e4d9950581fb1f15245843856c23)

Merge "Revert "HIDL Java getService now the same as C++.""
am: 65d1e549b6

Change-Id: I545166bd11ee2a38b43743ebb4559429e1b22349

Merge changes from topic "keystorebp"
am: a12eab69d1

Change-Id: I4a6f7f84d683582d126a6370e1cf3ae6ee9f4b19

Export AIDL for keystore Android.bp.
am: 01478ec183

Change-Id: Ie2308e809410e1c8237095c7f74bce08b8f76397

Merge "Revert "HIDL Java getService now the same as C++.""

Revert "HIDL Java getService now the same as C++."

Appears to be causing problems on 2017 devices.

This reverts commit c1f00f8cd895e38c74ed1cb1adff33065ccecd06.

Change-Id: I6c1717a612bfa3fe63f08160a1db652c284f6ba4

Merge changes from topic "keystorebp"

* changes:
  Convert lockregioncodeinjection to Android.bp
  Export AIDL for keystore Android.bp.

Remove Parcelable interface from DefaultNetworkEvent

This patch takes advantage of the direct DefaultNetworkMetrics interface
between ConnectivityService and IpConnectivityMetrics and removes the
Parcelable interface from DefaultNetworkEvent.

IpConnectivityMetrics, IpConnectivityEventBuilder and associated tests
are updated as necessary.

Bug: 34901696
Test: runtest frameworks-net
Change-Id: I59b6e04fc126051320d08a422cfbd4d27042123e

Extract logging of default network events

This patch extracts the logging of DefaultNetworkEvent from inside
ConnectivityService and move it to a new DefaultNetworkMetrics class.

The DefaultNetworkMetrics is a singleton owned by the
IpConnectivityMetrics singleton implementing the metrics service for
core networking. ConnectivityService has access to this singleton via
LocalServices.

This class layout will allow to remove the Parcelable interface of
DefaultNetworkEvent and will instead let the IpConnectivityMetrics
service grab metrics from the DefaultNetworkMetrics directly.

Bug: 34901696
Test: runtest frameworks-net
Change-Id: I55694d89124272732aba114198776462372de18b

Improve NetworkCapabilities docs.

Add IntDef for constants, and rely on new auto-documentation feature
to expand all of them at usage sites.

Test: docs-only change
Bug: 64133169
Change-Id: I8a6b5f54c8eb9d4fc7ae3d0d3fb673d52320664b

Merge changes I119cf83a,Icc5a1e38,If07a04d3
am: 61e3189ec7

Change-Id: I2ef7c49b23314a5cb94671bba0d3891b6b7c3920

Merge changes I119cf83a,Icc5a1e38,If07a04d3

* changes:
  Binder: Update to new ScopedLocalRef
  Binder: Clean up JNI code
  Binder: Change reaction to Error

Merge "HIDL Java getService now the same as C++."
am: 0c41ad872f

Change-Id: I9e161fd316d0bae5f3e4449adcb454924531fe11

Merge "HIDL Java getService now the same as C++."

Convert framework-protos to Android.bp

See build/soong/README.md for more information.

Test: m checkbuild
Change-Id: Id7992ff0cc5d631b64c1fa8e82e8a130ea508236

Binder: Update to new ScopedLocalRef

Follow-up to commit 625e0007ac07c5aefb4b3d486575ab916116c129 and
commit fac8cb2405999d18aefd75387a5bbdb0c6160e7f.

(cherry picked from commit 8571ec370da1b629aa8eb7267f139f1610627857)

Bug: 64689630
Test: m
Merged-In: I119cf83a5a54c15973ed3701ee448a9590bb404d
Change-Id: I119cf83a5a54c15973ed3701ee448a9590bb404d

Binder: Clean up JNI code

Factor out Error handling code and clean up. Use ScopedLocalRef,
fix minor logic issues.

(cherry picked from commit 625e0007ac07c5aefb4b3d486575ab916116c129)

Bug: 64689630
Test: m
Test: manual test
Merged-In: Icc5a1e38fa66aa79d87d0abb81acef6918ca78f5
Change-Id: Icc5a1e38fa66aa79d87d0abb81acef6918ca78f5

Binder: Change reaction to Error

Instead of aborting immediately with a native error, try to call
Thread.dispatchUncaughtException to trigger a Java-side abort.
Only fall back to the old strategy if this fails.

(cherry picked from commit 1cd76f5c93bc1e2409c880cade19a323cd999aed)

Bug: 64689630
Test: m
Test: manual test
Merged-In: If07a04d3769f3b9181c4736220333df9a2871a16
Change-Id: If07a04d3769f3b9181c4736220333df9a2871a16

Export AIDL for keystore Android.bp.

Test: builds, boots
Bug: 67041047
Change-Id: Iaff1de1f9c015a824ffc1f4946ced4a13d9bd7c1
Merged-In: Iaff1de1f9c015a824ffc1f4946ced4a13d9bd7c1
(cherry picked from commit 0c686e78ad2fbfc9e814bdd3abc5bd66c858c60f)

Convert lockregioncodeinjection to Android.bp

See build/soong/README.md for more information

Test: m checkbuild
Change-Id: I78e91a70e786df8e6722c6ec3fe762371e8f475a

Make it possible to dial an operator specific code

Added a new array config that indicating list of
FAC (feature access codes) to dial as a normal call.

Test: manual - Checked that special codes which configured in
CarrierConfig are dialed as a normal call.
Bug: 67664279

Change-Id: I1c4b60ae4f28c25e55b38563fe445f831793d466

Merge "Copy layout file of preference category from Settings."
am: 03c6030ce9  -s ours

Change-Id: Ibacd6e572b31105fe3afe1a904c15ed1d610fc41

Merge "Copy layout file of preference category from Settings."

Copy layout file of preference category from Settings.

Platform packages other than Settings may also use this layout to
be consistent with Settings.

Test: manual
bug: 62200881
Change-Id: I9e45772abac5aacfabbbafb38e287d0b523ef694
Merged-In: I9e45772abac5aacfabbbafb38e287d0b523ef694

Merge "Return a port from bindToPort() in IpSecService"
am: 9305fb7ea1

Change-Id: I51705d5db6af560f2423ef7861bfde6734632ec9

Merge "Return a port from bindToPort() in IpSecService"

Merge "Use ASM6 Opcodes everywhere in locked_region_code_injection"
am: 5aa4dafa58

Change-Id: Ic8f892399cdfa746a2fab4a841469f41f0fe98f5

Merge "Use ASM6 Opcodes everywhere in locked_region_code_injection"

Merge "Preserve order of shared library files"
am: 596bb64796

Change-Id: I95c1cd65ded3b3d14b7f916c8a10825399e7b8ec

Merge "Preserve order of shared library files"

Merge changes from topic "bt_get_cod"
am: 35dc02004c

Change-Id: Iec67ef3cc9c990b4d5285b23b51972915cbbd335

Merge changes from topic "bt_get_cod"

* changes:
  Add missing intents to protected broadcasts
  Get Bluetooth Class of Device

Merge "Persist Bluetooth CoD config value across reboot"
am: ba45409c92

Change-Id: I4d28d0b1b52e587285d3b92b66fdc4b6c1b38720

Add missing intents to protected broadcasts

Intents broadcasted by the Bluetooth stack need to be added to protected
broadcasts else they complain with errors in the log since Bluetooth is
considered a system process. A few of these intents, especially AVRCP
intents were missing in the list.

Bug: None
Test: Verified errors don't show up after introducing the intents while
playing on a sink device.

Change-Id: Ie283c50243d9a1bac3fd937f58bc1ddc20867d2f

Get Bluetooth Class of Device

Adds API to fetch Bluetooth CoD value of the stack. The API is hidden to
only be used by System code.

Bug: 36015415
Test: Verified fetching of COD from test app on flashed device, after modification
and after after reboot.

Change-Id: Ie35ecf141704c2aac46678da7cabdc7203a088f2

Merge "Persist Bluetooth CoD config value across reboot"