Merge "Move SDK Check for getNeighboringCellInfo"

Merge "Fix issue with sort_api.sh"

Merge "Start on DexBuilder"

Merge changes I66e40b4b,I599a2ff9,I7c210623

* changes:
  [PT16] Simplification of sendProxyBroadcast.
  [PT15] Tiny bugfix in setGlobalProxy
  [PT14] No-op refactoring of sendProxyBroadcast

Fix issue with sort_api.sh

Should use -ne not -neq.

Test: run script manually
Bug: 115609023
Change-Id: I337ed43be1f9250e6c2b2c88d97c68a5c9e8941c

Merge "Fix flaky ScheduleCalendarTest"

Merge "Add new "max-sdk-p blacklist"."

Merge "Network-related commands porting"

Move SDK Check for getNeighboringCellInfo

Move the SDK check for getNeighboringCellInfo() to
inside the phone process for security.

Bug: 117520186
Test: atest CtsPermissionTestCasesSdk28
Change-Id: I9ca059dad0ed5d2107af154e0a63c282133aba3d

Merge "Change ZipArchiveHandle from void* to ZipArchive*"

Change ZipArchiveHandle from void* to ZipArchive*

Bug: none
Test: m checkbuild
Exempt-From-Owner-Approval: owner is OOO for another week,
  it's a fairly minor change to this repository
Change-Id: If4cf57619034ab98b06115ca60beb2fb26c4cd19

Merge "Update robolectric dependency to v3.6.2."

Merge "Add Feature Flag for Hearing Aid Profile"

Fix flaky ScheduleCalendarTest

Accurately get the next day.

Test: atest com.android.server.notification.ScheduleCalendarTest
Fixes: 110355145

(cherry picked from commit fe099b59ec4d6e387837082310326a2a7564b2c3)

Test: Treehugger
Bug: 110355145

Change-Id: Idc20944679b44b30a40eb65d958df9fffec69375
Merged-In: Idc20944679b44b30a40eb65d958df9fffec69375

Start on DexBuilder

This change begins work on DexBuilder, which will allow us to generate DEX files
directly from layouts. This version is rather limited, but we will expand its
functionality in future CLs. The DexBuilder in this CL can create a DEX file
from scratch, define classes, and define methods on those classes. Within
methods, it supports extremely simple instructions, such as storing a small
constant in a register and returning a value from a register. Additionally,
there are tests to make sure that at least at a structural level, the generated
DEX files are valid.

DexBuilder and its associated builder classes use functionality from
libartdexfile and the Dex Slicer tool to support the actual encoding of DEX
files.

Test: atest, also manually tested by loaded a generated DEX file in an Android
app and verifying its behavior.

Change-Id: Iaa01aa7e3a0c7e4d5f4fa8dbce1492499c93c222

Merge "Fix overlap between operator name and heads-up notification"

Merge "Telephony light grey list clean up for Q"

Merge "Grant shell permission to manage accessibility"

Fix overlap between operator name and heads-up notification

The title of heads-up notification was shown incompletely
becaue it's covered by operator name.
To fix this issue, the operator name should be hidden
when a heads-up notification is shown.

Fixes: b/116533819
Test: atest SystemUITests
Test: manual - enable config_showOperatorNameInStatusBar
               and show a heads-up notification

Merged-In: Id3318cd7cb2c9b6950392fadd11d02ab873d84c5
Change-Id: Id3318cd7cb2c9b6950392fadd11d02ab873d84c5

Merge "Remove explicit dependencies on conscrypt"

Merge "Request lmkd to purge its list of pids after establishing connection"

Telephony light grey list clean up for Q

Bug: 112477300
Test: Telehony unit test
Change-Id: I5229224fe9a65b3647a42070f287e23f67084bda
Merged-in: I5229224fe9a65b3647a42070f287e23f67084bda

Request lmkd to purge its list of pids after establishing connection

lmkd keeps a list of pids registered by ActivityManager, however on rare
occasions when framework restarts and lmkd survives that list has to be
purged. Request lmkd to clear its pid list immediately after establishing
connection to it.

Bug: 116801366
Test: locally by killing zygote process
Change-Id: Id8c81e0cb0c4cf03b8faa0add5a9514fd3cd0c0c
Signed-off-by: Suren Baghdasaryan <surenb@google.com>

Merge "JavaBBinder: implement getInterfaceDescriptor"

Add new "max-sdk-p blacklist".

This will contain greylist entries accessible by apps with a
targetSdkVersion of less than Q. Currently empty, will be populated in
follow up CLs.

For now, these APIs are just merged into the light greylist at build time,
pending support in the runtime for implementing the correct runtime
behavior.

Also fix a bug in sort_api.sh so it deals with empty API lists
correctly.

