Merge "ConnectivityServiceTest: fix flakyness"
am: a82cac1452

Change-Id: I241192b2d9565c41f47bbe6227b50049898da679

Merge "ConnectivityServiceTest: fix flakyness"

Merge "ConnectivityManager: improve argument validation"
am: cdfd3a186f

Change-Id: I3adb40fe1bd1779d745bb122da06e33068ce48a3

Merge "ConnectivityManager: improve argument validation"

ConnectivityManager: improve argument validation

Using Preconditions and dedicated static methods for checking arguments
to improve error stack traces without error messages.

Test: covered by previously added unit test
Bug: 36701874
Change-Id: Id872b2c887a4bca43a8c3644622add1c2ee57c6d

ConnectivityServiceTest: fix flakyness

This patch attempts to fix the remaining spurious failures in
ConnectivityServiceTest, which have two causes:
 - waitForIdle() does not take into account the NetworkAgents handlers.
 - the deadlines in testRequestBenchmark are sometimes exceeded.

To fix the first issue, waitForIdle() is moved to a test level instance
method and also calls waitForIdleHandler on any non null
MockNetworkAgent. This is expected to fix spurious errors for the
following tests:
  - testMobildeDataAlwaysOn
  - testLingering
  - testPacketKeepAlive
  - testMMSonWiFi

To fix the second issue, the deadlines for testRequestBenchmark are
extended by 10ms. Also, the failure message is made more actionable by
providing the total time it took for the operation, instead of printing
the number of dispatches that were achieved before the deadline.

Bug: 32561414
Test: tests pass many times in a row (~500).
Change-Id: Id33c6ac1edfb0b89634fa7789dccb2da237e2709

Merge "Added IWLAN metered APN support"
am: 70fe11fc38

Change-Id: I12573353fca921d1454d6829302a379cd6b2cc8b

Merge "Added IWLAN metered APN support"

Added IWLAN metered APN support

If the device is in IWLAN mode, when checking if the APN
type is metered, we read the settings from the IWLAN
metered APN list.

Test: Telephony unit tests
bug: 38304530

Merged-In: I46c9e1a15dd0dd40811d5a959d558fc78608b127
Change-Id: I46c9e1a15dd0dd40811d5a959d558fc78608b127

Merge "Creat a new notification to avoid the mTetheredNotificationBuilder changed"
am: 99cf40a25a

Change-Id: I9d45fc368e746efa40df96c39719566a201dad45

Merge "Creat a new notification to avoid the mTetheredNotificationBuilder changed"

Merge "OBEX: Add required parameters TAG ID's for PBAP 1.2"
am: a2663b0060

Change-Id: I015534f9fb860fd0f875bab3d9130de029dc16b9

Merge "OBEX: Add required parameters TAG ID's for PBAP 1.2"

OBEX: Add required parameters TAG ID's for PBAP 1.2

Add required OBEX application parameters TAGID's and length
declarations required to support PBAP 1.2 specification.

Test: Connected with Remote PBAP Client supporting PBAP 1.2 and
verified that connection and transfer happens over L2CAP.
Connected with Remote PBAP Client supporting PBAP 1.1 and
and verified that connection and transfer happens over RFCOMM.

Bug: 33011817
Change-Id: I8a772b0dddf6ecf74b057e12d7596421c0e0f558

Merge "Address ConnectivityManager API comments."
am: b5d6e45e8d

Change-Id: I4a29a929e2482d664a969ca38e5f73c3ed816175

Merge "Address ConnectivityManager API comments."

Merge "Implement stopStreaming API for MBMS"
am: e245bf9067

Change-Id: I4b70f26a5579ddf3af7d426a04aea8ba33312790

Merge "Implement stopStreaming API for MBMS"

Merge "Add a method to start the captive portal login app."
am: 30f2544f0c

Change-Id: I7bc42e7f9cbc3635684257b920ea4a1ba7b55740

Merge "Add a method to start the captive portal login app."

Merge "Add test coverage for explicitlySelected networks."
am: 1bc1936ae9

Change-Id: I42ce887005325eee9fb1a60762cee375cff176a6

Merge "Add test coverage for explicitlySelected networks."

Add a method to start the captive portal login app.

