Merge "Fix formatting issue on Headset State Machine"
am: 61345026a7

Change-Id: I7058f53b20e94b75042ae1127e0a3b6f099d50b9

Merge "Fix formatting issue on Headset State Machine"

Fix formatting issue on Headset State Machine

Test: build, no user visible effect
Merged-In: I482661d95ac6e30a6f55afa9970627e889907125
Change-Id: I482661d95ac6e30a6f55afa9970627e889907125

Merge "Revert "jni: introduce ScopedBtAddr""
am: e4406bcc36

Change-Id: Ie136e2b870e7892bbfe8c62323c95f0578dca1e5

Merge "Revert "jni: introduce ScopedBtAddr""

Revert "jni: introduce ScopedBtAddr"

Causes crash on Fugu. Reverting for now until further investigation.

This reverts commit 2b233628767c0bf6ff21ae2acdeada8bea94e583.

Change-Id: I21daf6b75312695d7ccb83ae603aaeffc3c9e770

Merge "LE Maximum Advertising Data Length (2/4)"
am: 4cee47de11

Change-Id: I2a26d6eb8a9b8372dc2a171e798aec5f122794b0

Merge "LE Maximum Advertising Data Length (2/4)"

Merge "jni: introduce ScopedBtAddr"
am: 9b94cc9fe2

Change-Id: I53de9a0f7d17118088206ddd18aa706aa44de7fc

Merge "jni: introduce ScopedBtAddr"

LE Maximum Advertising Data Length (2/4)

Add ability to check maximum advertising data length.

Bug: 30622771
Test: manual
Change-Id: Ia912f9d738ec8dbf56205a1c22a21f687d9cb6cd

Merge "Bluetooth 5 AdvertisingSet implementation (2/4)"
am: 92b2f24308

Change-Id: I5d070edb82d2b1d51339bb68e9dc1fdb46e035e0

Merge "Bluetooth 5 AdvertisingSet implementation (2/4)"

jni: introduce ScopedBtAddr

ScopedBtAddr handles marshalling bt_bdaddr_t into jbyteArrays so we
don't have to.  It also complains when it can't allocate things and
deletes the local reference when it goes out of scope.

Test: compiles, sanity tests with various profiles

Change-Id: I78a6364cc5cac0e8664d2d04388a6e2ac4c7a58f

Bluetooth 5 AdvertisingSet implementation (2/4)

This patch wires up fist methods of AdvertisingSet, making it possible
to start advertising and stop advertising. It also replaces legacy
implemementation with calls to new implementation.

Bug: 30622771
Test: sl4a ConcurrentBleAdvertisingTest
Change-Id: I35e7db5df1945d101402299f78dd541e346d548c

Merge "AdapterService: Do not call System.exit() on onUnbind"
am: 5cc1112d65

Change-Id: I6893a7be8a7676092efdb5bf49921616aff041f8

Merge "AdapterService: Do not call System.exit() on onUnbind"

AdapterService: Do not call System.exit() on onUnbind

* Process termination should be handled by the OS
* System.exit() in onUnbind also prevents onDestroy() to be called
* Removed isMock() since its only use case is to check if System.exit()
  should be called

Bug: 35366908
Test: make, run java test, turning Bluetooth ON/OFF
Change-Id: Ieba927ce86d5dd27269f10188c74308fa76e66d9

Merge "Broadcast ACTION_PAIRING_REQUEST to everyone again"
am: 71da1cf06b

Change-Id: I57f8a7673d28fcbe1e60d51e192564bf13189682

Merge "Broadcast ACTION_PAIRING_REQUEST to everyone again"

Broadcast ACTION_PAIRING_REQUEST to everyone again

ACTION_PAIRING_REQUEST to be sent to all packages while new methods for
handling automated pairing are developed.

Test: AAP pairing && SL4A connection tests
Bug: 36024655
Bug: 35853357
Change-Id: Ib07adf5253c1e051ab2abc6e2ed711d8bbf107c9

Merge "Fix advertise data length parsing"
am: 91dca83714

Change-Id: I8328e4b789189d7f3320418553d9a5f5957e35c2

Merge "Fix advertise data length parsing"

Fix advertise data length parsing

Bluetooth 5 advertise data can be longer than 128bytes. The length bit
must be properly converted to unsigned value for such long data.

