Merge "Networking unit tests: fix some flaky tests"
am: 501bae6492

Change-Id: I6815e732f334d11208ff12af4023912593d96499

Merge "Networking unit tests: fix some flaky tests"

Merge "Pass all offload-exempt prefixes into OffloadController"
am: 7fc4f63f23

Change-Id: Ibbca37f65dc0a0398432f325bf362996b9d61519

Merge "Pass all offload-exempt prefixes into OffloadController"

Merge "Switch IpManager and IpReachabilityMonitor to use SharedLog"
am: a3ea0e2337

Change-Id: I380551a59a1278df4cb6a0e7e590edae38e79881

Merge "Switch IpManager and IpReachabilityMonitor to use SharedLog"

Networking unit tests: fix some flaky tests

 - less strict regex for SharedLogTest: the subsecond part of the
   timestamp can have 0, 1, 2 or 3 digits.
 - refactor NetworkStatsServiceTest and NetworkStatsObserversTest to use
   waitForIdleHandler facility of ConnectivityServiceTest.
   NetworkStatsServiceTest was using a flaky custom version of
   waitForIdleHandler.

Bug: 62918393
Bug: 32561414
Test: runtest frameworks-net
Change-Id: I634acfb5f4fe1bd5267e3f14b9f645edc32d5d12

Switch IpManager and IpReachabilityMonitor to use SharedLog

Test: as follows
    - built
    - flashed
    - booted
    - "runtest frameworks-net" passes
    - frameworks/opt/net/wifi/tests/wifitests/runtests.sh passes
    - frameworks/base/wifi/tests/runtests.sh passes
    - roamed from AP to AP watching IpRM logs accumulate
Bug: 62476366
Bug: 62162774

Merged-In: I4bef3b6b1975fa3c73cca0a0e95fad99708f061e
Merged-In: I265566158f3e5ebd21fb43e2eb0b8ba016fca76a
Change-Id: I155a26259420919eeed74166bacce3311caa49da
(cherry picked from commit 0fc10b2b1d7be649c80dda4ebdb2e31af216bfcc)

Merge changes I6779a835,I4e3d8322
am: 1fbe962482

Change-Id: I99af01094a2e1d905f47b12822a036a1de6e3611

Merge changes I6779a835,I4e3d8322

* changes:
  Only connect to one address per address family.
  Update our user agent to something newer.

Merge "[BUG] java.util.MissingFormatArgumentException: Format specifier '%d'"
am: 02309e9748

Change-Id: I1687f90ae7d042db2575ca10b168833e27adc7cb

Merge "[BUG] java.util.MissingFormatArgumentException: Format specifier '%d'"

Pass all offload-exempt prefixes into OffloadController

Additionally:
    - eliminate useless TetherMasterUtilState

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

Change-Id: I0482ac9dd9e8ba558b253643dbc01875ce9475ef

Only connect to one address per address family.

This ensures probes don't take too long when a host has
multiple addresses. So for example:

$ adb shell settings put global captive_portal_http_url http://play.googleapis.com/generate_204
$ host play.googleapis.com
play.googleapis.com is an alias for googleapis.l.google.com.
googleapis.l.google.com has address 216.58.196.234
googleapis.l.google.com has address 216.58.197.10
googleapis.l.google.com has address 172.217.26.106
googleapis.l.google.com has address 172.217.27.170
googleapis.l.google.com has address 216.58.199.234
googleapis.l.google.com has address 216.58.221.170
googleapis.l.google.com has IPv6 address 2404:6800:400a:807::200a

results in:

06-29 21:38:09.695 - PROBE_DNS play.googleapis.com 270ms OK 2404:6800:400a:806::200a,172.217.26.106

Bug: 36532213
Test: as described in commit description
Change-Id: I6779a8356bac8d4a3912b0822379d45dfc662541

Merge "Add boot profile support"
am: 3d901ec513

Change-Id: I37db2f4bd7e6412954d51e914f6e238db5027a6e

Merge "Add boot profile support"

Add boot profile support

If the boot profile file exists, its passed to the runtime as an
image compiler option. Also pass speed-profile for this case.

Bug: 37966211
Test: make, flash, and look at logcat during boot

(cherry picked from commit f99e70e93408bd6dfda595866f13ba0068d4798e)

Change-Id: Ia09ff0200186382e45151b7cabb53016f8e0aa3f

