Merge "ASM Priority Boost Tool"
am: 8ff8dd16cc

Change-Id: Ifb62ee3132e984340e4457b543c4f84f7c45995a

Merge "ASM Priority Boost Tool"

Merge "Fix code and test related to rules version check"
am: a3579d4b84

Change-Id: I2021e64f04e0e55de21dcdc0a932ce535e6a7f9f

Merge "Fix code and test related to rules version check"

ASM Priority Boost Tool

This tool is a replacement for the Jack plugin that allows injection
of static method calls before lock enter and exit.

A common use case would be to boost a thread's priority as soon as
it acquires a heavily contented lock and resetting the priority upon
release.

This tool is meant to be optionally invoked from /build/core/ rules
during the build process.

Test: JUnit Tests / resulting image on a bullhead.

Change-Id: If400414a0bf50f03768a1de2ebee42086a9d701f
Merged-In: If400414a0bf50f03768a1de2ebee42086a9d701f
(cherry picked from commit 93db63059ad5e9e1d59c8ce013559069d566ef4c)

Fix code and test related to rules version check

Fix code and test related to an incorrectly implemented
rules version check.

This meant that a device could not return the rules version
it shipped with.

Another unused (and also confusing) method and
associated test is being deleted here.

To run tests:
make -j30 FrameworksServicesTests
adb install -r -g \
  "out/target/product/angler/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"

Test: See above.
Bug: 31008728
Change-Id: I31b55e88072dd73055cb3d8cf252be6eac920322

Merge "Add a common call when wrapping an app."
am: 12e0975963

Change-Id: I81830a6d826cd109c99f6e857204f96da9d4a656

Merge "Add a common call when wrapping an app."

Merge "Ensure that android.test tests pass"
am: 156775fea8

Change-Id: I7fe4f45bd98acea016e02b44a4df38f0e907c0e4

Merge "Ensure that android.test tests pass"

Ensure that android.test tests pass

It looks as though the tests in test-runner/tests have not actually been
run successfully for over 7 years. As a result they have degraded. This
change fixes the tests so that they will pass and provides instructions
on how to run them.

TestCaseUtilTest.testGetTestCaseNamesForTestSuiteWithSuiteMethod
    This fails because it expected 2 names to be returned but only
    returns 0. The reason for that is although TwoTestsInTestSuite has a
    Test suite() method that does create a TestSuite with two tests the
    TestCaseUtil method does not actually call suite(). Instead, because
    TwoTestsInTestSuite is a TestSuite it just calls the tests() method
    on it which returns an empty Enumeration because TwoTestsInTestSuite
    is empty.

    The support for "static Test suite() {}" is broken but fixing this
    will affect the behavior of InstrumentationTestRunner which is used
    in thousands of different places both in and outside Google and so
    could cause untold problems.

TestSuiteBuilderTest.testIncludeAllPackagesUnderHere
    Reformat the list, one per line and then add missing name
    "testPublicConstructor".

ErrorTest/FailingTest
    These tests are not meant to be run on their own, only as part of a
    separate test. The RunAsPartOfSeparateTest annotation was added to
    allow these to be excluded using notAnnotation as shown in the
    instructions for running the tests.

Bug: 30188076
Test: followed new instructions in test-runner/tests/Android.mk
Change-Id: I60e7bee9cd08a9ab7777a2578fc58da772de5c1f

Merge "ConnectivityManager: fix style issue"
am: e74f3adba5

Change-Id: I0c41dedac1d1e947a3d614ccea85ef48dc011c34

Merge "ConnectivityManager: fix style issue"

Merge "Use RFC 7217 stable privacy addresses"
am: a88b12ce5e

Change-Id: I3b79ecd87233db0b4242e8c02355d83686d27e6e

Merge "Use RFC 7217 stable privacy addresses"

Use RFC 7217 stable privacy addresses

Ask Netd to setup WiFi interfaces to use stable privacy addresses. This
should fail benignly on devices with kernels lacking RFC 7217 support.

Test: as follows
    - built
    - flashed
    - booted
    - Manually observed persistent stable secret
    - Manually observed randomized IPv6 addresses (with flags = 800)
Bug: 17613910

Change-Id: I9465de16685b1eb0fd842446a530cf98d77a0c28

Add a common call when wrapping an app.

