net_test_bluetooth: replace with gtest variant am: e27d3133c2 am: a16a0266e4
am: 5f68c19c45

* commit '5f68c19c4512fa1bfb8dc99b1eae7046547eb586':

Fix SIGBUS crash when copying data am: 3f06e14fdd am: c4203a27fc
am: 5c5547d738

* commit '5c5547d738d2f97390907919f480dccc3fe205f2':

Indicate failed tests in run_unit_tests.sh am: 7ab59096c2 am: 08a960d795
am: 0a3f937949

* commit '0a3f93794972b291faad12570514b70da385e9b2':

Merge "Retry HID Polling when EINTR is returned" am: 95e0c57578 am: 3d1d6595ec
am: b161d37460

* commit 'b161d3746005b2d3f8496a43006d09b33d18bbc3':

Null-terminate hex string in uuid_new() am: addbe8f489 am: 5f2c3d86d2
am: 18b5bd160b

* commit '18b5bd160be41a15c9b11b2807cf84de1a516418':

Fix net_test_hci am: b0f59256c6 am: 0005a705cc
am: 7df0b466a0

* commit '7df0b466a0e94128d3005e1c2ea23d91b164df2d':

Merge "Fix broken test caused by misordered callback arguments." into cw-e-dev am: bedd614381
am: 11fc54d2c9

* commit '11fc54d2c90d6036fd1647a1cc41a07f1eb40189':

Merge "service/doc: Add Binder GATT client API definition" am: 88e68c2cd6 am: 6b0dadb3a3
am: ff65680624

* commit 'ff656806247f72c2509c32f1eda71231f7d1de5a':

Fix an error loading audio.a2dp.default.so library am: a3aed3bd53 am: 4402e3a447
am: 1e01259291

* commit '1e012592919e799d7408658951d0b1e1c0a20892':

Fix broken test caused by misordered callback arguments. am: 1b4c8b4ebb am: f5dee186f1
am: 09a1c8a17c

* commit '09a1c8a17cbf02c3c1547bda79aead7c2514fe55':

Treat empty/truncated GATT cache as error am: 1f656f15b1 am: 774defe11b
am: c369239337

* commit 'c3692393378059a3abfebfdaf1982db659dc1011':

Always update remote address type during LE scan am: 7b9fe757f5 am: 5c5d572459
am: a9517d17f3

* commit 'a9517d17f3b451c15a50cce894795b496ceaf36b':

Removed libpower-related references. am: 45a94e198e am: b8cde2c80d
am: c935cdf7bf

* commit 'c935cdf7bf4ef21175cff473ab0380297875959b':

Always update remote address type during LE scan am: 03d171170c am: 02ddf25bd0
am: 2de6e0fc1e

* commit '2de6e0fc1e537b95ec31ed8b200cdf80a00e32f4':

gn-build: Get net_test_bluetooth to compile am: 3e54b60e10 am: ee4744eed6
am: 047c633004

* commit '047c63300491240f6f3359337bc198de99f23ccd':

Merge "Do not mask out secure connections (SC) bit for BT <4.2" into mnc-dr-dev am: 15c1f685db am: 442e504115
am: 4560daf794

* commit '4560daf794a3820376d4953f05b4b9737e0d7aa7':

Fix bug in SDP 128-bit UUID lookup am: 7312c6d913 am: 50d8df7a33
am: 54e2c2be9a

* commit '54e2c2be9a68c8eef2995d239c41dddcbc2d17cf':

Fix bug in SDP 128-bit UUID lookup am: 526f3cc7e9 am: 1a1c20dd86
am: ae11858a72

* commit 'ae11858a72c94cb60de0cb3c698eea2310ad45e6':

Do not mask out secure connections (SC) bit for BT <4.2 am: 97305504e5 am: 5d30292aaa
am: fc8839fad4

* commit 'fc8839fad4f5b29ed09bce99dbf1efad0f3838ab':