Bug: 36203355
Bug: 36656914
Test: ConnectivityServiceTest (including new test) passes
Change-Id: Ide82858af67024623560ab79beea27c201b63447
Merged-In: I82a9a9a8da47870ba3f1bbef5941b37e970c844f

(cherry picked from commit 4734cdbbc38752765376aedb0d5208463674535e)

Implement stopStreaming API for MBMS

Test: testapps
Change-Id: Ic0477bf489a89a67697a318ff0fab065e0327e08

Merge "Also pin vdex file for camera"
am: d4ae914be2

Change-Id: I0d1a687ea2d90e968fe4e9705de223dc8b181238

Merge "Also pin vdex file for camera"

Merge "Consolidated all data allow/disallow logic into one method"
am: 449d73251f

Change-Id: I861d75dad60ab83039aa06565141a2eb5b294386

Merge "Consolidated all data allow/disallow logic into one method"

Consolidated all data allow/disallow logic into one method

Consolidated all data allow/disallow logic into one method
and got rid of the obsolete isDataPossible. This is the
preliminary work for checking if data is allowed just right
before data connection entering connected state.

Test: Telephony sanity tests and unit tests
bug: 33847610
Merged-In: I24f1415bbb59990e9e8e9920724be9e3f17e3e60
Change-Id: I24f1415bbb59990e9e8e9920724be9e3f17e3e60

Merge "Expand logging capabilities of Tethering subsystem"
am: 0373340b07

Change-Id: I1381b8457781dcfafe52371a5dc7628282cb8efc

Merge "Fix NetworkNotificationManagerTest"
am: c13a80deee

Change-Id: I8f02be13da5790d0fd60f04886e87de04c96de08

Merge "Expand logging capabilities of Tethering subsystem"

Merge "Fix NetworkNotificationManagerTest"

Merge "Captive portals: reinspect http result after fallback probe"
am: 3a7c4f3337

Change-Id: Ia9dfbb764909aeb8586066121ddc8106fa14bb7e

Merge "Captive portals: reinspect http result after fallback probe"

Merge "Move NetworkNotificationManagerTest to tests/net"
am: c01fbf0f0c

Change-Id: I10ce85211ac0305e8d05ca02b58b819297f59bbf

Fix NetworkNotificationManagerTest

The testNotificationsShownAndCleared test was not doing anything
because the list of notification to show was always empty.

This patch fixes this issue and actually makes the test loop on
non-empty collections, and also fixes another ordering issue in
assertions themselves, hidden until now by the first issue.

Bug: 32561414
Test: runtest -x frameworks/base/tests/net/java/com/android/server/connectivity/NetworkNotificationManagerTest.java
Merged-In: I4837b1175d7c9133e9156e33acaa1e7e3341cc62
Change-Id: Ia30587bd68a87b83b62f57eeeb28fef7d95dbf81

(cherry picked from commit f8b804be5aeb5d0a36ff88649d7c5ecdc47c3ad1)

Creat a new notification to avoid the mTetheredNotificationBuilder changed

Now the method NotificationBuilder.build() is different with m.
it is not creat new notification. But here it need to creat a
new notification to avoid the mTetheredNotificationBuilder
changed. So we use buildInto to replace build

Change-Id: I274ec833de4ade4a778e1937a09d1e6242d92a7f
Signed-off-by: zhouzhijie <zhouzhijie@xiaomi.com>

Merge "Move NetworkNotificationManagerTest to tests/net"

Expand logging capabilities of Tethering subsystem

Introduce a SharedLog that can be shared among all Tethering
components.  All log entries go to the same log, and each includes
its subsystem name.  Additionally, all entries emitted to the
system log shared the same tag, so can easily be watched.

More subsystems can inherit a SharedLog, but to minimize cherry-pick
conflicts we limit ourselves to these changes only.

Test: as follows
    - built
    - flashed
    - booted
    - log output observed via dumpsys
    - runtest frameworks-net passes
Bug: 32163131
Bug: 36988090
Bug: 38218697
Change-Id: I1229d4c55987402d84b0d4a2c8a595e0c829aa3e

Merge "Finish moving NetworkStats tests to tests/net"
am: 1ae1b6b960

Change-Id: If85e483d5b44872c10e42ef55ddaf209601c7cfa