Bug: 115609023
Test: m
Change-Id: I213874062f393f96d120648a934ae5ad7aba93af

Remove explicit dependencies on conscrypt

The dependency is unnecessary since conscrypt is
now included in the "standard libraries".

Test: build
Bug: 113148576
Change-Id: I5d6ae56af35fddb452fdd5b6abbb47178c1bfdc0

Merge "Change permissions checks for setAirplaneMode."

Merge "Prevent NFE in SystemUI when parsing invalid int"

[PT16] Simplification of sendProxyBroadcast.

sendProxyBroadcast is always called with the same argument, and
it would make no sense with another argument anyway. Remove it.

This concludes the ProxyTracker refactoring with 227 lines removed
from ConnectivityService, a lot clarified, and some bugs removed.
Things can still be improved, but presumably at a much higher cost.

Next steps are : write tests, now that ProxyTracker is both testable
and mockable. And try to pour some gasoline on the PROXY_CHANGE_ACTION
broadcast, see if it burns well.

Test: runtest
Change-Id: I66e40b4bf5cfd0b2dc4fa37ea97b3429fe1b7e6c

[PT15] Tiny bugfix in setGlobalProxy

This bug has existed for a long time. If mDefaultProxyEnabled is
false, then the mDefaultProxy member should obviously not be used
in the broadcast.

Test: runtest
Change-Id: I599a2ff9f96d4667e824cf000c2125f86010bb02

[PT14] No-op refactoring of sendProxyBroadcast

If mGlobalProxy is non-null, then getDefaultProxy returns mGlobalProxy
so the first change is a no-op.
If mGlobalProxy is null and mDefaultProxyEnabled is true, then
getDefaultProxy returns mDefaultProxy, which has just been set to
proxyInfo, so the second change is a no-op.
If mGlobalProxy is null and mDefaultProxyEnabled is true, then
getDefaultProxy returns mDefaultProxy ; if mGlobalProxy is null and
mDefaultProxyEnabled is false, then getDefaultProxy returns null,
therefore the third change is a no-op.

Test: runtest
Change-Id: I7c21062302bf54f4fc917c82e0175975051a55ec

Merge "Build com.android.media.remotedisplay with java_sdk_library"

Add Feature Flag for Hearing Aid Profile

Using the Settings App-Developer Options-Feature Flag, allow the user to
enable or disable the Hearing Aid Profile.

Bug: 116317072
Bug: 116044083
Test: Manual testing using Settings App
Change-Id: I58a9d339941e235242c443c85b6f4194b5a296c9
Merged-In: I58a9d339941e235242c443c85b6f4194b5a296c9
(cherry picked from commit fe8c8337edc5e9d6b61d0942873538912d523db7)

Merge "Drop unused libcore.io imports."

Drop unused libcore.io imports.

This removes the last import of libcore.io.Libcore from frameworks.
This means that it can be safely hidden from frameworks in a follow-up
CL. Both of the unused imports in this CL were already removed on the
internal master branch (commit a05a994610a8e55b5cda1059fa816f8796135d9c),
but that commit doesn't cherry-pick cleanly onto AOSP master.

The IoUtils import also appears unused, but it wasn't touched by the
above CL so this CL also leaves it alone.

Bug: 117548625
Test: Treehugger
Change-Id: Icc68c64ef2907372af848d74d9cf96070c89d52b

Merge "Remove unused imports of libcore.io.Libcore."

Merge "Enable lint_as_errors for frameworks/base docs"

Merge "Fix implicit fallthrough warnings."

Grant shell permission to manage accessibility

Bug: 116118615
Test: make
Change-Id: I6caa663a09a9119cd62561ed746cd139025edf44

Fix implicit fallthrough warnings.

Test: make checkbuild
Bug: 112564944
Change-Id: Ia33bf816fb6b179ac0a5a6a20bcb50c11b47431d

Enable lint_as_errors for frameworks/base docs

Add back various error/warning related args while running doclava for
frameworks/base docs.

Test: m -j docs
Bug: b/116163454
Change-Id: I9cbe3f0bd1b6acfe0b6e2c6567c78bb70f892c77

Merge "refactor data/voice roaming states"

Merge "Remove BluetoothDevice.ACTION_DISAPPEARED"

Update robolectric dependency to v3.6.2.

This is required for support of v53 .class files.

Test: The following tests pass both before and after this CL:
      make RunFrameworksServicesRoboTests RunBluetoothRoboTests \
           RunCarMessengerRoboTests RunKeyChainRoboTests \
      RunManagedProvisioningRoboTests
