HealthService: Fix not adding channel to channel list

Channel is not added to channel list on success of connect channel
request. Upon connection status (connecting) callback it searches
for channel id but id does not exists and treating it as incoming
connection.

Change-Id: Id1b2bcbbf9b5eaf958d391055dd0ffeaf9aa0a64

PBAP: Add strict phonebook name checking

This change adds strict phone book name checking
for every Get request for getting phonebook.

This is affecting TC_PSE_PBD_BI_01_C qualification PTS test.

This patch is based on ae6e8004f263f1f4c7e2da64d3aed93188956242
and 3a3453527ced6aef659151fa9453d8ab7e3f3a68 from CodeAurora but
was adjusted to fit AOSP.

Change-Id: I6adc09f6890300057ab26b391c87b59025833b1c

GATT: Fix parameters order for gattClientGetIncludedServiceNative

inclSrvcInstId and inclSrvcType were in reverse order.

Change-Id: I28c1f464344af2893c1a63f72cae8d49eb7bb990

HFP: Fix missing OK for AT+CLCC in virtual call

When AT+CLCC is received from HF while we're in virtual call mode there
is only non-zero index call sent without final zero index call. This
makes AT command response incomplete since no OK is sent.

From HFP specification, section 4.31.1:
"The AG shall always send OK response to HF."

Change-Id: I2d759bd3d65f2adb9a364d40ffe23d6197c6375b

MAP: Fix reply code for UpdateInbox function

From MAP 1.1 specification, section 5.9:
"If MSE does not allow the polling of its mailbox it shall answer
with a "Not Implemented" error response."

This was affecting TC_MSE_MMB_BV_16_I qualification test case.

Change-Id: I19c2a0627826f27150b44f2f98e56c7663e1e7f9

DO NOT MERGE: Add status callback for start/stop advertising.
cherrypick from master to fix b/13289050

Change-Id: I666969442b756ce461ff64eff0d09f5283431fb3

Merge "DO NOT MERGE release wakelock if no activity in Bluetooth MAP profile to avoid waste power." into klp-dev

Merge "DO NOT MERGE Add Bluetooth privileged permission to bluetooth app. fix b/13226130." into klp-dev

DO NOT MERGE release wakelock if no activity in Bluetooth MAP profile to avoid waste power.

bug:11138488
Change-Id: Ie7d8903cc917aee51711fa47f275863e4046f3d8

DO NOT MERGE MAP: fixed missing filtering on Priority

FilterPriority was ignored in the code as SMS does not have priority.
This fixes the support for the priority filter for sms + mms.
NOTICE: sms messages does not have priority and hence will return
an empty messagelisting if the client requests a list with priority
filter set til high-pri only.

Bug: 11168636
Change-Id: If028e47850323d20d1e9a23bc947782ae8da1df0

DO NOT MERGE update play status based on A2dp audio state from stack.

this is to work around the problem:some video/audio player didn't
update the play status by calling setPlaybackState.
The headset(HBS730) register the play status update on the phone.
It will depend the play status reported from the phone to send
play or pause command.

bug:11325212
Change-Id: I7fde5960edd4a5b3474fa17456aacd5b4b432d4d

DO NOT MERGE Add Bluetooth privileged permission to bluetooth app. fix b/13226130.

Cherry picking ble advertising changes from master to KLP MR2.
b/13137996

Change-Id: Iba48f4757a6b54911277900b875d83eaa81488b7

Merge "DO NOT MERGE BLE peripheral mode (2/4): Gatt Service Change." into klp-dev

DO NOT MERGE BLE peripheral mode (2/4): Gatt Service Change.

Add advertising data to Gatt Service/

Cherry picking ble advertising changes from master to KLP MR2.
b/13137996

Change-Id: I0ea7df8c159ad388d8e58768c49e785e9a4694bb

Device cannot go to sleep while connected to HBM-570 BT headset

The headset never disconnect the Obex connection for PBAP,
so in this case, we never release the wakelock.
Our PBAP implementation won't release wakelock
until the Obex is disconnected by the remote device
To better control the wake lock, we should
only acquire the wakelock when we receive the reqeust from the remote device
will release the wakelock after we send the response packets.

