Merge changes Ie762ce75,I611fd791
am: 21a57f263f

Change-Id: Ibb291720f6c257a8ead2c039f584f4b3ecc69be5

Merge changes Ie762ce75,I611fd791

* changes:
  ConnectivityManager: unit test for argument validation
  ConnectivityManager: uses service error codes and exceptions

Merge "Remove obsolete DEXOPT_SAFEMODE."
am: 383b2af3ec

Change-Id: I97ccd9f087366d93495e3aeb8b26780039fd417b

Merge "Remove obsolete DEXOPT_SAFEMODE."

ConnectivityManager: unit test for argument validation

Bug: 36701874, 37107940
Test: new test passes
Change-Id: Ie762ce758b3d94052b7438a67fc55bef4690cbbb

ConnectivityManager: uses service error codes and exceptions

This patch introduces between ConnectivityManager and
ConnectivityService a mechanism for propagating back to clients
informative errors in the form of error codes and associated custom
runtime exceptions.

Without error code, the service can only throw a limited number of
different exceptions over Binder. Furthermore the throw site stack
traces are always loss. Although for individual instances of a throw,
the error message can be inspected, aggregations of stack traces from
app crashes sanitize error messages and only leaves the stack traces.

This makes debugging dificult for some service calls such as
requestNetwork that can have a variety of failure modes.

In this patch only one failure mode is codified. More can be added later
at a light cost by: 1) defining an error code, 2) defining an
associated exception, 3) mapping the code to the exception. This patch
can serves as a template for doing so.

Test: $ runtest frameworks-net,
      #testNetworkRequestMaximum() detects the new exception type.
Bug:  36556809, 36701874
Change-Id: I611fd7915575c9e418f7149fcdc8a879d2a3716d

Merge "Bluetooth: HID: Add support for Set Idle and Get Idle Commands (3/4)"
am: 51876abe82

Change-Id: Ia3242c0c2d7111b356c3ad2c70e8b1bea4572e9a

Merge "Bluetooth: HID: Add support for Set Idle and Get Idle Commands (3/4)"

Merge "Notify ImsService Status Correctly"
am: 1bb9181380

Change-Id: I694072b5125151cd1db6b88774c09126199e7cc0

Merge "Notify ImsService Status Correctly"

Merge "Add Save and Restore of BluetoothOn setting"
am: bbcc641317

Change-Id: Ie967141aa0d8f43750f84fdd6071c26e0a26958a

Merge "Add Save and Restore of BluetoothOn setting"

Merge "Increase limit of locale pairs"
am: aa9a59ab32

Change-Id: I4f9d25169bbd7e53c5396e944613e8c15fe4e308

Merge "Update startStreaming to return status code"
am: 6476ceeaee

Change-Id: I92f8fad5bff456b158ba2afd55c3065678b0d5cf

Merge "Increase limit of locale pairs"

Merge "Update startStreaming to return status code"

Notify ImsService Status Correctly

Modifies ImsService to allow multiple status callbacks in
for one ImsFeature. This better handles one ImsFeature
for normal/emergency calling.

Bug: 38001858
Test: Unit Testing
Merged-In: I70ae6f5349aef75aa86d54fe37a3c32459ea3afa
Change-Id: I70ae6f5349aef75aa86d54fe37a3c32459ea3afa

Add Save and Restore of BluetoothOn setting

This change will automatically save the Bluetooth On setting when
the user chooses to backup the phone settings into the cloud. This
setting is restored by the Setup Wizard (SUW) when configuring the
phone and this change will enable or disable the Bluetooth based
on this restored setting.

Bug: 35657817
Test: Manual test with Sailfish
Change-Id: Ie4518593af63f96f8c363f98941ca5260a3ec4bb

Bluetooth: HID: Add support for Set Idle and Get Idle Commands (3/4)

Provides an interface for application to send Set Idle
and Get Idle commands to remote HID Device. Support for these
two commands was missing from existing code, so existing code
design is reused to add support for these two commands.

Without this support following mandatory PTS test cases for HID 1.0
cannot be passed, TC_HOS_HID_BV_05/06.

Test: Executed PTS tests TC_HOS_HID_BV_05/06 and confirmed if they can
pass