Merge commit \'8b71f990de2b2ca9354726b56f0bd41bb9082ad3\' into HEAD am: abb36da490
am: b59bc16569

* commit 'b59bc1656929b3324844dd462a4b4f0067d381ae':

Merge "Rename net_test_bluedroid to net_test_bluetooth" am: c2ad136c3e am: 028f32f35f
am: 65a55b1763

* commit '65a55b17638b0f88cdc5ba6b719a9dda6fb36500':

Fix interop database unit test am: f2a7e6c4eb am: da16e1c6a4
am: 26d36ea465

* commit '26d36ea4651bcbd810b7d70a2d1011f411068ae9':

Merge "Serialize remote version query over LE" am: a0a3dacd68 am: 65ba9b572b
am: dc0bb24d1d

* commit 'dc0bb24d1d40ed23bf33bcf6d0db3d28ea357e33':

Remove unused HCI_PROTO_REVISION am: 879aeec727 am: e7a14f9a7f
am: 6fad4e7234

* commit '6fad4e723449dfe3fb42e90d968028753474db62':

Serialize remote version query over LE am: 864e7734f2 am: b2bf772af1
am: a5d20f2f3b

* commit 'a5d20f2f3b63297b22ba2f72bf078130fcf91a2a':

Check remote LMP version before enabling secure connections am: 4f7c4f6732 am: 406a266f13
am: bbcc5f4fb9

* commit 'bbcc5f4fb94cdfa7522162fa392fc2692de674ef':

Merge "net_test_bluetooth: Improve set/get name logic"
am: da8680df37

* commit 'da8680df37ed3606b484cab11dc3d5c0f0dc1154':
  net_test_bluetooth: Improve set/get name logic

Merge "net_test_bluetooth: Improve set/get name logic"

Fix BT connection failures
am: 78500516e5

* commit '78500516e502944f7e3f86fcfbe70f797a2e45e5':
  Fix BT connection failures

Remove "block" parameter from eager_reader_read()
am: b9757ee3c9

* commit 'b9757ee3c9f1bb4d521624606db5b2e7ba722eeb':
  Remove "block" parameter from eager_reader_read()

Properly reset NONBLOCK flag in semaphore_try_wait()
am: 48df46b839

* commit '48df46b8398cf224a5209002cce3d534c4480d99':
  Properly reset NONBLOCK flag in semaphore_try_wait()

Fix BT connection failures

When HFP connection failed establishing between phone and watch, it can't
recover RFCOMM status which causes that HFP connection will not be made
unless BT radio goes off and comes back on. RFCOMM and HFP both trying to
open RFCOMM control channel at nearly same time, at some point, there's
a possibility that SABM packet didn't reach out to BT controller to connect
to phone from watch with HFP while phone tries to connect to watch with
AW RFCOMM.

This will take care of pending SABM packet to reach out to controller
properly so that RFCOMM status will not be messed.

Bug: 24285000
Bug: 25579482
Bug: 25579423
Bug: 25578997
Change-Id: Icc26b86c644ebd8d1c12a2fb4e94d04ba16cc2dd

Remove "block" parameter from eager_reader_read()

Setting this parameter to true did not work and did not behave as
expected. The functionality provided by this parameter is also not
necessary, thus removing code.

Change-Id: I29e60da4adf1d1fc84d8ec9a590de89e94bb7900

Properly reset NONBLOCK flag in semaphore_try_wait()

Without this fix, calling semaphore_try_wait() on a semaphore that
wasn't currently set, would leave the NONBLOCK flag on the file
descriptor as a side-effect.

Also added a unit test for semaphores, including a test specifically for
this condition.

Change-Id: I0ea37bb68b14c76febaab25b3aee1bb4f5acee8c

net_test_bluetooth: Improve set/get name logic

Without this CL semaphore_wait would instantly return due to the fact
that enabling the adapter would cause the properties callback to post
immediatly. Also prevented the test from returning a false positive if the
original device name was the same as the test name.