Merge "hidl getService throw correct exceptions."
am: 5d5ace5e11

Change-Id: I16bcf1b32914ef42bbe788a8b67e92f6fe9d04a2

Merge "hidl getService throw correct exceptions."

Merge "Track move of distro installer code"
am: 8558d93db8

Change-Id: I2d27a58c21b7b8931b1d058d6a6bc88eeb8f7709

Merge "Track move of distro installer code"

[BUG] java.util.MissingFormatArgumentException: Format specifier '%d'

Bug: https://issuetracker.google.com/issues/63121293

Test: null

Change-Id: I2745c415e21a7f26535605cc5bef7eaf2109ae8e
Signed-off-by: liangweikang@xiaomi.com <liangweikang@xiaomi.com>

Merge "Implement cancelDownload and remove appName"
am: 9b43ffcf2f

Change-Id: I98914bf79ad6178e1ef8689acf5d0a18c78a7735

Merge "Implement cancelDownload and remove appName"

hidl getService throw correct exceptions.

NoSuchElementException needs to be thrown when getService is
called and the service is not available.

Test: hidl_test_java
Fixes: 63087967

Merged-In: I207098cfb732e888f91704f623e888e72e6ce7d9
Change-Id: I207098cfb732e888f91704f623e888e72e6ce7d9

Merge "Support battery level reporting via Apple VSC (1/2)"
am: 874af710b7

Change-Id: Id65e6e11308c66893cb701af8909df9fee3a131f

Merge "Support battery level reporting via Apple VSC (1/2)"

Support battery level reporting via Apple VSC (1/2)

* Add related constants to BluetoothHeadset

Bug: 35874078
Test: make, test with supporting headsets
Change-Id: I9f49ea28efdf6de0e751a377d879fe13a47655d5

Merge "Support battery level reporting via Plantronics XEVENT (1/2)"
am: c79c4240ec

Change-Id: I6aa50e541763adfa9ebe738d98d86152cf0cd1d5

Merge "Support battery level reporting via Plantronics XEVENT (1/2)"

Support battery level reporting via Plantronics XEVENT (1/2)

* Add related constants to BluetoothHeadset

Bug: 35874078
Test: make, test with supporting headsets
Change-Id: Ied089a065bf9caa0d03f681950fefa2ea990bb3e

Merge "No need to explicitly list ethernet upstream type"
am: 01017959fe

Change-Id: I93a74d80db134e832cef16d61abc057c88e86675

Merge "No need to explicitly list ethernet upstream type"

Track move of distro installer code

Track move of distro installer code; reflect the new
build target.

Tested with:
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"

Bug: 31008728
Test: See above
Change-Id: Iad987d404af75af3694d50e25ed419889a9557f2

Merge "Use sendBroadcastAsUser rather than sendBroadcast"
am: d129a14647

Change-Id: I76324b2d983887dbdac4b26e518a68d8c4035a47

Merge "Use sendBroadcastAsUser rather than sendBroadcast"

Merge "HFP: Modify comments for ACTION_HF_INDICATORS_VALUE_CHANGED (1/2)"
am: 5790bae2f4

Change-Id: I480ef38d5aa61aa153ee53d325e8b249c1eab222

Merge "HFP: Modify comments for ACTION_HF_INDICATORS_VALUE_CHANGED (1/2)"

HFP: Modify comments for ACTION_HF_INDICATORS_VALUE_CHANGED (1/2)

* Per HFP 1.7.1 spec page 102 of 144
  EXTRA_HF_INDICATORS_IND_ID should be int
  EXTRA_HF_INDICATORS_IND_VALUE should be int

Bug: 35874078
Test: make, PTS test, unit tests
runtest -c com.android.bluetooth.btservice.RemoteDevicesTest bluetooth

Change-Id: Idd316a82bab164b6ea1701bb261b1cc3dee196f1

No need to explicitly list ethernet upstream type

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

Merge "Use Bundle for inter-process message"
am: db8e4efd5b

Change-Id: Ib93e91969063a5d2276c99b8b0fee4dd1a398771

Merge "Use Bundle for inter-process message"

Implement cancelDownload and remove appName

Implements the cancelDownload call
Removes appName as an argument from all EMBMS-related apis
Implements the cross-filesystem move method in MbmsDownloadReceiver

Change-Id: Ie8ebfccca4fbb3d69289beb2ed77101fe2c7727f

