Merge "Adds APIs to extract/import arrays of primitive Java types out of/into a HwBlob."
am: a546c38130

Change-Id: I969ec51ed8fc6c7e11e7cfe7c4e1a3d6aa5802de

Merge "Adds APIs to extract/import arrays of primitive Java types out of/into a HwBlob."

Merge changes I2ef3737f,If080281f
am: bb3de35e5f

Change-Id: Ibe1af669d200057f1d91142299055694b55b5660

Merge "Change storage migration to use quota APIs."
am: 0fccecc6a0

Change-Id: I15f1aa9f01b5cef928b8b68e2568bf67e1e387b9

Merge "Reland "HIDL Java getService now the same as C++.""
am: c1616f6820

Change-Id: I5d53c369ae9cb1bc963b06e734cf9c47ad61265f

Merge changes I2ef3737f,If080281f

* changes:
  Add property for system server compiler filter
  Compile secondary dex at boot after OTA

Merge "Change storage migration to use quota APIs."

Adds APIs to extract/import arrays of primitive Java types out of/into a HwBlob.

Bug: 68654583
Test: hidl_test_java
Change-Id: I6ec799b423396f9212e4c221e41791adc9086410

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

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

Add getService(retry) for Java for C++ feature parity.

Test: hidl_test_java, boot
Bug: 67981006

Change-Id: I4e6711f8c2d7ad0ef6994ce9c480118320e1bde8

Add property for system server compiler filter

The property is dalvik.vm.systemservercompilerfilter, this changes
if dexopt is needed.

(cherry picked from commit 5e07a0d9cacb1bf943f4617b1edcb930e1578dad)

Bug: 62356545
Test: adb shell setprop dalvik.vm.systemservercompilerfilter quicken
Test: adb shell stop && adb shell start

Merged-In: I2ef3737f24816a4cb75e718abf274a891eb9fda1
Change-Id: I2ef3737f24816a4cb75e718abf274a891eb9fda1

Compile secondary dex at boot after OTA

(cherry picked from commit ce54397368dc98182d7b4eb2ff3c142bbd87e39d)

Bug: 38413085
Test: build successful
Merged-In: If080281fb3147fea625bcafce91167e8c1439736
Change-Id: If080281fb3147fea625bcafce91167e8c1439736

Change storage migration to use quota APIs.

New quota APIs are much faster than trying to measure manually, and
removing this last user of calculateDirectorySize() means we can
remove it once and for all.

(cherry picked from commit c8b29ac6f0340bd0c3c9eb388c4be2d446de9a59)

Bug: 36056324
Test: builds, boots
Merged-In: Ibdf1ee4e8885680e106df6a9269b6309ddc61af8
Change-Id: Ibdf1ee4e8885680e106df6a9269b6309ddc61af8

Merge "Put libcore generated source files into LOCAL_INTERMEDIATES_SOURCES"
am: 834be81189

Change-Id: Iec1547cc91bf3cb08a47194e91838eb77b4126d7

Merge "Put libcore generated source files into LOCAL_INTERMEDIATES_SOURCES"

Merge "OWNERS: Prefer Jeff Sharkey's @android.com account over @google.com"
am: b519569eb3

Change-Id: Ib9099673a4b643b10f38a0c94ddea49949f796eb

Merge "OWNERS: Prefer Jeff Sharkey's @android.com account over @google.com"

Merge "Fix Robolectric Bluetooth tests"
am: 01c9c0e4ba

Change-Id: I7987ddaa520c202a15af03bf0791c39a1720d3b3

Merge "Fix Robolectric Bluetooth tests"

Fix Robolectric Bluetooth tests

The Bluetooth HID profile name BluetoothInputDevice is recently changed
to BluetoothHidHost, but it is not updated in the SDK for Robolectric
tests, and it causes problems with SettingsLib tests. Thus, we need to
replace the HID profile use case with PAN. This change will not
influence the test coverage.

Test: make RunSettingsLibRoboTests
Bug: 68213274
Change-Id: If30ef0e82252d8f7229815b80e403ae63ab10643

OWNERS: Prefer Jeff Sharkey's @android.com account over @google.com

He prefers to use his @android.com account.

Test: Treehugger
Bug: 63673347
Exempt-From-Owner-Approval: Allow @android.com to approve this CL :)

Change-Id: Ie80cc7864de7b91ae44ad57f0bae9b859d034803

Merge "Track moving of OkHttp facade API to new package."
am: 45183fc5a7

Change-Id: Ied0bf8a62be2dd2038adf3de2251419bf5f3e964

Merge "Track moving of OkHttp facade API to new package."

Track moving of OkHttp facade API to new package.

Another CL in this topic moves the classes from
  libcore.net.http
to
  com.squareup.okhttp.internalandroidapi.
In jarjar'ed build targets, this becomes
  com.android.okhttp.internalandroidapi.

This facade constitutes the API via which non-libcore parts
of the Android platform (currently framework) may access
OkHttp. It's moving because libcore.net.http is already
part of libcore, and the overlap of packages is problematic
for builds with EXPERIMENTAL_USE_OPENJDK9 set to true.

Bug: 68220880
Test: Treehugger
Change-Id: Ia79966563cc0b5ab0923d54c21e54b6192d8c990
Exempt-From-Owner-Approval: Jeff Sharkey is an owner, but only one of his accounts is listed as an owner (@android.com vs. @google.com)

Merge "Quick Settings: Avoid expand indicators shortly showing on edit"
am: 47c83a5f9d

Change-Id: Id07c6a9413fb3f647d8ead5a77012800ce5b1bc8

Merge "Quick Settings: Avoid expand indicators shortly showing on edit"

Merge "Define OWNERS for permission grant changes."
am: 8d2a5bd42a

Change-Id: Ib91c1ecc3d673ba103f966008609d2444107eb7e

Merge "Define OWNERS for permission grant changes."

Define OWNERS for permission grant changes.

Permission grant changes need explicit approval from product and
SWE leadership to ensure that CTS/CDD requirements are met.

Change-Id: I0a84712b5c5b4e11e72bf5921b72ef6a47e7fdc1

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

Change-Id: I29ffbb64c35214761b7f6921212920cf8ffc8599

Merge "Add support for send Message with messaging options"

Quick Settings: Avoid expand indicators shortly showing on edit

 * Upon entering of the Quick Settings edit view, during
    the animation, all the expand indicators are rarely
    visible for a short period of time until hidden

 * To avoid this visual glitch, hide the expand_indicator
    by default and let the code show it when necessary
    instead of hiding it for most cases

Change-Id: Ia14d9a8d63178ccb54fbcb969ddf51f06c88f35c
Signed-off-by: Adrian DC <radian.dc@gmail.com>

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

Put libcore generated source files into LOCAL_INTERMEDIATES_SOURCES

LOCAL_SRC_FILES is expected to be a list of files relative to the
Android.mk files.  If OUT_DIR is absolute then adding generated
files to the list produces bad paths when $(LOCAL_PATH)/ is prepended
to it.  Put libcore's generated sources into
LOCAL_INTERMEDIATE_SOURCES instead.

Bug: 64930165
Bug: 68375156
Test: m checkbuild
Change-Id: I4b36fd192570ea0cd52d8537a16c078c726d97fd

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++.""