Change-Id: I987efcb9a5ef58209d37d2fc21f3c149fb3e556c

Blacklist devices for absolute volume control
am: d0aa6cc53a

* commit 'd0aa6cc53abaf122a2426c20691ccfa025ca7369':
  Blacklist devices for absolute volume control

Blacklist devices for absolute volume control

Some devices do not play well with absolute volume control. They either
play back music at unacceptably loud volume levels or provide
insufficiently granular volume control where a single "step" of volume
change results in unexpectedly large jumps in volume.

Bug: 26070064, 26060431
Change-Id: I5a336fba87119325b47a5ad385e9c48a631f2121

net_test_bluetooth: fix set and get name tests
am: ddf67ab9aa

* commit 'ddf67ab9aa665e35302f6f0183d55787c87113cd':
  net_test_bluetooth: fix set and get name tests

net_test_bluetooth: fix set and get name tests

Fixed the get and set name tests so that they are more consistent and robust.

Change-Id: I4b76357eff05531c4e52cd6de03eb3b8b969062b

Merge "Remove dependency on base::Hash"
am: 17322144e3

* commit '17322144e3a5a1d66afd5926bf0b691b40c7ca44':
  Remove dependency on base::Hash

Merge "Remove dependency on base::Hash"

Remove dependency on base::Hash

This function is changing to just use std::hash anyway.  This is
slightly less efficient because of the copies, but doing it inline
makes the cost more explicit to bluetooth authors.

Bug: 26253162
Change-Id: I940ea8ce8aa27808cef8a8b9398a7756db3b5ca3
Test: Compiles

Fix memory leak in vendor layer
am: e17b696eed

* commit 'e17b696eedd55269d9b21c77be60ef08b9ddd1f9':
  Fix memory leak in vendor layer

Fix memory leak in vendor layer

When the vendor library transmits without giving a callback, the vendor
translation receives a buffer it's in charge of deallocating.

When there is a callback, it's the vendor library's job to deallocate
it.

Change-Id: I65b0a037b28103ba46a33a1baea942f7e724eb7c

Remove repeated code in HF BTIF init function.
am: 581622f834

* commit '581622f8349b7677db971eecca7f72c9d3598bf0':
  Remove repeated code in HF BTIF init function.

Remove repeated code in HF BTIF init function.

Change-Id: I25a07e9cdb29ef31696a156c6f7ff5b57f4f5cfb
Signed-off-by: yanlaijun <yan.laijun@gmail.com>

Bluetooth: Silence warnings
am: a80c731fd0

* commit 'a80c731fd0538866823668e49928cbc369d7bfd3':
  Bluetooth: Silence warnings

Bluetooth: Silence warnings

Silence unused-parameters warnings coming from libchrome.

Bug: 26228533
Change-Id: I08bc474c905cd2cf7b1c0492ee37187c499ddf52

net_test_bluetooth: GATT test refactor
am: f261cf8646

* commit 'f261cf86466c61cfbbaffecd82586855614d48c9':
  net_test_bluetooth: GATT test refactor

net_test_bluetooth: GATT test refactor

Properly integrating the GATT tests in net_test_bluetooth to utilize GUnit
correctly. Contains all the GATT tests and the base class the GATT tests use.
Also reorganized the files.

Bug: 25793348
Change-Id: I5a547a2528c20d6baa9b1fb6d0b956f718d736cc

test: Add bluetoothtbd_test and net_test_bluetooth to runner
am: 092cf9ec93

* commit '092cf9ec93d16416d2bfbee3bd55a4aec91705a4':
  test: Add bluetoothtbd_test and net_test_bluetooth to runner

test: Add bluetoothtbd_test and net_test_bluetooth to runner

Change-Id: Iee7365cc366793d3f88cd8b9a6be454c5b853863

service: Add scan structures
am: 57dcd7e476