Bug: 36086416
Test: manual
Change-Id: I7449ea99be2e78dd60dd90ec607b74f2bb02d213

Merge "Initialize the Media Player list at start"
am: af66394a61

Change-Id: I50bf13adf13e46737b6bc5ecc4bef1d5b01b446c

Merge "Initialize the Media Player list at start"

Initialize the Media Player list at start

If the browsable list builder fails to receive any connect callbacks when
connecting to media services, we could be left in a state where the media
player list is uninitialized. This is fixed by always initializing the
media player list at the start of AVRCP.

Change-Id: I15088c1f4e8ffa2ac8a101339c9b47989fc17a5e
Fixes: 36151636
Test: Manually tested connecting AVRCP with AVIC-8100NEX

Merge "Bluetooth 5 feature check implementation (3/3)"
am: af78536ab5

Change-Id: Ie3d08bc680f1e3a93fda7cf90d0d693f379104fb

Merge "Bluetooth 5 feature check implementation (3/3)"

Bluetooth 5 feature check implementation (3/3)

Wire the new feature check API to actual values received from the
controller.

Test: manual
Bug: 30622771
Change-Id: I0812ed3ed8e28b8be34090ffb09c27d7a6405b8b

Merge "PBAP PropertySelector"
am: e0f2564001

Change-Id: Ic85da3fffa537ec1651ea517ff7c8a0c41470009

Merge "PBAP PropertySelector"

PBAP PropertySelector

Fill out and transmit the PropertySelector when requesting to download
the phonebook.  Although the PropertySelector field is an optional field
and the default behavior is well specified when it is absent, some
phones do not adhere to the specification.  By sending the
PropertySelector field we can improve interoperability and eliminate
unneeded data.

Bug: 35322439
Change-Id: I25bcfce8bf4c7648c7cc07387bf02ebef600cf44
Test: Verification of filter bits set in wireshark and successfull
transfer of contact images, names, and numbers.

Merge "Use fixed size scan response data"
am: cc295bbfee

Change-Id: I83c212284fe05204ed25bf02c2a2a73221487c71

Merge "Use fixed size scan response data"

Use fixed size scan response data

We used to return fixed size scan response  data before commit ed3417a6,
and some applications rely on this behaviour. For now make sure that all
apps get fixed data.

Bug: 36003118
Test: manual
Change-Id: Ieff1a933c01e7fcb0ae36f740048ccba2b3481b9

Merge "Bluetooth 5 Advertising API"
am: dcffb67e16

Change-Id: I28998dc9ff77349eb219d0511ca8af797fd10c4f

Merge "Bluetooth 5 PHY change API"
am: d3e9d9f8b1

Change-Id: Ia9e2375ba0138e78077f4b4b7f9fac6ef8b65e9f

Merge "Bluetooth 5 periodc scan API (2/2)"
am: d84aac6a34

Change-Id: I322c05d22c1cbfeff33f9fa7238b001071c58269

Merge "Propagate Bluetooth 5 scan result properties"
am: 30241a4d72

Change-Id: I4fbbec83552071bd756b8aabd43d3e62b94874bd

Merge "Bluetooth 5 feature check API (2/2)"
am: b52e56f18b

Change-Id: I481dc74b86858e185798e65a48393470fc4d1496

Merge "Bluetooth 5 Advertising API"

Bluetooth 5 Advertising API

Bug: 30622771
Test: manual
Change-Id: I3a670756e26345833ce595c9b22b8b39ae0ba41f

Merge "Bluetooth 5 PHY change API"

Bluetooth 5 PHY change API

Bug: 30622771
Test: manual
Change-Id: I859d23e7db95585551675d09c0b009222a475dd2

Merge "Bluetooth 5 periodc scan API (2/2)"

Bluetooth 5 periodc scan API (2/2)

Bug: 30622771
Test: manual
Change-Id: I2bde2072f55949b195b64e0ce333a40b4f6d6673

Merge "Propagate Bluetooth 5 scan result properties"

Propagate Bluetooth 5 scan result properties

Bug: 30622771
Test: manual tests
Change-Id: Icbb353aa3435a97035c81384fc9031175a934239

Merge "Bluetooth 5 feature check API (2/2)"

