A2DP: Cleanup UIPC when no active device
am: f88552cdd8

Change-Id: If29bd610e0d1d9c63e1337363b33e8f1160b2ccb

A2DP: Cleanup UIPC when no active device

* Modified UIPC to support multiple users
* A2DP now calls UIPC_Close() when it has no active device, and calls
  UIPC_Init() when it has an active device

Bug: 72701090
Test: Carkits with A2DP
Change-Id: Ic1b4b1be2aa01c9896883e3cb2a668d7a43316f9

Merge "Add AVRCP Browse packets to the completed packets."
am: 37a143f49a

Change-Id: Idcfb9df419b035f7f4a7fe335b4a97bab00db29a

Merge "Add AVRCP Browse packets to the completed packets."

Merge "btif: Remove unused synchronization in event_cleanup_stack"
am: 001f5a00b2

Change-Id: I05c6256acc7d5da4cede7bd20fcc0b8e4ff11837

Merge "btif: Remove unused synchronization in event_cleanup_stack"

Merge "IBluetoothHearingAid.aidl"
am: 03fa65fc71

Change-Id: Id8bacf4d75ba1b27f1f496bda7095bb75bcb5192

Merge "IBluetoothHearingAid.aidl"

Merge "Adjust buffer length to fit in sdp_copy_raw_data()"
am: 29cb0b048b

Change-Id: Ia41425df56765dd9562f7ba65720751fd705dfc8

Add AVRCP Browse packets to the completed packets.

Also add class representations for the following packet types:
  AVRCP Browse Packet
    AVRCP Change Path
    AVRCP Get Folder Items
    AVRCP Get Item Attributes
    AVRCP Set Browsed Player
    AVRCP Get Total Number of Items
    AVRCP Play Item

Bug: 68854188
Test: run host native test net_test_packets
Change-Id: I0bc682b11ec9181cfe1ed92678bb869ff3ef5740

Merge "Adjust buffer length to fit in sdp_copy_raw_data()"

Factor out libchrome_support_defaults
am: 65d24fb451

Change-Id: I392d30ac3ee7fb85e4121600b351971da6888508

btif: Remove unused synchronization in event_cleanup_stack

Use Case: Repeated BT ON/OFF

Failure: FD leak is observed with ON/OFF stress test
which eventually leads to crash due to unavailability
of FDs.

Steps:
BT ON/OFF

Root Cause: During stack cleanup an unused future
structure is created which is not freed, leading to FD
leak.

Fix: Remove the creation of the future structure

Test: BT ON/OFF
Fixes: 36019324
Change-Id: Id5945da87f00eb7ab77a5f217c95cc43f0136e06

IBluetoothHearingAid.aidl

Bug: 69623109
Test: Spec was reviewed, tests will be included with client/server implementation
Change-Id: I8c972ac0e4aa908a14893c957ae07dd2244abbfe

Adjust buffer length to fit in sdp_copy_raw_data()

Align copy length with source buffer offset so that copy length will be
in boundary of the allocated buffer size

Test: Manual tested with DECK speaker.
Fixes: 35864526
Change-Id: If1a49e46e2948346226fbc96d7c2b56244cd60cc

Factor out libchrome_support_defaults

Disable Mac support for host binaries in one place, since
the dependency is based on libchrome support.

Test: build
Change-Id: I6641e18deffdff5205da181ad97e8b2892073a8f

Merge "Remove libbluetooth-system requirement."
am: a45d385323

Change-Id: I7e93d58e493338a6bbd01d68ad5c9aedd05107c9

Merge "Remove libbluetooth-system requirement."

Add more details in native Bluetooth debug logs
am: 273ffd858e

Change-Id: I3a1ab4082b78e319ed71468ce655afffc9123e0e

Make loghex properly count numer length
am: 78b3e7b303

Change-Id: I303112998d7629c8a26a698c548b80bad58157fd

Add more details in native Bluetooth debug logs

As part of debugging a security/pairing issue, these logs are modified
to give better debug information.

Test: compiled
Bug: 71364658
Change-Id: I742cc7dda0afc6350e881c22d03bdc1c69d37d88

Make loghex properly count numer length

Test: manual logs observation
Change-Id: I7ea8d4afa5ea980f11067a8dd920a1735cf28993

Remove libbluetooth-system requirement.

This module doesn't exist and it being missing was
silently ignored.

