Merge "L2CAP: Always use SUCCESS"
am: f256ea9c02

Change-Id: If78bace42679e10a74b8b3df98feda1970e29450

Merge "L2CAP: Always use SUCCESS"

Require a semicolon after ASSERT
am: 49496a3d06

Change-Id: I15734a820b5a67733babd0aae0263f8f97edc508

Require a semicolon after ASSERT

Test: atest --host bluetooth_test_gd
Change-Id: I304d097f6094eecdad04d99c9397475b4721115c

L2CAP: Always use SUCCESS

Test: bluetooth_test_gd --gtest_filter=*L2*
Change-Id: I6070168e2d9055b6d7f54726a773ab646dcb567c

Merge "HCI: Add classic interface definitions"
am: b8b18593c6

Change-Id: If2710b9c0327ac8456984722524d464512abfe81

Merge "HCI: Add classic interface definitions"

HCI: Add classic interface definitions

Test: bluetooth_test_gd --gtest_filter=*Hci*
Change-Id: I4206c818084302341d4e99f2cce8aad1c12ce4ae

[automerger skipped] Merge "DO NOT MERGE - Merge Pie Bonito/Sargo into "
am: 7bd0d9e0c3 -s ours
am skip reason: subject contains skip directive

Change-Id: I0a6eb0daaa03161d899b838f88945cc06bac056f

[automerger skipped] DO NOT MERGE - Merge Pie Bonito/Sargo into 
am: e39adc7f03 -s ours
am skip reason: subject contains skip directive

Change-Id: Ie61608aea918084208a6cc25b5f39c9567ee611e

Merge "DO NOT MERGE - Merge Pie Bonito/Sargo into "

Merge "Add Delay reporting for Hearing Aids"
am: 122ce6871f

Change-Id: If98f6b3faad73841924abdf92b2a4b09ac9411b9

Merge "Add Delay reporting for Hearing Aids"

DO NOT MERGE - Merge Pie Bonito/Sargo into master.

Bug: 131756210
Change-Id: I4009cad536b5c2d7a9f9d02769128dc5077e53b9

HCI: Rename ErrorCodes and consolidate Enable
am: 38ea5acca8

Change-Id: Ifa89f600275e8259c37fbddc02dd1e2f66b9112e

Add Delay reporting for Hearing Aids

Use the RenderDelay parameter from the Hearing Aid devices to implement
the Delay Reporting feature. This will helps in the lip-sync issues.
Please note that this change will requires a re-pair of the Hearing Aids
devices in order for the delay report to work well.

Bug: 132115153
Test: Manual test
Change-Id: If4bef049c18a3912817c2fa96b9bd890d5712c22

HCI: Rename ErrorCodes and consolidate Enable

Test: bluetooth_gd_test --gtest_filter=*HciTest*
Change-Id: Ic957968b5f0de1eac486b4e2ba103ec749819b0e

ModuleRegistry: Start() injected modules
am: 30fa258cf8

Change-Id: I096fdf186f33e9e690e5dd564f4b5ac28b71701b

ModuleRegistry: Start() injected modules

Call Start() on injected modules, and set registry_ and handler_
properly

Test: atest --host bluetooth_test_gd
Change-Id: Id655d537982eab285549ff797a907dabb15ca64d

Merge "Add benchmark for queue"
am: 7ec4793b50

Change-Id: I9586994386bbf51fae402424b66c3fa955c29a76

Merge "Add benchmark for queue"

Merge "Handling of incoming AVRCP Controller connections"
am: e7b536bc93

Change-Id: I7574cd7421edb6572472ffef685b70429f6d8d2a

Merge "Handling of incoming AVRCP Controller connections"

Merge "Close unexpected HearingAid open event if is not attempting to connect"
am: f612b732e1

Change-Id: Id6ca92d830e6e7f10d22260c15f6b47fea3b80fa

Merge "Close unexpected HearingAid open event if is not attempting to connect"

Merge "A2DP: Fix offload start VSC parameters"
am: da0b935bf4

Change-Id: I3425223bc0c0b49f49f04a164c10fdfa2a50bc57

