Replace NOT_SUPPORTED string constants with empty strings.

AVRCP was defining default state strings as NOT_SUPPORTED and hence the
UI would show NOT_SUPPORTED as the string for album/song info when no
player is selected on the phone. Replacing these strings with empty
helps UI make a more informed decision.

Bug: b/28120051

Change-Id: Ia8017dce784160d637a6653f70addb5258e7871e
(cherry picked from commit a525ff9f10b2ee0cee3269ab6082ddf09441199d)

Merge "A2dpSink sends CONNECTED instead of DISCONNECTED when connection timeout"

A2dpSink sends CONNECTED instead of DISCONNECTED when connection
timeout

When a timeout happens for an outgoing A2DP Sink connection we are (most
likely a typo) sending a CONNECTED instead of DISCONNECTED event. This
confuses the UI and shows it as connected instead of disconnected.

Change-Id: I2fc5a9df86781b45e5c5ff9368a7c49c6c910294
(cherry picked from commit 688344c5ed34758c1e96a1dec9212e10107a7263)

Merge "Separate LE scanner from GATT client (2/4)"

Separate LE scanner from GATT client (2/4)

Right now, LE scanning functionality is combined with the GATT client.
This is the source of various bugs, like scans suddenly stoppinging when
a GATT client is killed. It also increases memory consumption, because
we associate many structures with a GATT client, which are not necessary
when just scanning.

Test: sl4a BleScanApiTest ConcurrentBleScanTest
Bug: 30622771
Change-Id: Ie4a20abe7fa112bf501619651e288ae46d28e209

Merge "Define cover art attribute id"

Define cover art attribute id

Test: Tested manually
Bug: 32081388
Change-Id: Ib9df511a36d19f1bd5197836ad34b19edba73748

Merge "Add AVRCP unit tests and switch to using mockable interfaces"

Add AVRCP unit tests and switch to using mockable interfaces

Add a mockable interface for MediaController in order to add unit
tests for AddressedMediaPlayer.

Bug: 32416674
Test: Tested manually
Change-Id: I081c37915fc15e58f3de37c3a16e03747503860c

Merge "Restore GATT unit test"

Merge "Change how Advertise Data is passed to BTIF (3/3)"

Merge "Remove all old Bluetooth tests"

Restore GATT unit test

Bug: 32414937
Test: Test compiles and runs
Change-Id: I4d2a575a49da64f878c2d1d110e0d4fb5931c06a

Remove all old Bluetooth tests

 - Remove all old Bluetooth tests that do not compile
 - Change Android.mk to compile the tests as "tests" instead of "optional"

Bug: 32402820
Test: Tests now compile
Change-Id: Iadafae095a8ba9602a632ed55bd0208fcf64e4a8

Change how Advertise Data is passed to BTIF (3/3)

The way that AD data is passed right now put some additional
limitations, i.e. only one manufacturer specific data can be set, or
only one service UUID. By moving AD generation to upper layers, aother
set of limitations is removed.

Test: Covered by BleAdvertiseApiTest sl4a test
Bug: 30622771
Change-Id: I5444455a89ee085dcccfdc98846955374d58c374

Merge "jni: Use RAII to manage callback environment"

jni: Use RAII to manage callback environment

CallbackEnv is a new RAII object which:
 - Retrieves the callback environment when instantiated
 - Clears exceptions when it goes out of scope
 - Centralizes checks that we are in the correct environment

It can be used the same as a pointer to JNIEnv with the -> operator and
the JNIEnv pointer can be retrieved with get();

Test: basic functionality tests
Change-Id: I004f2655e2e02847782fec794faa62da85c85d49

Merge "OPP: Fix "0%" progress even after file transfer finished"

Merge "AVRCP 1.6 CT implementation"

AVRCP 1.6 CT implementation

Bug: b/28791287
Change-Id: Ie4cb76b301bb3f36a750658fb86caa0def1eb503
(cherry picked from commit 4b491c2c874395c436949183bcbd84ebb2493131)

Merge "Remove unnecessary error message"

OPP: Fix "0%" progress even after file transfer finished