Bug: 7456955
Test: no longer receive an error for this.
Change-Id: Ib84b592d25e19b2c6b0d20466015553fe57f5bf8

AVRCP: Don't fail in set_volume on disconnected devices
am: a7b668b6bc

Change-Id: I4083a6d60f506e1bc7bf882ad97cc39af7796ea1

AVRCP: Don't fail in set_volume on disconnected devices

Test: Change the volume on a headset with absolute volume
Bug: 72987704
Change-Id: Ia1acd28c51707e7b71e47140aaded0298ba4e8aa

Merge "Build: Remove DCHECK_ALWAYS_ON flag"
am: 51c084e434

Change-Id: Ia3fe3df2c4344d46844df275353668699dc029de

Merge "Build: Remove DCHECK_ALWAYS_ON flag"

Merge "A2DP: Construct BtWorkerThread's message loop on designated thread"
am: 199bf77055

Change-Id: Iada4a606411d0811bc60b3fc2cd63ef54e2bcb16

Merge "A2DP: Construct BtWorkerThread's message loop on designated thread"

Build: Remove DCHECK_ALWAYS_ON flag

* According to libchrome documentation, DCHECK_ALWAYS_ON must be set or
  unset on both our library and libchrome. Flag mis-match may cause
  hard-to-debug segfaults which we experience when using base::Thread

Change-Id: If3740196340b02d2d0bb41e08f0b551d47f49864
Fixes: 73147157
Test: make

A2DP: Construct BtWorkerThread's message loop on designated thread

* Moves BtWorkerThread's message_loop_'s construction to the
  Run() method run by the designated message_loop_thread_ so that
  message_loop_'s construction and execution are on the same thread
* Makes BtWorkerThread::StartUp() blocking until message_loop_
  starts running using std:condition_variable
* Modify BtWorkerThread::DoInThread() to return boolean

Detail:

Before this CL, BtWorkerThread.message_loop_ is created on Java main
thread and run on message_loop_thread_. During clean-up, A2DP shutdown
message_loop_thread_, rendering the posted_from field in current_pending_task_
invalid. However, although the runner thread is killed, the owner
thread, the Java main thread, is still active. Therefore, we we try to
PostTask() from Java main thread after message_loop_thread_ is killed,
we will try to access an invalid Location object. This results in a
SEGV_MAPPER SEGFAULT. Thus, the fix would be creating and running the
message loop on the same thread so that Java main thread won't have an
invalid message loop after the worker thread is killed.

This can be detected earlier through adding "-DDCHECK_ALWAYS_ON" in both
libchrome and libbluetooth and libchrome will validate the thread for
us.

Fixes: 72831931
Test: Test with DCHECK_ALWAYS_ON in both libbluetooth and libchrome
      ProfileServiceTest stress tests

Change-Id: I7b196b8c1113c2758c2b82ddf4d13c46352bb620

Merge "Prevent spurious connect failures callback"
am: 29ff71470e

Change-Id: If8b053eaf419f3f44fcb8877f1e0a49f028d43cf

Merge "Prevent spurious connect failures callback"

btif_rc - make set_volume easier to comprehend
am: 884314c1d9

Change-Id: I71e44ac5ba5bd2762228a87bb6c61c5fb5f824d9

btif_rc - make set_volume easier to comprehend

Test: compilation
Change-Id: I73c01cb5e3a2da1cdb9acc3b7fe3114f4af7373e

Prevent spurious connect failures callback

Send connection failures only if the event's mac address matches the
currently bonding device.

Test: Run SL4A "BleCocTest:test_coc_secured_connection_throughput"
Bug: 71364658
Change-Id: I64243996ae37dd376ed36363a7d749b4b5ab36b2

Merge "Make sure task_runner is valid when we call PostTask on it"
am: c0df1edbf1

Change-Id: Ic38745138958a7e9c5d58a6e81742deb101c5d43

Merge "Make sure task_runner is valid when we call PostTask on it"

Call BTA_GATTS_AddService on correct thread
am: 541862ce72

Change-Id: I40487af3885e01b95fddb9f8260992e5995c11b2

Call BTA_GATTS_AddService on correct thread

Right now BTA_GATTS_AddService is incorrectly called on btif thread.
This might lead to situations where it is executed while bta thread is
already cleaning up, and deleted the control blocks, resulting in crashes.
From now on, execute it on bta thread. This ensures that it is executed
before control blocks are freed.