bug:10957699
Change-Id: I85e0609e36788d9c36334dd804ec52c90b90615b

am e9428c35: am 7677468e: am 884362fe: am ada9e950: am 727741cd: Merge "Use pathPrefix filter instead of path for /btopp content" into jb-mr1-dev

* commit 'e9428c35344af989f9e643964a81e2dc0ce1ffe1':
  Use pathPrefix filter instead of path for /btopp content

am 7677468e: am 884362fe: am ada9e950: am 727741cd: Merge "Use pathPrefix filter instead of path for /btopp content" into jb-mr1-dev

* commit '7677468ea5a2a71780e8bcf69a97c939b2ebc39b':
  Use pathPrefix filter instead of path for /btopp content

am 884362fe: am ada9e950: am 727741cd: Merge "Use pathPrefix filter instead of path for /btopp content" into jb-mr1-dev

* commit '884362fee9df9f2c61d61bc0926033de82e58f14':
  Use pathPrefix filter instead of path for /btopp content

am ada9e950: am 727741cd: Merge "Use pathPrefix filter instead of path for /btopp content" into jb-mr1-dev

* commit 'ada9e95000f65d4ad7e5c697d48c76e6b9e54cf0':
  Use pathPrefix filter instead of path for /btopp content

am 727741cd: Merge "Use pathPrefix filter instead of path for /btopp content" into jb-mr1-dev

* commit '727741cd610b35e675e7b7769ddf431e5763344d':
  Use pathPrefix filter instead of path for /btopp content

Merge "Use pathPrefix filter instead of path for /btopp content" into jb-mr1-dev

am c4d45b2f: am e6643ed4: am 12533801: am 8e13dea5: am fdcf4028: am a162ec5d: am deb7bdad: Use pathPrefix filter instead of path for /btopp content

* commit 'c4d45b2f898d77b61f8614e22645bfb8ecd00930':
  Use pathPrefix filter instead of path for /btopp content

am e6643ed4: am 12533801: am 8e13dea5: am fdcf4028: am a162ec5d: am deb7bdad: Use pathPrefix filter instead of path for /btopp content

* commit 'e6643ed4cbe6d5ebe4f1a4979a78f81f6aeff1f0':
  Use pathPrefix filter instead of path for /btopp content

am 12533801: am 8e13dea5: am fdcf4028: am a162ec5d: am deb7bdad: Use pathPrefix filter instead of path for /btopp content

* commit '125338015bd96550c2d15d87a506640499820885':
  Use pathPrefix filter instead of path for /btopp content

am 8e13dea5: am fdcf4028: am a162ec5d: am deb7bdad: Use pathPrefix filter instead of path for /btopp content

* commit '8e13dea5f919737738db6e38c56cc83f43a7d555':
  Use pathPrefix filter instead of path for /btopp content

am fdcf4028: am a162ec5d: am deb7bdad: Use pathPrefix filter instead of path for /btopp content

* commit 'fdcf402834c3f108a085a9c9b2f3be4848174f3b':
  Use pathPrefix filter instead of path for /btopp content

am a162ec5d: am deb7bdad: Use pathPrefix filter instead of path for /btopp content

* commit 'a162ec5d684e4bf3e14ef1ddbd4a910c82915b93':
  Use pathPrefix filter instead of path for /btopp content

am bf0aa27e: am 14b74c3c: Use pathPrefix filter instead of path for /btopp content

* commit 'bf0aa27ec7e106deb4fe61a10deebffcfe68b4bd':
  Use pathPrefix filter instead of path for /btopp content

am 14b74c3c: Use pathPrefix filter instead of path for /btopp content

* commit '14b74c3c20e06cb69b4b624c3ebb9b72bb14090d':
  Use pathPrefix filter instead of path for /btopp content

am deb7bdad: Use pathPrefix filter instead of path for /btopp content

* commit 'deb7bdad8cbe0501d414fdec698387714837f927':
  Use pathPrefix filter instead of path for /btopp content

Merge "Use pathPrefix filter instead of path for /btopp content" into klp-dev

Import translations. DO NOT MERGE

Change-Id: I8452ddf1d6c278a8c0e8de1f501f000d5bcc8525
Auto-generated-cl: translation import

Use pathPrefix filter instead of path for /btopp content