When the file is small enough to be sent together with header in one
packet, PrivateOutputStream#write() never calls back ClientOperation,
because OBEX Client has no more data to send.
In this case, response code in ClientOperation is not updated from
default value (-1) until the PrivateOutputStream is closed.
It causes progress bar not updated from 0%.

The patch adds PrivateOutputStream#close() call after the first write
operation when file and header are sent in one packet, and ensures
progress bar to be updated properly.

Bug: 31895670
Test: manual

Change-Id: I735a872b756ab1c8892c2f8bbf2dce51ae24fc5a

Remove unnecessary error message

Test: compiled successfully
Change-Id: I85591142ccd6ac4d53e7bf62100ad4ae5ef473e2

Merge "Fix error where root folder depth was set as 1 instead of 0"

Merge "Reduce AdapterState log spam on isTurning*() functions"

Reduce AdapterState log spam on isTurning*() functions

Bug: 31857542
Test: manual
Change-Id: Ia3658509af727714820a38c9f94398d714813046

Fix error where root folder depth was set as 1 instead of 0

Test: Tested by hand
Bug: 32082139
Change-Id: Id46c3dab1e21d166691620cb762d64893ce5d41d

Merge "Prevent Null Pointer Exception caused by race condition in AVRCP init"

Prevent Null Pointer Exception caused by race condition in AVRCP init

The object |mMPList| could be null if AVRCP started handling messages before
the phone was unlocked. Now we always initialize the Media Player List.

Test: Manual testing
Bug: 32060313
Change-Id: I66cba51159586da8abe51297dba94a3e1dc1b067

Merge "Convert advertising HAL from struct into class (2/3)"

Convert advertising HAL from struct into class (2/3)

Bug: 30622771
Test: all related tests were updated
Change-Id: Ib32b7ef9483b14dd92a9555856d1b62b85e87c0a

Merge "style: Remove commented-out code"

style: Remove commented-out code

Test: continues to compile
Change-Id: I94f3e869621bcffd1aba678894ac14fc3800184f

Merge "Make AVRCP encryption aware"

Make AVRCP encryption aware

AVRCP can't access the different media players while the user hasn't
unlocked the phone or if Bluetooth is turned on before the phone
has booted all the way.

Bug: 31966072
Change-Id: Id022e2a8068257885ddb06c5be3d4b04c026ac53

Merge "Ensure LE scan clients are removed when the related app dies"

Ensure LE scan clients are removed when the related app dies

Use the passed in ScanClient in handleStopScan() to ensure that the
client appDied flag is in the correct state to ensure proper removal of
the client application reference.

Bug: 30681588
Test: manual
Change-Id: I0e0067a8e21ebd5f89870a7054b1f4092fe63ae1
(cherry picked from commit 5185d1989c93faa3ce4b4410ef49a0205f43b239)

Merge "Fix connecting to profiles when bonding from local device (2/3)"

Fix connecting to profiles when bonding from local device (2/3)

When bond is created from Bluetooth Settings, profiles gets properly
auto connected when services are discovered. If pairing is done from any
other app, i.e. NFC app, the bond is not being recognized as initiated
from local device, and profiles are not connected.

This patch makes sure that if bonding is initiated, no matter from which
app, it will cause proper profiles to connect.

Bug: 30211618
Change-Id: Icda038447554f429b9cbfc74d68c679e63572bdb

Merge "Offset browsing media entry UID by one"

Merge "Re-enable browsing support for AVRCP"

Re-enable browsing support for AVRCP

Bug: 31941615
Change-Id: Ia6846b3d6133c42863ae3cc630c980bea3e6bf20

Offset browsing media entry UID by one

Offset AVRCP UID by one for browsing since UID 0 is reserved for
getElementAttributes.

Bug: 31962824
Change-Id: I7efcd0546ebb0d02b4b1bfb05038a86d9fa0835f

Merge "Cleanup BluetoothPbapReceiver log messages"

Cleanup BluetoothPbapReceiver log messages