Bug: 34344715
Change-Id: I548362cc328498920b2dae740f1a76b2cc2d6a67

Remove obsolete DEXOPT_SAFEMODE.

PackageManager side of the change.

bug:37929796
Test: build

(cherry picked from commit 56123bacff220e6ac77a1179197578d3a62ecdd2)

Change-Id: I3a789d49121ca383f7931f3803b6140be456340f

Merge "Statically include android.test.runner classes in legacy-android-test"
am: 9588b607f0

Change-Id: I17d3c541190c3301b27ee09806681615d97a6240

Merge "Statically include android.test.runner classes in legacy-android-test"

Merge "NetworkStatsFactory: no double looping on stats + stacked interfaces"
am: 1683224416

Change-Id: Id4987bc7f3cd486d0a0eeb1c2e5069076377de0a

Merge "NetworkStatsFactory: no double looping on stats + stacked interfaces"

Merge "Bluetooth: Fix Periodic Adv interval check"
am: cb70647418

Change-Id: I2bd2aec1b09401ea66d8e3df5654248d676d2fb7

Merge "Bluetooth: Fix Periodic Adv interval check"

Bluetooth: Fix Periodic Adv interval check

The MIN and MAX values need to be swapped else
the parameter check in setInterval would always
fail

Bug: 37579882
Change-Id: I48538c2a61be88caaf04abd94074b3d9eb6dde96

Merge "Changes to MbmsStreamingManager for test app"
am: df57734354

Change-Id: I661e2cb4609f556d31e464d1cadff459b9b80354

Merge "NsdService: more cleanups"
am: 903d64634a

Change-Id: Id61e1e2e240134c44e06bc9d78075ffcff8b5ff6

Merge "Changes to MbmsStreamingManager for test app"

Merge "NsdService: more cleanups"

Merge "NetworkStatsFactory: fix double accounting on IPv6 only networks"
am: ba666e0621

Change-Id: I6c25c2f5f95ac2ffefe263b34e95009b9823c254

NetworkStatsFactory: no double looping on stats + stacked interfaces

This patch uses the associative structure of the stacked/base interface
collection to avoid the two nested for loops over stacked/base
interfaces and all stats entries when correcting 464xlat traffic stats.

Consequently the list of stats entries is iterated only once.

Test: runtest -x frameworks/base/../NetworkStatsFactoryTest.java
Bug: 33681750
Change-Id: I84ae997fb693b909f431764697627b9957131732

Merge "NetworkStatsFactory: fix double accounting on IPv6 only networks"

Merge "Bluetooth: Fix interval low/min values"
am: 66deee2bd4

Change-Id: I42d9a56ca85715dae2093bada1ecadd0a4c0c74e

Merge "Bluetooth: Fix interval low/min values"

Bluetooth: Fix interval low/min values

Bug: 38175945
Test: none
Change-Id: Idf487c7aafc518f2c57be78a1fa979b2c4a383d6

Merge "Switch from libcrypto_static to libcrypto target"
am: 4ad5256547

Change-Id: Iba529dac6e8c3fc62e0f6a700604299fa9e1f629

Merge "Switch from libcrypto_static to libcrypto target"

Merge "Merge "Bluetooth: Move Bluetooth battery stats tracking (1/2)" am: 7df4093371" into stage-aosp-master

Merge "Bluetooth: Move Bluetooth battery stats tracking (1/2)"
am: 7df4093371

Change-Id: I6f3fa3a9f48113c20a8dc4912b782eab2c4ef39a

Merge "Bluetooth: Move Bluetooth battery stats tracking (1/2)"
am: 7df4093371

Change-Id: Ie6a4af6f5ac48e529aaeee34d92f893e710d8278

Merge "Bluetooth: Move Bluetooth battery stats tracking (1/2)"

Bluetooth: Move Bluetooth battery stats tracking (1/2)

Move Bluetooth battery stats tracking to AppScanStats for a more
unified tracking system.

Bug: 37720787
Test: cts-tradefed run cts-dev -m CtsIncidentHostTestCases -t com.android.server.cts.BatteryStatsValidationTest#testBleScans
      Perform BLE scan and check battery stats
Change-Id: Ie6c682374e6e258c291d3a11eede649c7747ef40

Statically include android.test.runner classes in legacy-android-test