path is doing a full path match. It did not guard /btopp/
also fix a minor problem in tests/AndroidManifest.xml
bug 11148516

Change-Id: I8f4a614f999e643e8ae122baf0bf898b2b3008d5
Conflicts:
tests/AndroidManifest.xml

Use pathPrefix filter instead of path for /btopp content

path is doing a full path match. It did not guard /btopp/
also fix a minor problem in tests/AndroidManifest.xml
bug 11148516

Change-Id: I8f4a614f999e643e8ae122baf0bf898b2b3008d5
Conflicts:
tests/AndroidManifest.xml

Use pathPrefix filter instead of path for /btopp content

path is doing a full path match. It did not guard /btopp/
also fix a minor problem in tests/AndroidManifest.xml
bug 11148516

Change-Id: I8f4a614f999e643e8ae122baf0bf898b2b3008d5
Conflicts:
tests/AndroidManifest.xml

Use pathPrefix filter instead of path for /btopp content

path is doing a full path match. It did not guard /btopp/
also fix a minor problem in tests/AndroidManifest.xml
bug 11148516

(cherry picked from commit 2819c1dbef39a712ab8adfcd36d2cb01f2379259)

Change-Id: I00b23e5c3a98422c2dd1cc220f2c121578ff5c8f

Merge "remove a read lock to work around a platform deadlock problem." into klp-dev

remove a read lock to work around a platform deadlock problem.

remove the lock in getStatefunction to work around a platform
deadlock problem.and also for read access, it is safe to remove
the lock to save CPU power

bug:11655066
Change-Id: Iddc0ada04010d95ae7d5fbc93ab837ebfc025b7a

Merge "LE: Fix stack crash on invalid descriptor write request" into klp-dev

LE: Fix stack crash on invalid descriptor write request

Fix stack crash when an application attempts to writes a NULL array
using the writeDescriptor() function.
bug 11574019
Change-Id: Id2d7e49216fcc6e3af369047005fb7e67e0923db

Merge "cancel device discovery before start pairing process" into klp-dev

cancel device discovery before start pairing process

During pairing process, SDP search is needed which cannot be handled
if we are doing device discovery
bug 11294130

Change-Id: Ic025fa3000af4f2d4c2c2ac411d26b6a99ad097c

Merge "FF/REW is not correctly for AVRCP." into klp-dev

Handles AT+BVRA=1 received during an open SCO session.

Bug: 11072505
Change-Id: If665517cd9efa087154edd57b23163e7cc098f6f

Import translations. DO NOT MERGE

Change-Id: I5ca5b6507989bc70ccc1cb5790e9a5b8b45caeb4
Auto-generated-cl: translation import

Merge "NPE in bluetooth MAP cause bluetooth process crash" into klp-dev

FF/REW is not correctly for AVRCP.

While a music is being played, if FF/REW is pressed on the connected device,
it remains in the FF/REW state constantly (on the Porsche / Audi A8 carkit)

bug:11377223
Change-Id: Iee5468308f2786839c3a047c95dfa0a49ef28efb

NPE in bluetooth MAP cause bluetooth process crash

bug:11448104
Change-Id: I30045162c89e2150a9ef476f33e1b322e7cc9a44

Import translations. DO NOT MERGE

Change-Id: I75b1aeb7c88fa9a3c678f91a2d88a95fa63f80df
Auto-generated-cl: translation import

MAP: fix problem with MNS not disconnecting correctly

MNS is needed to disconnect with the MSE but did not.
This patch fixes that by re-adding a disconnect() that was wrongfully removed during review.
Patch set 2:
-----------
 - Split disconnect and shutdown for MNS connection
bug: 11161674
Change-Id: Idd09fc774584dfdb171b0d00be0f7711ae57b853

Merge "Change pairing api/intent permission from PRIVILEGED to ADMIN" into klp-dev

Change pairing api/intent permission from PRIVILEGED to ADMIN

The APIs are createBond, setPin, setPairingConfirmation
The intent is ACTION_PAIRING_REQUEST
bug 11101076

Change-Id: I3b065ab0348a964bb557e57527491b2f34e68a81

Bluetooth Application Crash while try to pairing with laptop - IKDVXPSE-345