Test: The above tests also pass when EXPERIMENTAL_USE_OPENJDK9=true
      (they didn't pass with v3.6.1).
Test: The following tests fail both before and after this CL (error
      trying to mock a final class):
      make RunStorageManagerRoboTests RunEmergencyInfoRoboTests

Bug: 116872534
Change-Id: I903a3448af1c0be55cc4af098521b3ba422c5579

Merge "Add DumpstateOptions java implementation"

Merge "Remove explicit dependencies on "core" libraries"

Merge "NetworkMonitor: add log of CaptivePortal result"

Change permissions checks for setAirplaneMode.

Require NETWORK_SETTINGS (or NETWORK_SETUP_WIZARD) instead of the
legacy CONNECTIVITY_INTERNAL permission. The users are as follows:

- The system callers (Phone, Settings, SystemUI, VrSettings) all
  have NETWORK_SETTINGS.
- SetupWizard has NETWORK_SETUP_WIZARD
- sl4a has NETWORK_STACK

Bug: 115302596
Test: builds, boots, airplane mode via SystemUI works
Change-Id: I8ca40182bd8b5e3fd9a82296c0cc28de30ed4baf

Merge "[PT13] Simple cleanup of ProxyTracker."

Merge "Fix incorrect javadoc link"

Merge "[PT12] Factor some code into ProxyTracker."

Merge "Add log for debugging ConnectivityManager"

Merge "Add the metalava_enabled property"

Fix incorrect javadoc link

Test: m -j docs with -lerror enabled for frameworks/base docs.
Bug: b/116163454
Change-Id: I02a164b676397dd2d9e8a52ae344c1033b1a4029

Merge "Introduce AlternativeNetwork APIs"

refactor data/voice roaming states

1. migrate set/get roamingType from ServiceState to
NetworkRegState.set/get roamingType
2. migrate set/get roamingStateFromRegstate from ServiceState
to NetworkRegState.regState
3. new System API for neworkRegistration.isRoaming().

Bug: 116127096
Test: ServiceStateTest, ServiceStateProviderTest

Change-Id: Ib89744089c98713819c8d313240d629ddb77d0bf

Remove BluetoothDevice.ACTION_DISAPPEARED

This intent was never sent by the Bluetooth stack

Fixes: 112115770
Test: make
Change-Id: Iaf71aeafa21f44ec9b59c1b766c936ab7dde39f1

Merge "Camera: Trivial doc update"

Merge "getTelephonyProperty returns defaultVal"

Merge "Update PROPERTY_HIGH_DEF_AUDIO API docs."

Merge changes I7070326d,I8346e53c

* changes:
  Add OWNERS file to AndroidKeystoreSPI implementation
  KeyguardStateMonitor looses connection with keystore if keystore dies

Merge "Create URI for individual user settings in SIMINFO"

Update PROPERTY_HIGH_DEF_AUDIO API docs.

Make it more clear that the intention of the property is to convey whether
a call is high definition on the UI or not, and NOT to report the actual
audio codec in use.

Test: Build
Bug: 78550833
Change-Id: Id002cd1c9cc1d5aea6f8d9ab4e3d99956b7c42b2

Introduce AlternativeNetwork APIs

Introduce AlternativeNetwork APIs

Bug: 113106744
Test: Verified using test app to make api calls
Change-Id: I134d1aef40f014378fd1364e0bfec6926fe2093e
Merged-In: I7f470cd6028a12cc66a660d58720f803271d38eb

Merge "Remove ResponseCache from apache-http"

Add OWNERS file to AndroidKeystoreSPI implementation

Change-Id: I7070326d680c6bff00839d847f8fadbbe2b16fa7

Camera: Trivial doc update

Change-Id: I63198c6d2cb348380e718974dd356d48137ae376

Add DumpstateOptions java implementation

DumpstateOptions will be used to pass options to Dumpstate service
for specifying arguments for taking bugreports.

Test: m
Test: adb bugreport

Change-Id: I4dc9511f0be2f86239ea9094a3c527b6ce0d1d41

KeyguardStateMonitor looses connection with keystore if keystore dies

When keystore dies it no longer gets information about the lock screen
visibility state. This state is vital to enforcing the "unlocked
device required" authorization of keymaster keys.

With this patch KeyguardStateMonitor tries to reestablish the connection
to keystore if communication fails.

Test: run atest android.keystore.cts.CipherTest#testKeyguardLockAndUnlock
      after killing keystore
Bug: 117552147

Change-Id: I8346e53c342bdba0f5960b1feba7c26db5cef33e

Remove explicit dependencies on "core" libraries

The dependencies are unnecessary since bouncycastle, okhttp
and conscrypt are included in the "standard libraries".

Test: build
Bug: 113148576
Change-Id: Ic8d15d911decbdd6391db8c1231b845353586a54

Merge changes Icd51c4c7,I836ecb94

* changes:
  Add entries to hidden api greylist
  Add method to hidden api greylist

Merge "Frameworks: Add VMRuntime.hasBootImageSpaces check"

Merge "Revert "DO NOT MERGE: Move AM/WM unit tests out of FrameworksServicesTests to WmTests""

Merge "Whitelist free short code batch for 2018 Q3"

Merge "[PT11] Move handleDeprecatedGlobalHttpProxy to ProxyTracker"

Add entries to hidden api greylist

Bug: 115387204
Test: m appcompat
Change-Id: Icd51c4c7d446ea72211804fbb050f4fee742db17

Add method to hidden api greylist

Bug: 114488777
Test: m appcompat
Change-Id: I836ecb947390f7fc6986f6848299375c7599fc97

Merge "Remove unused import"

Revert "DO NOT MERGE: Move AM/WM unit tests out of FrameworksServicesTests to WmTests"

This reverts commit dee5a4dc2a5a6829e5509fc0e94903e30dcc238f.

Reason for revert: CLs on upstream branch are on hold.

Change-Id: I253294d080efdfd1dbf377812f12880e03dff1f6

Merge "Refactor tetherGetStats by using stable aidl structure"

Merge "Remove BluetoothDevice.ACTION_DISAPPEARED"

Merge "Make setAirplaneMode @SystemApi for SetupWizard."

Remove BluetoothDevice.ACTION_DISAPPEARED

This intent was never sent by the Bluetooth stack

Bug: 112115770
Test: make
Change-Id: I6c2b87529af7e6a7123cde8141fbdb9d1822ec9f
Merged-In: I6c2b87529af7e6a7123cde8141fbdb9d1822ec9f
(cherry picked from commit dd1c592fcd11e2bee25db0077da0ef3f59071d5d)

Merge "Fix incorrect javadoc links"

Merge changes from topic "aapt2-manifest-elements"

* changes:
  Remove workarounds for b/79755007
  AAPT2: Add the rest of supported manifest elements
  AAPT2: Add supported manifest elements

Merge "Avoid hidden DataFormat.is24Hour field access"

Merge "Phone service state change atom."

Phone service state change atom.

Background: BatteryStatsService tracks the phone scanning time metric. In Q,
we are migrating the dumpsys-based collection to the new infra. This CL is to
instrument the new logging mechanism.

Test: statsd test
FIX: b/116748990

Change-Id: I3cf5e0026bfc6f594c5f7b524f14b3a43b702afb

Merge "dex/TEST_MAPPING: referenced non-test modules"

JavaBBinder: implement getInterfaceDescriptor

JavaBBinder previously only implemented transact (which works because
it calls into Java and the Java binder will call checkInterface).

However, when ibinderForJavaObject is used, the binder returned will
not properly report its interface descriptor. In this CL,
getInterfaceDescriptor is implemented so that this is reported
correctly.

Bug: 112664205
Test: atest android.binder.cts (fails without this)
Change-Id: Ic8f7d202021688287edafdc27db25fc8ec2f075c

Merge "Mark getInstallerPackageName @Nullable"

Avoid hidden DataFormat.is24Hour field access

DateFormat.is24Hour is not part of the CorePlatformApi but
DateFormat.set24HourTimePref(Boolean) is. Switch over.

Bug: 113148576
Test: build
Change-Id: I27103890f94cfe3679f386ea33b0450b453a7625

Remove unused import

StructGroupReq is imported but not used.
The import can be removed.

Bug: 113148576
Test: build only
Change-Id: I6b2ec5eb3f6c6e298fcde8e47dfd0979f889ca1d

Merge "Remove usage of StringFactory"

Remove unused imports of libcore.io.Libcore.

Test: Treehugger
Bug: 115503977
Bug: 117548625
Change-Id: I122fa68ddc2bd531f627dcac00834b47249bd56e

Merge "Remove dependency on internal libcore method"

Remove usage of StringFactory

Code outside of the runtime should not use StringFactory.
The java.lang.String API works fine.

Bug: 113148576
Test: build only
Change-Id: I2912e02265306dcd5c5923de019b2c376349ea29

Merge "Switch test to TestIoUtils from IoUtils"

Remove dependency on internal libcore method

The test already uses FileUtils.deleteContents() in several
places so the one usage of IoUtils.deleteContents() which is
functionally equivalent can be switched over to that instead.

Bug: 113148576
Test: build only
Change-Id: Ic5569c82701e89ae9178c1438caf1e8885861a76

Make setAirplaneMode @SystemApi for SetupWizard.

Expose ConnectivityManager#setAirplaneMode and the
NETWORK_SETUP_WIZARD permission so that SetupWizard can call
setAirplaneMode.

Delete setAirplaneMode from the greylist because the permissions
checks on it, since at least 2013, have been for a privileged
permission (CONNECTIVITY_INTERNAL) that normal apps don't have.

Test: m
Bug: 115302596
Change-Id: I28f24d32c4b4b386407e9e0e3514ea9f8854f4a8