Bluetooth 5 feature check API (2/2)

Bug: 30622771
Test: manual
Change-Id: I90e2efe989745c07c2f2fb8f4ea5bc3b718382f6

Merge "OPP: Add OPP 1.2 feature."
am: 3361ec466c

Change-Id: I861f5aecb8721baa8cc7a8ffd4e215f0d9de1ddd

Merge "OPP: Add OPP 1.2 feature."

OPP: Add OPP 1.2 feature.

Add changes to support OPP 1.2 which uses OBEX over L2CAP.

Test: Connect with Remote OPP Client supporting OPP 1.2 and verify
that connection and transfer happens over L2CAP. Connect with Remote
OPP Client supporting OPP 1.1 and verify that connection and transfer
happens over RFCOMM.

Bug: 33010988
Change-Id: I44c2f2f01fb04f4306d0eb121d66dc08954966c0

Merge "HID: Access JNI callback object when valid."
am: 9257decc86

Change-Id: I9ef4dbbe1e8c534e62a89d009f804511fc46d12c

Merge "HID: Access JNI callback object when valid."

HID: Access JNI callback object when valid.

Usecase:
Perform BT off/on with HID connection in stress loop overnight.
Expectation:
BT should not crash, and should work normally after stress test.
Observation:
BT crash observed due to dereference of JNI object whch was NULL.
Coredump
=========================
pid: 4629, tid: 4711, name: BT Service Call  >>> com.android.bluetooth <<<
signal 6 (SIGABRT), code -6 (SI_TKILL), fault addr --------
Abort message: 'art/runtime/java_vm_ext.cc:470]
JNI DETECTED ERROR IN APPLICATION: obj == null'
=====================
bt_btif : btif_hh_upstreams_evt: BTA_HH_OPN_EVT: handle=255, status =7
bt_btif : in, bd addr:00:1f:20:ea:ca:8b, prop type:4, len:4
bt_btif_dm: get_cod remote_cod = 0x00002540
I bt_btif : HAL bt_hh_callbacks->connection_state_cb
JNI DETECTED ERROR IN APPLICATION: obj == null
    in call to CallVoidMethodV
"BT Service Callback Thread" prio=5 tid=12 Runnable
  | group="main" sCount=0 dsCount=0 obj=0x12ca8b80 self=0xe7a84400
  | sysTid=4711 nice=0 cgrp=default sched=0/0 handle=0xd8f88920
  | state=R schedstat=( 75150605353 59660157373 74582 ) utm=3592
  stm=3923 core=4 HZ=100
  | stack=0xd8e8c000-0xd8e8e000 stackSize=1014KB
  | held mutexes= "mutator lock"(shared held)
  native: #00 pc 00377955  /system/lib/libart.so (_ZN3art15Dump
  NativeStackERNSt3
  __113basic_ostreamIcNS0_11char_traitsIcEEEEiP12
  BacktraceMapPKcPNS_9ArtMethodEPv+128)
  native: #01 pc 0035795d  /system/lib/libart.so (_ZNK3art6Thread9
  DumpStackERNSt3
  __113basic_ostreamIcNS1_11char_traitsIcEEEEbP12BacktraceMap+304)
  native: #02 pc 00250d79  /system/lib/libart.so (_ZN3art9JavaVMExt8
  JniAbortEPKcS2_+812)
  native: #03 pc 0025103d  /system/lib/libart.so (_ZN3art9JavaVMExt9
  JniAbortFEPKcS2_z+72)
  native: #04 pc 00284127  /system/lib/libart.so (_ZN3art3JNI15Call
  VoidMethodVEP7_
  JNIEnvP8_jobjectP10_jmethodIDSt9__va_list+478)
  native: #05 pc 00005f81  /system/lib/libbluetooth_jni.so (???)
  native: #06 pc 00010d27  /system/lib/libbluetooth_jni.so (???)
  native: #07 pc 000657bf  /system/lib/hw/bluetooth.default.so (???)
=======================

Root cause:
For quick BT turn on/off cases there may be instances where callback from btif
is triggered even before callback object is created or cleanup path may clear
callback object just before stray btif callback call tries to access the same.

Fix:
Check if callback object is valid before trying to access the same

Test: Crash not observed in stress test with hid connection and BT on/off.