Merge "Finish moving NetworkStats tests to tests/net"

Merge "Convert libsoundpool to Android.bp"
am: dde9ee9edd

Change-Id: Ia7bb54b658031c68036ee2a218e5b229d32b1d25

Merge "Convert libsoundpool to Android.bp"

Convert libsoundpool to Android.bp

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

Test: m -j checkbuild
Change-Id: If0b7cb1e2c86ff1be41c00d98fd9ddec31287a53
Merged-In: If0b7cb1e2c86ff1be41c00d98fd9ddec31287a53
(cherry picked from commit 98c127f8cc652d1179871ad47cac99d15ea1a5a0)

Merge "Fix for Bluetooth not restarting on User Switch"
am: 196b72aae5

Change-Id: Id0ef82b93b047f13683f7538740f0544549b0eca

Merge "Fix for Bluetooth not restarting on User Switch"

Add test coverage for explicitlySelected networks.

Also, make all tests start with mobile data always on disabled.

This is because some tests only pass when mobile data always on
is disabled. This doesn't cause any problems when running all
the tests in the file, because these tests are always run after
one or more calls to tearDown, which disables mobile data always
on. However, it does cause issues when those tests are run alone.

Test: new test passes 50 times in a row
Test: ConnectivityServiceTest passes
Change-Id: I1eef5d7f5ec5464e0f9a1d7f1130d9ba6dea4557

Merge "Convert libmedia_jni to Android.bp"
am: 85faffa7ef

Change-Id: I7afbf6dda3dab6a3c36576e2c118b1accdcc2a17

Merge "Convert libmedia_jni to Android.bp"

Move NetworkNotificationManagerTest to tests/net

Bug: 32561414
Test: no functional change
Change-Id: Iad17e3336a4a589da5cf88fd6359323e34b747e1

Finish moving NetworkStats tests to tests/net

Follow-up of commit c86013be936f36c61fa0cdfad95141f6de3c6ef7.

Also add @SmallTest annotation to classes moved to tests/net.

Test: no functional change
Bug: 32561414
Change-Id: I0a0e8865d37ba7bae06ce352d3fc385989adc300

Captive portals: reinspect http result after fallback probe

When DNS queries that precede the http and https probe take a long time
to complete, it is possible that the fallback probe fires earlier than
either the http and https probes.

This causes the detection logic to ignore the result of the http probe,
which may cause the system to miss captive portals advertising
themselves on the url used by the http probe but not on the url used by
the fallback probe.

This patch fixes that issue by forcing the detection to wait on the
result of the http probe if the fallback probe does not find a portal.

Bug: 38259299
Bug: 36532213
Test: manually tested with nearby captive portal networks
Change-Id: I07ea23a2b5f694c5ada9633169af98409efedff1

Address ConnectivityManager API comments.

This patch is a cherry pick of the two following commits:
  - 15fd4395e13b4662a262ee67a5f75f5c521c62ef which addresses several
    issues in the public api of ConnectivityManager.
  - e2d48ff57c5529b32a9cd7bfdcf21d29df539f73 which fixes the documentation
    of several methods in ConnectivityManager public api.

Because the first commit change the public api that is referenced in
the documentation fixed by the second commit, it is not possible to one
without the other. In both cases trying to cherry pick only one of them
results in a build error.

The first commit was submitted successfully on an internal branch before
the checks done in the built got stricter.

Bug: 36370941
Test: marlin builds and boots
Change-Id: I86dcf056e6b165e527c3ee88dbabc2764ac09a08
Merged-In: I693ee5270bf186c88c7c5056293519f7237504ff

(cherry picked from commit 15fd4395e13b4662a262ee67a5f75f5c521c62ef)

Merge "Add tethering offload HAL call via JNI"
am: 8f65e470b0

Change-Id: Ib8403f685371b44f0fdad6d15ae372787d457d13

Merge "Add tethering offload HAL call via JNI"

Convert libmedia_jni to Android.bp

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

Test: m -j checkbuild
Change-Id: I1817c75ee45537ca0d5fc40a035f94a3dfeccc11
Merged-In: I1817c75ee45537ca0d5fc40a035f94a3dfeccc11
(cherry picked from commit 76de4f6b4d09423b3a5d3603527ce79acae4ed32)