When an application is wrapped using setprop wrap.<application>, it
doesn't set the special zygote flag, or make the mallopt call. This
means that a wrapped application will behave differently, and not
be able to track zygote native allocations versus app native allocations.

This CL adds a nativePreApplicationInit call that is used after the
Zygote is forked, and when a wrapped application is called.

Bug: 62068500

Test: Booted sailfish system. Verified that the zygote has not called
Test: the new function. Verified the function is called when a wrapper
Test: is used. Verified the function is called when the Zygote forks
Test: a process.
Change-Id: I392e1b5429be77def63f7815b072d68e9408cc7f
(cherry picked from commit 2980de4a59db048851fbbac9cd847a96a12fa4ec)

resolve merge conflicts of 4ff3cf1eca71 to stage-aosp-master

Change-Id: Id8b10a739e6a313d74908d7fd7068d53ccf994a5

Merge "migrate roaming operator configs to carrierconfig"

migrate roaming operator configs to carrierconfig

Bug:37885668
Test:runtest --path
packages/apps/CarrierConfig/tests/src/com/android/carrierconfig/CarrierConfigTest.java
Change-Id: Ic669581243a7a640f9ac1b5155954514168cd14b
Merged-In: Ic669581243a7a640f9ac1b5155954514168cd14b

Merge "Cleanup a.t.TestGrouping"
am: 01dcf54e9a

Change-Id: I58a278b7c45afe27309cf0029e9fb8fd452b22f2

Merge "Cleanup a.t.TestGrouping"

Merge "Remove unused a.t.InstrumentationUtils"
am: 0e4f1358d6

Change-Id: Id4f1845ec46d9b79fbfbc8a7ca6f75279fef69f0

Merge "Cleanup a.t.InstrumentationTestRunner"
am: 9d5576e081

Change-Id: I376079be6f5e56adf51fec62ed3f89969029c0a1

Merge "Remove unused a.t.s.InstrumentationTestSuiteBuilder"
am: 1935dce8eb

Change-Id: If6e86d76bd8dfca6e91cc6eeda20cd1492980b1e

Merge "Remove unused android.test.DatabaseTestUtils"
am: 3724e7e5e9

Change-Id: Iebf38b15aaae0317935fe00e07d00cf806ab5942

Merge "Remove unused a.t.InstrumentationUtils"

Merge "Cleanup a.t.InstrumentationTestRunner"

Merge "Remove unused a.t.s.InstrumentationTestSuiteBuilder"

Merge "Remove unused android.test.DatabaseTestUtils"

Merge "Always allow Ethernet to be an upstream"
am: ecbe6fa147

Change-Id: Ic9fd4bfd6ba7ecc00552019f804b87bc203ea98a

Merge "Always allow Ethernet to be an upstream"

Always allow Ethernet to be an upstream

If there are any upstream types defined at all, make sure that
either TYPE_ETHERNET is included somewhere within the sorted list
or force it to be at the front.

Test: as follows
    - built
    - flashed
    - booted
    - runtest frameworks-net passes
Bug: 32163131
Bug: 36076442
Change-Id: Ie61d1358f73d518de23f6ca48ca2765ca14a1067

Merge "Remove unused a.t.InstrumentationCoreTestRunner"
am: 1bfccadc81

Change-Id: Id624c17f7d5da6fa81cfac9a21726042b793c270

Merge "Remove unused a.t.InstrumentationCoreTestRunner"

Cleanup a.t.TestGrouping

TestGrouping is not part of the API but is used by classes that are and
so will need to be included in the static library we ship as part of the
SDK. As that library will be built directly from the source (as opposed
to android.jar which is built from stubs) developers will be able to see
classes and methods that are hidden and so not present in the stubs.

This change makes TestGrouping and its members as inaccessible as
possible. Unused methods and fields were removed and the TestGrouping
class was simplified by passing the ClassLoader into the constructor,
initializing the classLoader, making the classLoader field final,
and removing the setClassLoader(ClassLoader) method.

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

Remove unused a.t.InstrumentationUtils

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

Cleanup a.t.InstrumentationTestRunner

ARGUMENT_TEST_CLASS, ARGUMENT_TEST_PACKAGE, ARGUMENT_TEST_SIZE_PREDICATE
and ARGUMENT_DELAY_MSEC were made package private because they are not
part of the API and are only used from within the same package.

