Merge "Initialize LDAC to ABR mode"

Initialize LDAC to ABR mode

Bug: 35381097
Change-Id: Idcc2fabe86acaf441f5a71f5d2bb66ad29252cdb

Merge "Revert "Development Options: toggle binderized hals""

Merge "[VpnSettings]Crash when activity has been recycled"

Merge "Show correct Wi-Fi status when switching to multi window mode"

Merge "Add default value to Bluetooth AVRCP version property get"

[VpnSettings]Crash when activity has been recycled

This is a case we found in monkey test, the main thread handles message while the activity has been recycled. It will raise a NullPointerException when we use getActivity() in following lines.
Assumpt the situation is: settings run in background and activity been recycled quickly, but handler still in the process.

Test: NA

Change-Id: I19db4f13e13294618ec8e42e4d9c54ce23504344
Signed-off-by: liurong <liurong@xiaomi.com>

Add default value to Bluetooth AVRCP version property get

When the property isn't set a crash can occur since we try to look up an
empty property value.

Bug: 36531010
Test: Open Developer options after a factory reset
Change-Id: I82c0fde00774ab1a4f3e4c44a715c74d4a7804e9

Merge "AVRCP version interop pereference (2/3)"

Revert "Development Options: toggle binderized hals"

This reverts commit 73dd0ae30e2779cb845d1ac6e785d619e1e1e310.

Reason for revert: b/34274385 - This was a temporary setting.

Change-Id: I98b2d05c431bb5361b11cd3b05b7f5ac4e939e0e

Merge "Update SubscriptionManager API by replacing 'id' & 'idx' with 'index'."

AVRCP version interop pereference (2/3)

Add a mechanism to allow the user to select the AVRCP version broadcasted
through SDP in the developer options. This helps solve some interoperability
issues such as with Honda Carkits where broadcasting AVRCP version 1.6 causes
media metadata to fail.

Bug: 35956792
Test: Select AVRCP version through developer option
Merged-In: I7d4b1bd23342d1a96ffdf08bb7d6d3c5e5fedca6
Change-Id: I7d4b1bd23342d1a96ffdf08bb7d6d3c5e5fedca6

Update SubscriptionManager API by replacing 'id' & 'idx' with 'index'.

Bug: 35767068
Test: No build failure on update.
Change-Id: Ib7211a9d0c10a717bfd5a7b22b8bea2b3973d12f
Merged-in: Ib7211a9d0c10a717bfd5a7b22b8bea2b3973d12f

Merge "Bluetooth: stop pairing service correctly"

Bluetooth: stop pairing service correctly

If BluetoothPairingService was stopped and restarted, it would receive
a null intent, and it also would not signal correctly when it needed to
be stopped.

Since restarts can happen, check that we are still bonding when starting.

Also prevent crashes when we might be called with a null intent.

Test: initiate pairing remotely, adb am kill service, notice no crash
Bug: 36130262
Change-Id: I408811935fdb36591826f4481d103cf4799db27e

Merge "Optimized Code: call direct method to get integer extra"

Optimized Code: call direct method to get integer extra

Description: Make the default preferred sim slot as the first one(0) when get PREFERRED_SIM.

Test: NA
Change-Id: Id54bdbefd413c58ecd637cb8ff522861ead79331
Signed-off-by: liurong <liurong@xiaomi.com>

Merge "Bluetooth: leave screen off for permission request"

Merge "Improve Wi-Fi Settings UI in guest mode"

Improve Wi-Fi Settings UI in guest mode

Progress bar is still seen along with a permission forbidden tip in
guest mode and it's confusing. It should be hidden because actually
nothing happens there.

When Wi-Fi is off, Wi-Fi OFF tip should be shown even in guest mode
becuase it should be consistent with Wi-Fi status.

Improves d8c3ca43163427a75c8b70114096238177f97af0.

Bug:31235488
Test: manual inspection.

Change-Id: I611719db253a0aec03d61e881af0e5dc3869d50b
Merged-In: If89728e293721cea45dcd717316d87f6df17ea7c

Merge "Fix network name for Passpoint network"

Bluetooth: leave screen off for permission request

Test: pair to a carkit, disable pbap, reconnect from carkit
Bug: 32555940
Change-Id: I06d6cc0873f9f991e68f1de9b629dbe3a7bb7701