Bug: 31442739
Bug: 31457305
Test: manual - checked logcat output for (lack of) excessive logs
Change-Id: I76f32dd950afd0a2949f38bfa6c0d4212b83357c

Merge "Don't try to stop advertising on death of GATT clients"

Don't try to stop advertising on death of GATT clients

We should no longer stop advertising for GATT clients. Advertisers
are kept in AdvertiseClient, and their death is handled in
AdvertiserDeathRecipient.

Bug: 30622771
Test: Kill app with GATT client registered
Change-Id: I21f246042d1632824ed3d2d8937f13bcafe380ab

Merge "Enable debug logs for Bluetooth AdapterService"

Merge "Enable debug logs by default in BluetoothPbapReceiver"

Merge "Do not downgrade filtered scans to opportunistic"

Enable debug logs for Bluetooth AdapterService

Enabled log messages that are useful to debug the AdapterService state.
Remaining log messages are controlled by AdapterService.VERBOSE flag
and are disabled by default.

Bug: 31442739
Test: manual, and "frameworks/base/core/tests/bluetoothtests" unit tests
Change-Id: I7d81fd605f64e64f29aee37314821384234b286d

Enable debug logs by default in BluetoothPbapReceiver

Enabled more logs to help identify issues when Bluetooth
is started because of PBAP.

Bug: 31442739
Bug: 31457305
Test: manual - checked logcat output for (lack of) excessive logs
Change-Id: I6348fe57a8b3546a8152416a5db95b8b2226e166

Do not downgrade filtered scans to opportunistic

Fixes: 31308591
Test: manual
Change-Id: I0d8bcffef9951c03607f7f3b25b79557062a9bec
(cherry picked from commit 8d6abb0a65bd4e9f4726a9c565b0c74fb6aed1a1)

Merge "Update deprecated Notification builder to new API"

Update deprecated Notification builder to new API

* Replace Notification.setLastEventInfo() and various variable
  assignments with the corresponding Notification.Builder API calls.
* Explicitly choose theme for getColor() using Context.getTheme() as
  previous getColor() method is deprecated as well.

Bug: 31218737
Test: TestTracker/63162/3975
Change-Id: I696cb7f07b50b04551e56c3b4a02fea87f23589c

Merge "Bluetooth: split setting params and enabling  advertising (1/3)"

Bluetooth: split setting params and enabling  advertising (1/3)

Proper order of commands when starting advertising:
* set parameters
* set data
* enable advertising

This cannot be achieved when setting advertising parameters and enabling
advertising is put together in one function. Enabling before setting
data might cause first advertisements to be empty. If a device with
hardware filter pick up such advertisement, it might fail to properly
recognize our device.

Bug: 30622771
Bug: 19372779

Change-Id: I66c71f1b2c07c832eda0983ead816741798e923b

Merge "AVRCP 1.6: media browsing support on Target(3/3)"

AVRCP 1.6: media browsing support on Target(3/3)

Description:-
This patch provides Media browsing support for AVRCP using
new Media Browsing APIs provided from Android 5.0

Features:-
1) setAddressedPlayer
- Change the control to the specified player
2) setBrowsedPlayer
- Change the player with which fileSystem is being browsed
Browsing commands:
3) getFolderItems scope=MediaPlayers/VFS/NowPlaying
4) changePath
5) getItemAttributes
6) playItem
7) getTotalNumberOfItems

Handling notifications:-
1) ADDRESSED_PLAYER_CHANGED
2) AVAILABLE_PLAYERS_CHANGED
3) NOW_PLAYING_CONTENT_CHANGED

Added support for dual RC

New files added:-
AvrcpHelperClasses.java
- Helper classes used for callback/response of browsing commands.
AvrcpConstants.java
- Group all the constants used in Avrcp.
AddressedMediaPlayer.java
- Interface to communicate with media controller for NowPlayingItems.
BrowseMediaPlayer.java
- Interface to communicate with MediaPlayer for browsing FileSystem.
AvrcpMediaRspInterface.java
- Interface to communicate with the native layer.

Verification:-
AVRCP 1.5/1.6 functionality has been verified with
google MediaBrowserService sample app.