* commit '57dcd7e47681d3f452ab0f2b34f413af9be51e9e':
  service: Add scan structures

service: Add connection state tracking to Adapter
am: 0f29c005ee

* commit '0f29c005ee0a1a5c68c9b8e33f099fffefb5bab8':
  service: Add connection state tracking to Adapter

service: Add scan structures

Added the ScanFilter, ScanSettings, ScanResult structures that are
used in the BLE scan API.

Bug: 25744656
Change-Id: Id1bb28e7ebe85fd40013876aa097e1d0f0a4e471

service: Add connection state tracking to Adapter

Added the IsDeviceConnected method to Adapter and also an
Observer method that other components can use to track the ACL
state between the local adapter and a remote device.

Bug: 25157450
Change-Id: If9bc25ab58b4a9b5430753405fb64834c0946050

Merge "Rename libchrome-host to libchrome."
am: 6e7f114917

* commit '6e7f114917c3cf6b2c801afcf599c2c962bca019':
  Rename libchrome-host to libchrome.

Merge "Rename libchrome-host to libchrome."

Clean up A2D_ParsSbcInfo indentation
am: 76198e72bf

* commit '76198e72bf5b8942dbc97bb53c97dba62bf30da1':
  Clean up A2D_ParsSbcInfo indentation

Merge "Fix p_info increment in A2D_ParsSbcInfo"
am: 8520286cea

* commit '8520286cea654221b5577b8ed4a2e0272bf7d28c':
  Fix p_info increment in A2D_ParsSbcInfo

Clean up A2D_ParsSbcInfo indentation

This should introduce no change in behavior.

Change-Id: I14faa73d898f69bd15aff7c3857c671ebe1628c9

Merge "Fix p_info increment in A2D_ParsSbcInfo"

Add default implementations for observer callbacks
am: c2a61ac0a9

* commit 'c2a61ac0a96ba21ff99c396e14a0e6b42ccf9e79':
  Add default implementations for observer callbacks

Add default implementations for observer callbacks

Added defaults functions that do nothing for the bluetooth observers. Also
removed empty definition of the discovery callback in adapter. A little cleanup
was done too.

Change-Id: I93b6134f25e30e30cc2072cb4c99ce06aa781849

Fix the fixed_queue_try_remove_from_queue() internals
am: 153bdfb978

* commit '153bdfb9784b21e235676508c75bf2f4ee2b937f':
  Fix the fixed_queue_try_remove_from_queue() internals

Fix the fixed_queue_try_remove_from_queue() internals

Calling fixed_queue_try_remove_from_queue() should update
queue->dequeue_sem and queue->enqueue_sem as appropriate.

Also:
 * Removed function fixed_queue_init() because it is not used
 * Added fixed_queue_test.cpp unit tests for fixed_queue
 * Updated some of the fixed_queue API description

Bug: 25973053

Change-Id: I3a0931f34fed34a2d60f82904601ef4157ee8ff9

Rename libchrome-host to libchrome.

Update the name of the libchrome-host library to the match the one used
in the target.

Bug: 24619596
TEST=make dist

Change-Id: I73e906a0683142cc7e43237d4c5fa20c27acc6cc

Fix crashes with lots of discovered LE devices
am: d77f1999ec

* commit 'd77f1999ecece56c1cbb333f4ddc26f0b5bac2c5':
  Fix crashes with lots of discovered LE devices

Fix crashes with lots of discovered LE devices

When loads of devices are discovered a config file which is too large
can be written out, which causes the BT daemon to crash on startup.

This limits the number of config entries for unpaired devices which
are initialized, and prevents a large number from being saved to the
filesystem.

Bug: 26071376
Change-Id: I4a74094f57a82b17f94e99a819974b8bc8082184

Add discovery state callback to adapter
am: ade7583ddd

* commit 'ade7583ddd1877d25d3605be111319e5d8d760e4':
  Add discovery state callback to adapter