Show correct Wi-Fi status when switching to multi window mode

In WifiSettings, when switching to multi window mode,
WifiSettings fragment is recreated and "onPause()" is called,
then WifiTracker stops and wrong Wi-Fi status is shown.

Move "WifiTracker.startTracking()" to "onStart()",
 "WifiTracker.stopTracking()" to "onStop()"
to make sure to show correct Wi-Fi status.

Bug: 36078677
Test: Manual test OK

Change-Id: I215f180b1082f6039c03de8b4a3ca27d1f364c75

Merge "Changes event handling phases for DeviceListPreferenceFragment."

Changes event handling phases for DeviceListPreferenceFragment.

Bug: 31273140
Test: Manual inspection.

Change-Id: If4622b8d367813eb86a54a45f2e8fa4a55ca38b1
(cherry picked from commit e8d5c8fda16c75e4bcca3a218bc5cf7b254baa99)

Merge "Display the correct default for mobile data always on."

Merge "Unify the design of add icon in setting"

Display the correct default for mobile data always on.

Mobile data always on was enabled by changing the default in
ConnectivityService to 1. However, the default value in the
developer options setting was left at 0. This means that by
default (i.e., when the setting is empty), mobile data always on
is enabled but appears in developer options as disabled.

Fix this by changing the default in developer options. This is
simpler than the other options (adding database upgrade code to
settings, making the default be controlled by config.xml and
ensuring that when a config changed broadcast is received the
value is re-rease, hardcoding a default value as an @hide
constant in Settings, etc.)

Bug: 23113288
Test: setting is off after "adb shell settings delete global mobile_data_always_on"
Change-Id: I27e2c0af7a03e510e640dbcfa281449d00385154

Unify the design of add icon in setting

Add icon "+" displayed in APN setting differs from ones in other
settings like VPN settings, SIP accounts etc. Change the icon to
align with others.

Bug: 35936472
Test: Manually check icon display with fix
Change-Id: I99a57fbeb55bb04f911d28e85752ec04edab6f0e

Merge "wifi: use Passpoint API when forgetting a Passpoint network"

Merge "Change default long-press time to 400ms"

Merge "Show search results when query string contains trailing space"

wifi: use Passpoint API when forgetting a Passpoint network

Use WifiManager#removePasspointConfiguration when forgetting a
Passpoint network to remove the configuration.

This maintains the legacy behavior, where forgetting a Passpoint
network/AP will remove its provider's profile.

Bug: 34207710
Test: manual test
Change-Id: I8b7f6d9fb20fa9f0f0d90566f76c2a39308ce420

Show search results when query string contains trailing space

The query expression used to search for a term prefix is the prefix
itself with a '*' character appended to it. However, if query string
contains trailing space, the query expresssion (e.g. "screen *") doesn't
work. So to fix this issue, the unnecessary trailing space should be
removed.

Bug: 35231587
Test: manual - go to search in Settings and enter "screen "

Change-Id: Ifa5d96f5d38ad5454272f140da440b8ff55d5614

Merge "WiFiCallingSettings: don't start carrier app in onResume"

WiFiCallingSettings: don't start carrier app in onResume

onResume initialize UI before registering switch change listener.
So that listener is not invoked in UI initialization, but only real
switch change by user.

Bug: 33680334
Test: make
Test: manual - carrier app is not invoked in onResume
Change-Id: I7ca1eb1eab214b50c8de72946590c1ecb676d4df

Merge "Bluetooth: aim OPEN_RECEIVED_FILES at bluetooth"

Bluetooth: aim OPEN_RECEIVED_FILES at bluetooth

Bluetooth is the receiver for the OPEN_RECEIVED_FILES intent, so we
should aim the intent directly at them.

Test: go to BT settings and click "Open Received Files" in the context
Change-Id: I057215f1faf04c8c735a7e9340325716225ad22d
Fix: 35246073

Merge "move ACTION_CARRIER_SETUP intent to public api"

move ACTION_CARRIER_SETUP intent to public api

As per CDD: The "android.*" namespace for intent constants is reserved
for public
Android API in AOSP. (Whether public to the full SDK, @SystemApi or
defined in AOSP support libraries.)
ACTION_CARRIER_SETUP intent is generally useful for carrier privileged
apps which is unbundled carrier apps, thus move to public APIs 33679956