Merge changes from topics 'embms-cleanup-action', 'embms-download-token'
am: c2f33f18eb

Change-Id: I7c2b0b34ed56bc506ce6139fb39649aec8319e81

Add download request tokens and enforce destination clear
am: 2043f70acb

Change-Id: I91e61d8f15815efc9f4d08d71d01944c096dc05f

Merge changes from topics 'embms-cleanup-action', 'embms-download-token'

* changes:
  Implement the cleanup action in MbmsDownloadReceiver
  Add download request tokens and enforce destination clear

Use Bundle for inter-process message

The message sent from NetworkScanRequestTracker to TelephonyScanManager
might be across different process, so we have to wrap the List<CellInfo> into
Bundle because List<> is not parcelable.

Cherry-picked cleanly from:
https://googleplex-android-review.googlesource.com/#/c/2467157/

Test: Telephony sanity tests
Bug: 30954762

Merged-in: I77945b247530b85c0b82876d528549498a711550
Changg-Id: I77945b247530b85c0b82876d528549498a711550
(cherry picked from commit 56049437095d3cb8334f667ff73061f2926b5416)

Change-Id: Iec4043e611a8f58733a361ec9a52fc74e1c974d5

Merge "Switch PackageStatusStorage to XML-based storage"
am: 4e77ae6d1a

Change-Id: Ib30177ef93ecacfbd6ab0fb98f933fbf8af0e270

Merge "Switch PackageStatusStorage to XML-based storage"

Update our user agent to something newer.

Bug: 36532213
Test: aosp_bullhead-eng builds
Change-Id: I4e3d832248a746c9fef5302a1eafbe66ade8ed25

Merge "Add APIs to get remote device's battery level (1/2)"
am: ceaccbfe70

Change-Id: Ic3253ef666632aaaffe8e1547dc563cd3bca9e37

Merge "Add APIs to get remote device's battery level (1/2)"

Implement the cleanup action in MbmsDownloadReceiver

Implement ACTION_CLEANUP, add result codes for the ordered broadcasts,
and remove the download request extra for file descriptor requests

Change-Id: I5ee19ad0aefbdceacb636b3d17e197a6fb1bdaa6

Add APIs to get remote device's battery level (1/2)

* Add BluetoothDevice.getBatteryLevel() API to retreive battery level
  information of remote device
* Add BluetoothDevice.ACTION_BATTERY_LEVEL_CHANGED intent to notify user
  that remote device's battery level has changed

Bug: 35874078
Test: make, pair with devices and use them
Change-Id: I41051ee25383f5f3a1e505aef6f8c526385f58bd

Use sendBroadcastAsUser rather than sendBroadcast

According to messages in the logcat sendBroadcastAsUser
is the better option.

Test: Manual
Bug: 31008728
Change-Id: I06bcd3c53878fc2b1b817337ea8719fcc4975501

Merge "IMS: Check for binderstate before calling endSession"
am: c49534cf09

Change-Id: I9d173693c466b7d8512d11f266af48f5307669d5

Merge "IMS: Check for binderstate before calling endSession"

Merge "Switch to streaming data for time zone update"
am: f093d4e13e

Change-Id: I3fd22fe47e1d7b86d4f88d527940c6812ecfa31b

Merge "Switch to streaming data for time zone update"

Merge "Clean up Predicate related code in android.test"
am: 1230b249f1  -s ours

Change-Id: I4836793deccd83e06129ad18035f4f4acf5f8617

Merge "Clean up ClassPathPackageInfoSource"
am: 8c7026c5a3

Change-Id: I503ae16c4d47ce3cecb79987da919c7f579347da

Merge "Clean up Predicate related code in android.test"

Merge "Clean up ClassPathPackageInfoSource"

IMS: Check for binderstate before calling endSession

ImsRegistrationListenerProxy listener, which is registered
as part of ImsManager.open, is not cleared even after moving
feature state to NOT_AVAILABLE because end session
is never called.

Check for binder state instead of feature state READY before
calling end session. End session will be triggered after
moving feature state to NOT_AVAILABLE.

Notify Ims state change only for the supported feature state
change to clients.

Test: Manual
Bug: 62816014
Change-Id: I1bb588ce2be9c181355ff744f43dad4e784e4ec5

Merge "[Resources] house keeping in TypeList"
am: ff2e03cfce