Bug: 19361366
Merged-In: I0ab7f0c7d87c06fe4f454151d20494c56aceae12
Change-Id: I40b9e7aae81d994a709559844928b0749c603ea8

Merge "Mercedes Benz MAP message listing workaround"

Mercedes Benz MAP message listing workaround

Use case:
Sync MAP message to Benz NTG 4.5

Precondition:
1. One of SMS message contains any special characters not ASCII

Steps:
1. Connect with Benz NTG 4.5.
2. Sync SMS message from carkit.

Failure:
Carkit always shows "No Message".

Root Cause:
When this carkit requests the message listing, and any message subject has
non-ASCII special characters in it, the carkit will stop all MAP activity
and stay idle.

Fix:
Strip special characters in the subject tline in message listing
for this carkit.

This change also introduces a Java bases interop database for future
use/expansion.

Bug: 29025011
Change-Id: I4255cbeb068c82f32a68b1022285dfa723e199ec
(cherry picked from commit 4786e5fffff14d92b795084b4470b785de66dfd0)

Merge "Fix ConcurrentModificationException in GattService.onScanResult"

Fix ConcurrentModificationException in GattService.onScanResult

If a scan client is added or removed while iterating over
mRegularScanClients, a concurrent modification exception will be thrown,
because we use a regular hash map.

Use a ConcurrentHashMap instead to fix this issue.

Bug: 31122137
Change-Id: Icbd427ebbe63adc6b5bd4ee950ef3a874ce6067b

Merge "Change MAP to send one new message event upon storage unlock"

Merge "ContextMap: Remove connection references when an app is removed"

Change MAP to send one new message event upon storage unlock

The MAP service will now send one new message event for all messages
when device storage is unlocked instead of sending an event per message.

Bug: 30809925
Change-Id: I1c9e10125c9a754bf4f6c25718fc0422d79cf37e
(cherry picked from commit 8b02ffef2611c8f21d751e5a196bcb765a9e997e)
(cherry picked from commit ac3a80eed68e2374151abb59e66d95bb2e66cf61)

Merge "Separate advertiser from GATT client (2/4)"

ContextMap: Remove connection references when an app is removed

This fixes an issue which could cause a stale Bluetooth address to be
used. The issue would manifest itself when an app is added with a
connection ID, then the app is removed and app is re-added, causing
the original connection ID (which could be stale) to be used.

Bug: 30765855
Change-Id: Idacbbfb07d895bf9276fdd986100e832a54d81b0
(cherry picked from commit 660208ad2ee2ea394adab0d1c1cea9b068bb70c6)

Separate advertiser from GATT client (2/4)

Bug: 30622771
Change-Id: Icacf1fa6a526d67ffadb8acc056d2acfefc5e895

Merge "Don't use SMS database until after the phone is unlocked"

Don't use SMS database until after the phone is unlocked

When the device is encrypted and a passcode is set,
the SMS database isn't available and the MAP service
crashes when trying to access it. The service will now
respond with OBEX_SERVICE_UNAVAILABLE until the phone
is unlocked. At this point, the SMS database will
notify the content observer that all the messages are
available, and send corresponding events.

Bug: 29990836
Change-Id: I53f12d0e114212d94f16b56df7c7dbe73f6254df
(cherry picked from commit db8d8ae565b3db6a5e3187170dcb7b281a79f9da)
Merged-In: I53f12d0e114212d94f16b56df7c7dbe73f6254df

Merge "Set the Adapter Service onCreate()"

Set the Adapter Service onCreate()

We should call setAdapterService(this) inside onCreate() instead of
inside BleOnProcessStart(). The former is more robust.

Change-Id: Ide1d98dfdde148645f3efe246e6fbf95659195a7
(cherry picked from commit 39d7556b76036e02b87a6c4288a9a090e2bd6686)

Merge "Allocate CallbacksObj Before Initializing A2DP JNI Interface"
am: b1b9ae5d4b

Change-Id: I2f4cb3851fd7941bb4192aa6eded60403d67201d