The JavaDoc comment was changed to a multi-line comment because it is
not and should not be associated with any member.

Fixed a couple of minor JavaDoc issues.

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

Remove unused a.t.s.InstrumentationTestSuiteBuilder

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

Remove unused a.t.InstrumentationCoreTestRunner

The only usage of this was in the unused
cts/tests/vm-tests-tf/AndroidManifest.xml which was deleted in
https://android-review.googlesource.com/#/c/418900/.

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

Remove unused android.test.DatabaseTestUtils

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

Merge "Track TimeZoneDistroInstaller method changes"
am: ba6b96d978

Change-Id: I9352872ceb2d11739a4372589f69bb473f3457d3

Merge "Track TimeZoneDistroInstaller method changes"

Merge "Create a TetherInterfaceStateMachine when told the interface name."
am: 51ee00785c

Change-Id: If79a096375e6b9665d5ba4aa0bcd9dbec6a8adab

Merge "Create a TetherInterfaceStateMachine when told the interface name."

Create a TetherInterfaceStateMachine when told the interface name.

Test: as follows
    - built
    - flashed
    - booted
    - runtest frameworks-net passes
Bug: 32163131
Bug: 62343300

Change-Id: I93f3bdcf20dc4f03cd621d7ab6066344642550e3

resolve merge conflicts of 48bc4c1f420d to stage-aosp-master

Change-Id: I41889b0497463e8a63cf2d1e983e7e54298c5c2e

Merge "Better errors from unregisterNetworkCallback"
am: 04e89ad631

Change-Id: Icbd68ff14db55c6546c7de6353774a3622b32f9e

ConnectivityManager: fix style issue

Simple style fix flagged by linter.

Test: no functional change.
Change-Id: I2cb19912cba149eeaffdd157616d210259d42959

Merge "CaptivePortalLogin correctly unregisters callbacks"

Merge "Better errors from unregisterNetworkCallback"

Merge "DO NOT MERGE Instrument captive portal login activity"
am: 778ade9073  -s ours

Change-Id: Ie493a705d3c60958037b0128195ee4ac85329d8b

Merge "DO NOT MERGE Instrument captive portal login activity"

Merge "Add an extra level of indirection to make sure that native IHwBinder"
am: b6dde12ef0

Change-Id: I342d0d1a09ed1677b7619df7df033471b48c937a

Merge "Add an extra level of indirection to make sure that native IHwBinder"

Merge changes from topics 'embms-init-fix', 'embms-download-2'
am: e601160a96

Change-Id: Icb84816dc98198ad6ed85dd8b030fbd0198d73a4

Merge changes from topics 'embms-init-fix', 'embms-download-2'

* changes:
  Add callback for initialization done
  Embms download part 2

Merge "Remove legacy advertiser from map after it's no longer used"
am: 42b1578867

Change-Id: I8f6f49f0d75c840faa8a7dbdb56a1debecd28d97

Merge "Remove legacy advertiser from map after it's no longer used"

Add an extra level of indirection to make sure that native IHwBinder

objects, while alive, maintain a strong reference to their Java
counterpart.

Bug: 33042939
Test: hidl_test_java
Change-Id: I69179772b3218705e9ef46cbb10b21b6473280d6

Remove legacy advertiser from map after it's no longer used

Test: manual
Bug: 62597369
Change-Id: Ief1e1b054f29b59b1ed72514b138d4f209f9847a

Track TimeZoneDistroInstaller method changes

The TimeZoneDistroInstaller now takes a TimeZoneDistro object
not bytes, which will help to hide whether the distro is in memory
or in storage. See the associated libcore change for details.

To run tests:
make -j30 FrameworksServicesTests
adb install -r -g
  "out/target/product/angler/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"

Test: See above.
Bug: 31008728

Change-Id: Ieead1bb87b9ae196abab93342ba5c5747ca45116

CaptivePortalLogin correctly unregisters callbacks

The NetworkCallback registered by the CaptivePortalLogin activity in
onCreate was unregistered in both onDestroy() and done(). In addition
done() can be called concurrently from different places (from the
webview, from the captive portal test probe, from the activity menu),
resulting in incorrectly unregistering the callback more than once.