Test: Manual
Bug: 33679956
Merged-in: I85dfa99d0b20bb1b534dfe8ead3c274a5c739786
Change-Id: I84ffd5f2c8c3418689ee3d5ff5ddc3f9f7959763

Merge "Bluetooth: service for pairing notification"

Bluetooth: service for pairing notification

Use a foreground service to show the notification, separating dialog
creation from the receiver.

Test: pair request from remote device while the screen is off
Bug: 35234069
Change-Id: Ia3be3e8efdacf465e095af5a498ef5eb0fca83cd

Change default long-press time to 400ms

The default long-press time has been changed from 500ms to 400ms by
b/30159825. so the values for the list of long-press timeout options
in Settings app should be changed.

Bug: 35226519
Test: manual - go to Settings > Accessibility > Touch & hold delay

Change-Id: I8e2421ea4119742bb6bf0e27688c65e3de3099dc

Merge "Add null pointer checks when updating A2DP configuration values"

Add null pointer checks when updating A2DP configuration values

Under expected circumstances, those null pointer checks are not
needed: onCreate() is always called before onCreateView(),
and the former creates the objects: mBluetoothSelectA2dpCodec,
mBluetoothSelectA2dpSampleRate, mBluetoothSelectA2dpBitsPerSample,
mBluetoothSelectA2dpChannelMode, mBluetoothSelectA2dpLdacPlaybackQuality .

For some unknown reason, somehow the following chain of calls is executed
before onCreate() :
  onCreateView() ->
  getActivity().registerReceiver(mBluetoothA2dpReceiver, ...) ->
  mBluetoothA2dpReceiver.onReceive(...) ->
  updateBluetoothA2dpConfigurationValues()

Adding the null pointer checks until the issue is understood.

Test: Code compilation
Bug: 35204511
Change-Id: I7c1d81015e66b26544b7b26f36cbc75f11713496

Merge "Remove Bluetooth autoconnect on dock"

Merge "Provide calling package name in cell info APIs"

Remove Bluetooth autoconnect on dock

Test: compiles
Change-Id: I31305c70269c685893132f14975c32a051280afc
Fix: 35219649

Provide calling package name in cell info APIs

Provide calling package name in cell info APIs

BUG=31041780

Merged-In: Iad6e5c67ead2bf50d83d1ccad48c3f0341f53712
Change-Id: Iad6e5c67ead2bf50d83d1ccad48c3f0341f53712

Merge "Catch IllegalStateException when updating Bluetooth A2DP codec info"

Catch IllegalStateException when updating Bluetooth A2DP codec info

Add an explicit check when calling getResources() inside
updateBluetoothA2dpConfigurationValues().
This fixes a potential race condition - the latter could be called
in background when processing intent update.

Bug: 35021206
Test: Manual UI testing
Change-Id: I38117c5b1e08f4b2ecfd7637bb3e34920aa0d296

Merge "Add missing null pointer check for Bluetooth A2DP Codec Settings"

Add missing null pointer check for Bluetooth A2DP Codec Settings

Bug: 34885602
Test: Code compilation
Change-Id: I13443390e7c6ffa34b5957a588fc0cefdf5b80f5

Merge "Bluetooth: remove unused SINK_STATE_CHANGED action"

Merge "Update the A2DP Codec Config API"

Update the A2DP Codec Config API

Previously, the JNI upcall would contain only the current codec config.
In the new API, the upcall contains:
 1. The current codec config
 2. The list of codecs containing the local codecs capabilities
 3. The list of codecs containing the selectable codecs capabilities.
    This list is the intersection of the local codecs capabilities
    and the capabilities of the paired device.

Also, refactored the Java internals to accomodate the extra information:
 * Added new class BluetoothCodecStatus that contains the extra info:
   current codec config, local codecs capabilities and selectable
   codecs capabilities
 * Renamed method getCodecConfig() to getCodecStatus() and return the
   corresponding BluetoothCodecStatus object.
 * Updates to class BluetoothCodecConfig:
   new methods isValid(), getCodecName(), and updated toString()
   so it is more user friendly
 * Removed BluetoothCodecConfig.EXTRA_CODEC_CONFIG and
   EXTRA_PREVIOUS_CODEC_CONFIG.
   The former is superseded by BluetoothCodecStatus.EXTRA_CODEC_STATUS;
   the latter is not really used.