Merge "A2DP: Fix offload start VSC parameters"

Close unexpected HearingAid open event if is not attempting to connect

When HearingAid bta layer receive unexpected HearingAid open event,
should close it if is not attempting to connect.

Bug: 131863877
Test: 1.Forget/Repair HearingAid device from Bluetooth UI.
2.Disconnect/Reconnect HearingAid device from Bluetooth UI.
3.HearingAid device power off/on, check them can reconnect.

Change-Id: I75d63d0441dcc6b7338b1e30453021eb271f4872

Add a signaling socket from facade/cert to tester
am: 663ee01503

Change-Id: I2f2dfb01de8ac8ad132ba7ec9b4e677bdb256277

Add grpc root service for test configuration
am: 30bf869b83

Change-Id: I9226c028d4a40b7aee39b714e19c9d33b94d36ab

Add a signaling socket from facade/cert to tester

The tester needs to wait for grpc server to start up before sending any
RPC. Use a UNIX domain socket to indicate when the grpc server is ready

Test: cert/run_cert.sh
Change-Id: Iaf5e9dfaedaa30a09448da0632ae7d362845c87a

Add benchmark for queue

Test: run bluetooth_benchmark_gd
Change-Id: I074f7a6f0a7927f9bf35f92cbc0bb4f3f10adfc8

Add grpc root service for test configuration

When facade_main and cert_main is started, it only starts grpc root
service, and listens for StartStack() request before brining up required
modules. Then, it listens for StopStack() request before cleaning up
modules. Defining new rootservice.proto for facade and cert.

Test: cert/run_cert.sh
Change-Id: I10fa745b9a235080be7a0c82fe7f3a297c1a112d

Break GdDevice into GdDevice and GdCertDevice
am: 87ddd5b26f

Change-Id: I9fb0d8bdb0b718a8846de25f5aaa8fbdaa33c1e4

Consolidate facade and cert entry point naming
am: 4999504f61

Change-Id: Ia48e8a6f3b4c6849e2e5cb5c0751e8e156f5c306

Add BidiQueue
am: d5b7ba71ba

Change-Id: I00a6898e46cb7e8ad550944fe63bc0cd3a9105ca

Break GdDevice into GdDevice and GdCertDevice

We need distinct devices, since the gRPC capabilities are
different from each other and we need to tell them apart
from each other in tests.