This patch fixes the lifecycle management of the NetworkCallback
registered by the CaptivePortalLogin activity so that it is unregistered
once only in onDestroy.

In addition the done() method is made robust against multiple calls and
becomes a no-op after the first call. This avoids multiple calls
to CaptivePortal for the same captive portal.

Bug: 62497809
Test: tested manually with captive portal setup in the office
Change-Id: I77fbeb55cf91d3b44e91d2fecb800dae40279652

Merge "Make interface IP serving code set LinkProperties"
am: 1dae9a4709

Change-Id: I84e4b3fb0be210f59eb663597a6c81c95d3e51d6

Better errors from unregisterNetworkCallback

This patch changes the validation of unregisterNetworkCallback in
ConnectivityManager so that the caller can better distinguish the case
of a callback that was never registered from the case of a callback that
has already been unregistered.

Bug: 62497809
Test: runtest frameworks-net passes
Change-Id: I58eda22725dd4e67dc4b64207e38cf482032df10

DO NOT MERGE Instrument captive portal login activity

Also add CAPTIVE_PORTAL_LOGIN_ACTIVITY_SSL_ERROR constants to
metrics_constants.proto for counting ssl error pages shown by the
captive portal login activity.

Bug: 36203355
Bug: 34901696
Bug: 36532213
Test: manually tested with captive portal testing setup while watching
      event logs: adb logcat -b events | grep captiveportallogin
        06-08 16:25:20.279 21558 21558 I sysui_action: [1004,com.android.captiveportallogin]
        06-08 17:01:04.052 21558 21558 I sysui_action: [1007,com.android.captiveportallogin]

Merged-In: Ia05853506d424ad4cb2c11770ae7718fbef3f01c

(cherry picked from commit c61dc92fc0619d101c4b789f6305012701cc47a0)
(cherry picked with DO NOT MERGE because oc-dev-plus-aosp already
contains a version of this commit compatible with proto2).

Change-Id: I55ea3c61842f16bf6a98e812fb6dc940f96f895c

Merge "Make interface IP serving code set LinkProperties"

Merge "Documentation: changed tid parameter description for getThreadPriority()"
am: 663b02dd28

Change-Id: I4953cf4bf4af592e080114bf937453b84b4a3e4e

Merge "Documentation: changed tid parameter description for getThreadPriority()"

Documentation: changed tid parameter description for getThreadPriority()

Wrong description of the "tid" parameter used by getThreadPriority():
the function does not "change" the priority of the thread associated to
"tid".
Defined also the meaning of setting tid parameter to 0.

Test: None

Change-Id: I8552f3b110eaab1aefa51477e82d3e295547be34
Signed-off-by: Alessio Balsini <alessio.balsini@gmail.com>

Make interface IP serving code set LinkProperties

Additionally, clean up awkward IPv6TetheringInterfaceServices
instantiation mechanics.  In future, this class will be absorbed
by TetherInterfaceStateMachine (prior to its renaming to IpServer).

Test: as follows
    - built
    - flashed
    - booted
    - runtest frameworks-net passes
Bug: 29337859
Bug: 32163131

Change-Id: Ib620e3df182f9f8e2c019aae1cd8981eb0b29376

Add callback for initialization done

Add callback for initialization done in the framework, and listen to it
in the testapps. Make initialization asynchronous as well for both
download and streaming.

Change-Id: Iea7f803df9d2752401b2eca9f6c7375007cac35e

Merge "Simplify NfcCommand and avoid NullPointerExceptions"
am: b5c4e80ecd

Change-Id: I2dcff4d7d0b0bdc5b344f08ce5d563da0a272ddf

Merge "Simplify NfcCommand and avoid NullPointerExceptions"

Embms download part 2

Add support for multi-part file downloads.
Improves destination directory handling in the download process.

Change-Id: Ibad57bab8804530ce09305424790d5520cd02071

Merge "Add a svc command for enabling/disabling Bluetooth"
am: 7d136b777a

Change-Id: I62158ee728e6c431c21e5de30ecd9b7b33bbc73a

Merge "Add a svc command for enabling/disabling Bluetooth"

Add a svc command for enabling/disabling Bluetooth

Bug: 62067926
Test: svc bluetooth disable
      svc bluetooth enable