Merge "Allocate CallbacksObj Before Initializing A2DP JNI Interface"

Allocate CallbacksObj Before Initializing A2DP JNI Interface

Bug: 31016704
Change-Id: If3dd4fdbc92e1420bc60910c39771aa549ac5dab

Merge changes from topic 'ble-scan-accounting'
am: 725d2dac7b  -s ours

Change-Id: Ic992092816435f0b1b24c762abd06722e32b7c58

Merge changes from topic 'ble-scan-accounting'

* changes:
  Reset ble scans in BatteryStats on start up
  Use the incoming WorkSource to notify batterystats of BLE scans

Merge "Fix a deadlock in service discovery callbacks"
am: 6fcd0081eb

Change-Id: I6e343bbc3ef5c54e372687ee989e1baa133a01c8

Merge "Fix a deadlock in service discovery callbacks"

Reset ble scans in BatteryStats on start up

Bug:22718669
Change-Id: Id31431bf35db3a52c8f1af1f210247d7ea20f6ed

Use the incoming WorkSource to notify batterystats of BLE scans

BatteryStats is going to start tracking bluetooth scanning, so
we start by passing along the WorkSource we were given (or we created
if none was specified).

This is only done for BLE scanning at the moment, and only for regular scans.
Batched scans will also need to be considered.

Bug:22718669
Change-Id: I15dd7c912320552323aaf9d713b4bb3224ffffec

Fix a deadlock in service discovery callbacks

onSearchCompleted calling gattClientGetGattDbNative directly, might
cause deadlock if the jni_workqueue is full.

Bug: 30835367
Change-Id: I05de735aa189dd2d250f2c5816d38c2ddcabb864

Merge "Fix NPE in permissionCheck"
am: 64ed2357e3

Change-Id: Ief4258ed339822801b97c9a85309608d59e98286

Merge "Fix NPE in permissionCheck"

Fix NPE in permissionCheck

permissionCheck can throw NPE when notification is received before
GATT database is discovered. Assume that permission is granted when
receiving notifications for unknown characteristic.

Bug: 30853729
Change-Id: I02aab1197b036584b2021d76f290c2c57db24a19

Merge "Add LE Secure Connection data parsing (2/4)"
am: 29101f7fd4

Change-Id: I962b635c44d2573a62936ab58bf8577ef19000cc

Merge "Add LE Secure Connection data parsing (2/4)"

Add LE Secure Connection data parsing (2/4)

Bug: 30460956
Change-Id: Ie5c3ad96e7a834c9dc0a6791cf2cd24ecaeb927e

Merge "Prevent LE scanning apps from being wrongflully downgraded to opportunistic"
am: c9db1189d8

Change-Id: I216d43f760661274145b7140efb68e5c01528073

Merge "Prevent LE scanning apps from being wrongflully downgraded to opportunistic"

Prevent LE scanning apps from being wrongflully downgraded to opportunistic

Bug: 30648104
Change-Id: I9d749a20eefdbeb126fa25a425d636205a22b2ef
(cherry picked from commit a1c7315ed05df04599e7cdc5f4bcf7d3ec8d3392)

Merge "Change URI accessed to properly update read status"
am: f337ee1fe0

Change-Id: Iad683df58a9067e275bc07c0701cce2dfb94fd83

Merge "Protect against SecurityException generated by query"
am: 369f6662fa

Change-Id: I9e379db4c8d54a7733bba873da2cb6abc73493e6

Merge "Change URI accessed to properly update read status"

Change URI accessed to properly update read status

Bug: 28970262
Change-Id: I07d9ab24d10396baa4ddbfbdfad5b1eec1b8e9d5

Merge "Protect against SecurityException generated by query"

Protect against SecurityException generated by query

Bug: 30444661
Change-Id: I1d93645f1d4475cf603cda8bb28df0b960ccb445

Merge "HFP 1.7 profile update (4/4)"
am: 1b9d2081e0

Change-Id: Ib18e452187c7d673e8ae387da99bf9e6faf43bda

Merge "HFP 1.7 profile update (4/4)"