Test: sl4a GattClientTest
Bug: 73054849
Change-Id: Ie2bf80414bceba2590c21d3825c78fbb58449520

Make sure task_runner is valid when we call PostTask on it

Test: runtest -j32 bluetooth -c com.android.bluetooth.btservice.ProfileServiceTest
Change-Id: I89426f7eb1204af9def0fa927dfeef1d5aec689f

HBS1098 Paring failure after connection with Car-Audio
am: 0c0d2975d7

Change-Id: Iee1c51c7581f2236788eab88f74912ba47ebc3ab

HBS1098 Paring failure after connection with Car-Audio

Use Case: HBS1100 Paring failure after connection with Car-Audio

Steps:
1. Establish connection with Volkswagen CK, and then playback music
2. Try to pair with HBS-1100

Failure: DUT going for temporary pairing with HBS1100 when request received
while remote name is not present

Root Cause: As auth_req is not handled properly in bta_dm_pinname_cback during
RNR, auth_req variable got reset and DUT is going for temporary pairing.

Fix: local and remote auth_req and local and remote IO capabilities stored and
retained in call backs properly.

Test: Manually tested

Fixes: 35735338

Change-Id: I83382aac9fc1197d7fb31feedbd0fe3b35de87e8

Merge changes from topic "coc-test-increase-data-len"
am: b46c3bf557

Change-Id: Ic9340baf7dde702a26e954395acf5e64314d1b5d

LE Read Maximum Data Length during startup
am: 4f0c5b5838

Change-Id: If6d467f483132101f649e4ebdb1d26434b74ca12

Merge changes from topic "coc-test-increase-data-len"

* changes:
  Add function to change LE Tx Data Length
  LE Read Maximum Data Length during startup

Unify logging in btif_sock_l2cap.cc
am: 35ee023655

Change-Id: If90184c2211b7b3ce1bd405611bb83f929b6274a

Add function to change LE Tx Data Length

As part of new SL4A tests for LE CoC to measure data throughput, this
commit adds a function to modify the LE Tx Data Length parameter to its
maximum.

Test: Ran the new ACTS Tests for LE CoC; BtCocTest and BtCoc2ConnTest
Bug: 70683224
Change-Id: I06d3f95a339dcdc310a18bcf17fbca8623f849d6

LE Read Maximum Data Length during startup

Test: verified in sl4a test for LE CoC throughput
Bug: 68359837
Change-Id: Ie424fef50f6f355d2919d9304bd062e4604c3341

Unify logging in btif_sock_l2cap.cc

Test: compilation
Bug: 68359837
Change-Id: I71edd3d04734471b6cd586c323aa6b335f903fae

Remove Mac host libraries build depending on libchrome.
am: 30aa9c9447

Change-Id: I6b5463133e13ff3ac05141cdd0a70227ce3bd8de

Remove Mac host libraries build depending on libchrome.

Libchrome will drop the support for Mac host.
This is the preparation.

Bug: 72618679
Test: Built on Mac.
Change-Id: I37982210a1da74d56c1da3227c5887685cd8b6cb

Merge "Add function to update LE connection parameters"
am: 2fc4777ff1

Change-Id: I323c50150dc691d12ffd80233238490d2d1346c9

Merge "Add function to update LE connection parameters"

Maximize L2CAP socket MTU
am: 622bd222e4

Change-Id: Ia1a74b45b0fdea696c17ff23b39503627aac262a

BTA_JvL2capConnect simplification
am: 3538b64d97

Change-Id: I0ac5d16c8bab410f2fb6d2ca99811ae2d2768e31

Simplify btSock_start_l2cap_server_l
am: e5e2dd556b

Change-Id: I0e6757ebdfb4b85cfaa60285960c70b82f669044

Add function to update LE connection parameters

As part of new SL4A tests for LE CoC to measure data throughput
performance, this commit will add a function to enable the CoC Facade
to modify the LE Connection Parameters especially the Connection
Intervals. Also, add some debug logs.

Test: Ran the new ACTS Tests for LE CoC
Bug: 70683224
Change-Id: If7665cf70a18756a58556c60cb9ce2a587ae691f

Maximize L2CAP socket MTU

Introduce L2CAP_SDU_LENGTH_LE_MAX constant, and use it for LE CoC
sockets, while still keeping the old value for Classic sockets.

This also require change in how we send the rx_mtu up the stack - it
must be read from the socket property.