In preparation for removing junit and android.test classes from
the Android API this change adds the android.test classes that are in
the android.test.runner library into legacy-android-test so that they
can be statically included into the application.

The junit.... classes from android.test.runner can already be
statically included from the junit library.

Bug: 30188076
Test: make checkbuild
Change-Id: Id8d32da9e46f0d9142b60ef28998b40e8d237b32

NetworkStatsFactory: fix double accounting on IPv6 only networks

For 464xlat scenarios on IPv6 networks, the clatd interface setup
introduces double counting of apps ipv4 traffic. NetworkStatsFactory was
accounting for this on the tx path, but not on the rx path. Also it did
not accounted for the 20 bytes added by the IPv6 header.

This patch subtract correctly the rx and tx traffic from the root uid on
the underlying interface, and also adds correctly the 20 bytes cost per
packet on the stacked interface for 464xlat traffic.

Test: added several new unit tests, based on synthetic data and real
      data also.
Bug: 33681750
Change-Id: I2675643b220acbc6110179fa937d4c313b6f5e32

Merge "Refactor and improve logging."
am: c3d41d5093

Change-Id: I8ef193a9ee0f5bf04ae18620c1a68228ed7ed929

Merge "Refactor and improve logging."

Merge "Prepare for removal of junit and android.test classes from Android API (step 1)"
am: b2ac85e76d

Change-Id: I2c06dbace44b4578a6cd94cb62a3ae67e558aaaa

Merge "Prepare for removal of junit and android.test classes from Android API (step 1)"

Refactor and improve logging.

Test: as follows
    - build
    - flashed
    - booted
    - "runtest frameworks-net" passed
    - "dumpsys connectivity" shows new log output
Bug: 32163131
Bug: 36504926

Change-Id: I14d6da18660223f7cace156cb6594ee18928a7b0

Merge changes I4872f8ba,I92039f29,Iaad13e13
am: 767ac317e2

Change-Id: Ibf618a27b2e5cfb3e8a69528dfecd275b33f0336

Merge changes I4872f8ba,I92039f29,Iaad13e13

* changes:
  NsdService: simple cleanups
  NsdService: test coverage for client requests.
  NsdManager: remove duplicated argument validation

NsdService: more cleanups

This patch does some simple refactoring around management of nsd service
enabled status.

Test: TODO
Bug: 37013369, 33298084
Change-Id: If1b124459a814cd0e821f683ccd568e30a698d2e

NsdService: simple cleanups

This patch replace some SparseArray<Integer> with SparseIntArray, and
simplify a value lookup function by using indexOfValue().

Test: TODO
Bug: 37013369, 33298084
Change-Id: I4872f8baa2bb2ff456c7f848d3afe2e7bcc9892e

NsdService: test coverage for client requests.

Adding coverage for:
  - NsdManager client disconnection
  - in-flight request GC

Test: new test passes
Bug: 37013369, 33298084
Change-Id: I92039f297cf99352bbf4196797933d89c0b819ff

NsdManager: remove duplicated argument validation

This patch simplifies argument validation in NsdManager public api and
regroup duplicated validation into common methods.

This makes stack traces more actionable as now specific errors will
cause the api to throw exception from specific methods, whereas before
IllegalArgumentException would be thrown from inside the same api method
for different reasons.

This patch also includes a couple of other small cleanups.

Test: $ runtest -x frameworks/base/tests/net/../NsdManagerTest.java
Bug: 37013369
Change-Id: Iaad13e13976e9bf8f508d7188f823f8184ac414b

Merge "Remove unused lambda captures"
am: 1101f45613

Change-Id: I371bd344d87a213393787a528121308c96c12d83

Merge "Remove unused lambda captures"

Increase limit of locale pairs

This is just to avoid people injecting arbitrarily large
numbers here and causing crashes due to memory limitation.
Raised to a high enough level to avoid chance of hitting in practice.

Test: build
Change-Id: I1d2a245f9c391a9d6aec52bddd3dc086206d3e30

Update startStreaming to return status code

The StreamingService object will be constructed by the manager code.

Test: build
Change-Id: I53e321dbae14270d0df2c81774bf7f055baf216f

Switch from libcrypto_static to libcrypto target