Change-Id: I9a53d7ab1805df98d8cf1292cf1061eacc14655d

Merge "Track movement of some libcore/tzdata files"
am: 2ed8b06173

Change-Id: Icfe3e6bb1e0d19be462fb47c23cba1cf77c6bf20

Merge "Track movement of some libcore/tzdata files"

Merge "Fixes for tetheroffload crashes"
am: 5ac7a03f9f

Change-Id: I08406513912c9fbd383625f10837085abb5c0516

Merge "Fixes for tetheroffload crashes"

Fixes for tetheroffload crashes

Now that we can talk to the HALs (with some out of tree CLs and
"setenforce 0"), several crashes were encountered.

Fixes here include:
    - avoid hidl_handle move semantics
    - check HIDL method status return value (isOk())
    - convert Java short port numbers to ints
    - don't pass nulls to HIDL where Strings are required
      (limitations in parceling)

Test: as follows
    - built
    - flashed
    - booted
    - runtest frameworks-net passes
    - "setenforce 0" and start tethering
Bug: 29337859
Bug: 32163131

Merged-In: I5fa930be0c9eda491cf17bd4de9b55ab33672d25
Merged-In: Id1037d22826f4d426bccfa17dce0962c54518d64
Change-Id: I91314440c3a04e5f2502579b5f06dac9f25cf0cd
(cherry picked from commit 1185459c4169891bd7550655218e738c8bf1abe2)

Track movement of some libcore/tzdata files

Some files in libcore/tzdata are moving to system/timezone
under the package com.android.timezone.distro.

To run tests:
make -j30 FrameworksServicesTests
adb install -r -g "out/target/product/angler/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"

Test: See above.
Bug: 31008728
Merged-In: I2600dd4bee22a571b9eb7303733591b60d1a8b2a
Change-Id: I2600dd4bee22a571b9eb7303733591b60d1a8b2a

Merge "When updating a split app, copy compiled files from base.apk only."
am: 8a945baf4a

Change-Id: I7c2733ac9dfe60ed8a629d5d312b760c53baa76e

Merge "When updating a split app, copy compiled files from base.apk only."

Simplify NfcCommand and avoid NullPointerExceptions

Test: adb root
      adb shell
      stop
      svc nfc enable
      start
      svc nfc disable

Change-Id: Ie5170f0525f8830d1d80b5607191ce95e78a55ae

Merge "Merge "Use java.util.Objects.hashCode to get hashcode." am: 3890689bb2" into stage-aosp-master

Merge "Use java.util.Objects.hashCode to get hashcode."
am: 3890689bb2

Change-Id: I6b295dfe38e8c75ab6dd3add2521d63b62f4c8f1

Merge "Use java.util.Objects.hashCode to get hashcode."
am: 3890689bb2

Change-Id: I07e33a09f7ed2af457d2ad6261b5469b9dfd15c2

Merge "Use java.util.Objects.hashCode to get hashcode."

Merge "NfcTile: make sure NFC adapter is always available when needed"
am: 13ade8f910

Change-Id: I731c4d0f3460d1b2a24232a87afcd5c327ecb4b0

Merge "NfcTile: make sure NFC adapter is always available when needed"

NfcTile: make sure NFC adapter is always available when needed

Test: enable NFC->reboot->directly open QS panel, NFC tile will show disabled state. This commit fixes it.

Change-Id: Iabbe4ec11a38614b9c65627d58c6ee9ec93435f3

Use java.util.Objects.hashCode to get hashcode.

Test: Telephony sanity tests
Change-Id: I8b845f2e5e44b242278b8955807d1b40e67e1414

Merge "Merge IPv6TetheringInterfaceServices into TetherInterfaceStateMachine"
am: 6f029bbc82

Change-Id: I098bd78d29cf4119208d24977dab8494d3bf5fdb

Merge "Merge IPv6TetheringInterfaceServices into TetherInterfaceStateMachine"

Merge "Adds error codes and maximum RAN/Bands/Channels"
am: 70373f7928

Change-Id: I1c169772fb1335df4128b186e826f1bed1ca501d

Merge "Adds error codes and maximum RAN/Bands/Channels"

Merge "Do not CloseGuard KernelID or FieldID"
am: 6b70eafae1

Change-Id: I89390175e3404489991e599a74034e559a527435