Additionally in this patch, l2cap_socket.mtu is renamed to tx_mtu to
distinguish it from rx_mtu. Assignment of this variable is also added
when we receive it from the remote.

Bug: 68359837
Test: tranfer file using OPP, do obex, run LE CoC tests. Run tests
between device with this patch, and older android phones.
Change-Id: Iffa20a6c6a93d5787bada7bf28655e9880d8c1a0

BTA_JvL2capConnect simplification

Test: manual
Change-Id: I18ac5ac451490e0d3c0c9fc5daa9f8d64ff19bd5

Simplify btSock_start_l2cap_server_l

Test: run OPP, chekc LE CoC
Change-Id: I4091ea8065ba913966fcd14495b90b2c565fdba3

Merge changes I00560443,I2e39bd31
am: 70a0c9e436

Change-Id: Ib9a6ab8b75066a0b73dc3d1f39312ca0ed3905b9

Merge changes I00560443,I2e39bd31

* changes:
  Stricter check of the MTU size in the BTIF L2CAP socket
  Fix naming error in btif l2cap MPS -> MTU

Merge "Add more checks to LE Connection Parameters"
am: 85807ed9b6

Change-Id: I4ac8135b166abc3d49c0ed4f224b6da79021bba0

Merge "Metrics: Dump native metrics to Java as std:string"
am: a53c929550

Change-Id: Iafc3fc2b7aa484539961210e6bbf094594cf5367

Merge "Minimize amount of Flow Control packets for L2CAP CoC"
am: f262cb9f93

Change-Id: I4a0d546d06a53e98977982a179e4687ab0a5e9aa

Merge "PDU data lengty computation fix"
am: edde455514

Change-Id: I620a686465baca6f0ef65ff211a459076bca1b3d

Merge "Add more checks to LE Connection Parameters"

Merge "Metrics: Dump native metrics to Java as std:string"

Stricter check of the MTU size in the BTIF L2CAP socket

Test: manual
Bug: 68359837
Change-Id: I0056044388f8886f704a9ff2f30db09b3110d143

Fix naming error in btif l2cap MPS -> MTU

BTIF layer uses only MTU, maximum size of SDU, to do computations around
sizes. This error was introduced earlier during refactor.

Test: compilation
Bug: 68359837
Change-Id: I2e39bd31ce712d22f43e3059b6e1c73ef7b0fee7

Add more checks to LE Connection Parameters

This part of code is missing the necessary LE Connection Parameters
checks.

Test: Tested on R.A.T. LE mouse
Bug: 70683224
Change-Id: I966a3dd4fd2fa9dc54109872df3d4c43d34f0156

Merge "Minimize amount of Flow Control packets for L2CAP CoC"

Merge "PDU data lengty computation fix"

Metrics: Dump native metrics to Java as std:string

* Dump metrics to Java layer as std::string instead of going through a
  file descriptor so that Java layer can process metrics data directly
* Add new method dumpMetrics(std::string*)

Bug: 33693818
Test: adb shell dumpsys bluetooth_manager --proto-bin
      SL4A metrics tests: BtMetricsTest, BtFunhausMetricsTest
Change-Id: I67883c1371862d7e4ff33cec01d710d2d8f274c9

PDU data lengty computation fix

PDU data must also account for PDU length and Channel ID fields.
Otherwise each PDU is split into 2 BB packets, decreasing the link
speed.

Test: manual data transimssion test.
Change-Id: I026bf0e32c58eec620b5c55d748ecd81fa3ea67b

Merge "Set pending_discovery when doing GATT over BR/EDR"
am: 023f79e8ce

Change-Id: Ica7a4ed260edc7c8176ab7fccd5965bb2eea55a2

Merge "Set pending_discovery when doing GATT over BR/EDR"

Minimize amount of Flow Control packets for L2CAP CoC

In order to not waste bandwidth on Flow Control packets with credits:
1. Set Initial Credits in the connect request/response to the maximum value
2. Send credits only when the remote has less than 64 of them left,
   increasing the value back to maximum.

Bug: 68359837
Test: observe data flow in btsnoop log
Change-Id: I34e1fecc55422b34df0b1e82bb2b402eb06620bd

BUILD: Fix broken build for test-vendor_test_host
am: bc94521056

Change-Id: I9b4044f7b3faa54740b7cc089c26bb131c46543b