Change-Id: I22b0a87776d794f23f70f2f9d8d885761c0b8d14

Merge "[Resources] house keeping in TypeList"

Merge "Revert "Add ActivityInfo logging."" into oc-dev
am: 61196fef1f

Change-Id: I1ead0fd55eb35fd69a4c8e3de5a1449ce296edd9

Merge "Revert "Add ActivityInfo logging."" into oc-dev

Merge "Filter instant apps with different names" into oc-dev
am: adcc94f160

Change-Id: I855a58e6ea0655b4211f2aac0a4a1f9f94f639f2

Merge "Fix QS lock screen animation" into oc-dev
am: aa2286e417

Change-Id: I40a6e211fa02a83de09edcfac419b8e78a287e2f

Merge "Filter instant apps with different names" into oc-dev

Merge "Fix QS lock screen animation" into oc-dev

Merge "Fix TimePicker RTL issues" into oc-dev
am: 7b95c932d6

Change-Id: I9ec629e60882aec2b4ca84b0b81de4244d43afef

Merge "Fix TimePicker RTL issues" into oc-dev

Filter instant apps with different names

ag/2447596 introduced a regression that caused instant apps to be
made visible to one another.

Change-Id: I5db9a43f05dd5bec991cc0934b1c9a9406dfade6
Fixes: 62998487
Test: cts-tradefed run commandAndExit cts-dev -m CtsAppSecurityHostTestCases -t android.appsecurity.cts.EphemeralTest

Fix QS lock screen animation

Now that we have a footer, heights and clippings need to be adjusted
to be more accurate.
 - Clip QS within its layout box regardless of its y translation
 - Calculate height diffs correctly

Test: visual
Change-Id: Ic46996e342cd1000215aa9256da920a7856bb5a4
Fixes: 62489284

Merge "Fix the inconsistency of scan status." am: 41be35a1bd
am: ac2d00a195

Change-Id: Ibcdbe63349031ea95d137dd18465bec5de6efd7a

Merge "Fix the inconsistency of scan status."
am: 41be35a1bd

Change-Id: Ic92aa1aa9ecd91cd072257c93e12cdf030cdeba8

Merge "null check before dereference" into oc-dev
am: df8569787c

Change-Id: I41f9298e305f6f197120f2b3128187a4cbf6b28a

Merge "Fix the inconsistency of scan status."

Merge "null check before dereference" into oc-dev

Revert "Add ActivityInfo logging."

This reverts commit 7e84086387cba189bb0c20f9ed17784840a2904a.

Reason for revert: No Longer needed.

Change-Id: Id6a3c66908a0b7b2e6a666f86f30ce05d3964b4b

Clean up ClassPathPackageInfoSource

Part of the work of removing JUnit and dependent android.test classes
from the Android API involves providing a static library that developers
can include in their test applications to ease migration. That library
will be built directly from the source (as opposed to android.jar which
is built from stubs) and so developers will be able to see classes and
methods that are not present in the stubs. This change is one of a
number of similar changes that cleanup the existing non-API code in
order to minimize the additional methods and classes exposed externally.
The basic approach is to remove unused classes and methods, use least
visible access modifier possible and generally minimize the amount of
publicly visible code.

PackageInfoSources only provided a static field and accessor method so
they were moved into ClassPathPackageInfoSource and PackageInfoSources
was removed.

ClassPathPackageInfo was only used in ClassPathPackageInfoSource and in
TestGrouping. In the latter it was simply used as an intermediate value
between ClassPathPackageInfoSource.getPackage(String packageName) and
ClassPathPackageInfo.getTopLevelClassesRecursive(String packageName).
Moving that method into ClassPathPackageInfoSource allowed the
ClassPathPackageInfo to become an inner class of
ClassPathPackageInfoSource. As it is an inner class it no longer needed
an explicit reference to the containing ClassPathPackageInfoSource.

Bug: 30188076
Test: make checkbuild and ran FrameworkTestRunnerTests
Change-Id: Idb0b6a585030805b9cff8562abb93b7e5920c53a

Merge "Carrier Config changes to support IMSI encryption." am: 1caaa514c9
am: e26191a81c

Change-Id: I777010e6fea82f237ddf224525739a5687f13a2c

Merge "Carrier Config changes to support IMSI encryption."
am: 1caaa514c9

Change-Id: Ia9c4d93e8ababf87a0d6027a4322e73111651844