Add tethering offload HAL call via JNI

Additionally, try to locate java Control interface.

Test: as follows
    - built
    - flashed
    - booted
    - OffloadController log messages observed, but only ever
      "not enabled" messages (needs a supporting implementation)
Bug: 29337859
Bug: 32163131
Bug: 34361337
Merged-In: I5251d05f2d2fd732a33a8955a6c346b3a2401e46
Change-Id: I5251d05f2d2fd732a33a8955a6c346b3a2401e46
(cherry picked from commit 58ed1b051bdffd239d92919170a0bb41a4cc197c)

Merge "Add the new RIL requests and NetworkScanResult."
am: 0727abc423

Change-Id: I41560f522b3c46de4fe1a7a8377a6f2b7af04534

Merge "Add the new RIL requests and NetworkScanResult."

Merge "Zygote: Allow arbitrary-length wrap property names"
am: 11780f1ad3

Change-Id: Id1454ecd597347d5fca5f06a17df26e7303c59db

Merge "Zygote: Allow arbitrary-length wrap property names"

Add the new RIL requests and NetworkScanResult.

Those new RIL requests will be used to start and stop network scan, and
the scan result will be returned from RIL in the format of
NetworkScanResult.

Test: Telephony sanity tests
Bug: 30954762
Change-Id: I2bfd0fb8cbb3815cc62ec4385594dec9c5e4d279
(cherry picked from commit b36a4cb40222e1c0a61b13c1eca601ba761bd26e)

Merge "Discover primary service by UUID for PTS tests (4/4)"
am: abb732fc40

Change-Id: I09534a5b9b961f1ac1ecf93172e1ae5c33cf5df1

Merge "Discover primary service by UUID for PTS tests (4/4)"

Merge "startStreaming for MBMS apis"
am: 4b3c44be55

Change-Id: Id0e2e95387300147ba32ec3a418605901fdb241b

Merge "startStreaming for MBMS apis"

Merge "Support different SIM power states"
am: 0517b719bc

Change-Id: Icb6db4d805b4aea701a803eaaec716954f2ebf2a

Merge "Support different SIM power states"

Also pin vdex file for camera

Test: device boots
Bug: 33168521
Change-Id: I157e9684f9b8d64950b96c07e35b6101eccd4045

Discover primary service by UUID for PTS tests (4/4)

Bug: 38123054
Test: manual
Change-Id: I89e088e60f4325f1ece59d22efda0c907a3b716a

Support different SIM power states

Test: Verify SIM power states
Bug:  35844036
Merged-In: I968269e88eec40af6e3ca71cbb805a77e016da0f
Change-Id: I968269e88eec40af6e3ca71cbb805a77e016da0f

Merge "Set the decay timer for all forked apps."
am: 232ce7868d

Change-Id: Ia6e16a2f6b8e551a48d6f3ab4472d097284436f4

Merge "Set the decay timer for all forked apps."

Merge "Remove unused import in ConnectivityManager"
am: db6fc1afe8

Change-Id: I1a7ce964c66e0204bfcc69948f7bed23ab498c81

Merge "Remove unused import in ConnectivityManager"

Merge "Fix broken DUN evaluation logic"
am: b0daeeb879

Change-Id: Iae7450ef21485c5a13f45fab570bf658b2da8f28

Merge "Fix broken DUN evaluation logic"

Fix broken DUN evaluation logic

Test: as follows
    - built
    - booted
    - flashed
    - runtest frameworks-net passes
Bug: 32163131
Bug: 38152109
Bug: 38186915
Change-Id: Id4803504e825290d62c0b51a2b9bd664dcef30b1

Zygote: Allow arbitrary-length wrap property names

The restriction on system property key length has been lifted.
Update the invoke-with code to first check the full-length property.
Then fall back to the truncated version for backwards-compatibility.

Test: m
Test: manual with long package name (Maps)
Change-Id: I9f714af093a6017307cfef18c84de769f0de7c3e

Remove unused import in ConnectivityManager

This makes the pre-upload verification pass.

Test: no functional change.
Change-Id: I6beccea10ccfcb39d975cb8f6f34fc9d79f1e170