Bug: 35023377
Change-Id: Id9b526bfec2bf74d7110601f85e111088a1e7d76

Merge "Exception and death notification handling for ISap."
am: f2f1ecd69e

Change-Id: I83c55cc4d32d812415b164f89bffdbe58bb562ea

Merge "Exception and death notification handling for ISap."

Merge "jni: cleanup compiler warnings"
am: dc77303218

Change-Id: I03a6a68b3d4fd131d089f837b072f4b94c21bd07

Merge "jni: cleanup compiler warnings"

jni: cleanup compiler warnings

Test: compiles without warnings
Change-Id: I6f0b13648a624047aee0ac728e7a6c27e7bfad33
Fix: 35636708

Merge "HID: Send connection state change broadcast during cleanup"
am: 71f903cb3d

Change-Id: I35985711ac5708e021826a18a77ea3048d7bf829

Merge "HID: Send connection state change broadcast during cleanup"

Merge "Use base::Callbacks in BleScannerInterface (3/3)"
am: 00c0c74d0c

Change-Id: Idfe4df7bb59fa5d16ae3c55253fe0f25b0cbcec8

Merge "Use base::Callbacks in BleScannerInterface (3/3)"

HID: Send connection state change broadcast during cleanup

Usecase:
1) Connect HID device with DUT
2) Turn Off BT

Observation:
HID state change is not received by application leading to
application showing connected even when BT is turned off and
HID was connected.

Rootcause:
HidService does not send state change to application if BT is turned off
while connected

Fix:
Send broadcast for HID connection state change during cleanup.

Test: Verified that connection state change intent is received by upper layers
during BT off if HID was connected.

Bug: 35013637
Change-Id: I5d7989887f9089876c78e43d97d066f384cc237e

Use base::Callbacks in BleScannerInterface (3/3)

The remaining four native callbacks need more internal refactoring
before they can be turned into a callbacks.

Bug: 30622771
Test: sl4a FilteringTest
Change-Id: Ibe3cd66af9598a7d0294d36550bf35ab8ddfda43

Merge "PBAP: Fix Functionality failure for PBAP"
am: fe641c1e38

Change-Id: Ide835f7a3eb1dab26b7c9adff0a0b1710c9cff22

Merge "PBAP: Fix Functionality failure for PBAP"

Merge "OPP: Fix Functionality failure for OPP Rx"
am: 2cc49b962d

Change-Id: I6c139e1ca1e94dc8eeaaaf95912f56aed35474dc

Merge "OPP: Fix Functionality failure for OPP Rx"

OPP: Fix Functionality failure for OPP Rx

Usecase:
Send file from Remote BT device to DUT

Expectation:
File should be received properly

Observation:
File reception popup does not occur on DUT

Root Cause:
Socket listener not started for OPP

Fix:
Add changes to properly start listeners for OPP

Test:
1) Verified that SDP records for OPP are exposed and can be browsed
  by remote OPP Client.
2) File can be received without issues from OPP Client.

Bug: 35416619
Change-Id: I5b4f343f3fd7b957ce423ce2cb76cd0683e10c3a

PBAP: Fix Functionality failure for PBAP

Usecase:
Connect to DUT from PBAP Client (Carkit)

Expectation:
Carkit is able to browse PBAP SDP and connect to PBAP Profile
with DUT

Observation:
PBAP Connection is successful

Root Cause:
Socket listener not started for PBAP

Fix:
Add changes to properly start listeners for PBAP

Test:
1) Verified that SDP records for PBAP are exposed and can be browsed
  by remote PBAP Client.
2) Connection happens fine and phonebook can eb downloaded properly.

Bug: 35416619
Change-Id: I52acb1c10b91dd5ffecc44badcfd60f0e27b0a55

Merge "Add Loopers to tests that require them for message handlers" am: 6a231c7afd am: 5dd715ccd6
am: 78047e26a9

Change-Id: I3f79e4ae2cc4f9596ece598ab488b1e3b73f8367

Merge "Add Loopers to tests that require them for message handlers" am: 6a231c7afd
am: 5dd715ccd6

Change-Id: I3f3a8699e2bc3cce6b9c73036c7eda7671baea75

Merge "Add Loopers to tests that require them for message handlers"
am: 6a231c7afd