the handleRegistration will never run after we shutdown the handler thread first,
with this change, we can make sure the mObserver will never be null
when call handleRegistration.
bug:11148404

Change-Id: I93b351915b49cf59a7d2026b7d83a68a7e76dd38

merge in klp-release (no-op)

merge in klp-release history after reset to klp-dev

Fix for MAP/PBAP: Race-condition when requesting authorization from Settings app

When MAP and PBAP requests authorization from the Settings app via Intents,
it also sets a state isWaitingForAuthorization to indicate
that it should handle incoming replies.
This is however done in the code after the intent have been sent
and in some cases the reply for the intent is returned
before the state has been update = no handling of the reply.
This only happens on the second connect with a paired device and
only when the user have selected the "do not ask again" checkmark
in the popup requesting access.

FIX:
Move the update of isWaitingAuthorization to before sending the intent to settings

bug: 11179645
Change-Id: I25b019e32a01caa8509c8f97195712120823afa9

Allows another GMS core app to handle ACTION_PAIRING_REQUEST .

Now we use sendOrderedBroadcast for ACTION_PAIRING_REQUEST , and to
receive this intent, BLUETOOTH_PRIVILEGED is required.

Bug: 11087521
Change-Id: I6aa2fbc6538df39b24f141e7d75adc3658a7e7ac

merge in klp-release history after reset to klp-dev

Merge "Remove MMS support from MAP profile" into klp-dev

Remove MMS support from MAP profile

reject to push MMS message, will support it in the future
bug 10296324

Change-Id: Idf2aa081b049e323422223d7b24f6404de2d7184

merge in KQS81M

merge in klp-release history after reset to klp-dev

Merge "Update AVRCP implementation of IRemoteControlDisplay" into klp-dev

merge in klp-release history after reset to klp-dev

Import translations. DO NOT MERGE

Change-Id: I4aa104f811eb821da26ed448b93ae70bbb87712a
Auto-generated-cl: translation import

Update AVRCP implementation of IRemoteControlDisplay

Add implementation of new method in IRemoteControlDisplay.

Bug 8209392

Change-Id: Ic700be5476cb2f70b6a26098cbed3e8e130b3573

merge in klp-release history after reset to klp-dev

Merge "Change setPin, setPin, setPairingConfirmation permission level to BLUETOOTH_PRIVILEGED" into klp-dev

merge in klp-release history after reset to klp-dev

Change setPin, setPin, setPairingConfirmation permission level to BLUETOOTH_PRIVILEGED

bug 9987787

Change-Id: I2fe88f4c284d8403d79e08194bf7a720b305d029

Merge "Unable to receive Text Message to CAR Kit(nuvi3580LM)" into klp-dev

Unable to receive Text Message to CAR Kit(nuvi3580LM)

The problem is when we receive the SetNotificationRegistration,
the MNS thread is not ready yet.so the notification is not enabled.
so we won't send new message notification to Carkit.

bug:10898884

Change-Id: Ibaa4a92694bcbb1ea2fb4e8a3de0ff30acdafeb1

merge in klp-release history after reset to klp-dev

Merge "bluetooth crash caused by read phone book with invalid colummn number -1" into klp-dev

Import translations. DO NOT MERGE

Change-Id: Icd8a04f9f9c4d9564fa2dda13be067ce6cc868c8
Auto-generated-cl: translation import

bluetooth crash caused by read phone book with invalid colummn number -1

bug:10919257
Change-Id: I584d5c5c65e26cacfab227742b98c49282710f68

merge in klp-release history after reset to klp-dev

Merge "resource leakage in bluetooth MAP when empty message in phone." into klp-dev

Add MEDIA_CONTENT_CONTROL permission

Required to register IRemoteControlDisplay implementation

b/8209392
Change-Id: Ide88e9a11bcfa68e7d5e2095208e63afdb5c511e

Merge "Check null condition of number before contact query" into klp-dev

Check null condition of number before contact query

bug 10571304

Change-Id: Idb23e7085dc1dc7119e0e2aa8901743707e72285

Add MEDIA_CONTENT_CONTROL permission

Required to register IRemoteControlDisplay implementation

b/8209392
Change-Id: Ide88e9a11bcfa68e7d5e2095208e63afdb5c511e