Also, remove unused gd/cert/run_cert.py
Also, don't indicate rootcanal error at end of test if
return code indicates it was terminated by the SIGINT we sent it.
(-N return code indicates terminated by N signal, as per
https://docs.python.org/2/library/subprocess.html#subprocess.Popen.returncode)

Test: cert/run_cert.sh
Change-Id: Idf14a05d73f2f80f5b871b2c41118219c27a0f76

Consolidate facade and cert entry point naming

Test: atest --host bluetooth_test_gd and cert/run_cert.sh
Change-Id: I517391d2884d5cc8d5b77a9b9847637d83053a16

Add BidiQueue

An interface for passing a pair of queues (an up and a down),

Also adds BidiQueueEnd, so you can pass the end around without worrying
about your clients using your end of the queue by mistake.

Test: atest --host bluetooth_test_gd
Change-Id: Ib2d1904ec799af335db07fb474d0ba6b1d4d9949

Cert stack bring up and test Inquiry with two stacks
am: 90ec7400f0

Change-Id: I85e10a876b52922ca0ce558292b0f54d87752405

Cert stack bring up and test Inquiry with two stacks

* Add main entry point for cert stack.
* Add a test case where cert stack enables page scan and inquiry scan
and DUT starts inquiry. Verify the inquiry result.

Test: cert/run_cert.sh
Change-Id: I2deaf12b6f929301a3542c082a8f77513af8ce68

Merge "Add separate enqueue and dequeue interfaces for queue"
am: 55778d288e

Change-Id: I5f244be668f66c7940037f9360f125f1f43e1757

Merge "Add separate enqueue and dequeue interfaces for queue"

Merge "Don't read HearingAid properties if already have properties value."
am: 52d50e51e9

Change-Id: I6a8a532978d441b8cc5494d204ba5970aff073cc

Cert: implement assert function with event buffer
am: c3d8c16207

Change-Id: I0328031ca4c390eed86edc92ceb8d70a279856e1

Merge "Don't read HearingAid properties if already have properties value."

Add separate enqueue and dequeue interfaces for queue

Allows you to pass one around without exposing the rest of the queue surface.
Test: atest --host bluetooth_test_gd

Change-Id: I468c59208cf021feb8bcea9e3a3b0a72c35ead8d

Cert: implement assert function with event buffer

Test: run gd/cert/run_cert.sh
Change-Id: I237ecdee7e70b860f3e73d4825dd79b0235ae22b

Handling of incoming AVRCP Controller connections

Resolve edge cases on incoming AVRCP Controllers Remote Controller
cases, both caused the remote control component to either generate an
out of bounds error or fail to update peer features.

Test: receive incoming AVRCP connection
Bug: 130750865
Change-Id: I15abd2fd4bc963889d6cdfe9364a651e2b4c3c10

Merge "Module: Add TestModule for dependency injection"
am: c310d49b97

Change-Id: Ic68642cd4cfda8b7059a61aef63bb990e9f2d84a

Merge "Module: Add TestModule for dependency injection"

Module: Add TestModule for dependency injection

Test: HciLayerTest
Change-Id: I32dbfaa051be5cb42adb7a081b37966479df5268

A2DP: Fix offload start VSC parameters

* Add more codec specific parameters in VSC.
* Deliver AAC parameters to the audio HAL.

Bug: 131209515
Test: manual
Change-Id: I3908abcb8f7703e4b89513ed473f6949b3c1bbe3

Don't read HearingAid properties if already have properties value.

When read HearingAid properties, don't read it if already have the
HearingAid properties in config file. Will use these values in config
file since these values should never change.

Bug: 128464408
Test: 1.Forget/Repair HearingAid device from Bluetooth UI.
2.Disconnect/Reconnect HearingAid device from Bluetooth UI.
3.HearingAid device power off/on, check them can reconnect.

Change-Id: Id4c30947c75f0615514075ec36c50afb58fc3f14

Cert: add common helper for event stream on the py side
am: 89430fe73d

Change-Id: I4c0ce2ced3c292c316ee25f35e0d25fdfd972aed

Cert: add common helper for event stream on the py side

* add common helper for event stream on the py side, implement
  assert_event_occurs in the helper
* add event stream helpers for hci event, sco and acl in GdDevice
* add take_for() function in BlockingQueue for take data with timeout

Test: run gd/cert/run_cert.sh
Change-Id: Ia8168159ee47441ec332046627124e0ed3811d6b

Make SnoopLogger a Module
am: 5158bf5eeb

Change-Id: If905e7c85293c2b2abbf0f23c3efa3d77743bef7

Make SnoopLogger a Module

* Add API SnoopLogger::SetFilePath to set output file path
* Add return code check for GdDevice backing process and rootcanal

Test: cert/run_cert.sh
Change-Id: Iac33070d4ac880d50a0dd78ad75f7ac7ccc239b2

Rootcanal: Exit local loopback mode on HciReset
am: 843271b29b

Change-Id: Icb2bfeb0d50ae89fe787b16471bf35543d10296d

Rootcanal HCI HAL: Set read from socket blocking
am: 8c08ceb642

Change-Id: I1f48f1c7decbe6f85d08c6872559edf409cc753e

Merge changes I471a49f6,If5879a76
am: 4b62519c95

Change-Id: I766fc1e1fec734858dc50ee9be54f870f3816905

Merge "Create handlers for every module upon module start."
am: 37de441d0d

Change-Id: I9d0bd3aabd76603bd271b552eb00cc11d0c60da3

Rootcanal: Exit local loopback mode on HciReset

In Bluetooth code spec v5.1 Vol 2 Part E 7.6.2, it states "the commands
HCI_Reset and HCI_Write_Loopback_Mode can be used to exit Local Loopback
mode".

In HAL cert test, enter local loopback mode, reset, and check whether
local loopback mode.

Test: cert/run_cert.sh
Bug: 131625008
Change-Id: I79546b202affe8ccd48b10e230cf73465324b409

Rootcanal HCI HAL: Set read from socket blocking

In HciSocketDevice::SendHci, rootcanal first writes to HCI socket one
byte HCI packet type first, then HCI packet body. When the stack
receives the first byte, it tries to read the entire packet, but
sometimes the rest of packet was not delivered yet, so the stack will
crash. Instead, the stack should be blocked until all bytes are read,
with a 3 second timeout when rootcanal can't deliver all bytes to stack.

Test: try cert/run_cert.sh multiple times
Bug: 131625008
Change-Id: I99df53c658717a5c931ebc075eb75df447e10332

Merge changes I471a49f6,If5879a76

* changes:
  Clean up facade naming and paths
  Add enforcement of module dependency declaration

Merge "Create handlers for every module upon module start."

Merge "PDL: Check sizes before validating constraints"
am: 0f5c7cee85

Change-Id: I3974c61f92d4b8fb24c70b4f1a270651cd2bcea7

Merge "PDL: Check sizes before validating constraints"

Clean up facade naming and paths

Remove "cert" from build rules that apply to stack under test

hal/facade/* -> hal/
api.proto -> facade.proto

In proto -> [x]Facade
Module -> [x]FacadeModule
Grpc service -> [x]FacadeService

A few straggling references to HciCmd and HciEvt

Test: atest --host bluetooth_test_gd and ./cert/run_cert.sh

Change-Id: I471a49f65a216d6669b3cb7f5ffebc7c069e2d6d

Gracefully shutdown SUT in cert
am: c6467a9c11

Change-Id: Idc2047ad66ea685babec3e3ed27c63cc71eacd4e

HAL: Add facade for fetching incoming HCI packets
am: 4adb042422

Change-Id: Ifc54f58b4d1ef0306d5055c05936af727b5529ee

Fix BlockingQueue bug when using vector
am: 5041e4f269

Change-Id: I665ed4ef6ad47ad4b5b5f42c226d68b872fade7e

Remove redundant Bluetooth from internal class names
am: b45ecd2fd7

Change-Id: Id7b7adfe228e2729eaf542aba0ce0bcbf005e527

Create handlers for every module upon module start.

Test: atest --host bluetooth_test_gd and ./cert/run_cert.sh
Change-Id: I360556ef3bb2ccf34d5658f00b30cc33949a77c1

Add enforcement of module dependency declaration

Move dependency access into module, enforce dependency was listed
in ListDependencies.

Remove ModuleRegistry from Start/Stop.

Test: atest --host bluetooth_test_gd and cert/run_cert.sh
Change-Id: If5879a76aae764286f1b5bd7df2b497ea8f51998

Gracefully shutdown SUT in cert

* After each test suite, shutdown the stack before closing rootcanal
* Don't stop gRPC server twice
* Crash the stack when can't build gRPC server

Test: cert/run_cert.sh
Change-Id: Ie4c8d749079e99b46d63fb70d157db26d9aa2474

HAL: Add facade for fetching incoming HCI packets

Adding the event stream primitives. General event stream request is
defined in facade/common.proto. "grpc/grpc_event_stream.h" provides
helper class templates GrpcEventStreamCallback and GrpcEventStream to
help user handle EventStream request without writing boilerplate code.

Test: cert/run_cert.sh
Change-Id: I00b51fc7b1faefb7c97cc647876f41854872a415

Fix BlockingQueue bug when using vector

return value instead of reference when take

Test: run unittest
Change-Id: I268f5022306353493a348a52ed363cf60a125661

Remove redundant Bluetooth from internal class names

Test: atest --host bluetooth_test_gd and ./cert/run_cert.sh
Change-Id: I55b2c1ecc1ec254a197d6fde70968a9aa8c06c00

Add a persistent key to overwrite MIN_CE_LEN for Hearing Aids
am: 4900661c49

Change-Id: Iae6e1b3eec5a45a5923fafcbe01d00960ee73cab

Add a persistent key to overwrite MIN_CE_LEN for Hearing Aids

To help in debugging the noise issues on Hearing Aids, add a new
persistent key, persist.bluetooth.hearingaidmincelen, to overwrite the
MIN_CE_LEN parameter that are used for the Connection Parameter Update
operations.

Bug: 130916579
Test: Manual testing
Change-Id: I1c52b2f99235f8d88e0a1d278dbf3174f1c60709

Merge changes Ibcb21966,I026b9e89
am: 1e6d32cd3a

Change-Id: I47e78cde67529f6e1192cb28c29122af66485684

Merge changes Ibcb21966,I026b9e89

* changes:
  Migrate GD code to module
  Add basic concept of module and module registry + dependency graph

Implement blocking queue
am: 9c2f4d0dbc

Change-Id: I51ff86db093683bb7ba868e0b37e1b6851846583

Migrate GD code to module

Test: atest --host bluetooth_test_gd
Change-Id: Ibcb219666ff5552efc602bf063e69a6e6b7e4bd8

Add basic concept of module and module registry + dependency graph

Modules have protected lifecycle functions.
Modules are identified by a factory object that knows how to construct them.
Modules can indicate which dependencies they have.

Dependencies are started before their dependent modules.
Don't support incremental module stop for the moment - it's all or nothing.

Test: atest --host bluetooth_test_gd:ModuleTest
Change-Id: I026b9e893501506a6cea8387941b3a9424a9bec9

Implement blocking queue

Test: run unittest
Change-Id: I93c59a2d8c9bbe9bee762412b368773c4b35456f

PDL: Check sizes before validating constraints

Test: bluetooth_packet_parser_test  --gtest_filter=*WayTooSmall*
Change-Id: Ia43c066c11b83f1e9b651ffdd85c6e0196c45332

Add workaround for "Service Changed" indications breaking server-only remotes
am: 890c501e97

Change-Id: Ic0727d12da91674737a0b3300eb4a0aa1e686349

Add workaround for "Service Changed" indications breaking server-only remotes

Test: Connect with ruwido remote when magictether service is registered
Bug: 122994857
Change-Id: Ibb2c1272bb08c7a127c8f7d37ee6a05f15618fc2

Basic integration of GD with ACTS
am: facafb0849

Change-Id: Ib0aeae59c7f0e5b72bc36a456225a676e819d610

Merge "Revert "Revert "Cert for HAL adapter"""
am: 1bc6c42328

Change-Id: I8ccd9a439f0e30152290139049b3f1d24f26ba14

Basic integration of GD with ACTS

* Add GD base test, which can parse config and bring up rootcanal.
* Add GD device controller, which is only optimized for host at this point.
* Add host config for running tests,
* Add basic hal test to test end to end flows.

Missing from this patch:
* GD cert device controller (should be similar to GD device)

Test: after setting up acts (gd/cert/set_up_acts.sh): gd/cert/run_cert.sh

Change-Id: Ibde7c7b0fe85b64643ed11fffe833e9ff53ed48c

Merge "Revert "Revert "Cert for HAL adapter"""

Simple btsnoop logger implementation
am: e066082980

Change-Id: Ib5452a6cf4b9b00cadb35c9741ea33c5ba2363bf

Revert "Revert "Cert for HAL adapter""

This reverts commit 865d9d138f1099a0236847e39329ee641c62b1ae.

Reason for revert: Re-landing CL I3cf2f54605ce2709d1c149b1221dedc8dc5dca92

Change-Id: I63d60ab99d3a4a1b369f77d6c48eff0f463506e4

Simple btsnoop logger implementation

Test: run unit test and check btsnoop log
Change-Id: Ib5ea3fc6440c48e1a7b5d5f3bcc3a0b1c23ac9c2

Merge "Revert "Cert for HAL adapter""
am: a1cdf837cf

Change-Id: Idb0c25b945284e7f83467793a8afde35a3b2274f

Merge "Revert "Cert for HAL adapter""

Merge "HearingAid: add addToWhiteList NativeInterface API"
am: 4bafb40345

Change-Id: I6fa5a9d38d9de413f306e0c9bbb303ee227e237a