Test: A2DP streaming with headsets and switching the codecs

Change-Id: I490a70c82b686be7105862aeaeafcff495369dae

Bluetooth: remove unused SINK_STATE_CHANGED action

android.bluetooth.a2dp.action.SINK_STATE_CHANGED is unused.

Test: compiles
Bug: 34395439
Change-Id: I5c347886fdf9dbdf15feb82f2a9622dd22f010b6

Merge "Bluetooth: remove BluetoothDiscoveryReceiver"

Bluetooth: remove BluetoothDiscoveryReceiver

Use new function getDiscoveryEndMillis() to replace use of shared preferences
and intents.

Test: scan recently, then connect from external BT device
Bug: 34395439
Change-Id: Ia498b3bb4868b656cbbb9fbdc607a49af948343b

Merge "Integration of the AAC codec for A2DP source"

Integration of the AAC codec for A2DP source

Test: A2DP streaming to AAC headsets
Bug: 30958229
Change-Id: Icef9ce9fb8e41cad65bc57bf078513d0c7bf7b9a

Merge "Development Options: toggle binderized hals"

Development Options: toggle binderized hals

This allows for quick performance testing and also for a better
dogfooding experience.

Bug: 34256441
Test: Verification with getprop.
Change-Id: I81c5e0b1044d4569a6fe55a4b30f7e21c4465fc2

Merge "Make headset icons consistent before / after pairing"

Make headset icons consistent before / after pairing

Previously A2DP would be shown before pairing with a HFP device.

bug: 30677445
Change-Id: I0e9ab17197e6ee099d890d5368def36a14ca8344

Merge "Clear VPN after Clear credentials if VPN is being established"

Merge "Prepare for removal of legacy-test from default targets"

Prepare for removal of legacy-test from default targets

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

Bug: 30188076
Test: make checkbuild
Merged-In: I5d2c0685b74ea5ba368c777d9646b9d3ac24dca9
Change-Id: I980111b4b83d16ccd9d3a277ab2587bf638a2c8a

Merge "Update/cleanup Bluetooth Codec strings in Developer Settings"

Update/cleanup Bluetooth Codec strings in Developer Settings

* Use the summary strings instead of the titles strings as appropriate.
* Use proper translatable strings instead of hard-coded "Streaming: "
  embedded string.

Test: UI visual observation
Change-Id: I0222afc30c75e68e2568c33c781755802d86c502

Clear VPN after Clear credentials if VPN is being established

1. There is no way to disconnect VPN after 'Clear credentials'
without removing whole Internet connection because VPN profile to
disconnect is removed when Clear credentials.

2. This commit checks whether VPN is being established or not when
Clear credentials and disconnect VPN if VPN is being established.
Lastly, this shows a toast message to inform VPN disconnected.

Test: manual - took a photo

Signed-off-by : Sungmin Lee <insight.lee@lge.com>
Bug: 29093779

Change-Id: Id5ea01c8731b3b0fca2a31d9d84e8c103952b377

Merge "Moved the Bluetooth-related default config init within onCreate()"

Moved the Bluetooth-related default config init within onCreate()

This should avoid potential crashes in case somehow onCreateView() is
called without onCreate() being called first.

Test: Manual UI interaction with Developer Settings
Bug: 34192829
Change-Id: Iaf46d05f30d20f98727c4dbfb91feef8509c92f7

Merge "Some Wi-Fi Menus like WPS should be disabled when Wi-Fi is off"

Some Wi-Fi Menus like WPS should be disabled when Wi-Fi is off

When Wi-Fi is off, Advanced menus below should be disabled:
1. "Wi-Fi Direct"
2. "WPS Push Button"
3. "WPS Pin Entry"
in Settings->Wi-Fi->Advanced.

Bug:31102679
Test: Test with the device

Change-Id: I5af4c18512c991211eb42ef759e00917828f1b4c

Merge "Add Bluetooth Codec selection UI to the Developer Options"

Add Bluetooth Codec selection UI to the Developer Options

Test: A2DP streaming to headsets and manual UI interaction
Bug: 30958229
Change-Id: I75027200561e6c51fa767c427ed09b647757d0ab

Merge "Update Android Beam feature summary string"

Update Android Beam feature summary string

Bug: 32183480
Test: compiles
Change-Id: I750e998ea73b10645666c7d080afe7320b705d47
Signed-off-by: Ruchi Kandoi <kandoiruchi@google.com>