libcrypto target offers the same static library as libcrypto_static.
Thus, there's no need to use libcrypto_static.

Test: make pbkdf2gen
      Run pbkdf2gen with and without a password and confirm it works
Bug: 26160319

Change-Id: I89a60b7521e6c3df91421d0edf64a71b8b2c20e0

Merge "Fix a NPE with outlineProvider="none""
am: 46a4fbb2e4

Change-Id: Ib14f6b5369d60c06a716b99dd0274872fc0d2a51

Merge "Fix a NPE with outlineProvider="none""

Prepare for removal of junit and android.test classes from Android API (step 1)

In preparation for removing junit and android.test classes from
the Android API this change adds explicit dependencies on junit
and/or legacy-android-test to ensure that modules will compile
properly once it is removed.

Care was taken to ensure that any targets that depended,
directly or indirectly, on android-support-test did not also
depend on junit or hamcrest as they conflict with the versions
embedded within android-support-test.

Bug: 30188076
Test: make checkbuild
Change-Id: Ice80a29b420061f5d0af6eb46426455abf890e5e
Merged-In: Ifc3ee3dc804074a3c572bf2600cfb1fe1b6fa749

Remove unused lambda captures

Test: build
Bug: 37752547
Change-Id: Id102e93fef06e9c818b27fa8a5c6218c319bdd30

Merge "Destroy Allocation right away on exception"
am: 52bc790f61

Change-Id: I2724dd4adf68aa8330b10b8f596d9d7db3619e03

Merge "Destroy Allocation right away on exception"

Merge "Convert libandroid and libandroid_runtime to Android.bp"
am: d308803b82

Change-Id: I90804cba65863d6194d7dd683024b43297357f19

Merge "Convert libandroid and libandroid_runtime to Android.bp"

Merge changes from topic 'sms-files-move-opt-to-base'
am: abcc43d86a

Change-Id: Ib61b38a3fd065c3906e2e3f0863b83f2f4b740f4

Merge changes from topic 'sms-files-move-opt-to-base'

* changes:
  Move newFromRil() function to a utility class
  Remove dependency on telephony-common from services/core
  Move OTASP_* Consts from ServiceStateTracker to TelephonyManager
  Port Utility Functions from DefaultPhoneNotifier
  Remove dependency on IccConstants from SmsManager
  Move files from frameworks/opt/telephony to frameworks/base

Move newFromRil() function to a utility class

Remove a dependency on frameworks/opt from SmsMessage for CDMA

-Code move to frameworks/opt/telephony../cdma/SmsMessageConverter

Bug: 33414487
Test: Manually verified GSM and CDMA SMS: MO and MT
Change-Id: I730d292c4aad6524b03b18c0751738a73a71ecf7

Remove dependency on telephony-common from services/core

After migrating all the dependencies from frameworks/opt/telephony
to frameworks/base, the dependency is no longer needed and should
never have been created in the first place. Happily removed.

Bug: 33414487
Test: compilation
Change-Id: I261da4c6b819b3c960ce56d2cab5c80cb34e852e

Move OTASP_* Consts from ServiceStateTracker to TelephonyManager

-Move the OTASP_* constants from SST
-Update references to those constants in TelephonyRegistry

Bug: 33414487
Test: compilation
Change-Id: I9516a8dfa9623d69d7e051749815e91ed3d013fa

Port Utility Functions from DefaultPhoneNotifier

-Move static utility functions from frameworks/opt/telephony/
 ..DefaultPhoneNotifier to a new utility class in frameworks/base

-Update TelephonyRegistry to point to the new functions.

Bug: 33414487
Test: compilation
Change-Id: I36a113cb5122569d451ec3a2dfa71d72d590fa52

Remove dependency on IccConstants from SmsManager

-Redefine the length of the SmsRecord in SmsManager

Bug: 33414487
Test: compilation
Change-Id: I941085e96f45164152528c123102a59930859546

Move files from frameworks/opt/telephony to frameworks/base

Bug: 37688438
Test: compilation, boot on aosp_bullhead
Merged-In: I44d3d6231943ae0e7c3de4ab302e0cdff987aefe
Change-Id: I44d3d6231943ae0e7c3de4ab302e0cdff987aefe

Convert libandroid and libandroid_runtime to Android.bp

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