resource leakage in bluetooth MAP when empty message in phone.

bug:10844044
Change-Id: I60c1b0206d7ba4653d149bee2f9f0dd6da6a5afb

Change Volume step scale. Use setStreamVolume instead of avrcpUpdateVolume

bug 10681804

Change-Id: I553dac18d48c86bdb8b11428f56956f04ea87f27

Import translations. DO NOT MERGE

Change-Id: I585f487a522776ee1c529309d359452233abed15
Auto-generated-cl: translation import

Merge "Fix build." into klp-dev

Fix build.

Change-Id: I02a9d4dfa84c5778a1136d39b7bb14a2e523a9ca

disable broken tests

tests are causing build breakage

Bug: 10692365

Change-Id: Ieb4b8c6946cf2de4230f1320d940c89de53b100c

Merge "Fixes to the google review comments + spec 1.1" into klp-dev

Fixes to the google review comments + spec 1.1

- updated code to comply with MAP spec 1.1
- removed activity + strings.xml
- removed unused notification code
- fixed TODOs
- added more string validation and case insensitivity
- fixed internal+google review comments
- Added dump of incoming bMessages to /sdcard/bluetooth/log when verbose debug in enabled. Only the latest received message will be stored
- Fix functions msgListingSize and msgListingHasUnread to also consider mms filter message type
- Fix wrong tag length define for notification status parameter
- Re-added shutdown code to interupt the MNS
- removed map activity
- Added initial bluetooth map unit tests
- Fix map event report xml start tag to uppercase 'MAP'
- added support for using ProfileService class
- changed the way the Broadcast Receiver was implemented
- Fixed minor bugs found during Automotive Test Event
  - FilterPeriod application parameters can be present, but with zero length
  - For MMS the end-boundary were added too early
  - The FOLDER entry in bMessage can be empty for a message push
  - Wrong error value returned for a set status operation with a wrong handle
- In getMessage() exclude all binary content and smil.xml if the appParam attachment is set to "no".
- Set correct content id and content location for mms. Fix mms mime parser bug.
- moved disconnect to Handler thread
- fixed multipart-message split bug.
- added a few Unit tests for multi-part messages
- MMS parser optimized
- fixed exception in MNS obex Client
- fixed problem with Native PDUs not getting correct timestamp
- corrected mixup in ordinator/recipient for MMS

Change-Id: I3875762822a7f8ce0132065e0da5d0257e3850a1
Bug:10692365

Remove dependency on JNIHelp header side effects.

Bug: 10680559
Change-Id: If5c6e22dd89a41886bb72303a85ca674c60d1946

Stop scan when client binder dies

bug 10213756
Change-Id: I01b397070f729df7c6000c2dc2613efbe8801681
(cherry picked from commit 45ba247afa1cb3c61a871b0c13f6546411afe68b)

Import translations. DO NOT MERGE

Change-Id: I134988fe16520246474f92a6f7b035e30d0b6afb
Auto-generated-cl: translation import

Merge "Enables AT+ANDROID command in Bluetooth HFP." into klp-dev

Import translations. DO NOT MERGE

Change-Id: I0cccc9b44a66829a7f781986c6aee97c8744d17e
Auto-generated-cl: translation import

Enables AT+ANDROID command in Bluetooth HFP.

Adds another vendor-specific command AT+ANDROID in Bluetooth Hands-Free
Profile. And creates an API function that sends an unsolicited result
code +ANDROID .

Bug: 8736189
Change-Id: I7e5bba8a568f596565a1531948061e4d2fe456e0

Import translations. DO NOT MERGE

Change-Id: Ie4e94f0caf464133c4fb2e3c993d066ceeb8edb3
Auto-generated-cl: translation import

Adding support for Absolute Volume

Change-Id: Ie2ccaad7aaf56a89fe44b168026df3d84b373c06

Conflicts:
jni/com_android_bluetooth_avrcp.cpp
src/com/android/bluetooth/a2dp/Avrcp.java

Merge "Import translations. DO NOT MERGE" into klp-dev

Add support for ff/rew

Change-Id: I7ec60d94313b9ba5e4e8e62c82f19ae332b6fdd3
(cherry picked from commit 55123eff985f4d15ec198569a5db895ea086447a)