Merge "Carrier Config changes to support IMSI encryption."

Merge "Avoid use of TimeZoneDistro.equals() method" am: fe0b43cb1e
am: 6b5d71f7e5

Change-Id: Ib4d97f72a9fa936beb47267c9e9a888e306ed0e5

Merge "Avoid use of TimeZoneDistro.equals() method"
am: fe0b43cb1e

Change-Id: I8c263ce15bf29b8a14c862d9acae377ba9be77c3

null check before dereference

Change-Id: I6ae219b68920c160e104b307f60802fd75e705ce
Fixes: 62908319
Test: Manual

Clean up Predicate related code in android.test

Part of the work of removing JUnit and dependent android.test classes
from the Android API involves providing a static library that developers
can include in their test applications to ease migration. That library
will be built directly from the source (as opposed to android.jar which
is built from stubs) and so developers will be able to see classes and
methods that are not present in the stubs. This change is one of a
number of similar changes that cleanup the existing non-API code in
order to minimize the additional methods and classes exposed externally.
The basic approach is to remove unused classes and methods, use least
visible access modifier possible and generally minimize the amount of
publicly visible code.

The HasClassAnnotation and HasMethodAnnotation were never used
separately and only used by HasAnnotation. This merges the functionality
into a single nested class in TestPredicates, hidden behind the new
TestPredicates.hasAnnotation(). The HasAnnotationTest was renamed as
TestPredicatesTest and the HasClassAnnotationTest and
HasMethodAnnotationTest classes were removed as their tests provide no
additional coverage. The removal of the Has*Annotation.java files means
that the test-runner/src/android/test/suitebuilder/annotation/ directory
is empty apart from the package.html file so that was moved to
legacy-test to sit alongside the actual annotation classes.

The Predicates class, while part of the legacy-test module was only
ever used by the test-runner module and only its not() method was
actually used. So, the not() method and associated nested class were
moved to TestPredicates, the tests for not() were moved to
TestPredicatesTest and the Predicates* classes were removed.
That allowed for the removal of the legacy-android-tests as that is now
empty.

TestPredicates has a number of constants that were public. They were
hidden by moving them to the class that actually used them.

A minor generic issue was fixed in AssignableFrom.

Bug: 30188076
Test: make checkbuild and run FrameworkTestRunnerTests
Change-Id: I861da388a4146bb28e1e480d1b7ba9137b7b270e

Merge "Remove a.t.TestCase" am: 3c5ab7d324
am: 930ab689f0

Change-Id: I0d61b4e26c9551d177b5bb7bbdaab25885976c0d

Merge "Avoid use of TimeZoneDistro.equals() method"

Merge "Remove a.t.TestCase"
am: 3c5ab7d324

Change-Id: Id2f80c8bb95e3e64207d937929e027c49e602a72

Merge "Remove a.t.TestCase"

Switch PackageStatusStorage to XML-based storage

Based on feedback from a framework developer: an XML file can be used
instead of a Sqlite DB.

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: I1e6614d26df0e37ccea4dff82867e0b6aba39ca6

Fix TimePicker RTL issues

The TimePicker (seen when creating an alarm in the Clock app) has
two bugs in landscape mode:
- The hours/minutes switch places (hours should always be to the left
of minutes, regardless of RTL)
- The layout is completely messed up in 12-hour mode (the radial time picker
does not appear on the screen)

This fix addresses both issues by setting the layoutDirection="ltr"
attribute to force a couple of the layouts to be unaffected in RTL mode.
Specifically, the hours/minutes layout and the overall layout will always be
presented left-to-right.

Bug: 62846484  TimePicker layout is broken in RTL landscape mode
Test: manual (tested with/without fix in normal, forded-RTL mode, and
with Hebrew as the first locale).

Change-Id: I8ef3f2706660f60c162ba3d324a18b8f370f67f8

Merge "Clean up Predicate related code in android.test" into oc-dev-plus-aosp

Merge "IpManager: always log immediate provisioning errors" am: 98f2decdad
am: 2d2f66d277

Change-Id: If1e885c13ce7ec1f480add63940bf9bc929d5072

Merge "IpManager: always log immediate provisioning errors"
am: 98f2decdad

Change-Id: I6d745a4d6e9b932c16fe54fdabe191962389882d

Merge "IpManager: always log immediate provisioning errors"