Change-Id: I4e91d1c47ff62a61849aba0c9585ec9f094bbdee

Merge "Add Loopers to tests that require them for message handlers"

Add Loopers to tests that require them for message handlers

Bug: 34193424
Test: runtest bluetooth no longer fails
Change-Id: I4612cfb9c16b53293f52350e7f0abe9fb70c7543

Merge "Broadcast BOND_STATE_CHANGED to foreground." am: 6d3498fba2 am: 0864a05422
am: 90e812968b

Change-Id: I0125e259acb6b52180135fc8ddc1ca017ab4b92c

Merge "Broadcast BOND_STATE_CHANGED to foreground." am: 6d3498fba2
am: 0864a05422

Change-Id: Ia9999e7976b1b931637af472178df20ccf9ad489

Merge "Broadcast BOND_STATE_CHANGED to foreground."
am: 6d3498fba2

Change-Id: I4142caa728a8beb763d12d706836b8d841723401

Merge "Broadcast BOND_STATE_CHANGED to foreground."

Broadcast BOND_STATE_CHANGED to foreground.

Limiting BOND_STATE_CHANGED to settings app meant accessories apps can't
determine when a device completes pairing.

Test: pairing tests with various devices
Bug: 35234069
Bug: 35325574
Change-Id: If9fffb3410ec051b96e65b867571e93ae56ca706

Merge "Prevent getting Session Token if media browser isn't connected" am: cda194ce36 am: 5dd802d8d7
am: a0c275a817

Change-Id: I264408ec7a02231fa835f3d97865b3969410e8a6

Merge "Prevent getting Session Token if media browser isn't connected" am: cda194ce36
am: 5dd802d8d7

Change-Id: I2872bfbd6e7dcc64aa9a22629c4ef2125a1a968a

Merge "Prevent getting Session Token if media browser isn't connected"
am: cda194ce36

Change-Id: I30897364c0f001ece9d078e0760dcd6d66043e1a

Merge "Prevent getting Session Token if media browser isn't connected"

Prevent getting Session Token if media browser isn't connected

Bug: 35348574
Test: Sanity test browsing with VW Carkit
Change-Id: I795a19f4bd0a95d36f64bc767fdeda108c20f5ed

Merge "Services: start PBAP and OPP like other services" am: 14280ec46c am: 6686a52270
am: 4701d38aa1

Change-Id: I1b7a37e7741546441e07ea68c689082593c2e66f

Merge "Services: start PBAP and OPP like other services" am: 14280ec46c
am: 6686a52270

Change-Id: If279435e68c4f8bd65487ab3cbd7f1a73a94bc21

Merge "Services: start PBAP and OPP like other services"
am: 14280ec46c

Change-Id: I92c12c786ec4c5ce00dff9d26dcac23ce9a6a2f0

Merge "Services: start PBAP and OPP like other services"

Services: start PBAP and OPP like other services

PBAP and OPP are currently normal services that are started and stopped
by the system.  Have AdapterService start them just like all of the
other profile services instead.

Bug: 34395439
Fix: 35216901
Test: connect to a carkit that does PBAP
Test: use beam to receive a file
Change-Id: I84e769886f6aa4e698bd07fb9900f124111b50b3

Merge "Remove use of Contacts.IN_VISIBLE_GROUP" am: c8a531ab45 am: 2a5a9bee6e
am: 29621d90da

Change-Id: I00e01b54ba4c71c838764bf1a0622395213e98ff

Merge "Remove use of Contacts.IN_VISIBLE_GROUP" am: c8a531ab45
am: 2a5a9bee6e

Change-Id: Ia59221960abf72d4fb231b9b44c36773ec740081

Merge "Remove use of Contacts.IN_VISIBLE_GROUP"
am: c8a531ab45

Change-Id: I366027fcbefd385220e19af65196ed7ed4d174e4

Merge "Remove use of Contacts.IN_VISIBLE_GROUP"

Merge "Use resources for pairing ui package" am: 5c72d9a85a am: aa6795b9c9
am: 13cf7aa3d9

Change-Id: Idd042abdc9168ca5ce4c7c8967d761f687253bb2

Exception and death notification handling for ISap.

Test: Basic telephony sanity
Bug: 32020264
Change-Id: I3d710fc59a397fe0054b286d422066f9ff149262