Merge "Handle SubSettings instance leaks in Index.java"

Handle SubSettings instance leaks in Index.java

Index's constructor will hold context of caller.
context.getApplicationContext() instead of context to prevent activity
leaks.

Bug: https://code.google.com/p/android/issues/detail?id=223322

Test: manual-start a new activity call Index constructor ,then pressed
back.

Change-Id: Ic12af62f9b718d328610a57985f64ca4629bab99
Signed-off-by: gaochong <gaochong@xiaomi.com>

Merge "Update VPN preference state after disallow configuring VPN policy has changed."

Update VPN preference state after disallow configuring VPN policy has changed.

1. When Settings > More and change disallow configuring VPN policy
   and resume Settings, update VPN preference state.

2. When Settings > More > VPN and change disallow configuring VPN policy
   and resume VPN Settings, update its state.

Test: manual

Signed-off-by : Sungmin Lee <insight.lee@lge.com>

Change-Id: I4f3c85733ca6ba05cba46e2f4854f54a42b10c21

Merge "Always include BAND_MODE=AUTOMATIC in the Radio Band Mode List"

Always include BAND_MODE=AUTOMATIC in the Radio Band Mode List

To facilitate user recovery from issues where the band mode
list is incomplete and may not report the full list of bands,
the user should *always* have the ability to set the band mode
selection to AUTOMATIC.

Bug: 28124606
Test: compilation, UI check
Change-Id: Ib961d9b058ade5a51dfec0777971c8c98dee2033

Merge "New pref toggle for network recommendations."

New pref toggle for network recommendations.

Adding a new toggle to control whether network
recommendations are enabled.

Removed the "Use open networks" pref and its associated strings
and components as it's no longer needed.

Test: Built and ran.
BUG: 33549591
BUG: 33270391
Change-Id: I03069ca30d06089600ca958f8da117564189f839
Merged-In: I7248ac32e0598d82107c150372c1fe4e9e2243a5
Merged-In: I82eed6dc7daef1c622ea4e33ce42c331e0cd786d

Merge "Fix crash when rotating HighPowerDetail dialog"

Merge "DO NOT MERGE Add Bluetooth toggle prompts - settings"

DO NOT MERGE Add Bluetooth toggle prompts - settings

If permission review is enabled toggling bluetoth on or off
results in a user prompt to collect consent. This applies
only to legacy apps, i.e. ones that don't support runtime
permissions as they target SDK 22.

bug:28715749
Change-Id: I0b9065446f9d0cfb819065eaf2dc160eb1c1f7ec

Fix crash when rotating HighPowerDetail dialog

When rotating the device, the Activity of the target
Fragment become null. So NullPointerException occurs.
The dialog should check if the Activity is null
before calling onActivityResult().

Bug: 28333723
Test: manual - go to Settings > Apps > Clock,
               click Battery optimization and rotate device

Change-Id: I7f68d6ece7d169fd96d04c1788e55ee506aae1ae

Merge "Fix apn can't be updated when pressing back key"

Import translations. DO NOT MERGE

Auto-generated-cl: translation import

Bug: 32177112
Change-Id: Ib08dd2e3c6b87cc0fbdc16bb167f4e167e52b020

Merge "Refactor NetworkScoreAppManager for testing."
am: 512368a22e

Change-Id: I597c960d5e56014b23b280e1e73e324efa77f8fb

Merge "Refactor NetworkScoreAppManager for testing."

Merge "Link WIFI_SAVED_NETWORK_SETTINGS action with activity."
am: 666c9d7ea5

Change-Id: I1b193660453cbc492d83501f8e9fe3b544e6e4b2

Merge "Link WIFI_SAVED_NETWORK_SETTINGS action with activity."

Refactor NetworkScoreAppManager for testing.

Test: manually running app
Change-Id: Icd828886b7181529fa2bf414ba631bce6d1b4d44

Link WIFI_SAVED_NETWORK_SETTINGS action with activity.

Bug: 32738882
Test: gts test
Change-Id: Ie92e55fd1b0fbb7ba90b3f378a888b098dcf3287

Import translations. DO NOT MERGE

Auto-generated-cl: translation import

Bug: 32177112
Change-Id: Id8af490796464cb8422fa4be023af34baee73002