BUILD: Fix broken build for test-vendor_test_host

* Disable darwin build for test-vendor_test_host explicitly as
  it does not use "fluoride_defaults" in its build configuration.

Bug: 72618679
Test: make
Change-Id: Ia81bf705ad248e129df69c14e385ced118c6df58

Merge "BUILD: Disable build for mac-host target"
am: 61f4003e52

Change-Id: I7fbad3751db03d69edeea6a4085cdedc58bb4002

Merge "BUILD: Disable build for mac-host target"

BUILD: Disable build for mac-host target

* Move disable darwin (mac) options to "fluoride_defaults".
* mac-host target is disabled across fluoride, but developer can still
  build for non-mac targets on a mac host.
* Remove vendor_libs/Android.mk that is no longer used.

Bug: 72618679
Test: make
Change-Id: Ibad9fad3df169e9c4e418a4511c42066fcb10fdc

Set pending_discovery when doing GATT over BR/EDR

SDP discovery can return services before LE discovery, or after LE
discovery is already done. In both cases, pending_discover is not a
valid pointer, and must be set before call to bta_gattc_explore_srvc

Test: sl4a GattOverBrEdr
Change-Id: I2b43d2348edfa641c56a8085a92b6367b146030d

Get rid of L2CAP_MAX_BUF_SIZE, use variable-size buffers instead
am: f47bfde2ed

Change-Id: Icb7dc81a1a476705984e33c171d8d1b577bd03e6

Get rid of L2CAP_MAX_BUF_SIZE, use variable-size buffers instead

This is step towards increasing L2CAP_MAX_SDU_LENGTH value.

Test: transfer file using OPP, send data using CoC
Bug: 68359837
Change-Id: Ia4fb6b2325cb92db9c90c6a4d2ab17f271640ea8

Packet: Disable host tests for Mac
am: bf101fa2c5

Change-Id: I7cbd2e2b811cf95e813428029b1c00a440852b08

Packet: Disable host tests for Mac

Bug: 72618679
Test: Build on Mac
Change-Id: I85c1412b86a1fbbab01491a755be9bc2b8241d50

Merge "Drops unittest build on Mac host."
am: 2706fc2cd5

Change-Id: I62ee4f02dd8b3481420582e15f7048597b2271f6

Merge "Drops unittest build on Mac host."

Add Packet class as alternative to BT_HDR
am: 341ab2befa

Change-Id: If25a3b85486b871b99919429d37efc265a0b601c

Add Packet class as alternative to BT_HDR

Adds Packets, Iterators, and Packet Builders to Bluetooth to be used as
an alternative to BT_HDR.

 - Packet is a base class used to pass around data. It is intended to be
   immutable after creation and subclasses will implement all required
   functions and their own accessor methods to retrieve data.

 - Iterator provides a convienent way to traverse packet data.

 - PacketBuilder is used to construct Packets. This class is should be
   the only way to mutate packet objects.

Also add class representations for the following packet types:
  AVRCP (Base Packet)
    AVRCP Vendor Packet
      AVRCP Get Capabilities Packet
      AVRCP Get Element Attributes Packet
      AVRCP Register Notification Packet
      AVRCP Get Play Status Packet
      AVRCP Reject Packet
    AVRCP Pass Through Packet

Adds net_test_btpackets to run_host_unit_tests.py for pre-submit

Bug: 68854188
Test: run host native test net_test_packets
Change-Id: I0d385710178e5feb2d5089847580754ad6308709

Drops unittest build on Mac host.

Bug: 72618679
Test: Built on Mac.
Change-Id: If6e2f8cf39cd80e80a419fbbed0ce34931ede915

Report LE CoC credits to GAP clients
am: b9c95d5a61

Change-Id: I28db97500b79b8e4d90ab31334310deb2779c158

Report LE CoC credits to GAP clients

Test: manual with HA
Change-Id: Ic4552b05b6e6c7dad5311e0b893da3166bf1895c

Merge "Report credit count change, when credits are received for LE CoC"
am: f6f158713a

Change-Id: I6bc20a0b10414c9e73333bce9e550e8b32390d74

Merge "Report credit count change, when credits are received for LE CoC"

Merge "HFP: Fix NPE in error logging for voice recognition events"
am: 7676c7df90

Change-Id: I03f15ceda56a53616ec514fc46419288a624cdf3

Merge "HFP: Fix NPE in error logging for voice recognition events"