Fix for Bluetooth not restarting on User Switch

The flag that keeps track of the Bluetooth adapter status(mEnable) is
not guarded correctly and could go out of sync with the adapter status.
This causes User Switch handling to not function correctly since it
checks for the status of the mEnable flag before restarting Bluetooth.
This fix queries the Bluetooth Adapter for the status instead of using
the mEnable flag.

Bug: b/37648701
Test: Tested switching users multiple times in both phone and carkit.
Also tested pairing and connection for sanity.

Change-Id: I17facecf8e1de90df11c11d5b0a184f48a0356e4

Set the decay timer for all forked apps.

This will cause native memory frees to be less expensive, but it will cost
a bit of PSS that might hang around.

Bug: 36401135

Test: Built and booted bullhead.
Test: Ran jemalloc unit tests.
Test: Ran bionic unit tests.
Test: Ran a test that allocated and free'd a large piece of memory,
Test: and verified that after changing the parameter, the PSS
Test: sticks around (decay timer set to 1), the PSS is purged (decay
Test: timer set to 0).
Change-Id: I140b26986f3f81fddd5b9ed9c9e4f2d4977b819b

startStreaming for MBMS apis

Clean up MBMS APIs a bit and make things functional for the
startStreaming/getPlaybackUri APIs.

Test: testapps
Change-Id: I141a4114cc912b98ae22aefb760346061e286f1a

Merge "Change signalStrength to int."
am: 73818566d6

Change-Id: I8a9afddc749afc216dc1dc16f0465df1d0dab24f

Merge "Change signalStrength to int."

Merge "Remove switchStreams API."
am: f465d84ebe

Change-Id: Ia2cd7338087ee5283d2241ec260d0649ccd3f2b2

Merge "Remove switchStreams API."

Merge "Moved c.a.i.u.Predicate[s] into legacy-test"
am: e476a30575

Change-Id: Ia0b3ebd30e045fe2b26fe9a78b2de665bf960b0c

Merge "Moved c.a.i.u.Predicate[s] into legacy-test"

Merge "Add API to support async network scans."
am: 625d5a9dd9

Change-Id: Iac369e26f707e59095ae7a89bfb11324e12634dc

Merge "Add API to support async network scans."

Moved c.a.i.u.Predicate[s] into legacy-test

These classes are only used by android.test classes that are
being removed. As their name suggests they should not be in the
Android API at all so it makes sense to remove them. Especially
as there is java.lang.function.Predicate available now.

It appears as though Predicate was only added in to the API
because it was used by a method in the API as the directory in
which it and Predicates live was not on the list of classes to
explicitly index. Moving it into legacy-test meant that they are
now being indexed explicitly which means that Predicates needs
to be hidden.

Keeps running the tests as part of the existing target.

At runtime apps targeted at the API version before these are
removed will have the legacy-test library automatically added
to their classpath so they should see no effect. Apps that
target a later API will have to include those classes from the
android.legacy.test.jar which will contain all the android.test
classes that depend on it as well.

Bug: 30188076
Test: make checkbuild
Change-Id: Ia8502ec77ac11f85e078d70b68df214a9435eee7
Merged-In: I6f6f5f16fe93bd80227a450c6254166632fc6813

Merge "update time while network is connected"
am: 465f8527dd

Change-Id: I863c775015079a21098efd32b570acf8a8b53efa

Merge "update time while network is connected"

Merge "ConnectivityService: minor formatting change"
am: 4c949ee07c

Change-Id: Icb420ee636270ca07553de64958b04759986d11f

Merge "ConnectivityService: minor formatting change"

Merge "CaptivePortal: login activity UI improvements"
am: cb570ec50a

Change-Id: I5ee0a2dbbb36e66822f4539f6b186df23a7438da

Merge "CaptivePortal: login activity UI improvements"

ConnectivityService: minor formatting change

This patch adds a space character between "synchronized" and
"(mTheLockVariable)" so that all synchronized block have a consistent
syntax. Basic stats shows that "synchronized (" is x10 time more likely
than "synchronized(".

This facilitates finding all locked sections by grepping.

Test: no semantics changes
Bug: 37119619
Change-Id: Id860700efa85ffebafed11af3a2997bc115d9c03