Disable bdtool compilation
am: 8f56b5e270

* commit '8f56b5e270790fdba9650338cbc41dbfd7280c5d':
  Disable bdtool compilation

Add discovery state callback to adapter

Change-Id: Icb3afacc5a23603762f2b0c42244dc408b245dcc

Disable bdtool compilation

For re-factoring purposes, the dependencies are broken. Need to
re-evaluate tool usage and port to new bluetoothtbd HAL layer if
necessary.

Change-Id: I27b2ad3d78c67a12c67d8db9f506f94888bea70f

net_test_bluetooth: adapter test refactor
am: fc616e01fb

* commit 'fc616e01fb674e5907e578ae8ecd72b189cfb534':
  net_test_bluetooth: adapter test refactor

net_test_bluetooth: adapter test refactor

Properly integrating net_test_bluetooth to utilize GUnit correctly.
Contains all the adapter tests and base class that all the other tests
utilize. Also removed all the old test code.

Bug: 25793348
Change-Id: Ie05d54e9d21f01e2d717bb79d187d7f0ff8a83f1

osi: Remove error_fclose goto and redundant fflush.
am: ed2267a0a6

* commit 'ed2267a0a66b1aec52d9039bc20a6c5099dbdae7':
  osi: Remove error_fclose goto and redundant fflush.

osi: Remove error_fclose goto and redundant fflush.

Change-Id: I327f9ce696bb68eb3b89b4a8b15dba189e027970

Fix the calculation of bta_sys_get_remaining_ticks
am: 865e736eea

* commit '865e736eea16629fcbb24aea18b250a0f51b31c7':
  Fix the calculation of bta_sys_get_remaining_ticks

Fix the calculation of bta_sys_get_remaining_ticks

Previously, bta_sys_get_remaining_ticks() calculated the remaining
ticks for all the items in the hash map, which is incorrect.
Instead, we need to calculate the remaining ticks only for the queried
item.

Bug: 26110234

Change-Id: Ia867c1d0da15809a643b946f7d8b9143e3882935

Added visibility to the UIPC_Read() errors
am: 242460a6e8

* commit '242460a6e8de083518f5d1c46384f9d0e818141b':
  Added visibility to the UIPC_Read() errors

Added visibility to the UIPC_Read() errors

Changed some of the BTIF_TRACE_EVENT() log messages
to BTIF_TRACE_WARNING() so we can get better visibility into the
UIPC_Read() errors.

Also, removed UIPC_ReadBuf() and UIPC_SendBuf(), because they are
not implemented / used.

Change-Id: Iccc6a4d73dfe3c3e57f21c538891d7cb429f11ac

Fix net_test_hci unit tests am: c3caf784c7
am: 6dbfc34218

* commit '6dbfc34218ae1fc25c507ddc40080be04f8e2729':
  Fix net_test_hci unit tests

Fix net_test_hci unit tests
am: c3caf784c7

* commit 'c3caf784c788af45fd6b657a09f4034b9d3387ba':
  Fix net_test_hci unit tests

Only initiate codec negotiation if feature is supported am: 1cb11bb9af
am: f846fe3496

* commit 'f846fe34967d0bb8a2742b3a76162d671b115cce':
  Only initiate codec negotiation if feature is supported

Update features pages after LE Host Support change am: ebc367a5a8
am: cfb9646a50

* commit 'cfb9646a50c607a5913507d98f7d0a268d38ab19':
  Update features pages after LE Host Support change

Only initiate codec negotiation if feature is supported
am: 1cb11bb9af

* commit '1cb11bb9af8c72437eaf364bd422715015e4534b':
  Only initiate codec negotiation if feature is supported

Update features pages after LE Host Support change
am: ebc367a5a8

* commit 'ebc367a5a8c90f70fd39096a2e658826d17e94a0':
  Update features pages after LE Host Support change

Fix p_info increment in A2D_ParsSbcInfo