Test: m -j checkbuild
Change-Id: Ie3c9c3d3c8c75ade36422d1e8ee0a5fe117e9091
Merged-In: Ie3c9c3d3c8c75ade36422d1e8ee0a5fe117e9091
(cherry picked from commit 3f8fd40f285dba5b3639ee1e3d4e4544048c1446)

Merge changes from topic 'libvintf_cp'
am: 9bd4c20764

Change-Id: I277b37789fdc78ed2dcb5c50c1fda9c40444a4e8

Add compatibility matrices to VintfObject.report()
am: 4463d991be

Change-Id: I26c78cca5e807ee291bdb097d11d5f4ac43bba29

Merge changes from topic 'libvintf_cp'

* changes:
  android.os.VintfObject: add API for EDI.
  Add compatibility matrices to VintfObject.report()

Merge "Fix Bluetooth GATT API default handler assignment"
am: 612b8bef0c

Change-Id: Ia71233576b7f8a712abf495164923a092cdc5677

Merge "Fix Bluetooth GATT API default handler assignment"

Changes to MbmsStreamingManager for test app

Add binding methods to MbmsStreamingManager
Add constants for synchronous binding exceptions

Test: testapp
Change-Id: Ie798c6008029c4201ec1bb3c98c11c4a949a048a

Merge "Update path for libmediandk headers"
am: 9d9aab48fa

Change-Id: I339300d2e0c230069af0a7f08caff6449de37c85

Merge "Update path for libmediandk headers"

Fix Bluetooth GATT API default handler assignment

Restores previous behaviour where GATT callbacks are invoked on the
binder thread and not the calling process main looper thread.

This fixes performance regressions as well as some
NetworkOnMainThreadException's for some applications.

Bug: 37544152
Bug: 37871717
Test: Covered by prior API tests.
Change-Id: Id8ab705dd4d7f00030e6ac29e056dde5180670e9
(cherry picked from commit 6bdc550e27e2c4e7b32bb91085eaac898d7d1156)

Merge "Merge changes from topic 'update-conscrypt' am: c497348c74" into stage-aosp-master

Merge changes from topic 'update-conscrypt'
am: c497348c74

Change-Id: Ifa261c6c003f3e0ca767356bc98c2c3033bd699a

Merge changes from topic 'update-conscrypt'
am: c497348c74

Change-Id: Ie3b9d74ff46be9e4b00c6efe6748feb5e096cd3a

Track updates to Conscrypt
am: 7fdce769c3

Change-Id: I691f1ed588814311363f67b8ce43cbe247e54b71

Merge changes from topic 'update-conscrypt'

* changes:
  Delete obsolete and unused KeyStoreTests
  Track updates to Conscrypt

Update path for libmediandk headers

Platform usages of libmediandk headers are now in media/.

Test: builds
Change-Id: I16385cab86ae9cac225304557df416bbb232ecf3
(cherry picked from commit c8ed45d38f3b42e06c9e0980e133a84181752b5e)

Merge "Reduce hwui CPU time by using glDrawRangeElements"
am: 057c7c3164

Change-Id: I32269ea47f637597efc73acc4e040c209b48a2ed

Merge "Reduce hwui CPU time by using glDrawRangeElements"

Destroy Allocation right away on exception

Bug: 28053584

To avoid holding the resource (surface) for too long.

Test: CTS on x86_64 emulator
Change-Id: I2fdec107c2a16b7dadbf78a9aee79c33423a3676

Merge "Convert libhwui to Android.bp"
am: 31fb17bbf9

Change-Id: I66401c892f90131e21efb36dc7316f905bd729d2

Merge "Convert libhwui to Android.bp"

Reduce hwui CPU time by using glDrawRangeElements

The CPU overhead of glDrawElements in the GPU driver is significant
with client-side vertex data (unique per draw call) as the driver has
to calculate the mininimum and maximum indices from the index buffer
in order to evaluate the range of vertex data required. This can be
avoided by keeping track of the min-max in hwui and passing it with
glDrawRangeElements. This requires OpenGL ES3.0 support (which is
already checked for elsewhere in hwui).

Test: manual - visual inspection on fugu (nexus player)

Change-Id: I57bb1ddd239a1032f74f1cd2683bbe0970e84bd9