This is supposed to adjust position in the passed buffer,
not adjust a value of the buffer.

It fixes A2DP sink pairing by not corrupting the codec mask.

Also add const qualification to the input buffer.

Change-Id: Ic83ba73e0ea90897c131b4c1417cb976903862ac

Fix net_test_hci unit tests

* Added new function API to hci_layer: hci_layer_cleanup_interface()
  It can be used to cleanup after hci_layer_get_interface()
  or hci_layer_get_test_interface()
* Fixed the unit tests inside HciLayerTest by removing usage of
  allocation_tracker_reset() and adding hci_layer_cleanup_interface() .
  The former was used as a hack to bypass the internal memory allocation
  of hci_layer_get_test_interface(). The latter is used to properly
  cleanup such allocation.

Bug: 26014740
Change-Id: I498b996a931c59dd3b4258f1a5741cfa6d302e42

Only initiate codec negotiation if feature is supported

Bug: 25634250
Change-Id: I97330c853f6cf90673b4ff28c72645f9d1c8ff9d

Update features pages after LE Host Support change

After "Write LE Host Support" command is sent, the number of supported features
pages increases. Make sure we have the correct number of pages because it will
be read later.

Change-Id: I23d57b2cc9e773e14b2ce30ccfa624ba433b191c
Signed-off-by: Frédéric Dalleau <frederic.dalleau@linux.intel.com>
Signed-off-by: Zhiquan Liu <zhiquan.liu@intel.com>

Fix race condition during simultaneous SDP am: 1b3ac36749
am: 55cc444ada

* commit '55cc444ada2bc3a122d7bd39b855d7fc6d64d5b6':
  Fix race condition during simultaneous SDP

Merge changes I7f9b2ea0,Ibb1c1d4c,I64cef032 am: cc79505e54
am: c24a397f4f

* commit 'c24a397f4ffae1c52356d0819e6c3f36c6487bee':
  Clean up run_unit_tests wordiness
  Add iterations option to run_unit_tests
  Clean up run_unit_tests a little

Fix race condition during simultaneous SDP
am: 1b3ac36749

* commit '1b3ac367492750439ebecd393af1799242767ef3':
  Fix race condition during simultaneous SDP

Merge changes I7f9b2ea0,Ibb1c1d4c,I64cef032
am: cc79505e54

* commit 'cc79505e542cab5d686f8dba298f6b0c224a8129':
  Clean up run_unit_tests wordiness
  Add iterations option to run_unit_tests
  Clean up run_unit_tests a little

Fix race condition during simultaneous SDP

Right now sdp_conn_id and p_sdp_db are stored in static global
bta_gattc_cb between call to bta_gattc_sdp_service_disc and
bta_gattc_sdp_callback. If multiple instances of SDP discovery
are running simultaneously, they override this field, and free
same memory multiple times. This patch fixes that by making sure
sdp_conn_id and p_sdp_db are unique for each SDP discovery.

Bug: 25801255
Change-Id: I8ec52229e906e6b8748db7504f77e1f4d7006fbe

Merge changes I7f9b2ea0,Ibb1c1d4c,I64cef032

* changes:
  Clean up run_unit_tests wordiness
  Add iterations option to run_unit_tests
  Clean up run_unit_tests a little

Merge changes I9333236a,Ie4e5992c am: 668c69faaa
am: befc395aa5

* commit 'befc395aa580a7047bc7854fb84a2372c15d5c92':
  Switch to osi_getbuf.
  A2DP SRC offload support

Synchronize Bluetooth stack init and cleanup am: c28849aa58
am: 9b35619839

* commit '9b35619839ab089356411ed9d61921628e808007':
  Synchronize Bluetooth stack init and cleanup

Merge changes I9333236a,Ie4e5992c
am: 668c69faaa

* commit '668c69faaa8e7b04ae77557a1cb5e615d3150c9f':
  Switch to osi_getbuf.
  A2DP SRC offload support