Use AddressWithType in PairingHandlerLe

Bug: 142341141
Test: updated unittests
Change-Id: Ic543148ca2d36f44e64ffb00744adf6d557b84c3

Expose Role through AclConnection, Link, and le::FixedChannel

Role is needed in SMP to properly bootstrap the pairing process

Bug: 142341141
Change-Id: If0815c7bc9d07180950e66dbaafd69220e933340

Merge "L2CAP: Allow opening connection, and add PTS request_connection test"

Merge "Add required header_libs"

L2CAP: Allow opening connection, and add PTS request_connection test

* Allow reserving a channel to send channel connection request
* Handle open channel request from user API
* Add PTS test_basic_operation_request_connection
(L2CAP/COS/CED/BV-01-C)

Test: run_cert_test
Bug: 141557006
Change-Id: Ia8245a3f944dd1e731808d1c99ac79047b9d67a8

Merge "Make FixedChannel and Link aware of address type"

Make FixedChannel and Link aware of address type

Bug: 140938432
Bug: 142341141
Change-Id: I120179eef18f3ac59e32e9196f700e51c47d1a84

Add required header_libs

Test: build
Bug: 143245419
Change-Id: Iadfb621e477e1cef325385f3f5b10285cb3fc752

Merge "HCI: Add fuzz test for HCI packets"

Merge "PDL: Parameterize the root namespace"

Merge "L2CAP: Handle initial channel configuration"

Merge "Connect inquiry results back to legacy layer"

Merge "Module: Mark getters as const"

Connect inquiry results back to legacy layer

Bug: 141963466
Test: Compile
Change-Id: I949b1d90ead14cf15baa2a74db979ec76151650a
Merged-In: I949b1d90ead14cf15baa2a74db979ec76151650a

Module: Mark getters as const

Test: bluetooth_test_gd
Change-Id: Ieea21b113154bac1539a1ff73f4ca46ad37dfe35

L2CAP: Handle initial channel configuration

On channel open, send an empty configuration request, and always
respond empty to configuration request, to allow L2CAP basic dynamic
channel setup for testing.

Test: run_cert_test
Bug: 141557006

Change-Id: I95774ea093d069c83313c3e6d00d0bc2c3e6c4ed

HCI: Add fuzz test for HCI packets

* Add fuzz test for a subset of HCI packets
* Modify the fuzz test generator to generate and register fuzz test at
  the same time

Bug: 142684649
Test: bluetooth_gd_fuzz_test
Change-Id: I1962fd497467bbf469d773823e3e6cf81f62772b

Store Handler in common::CallbackList, and execute callbacks on proper handlers

Bug: 143169747
Change-Id: Ib8b6901956ffde9969decbe0b4653758d6a9ce52

Merge "L2CAP: Call AclConnection.RegisterDisconnectCallback() in LinkManager"

Merge changes If88cfd69,Icea6e47b

* changes:
  Conditional entry between legacy Gd stack
  Gd shim layer neighbor modules

Merge "RootCanal: Use gd/os/log.h logging"

Merge "AVRCP Controller get play status"

Conditional entry between legacy Gd stack

Bug: 141963466
Test: Inquiry/Discoverable operational in Gd mode
Change-Id: If88cfd69d4eb60e615c1f84e8972676e27364c73

Gd shim layer neighbor modules

Bug: 141963466
Test: Compiles for realz

Change-Id: Icea6e47bd9539613ea441ae2aed604b59e4b692f

L2CAP: Call AclConnection.RegisterDisconnectCallback() in LinkManager

* LinkManager should get ACL disconnection callback from HCI since it
  needs to be able to free Link objects after tearing down Link states
* Previously, only Link is getting this callback, as result, there is a
  memory leak when ACL is disconnected as the Link object was never
  removed

Test: bluetooth_test_gd
Bug: 140938432
Change-Id: I28ed509cbaaa1c4917e1f632a6c35247f3ebe2ef

HCI: Use AddressWithType in various LE connection APIs

* Use AddressWithType object to ease object management and hashing
* Track classic and LE pending connections separately in AclManager
* Report peer resolvable address in LE enhanced connection complete
  event if it is not empty

Bug: 138261114
Test: bluetooth_test_gd
Change-Id: I2bea155485a5bdc3d087336c6276952f3d64f0ac

Merge changes I87ce02f0,I2c81de7a

* changes:
  L2CAP: Use AddressWithType in L2CAP LE APIs
  HCI: Create AddressWithType and its unit tests

AVRCP Controller get play status

Correct the parsing error in the GET_PLAY_STATUS message and call the
proper jni callback when it is received.

Bug: 139033614
Test: Connect AVRCP change media players, observe correct value parsed.
Change-Id: I8302b64efdd72bfa671cca5b8c512a436fa54ab3

L2CAP: Use AddressWithType in L2CAP LE APIs

* Bluetooth Low Energy connections always require explicit AddressType
  information
* Replacing existing Address and AddressType arguments in l2cap::le
  stack with hci::AddressWithType objects for easier hashing and
  managment

Bug: 140938432
Test: bluetooth_test_gd
Change-Id: I87ce02f0a0c867053a2b3560886924a566881801

HCI: Create AddressWithType and its unit tests

* Create hci::AddressWithType that has both hci::Address and
  hci::AddressType in the same copyable, hashable, and comparable
  object for easier management
* Create unit tests for the hash function

Bug: 139135297
Test: bluetooth_test_gd.AddressWithTypeTest
Change-Id: I2c81de7a3bd19656b687ef498f02d6380321f92d

Merge "Import base::CallbackList into bluetooth::common::CallbackList"

Merge "Add FUZZ test generator for packets and L2CAP fuzz tests"

Import base::CallbackList into bluetooth::common::CallbackList

We need a good mechanism for keeping lists of callbacks. Import if from
libchrome for now.

Test: compilation, libchrome already test that.
Change-Id: Ia88be4d80464b981e3794a6ca73909a1e7ce8c04

Merge "Make le LinkManager aware of AddressType"

Add FUZZ test generator for packets and L2CAP fuzz tests

* Add fuzz test generator for packet generator to generate macros for
  fuzz tests
* Add a few fuzz tests for L2CAP packets

Bug: 142684649
Test: bluetooth_gd_fuzz_test
Change-Id: I75bf726b40caef3c7895596039d01c395d34506f

Make le LinkManager aware of AddressType

Bug: 142341141
Change-Id: I53b0d0eb8140a6109c0d3f237059b790c76a2111

Update file group names

Bug: N/A
Test: builds
Change-Id: Ie33178a3397343d0a80959a9e39f2f4ba03d5734

PDL: Parameterize the root namespace

Bug: 140384404
Test: bluetooth_packet_parser_test
Change-Id: I08aa14159ac86faf9e4e130540ecab77b93526e4

RootCanal: Use gd/os/log.h logging

Bug: 140384404
Test: builds
Change-Id: I442c9629ace0c39f26de59aaab70efa955f34c78

Merge "Add a fuzzer in bluetooth gd for l2cap dynamic channel allocator"

Add a fuzzer in bluetooth gd for l2cap dynamic channel allocator

Initialized fuzz test for Bluetooth GD library. There is one fuzz target
gd/fuzz_test.cc and it will call all fuzz tests in gd.

Created a fuzzer for l2cap classic internal dynamic channel allocator to
test DynamicChannelAllocator::IsPsmUsed(Psm psm).

Bug: 142684649
Test: bluetooth_gd_fuzz_test
Change-Id: If7c66d6aefc5d52448824f7f96f3118607a37215

Connect gd/neighbor into build system

Bug: 142060811
Test: Compiles but for realz
Change-Id: I719133c6466aab5b1813c9c2e49e7c1a4f29ca6b

Fix initiating_PHYS bit in HCI_LE_Extend_Create_Connection

The initiating_PHYS bit in HCI_LE_Extend_Create_Connection should
be set properly according to the ability of bluetooth controller.
PHY_LE_CODED bit should be 0x04 instead of 0x03.

Test: Start to scan on settings then pick LE device to create bond

Bug: 142840872

Change-Id: Ia991ac2d4090629f0c20e41cdb2ab992aefcba72

Merge "Inquiry module implementation"

Merge "Add connectability module"

Merge changes Ib4c81ba3,I79bead5d

* changes:
  Add discoverability module
  Add page scan module

Merge "Make hci::AclConnection aware of AddressType for LE Connections"

Merge "HFP: remove the dropped HF connection request message in btif connect_queue and advance to the next scheduled connection request."

Merge "Classic inquiry scan API"

Add discoverability module

Bug: 142060811
Test: Compiles
Change-Id: Ib4c81ba3d4c30c30602d451c97609f7577e3e8e3

Add connectability module

Bug: 142060811
Test: Compiles
Change-Id: I0afa5f6a29b8c9c4d2b3e7dd8df427549abe01d5

Add page scan module

Bug: 142060811
Test: Compiles
Change-Id: I79bead5da3706520fd4c22c4e0e921d930bd5fb2

Inquiry module implementation

Bug: 142061208
Test: Compiles
Change-Id: Iead1e6dfe2c476ecd4671a99413527b9b628bb36

Classic inquiry scan API

Bug: 142061208
Test: API - Compiles
Change-Id: Ie7a5bda3f10bedefc412b50f61c593e496999596

Merge "Inquiry and page scan common module"

Make hci::AclConnection aware of AddressType for LE Connections

Bug: 142341141
Change-Id: I64e3e3265195d10e488e9c25d369d9c458ad67cc

Merge "Add Page Write commands"

Merge "Support inquiry commands and tests"

HFP: remove the dropped HF connection request message in btif connect_queue and advance to the next scheduled connection request.

* hf connect-request message does not be handled in btif connect_queue when remote headset has already reconnected hfp, but it did not be removed, either.
it will make subsequent connecting request messsages blocked in btif connect_queue and never be handled.
* Add unit test for btif_profile_queue to verify the case when connect_cb() returns non BT_STATUS_SUCCESS

Bug: 140846154
Test: Connected 2 headset (HS1, HS2) -> Disable Bluetooth and Turn Off
HS2 ->  Enable Bluetooth and Turn On HS2 -> Check hfp and a2dp profole
state after HS2 connected

Change-Id: I503fba1975054efe3180ecf1f7daa917fb6edf99

SecurityModule boilerplate code

This is boilerplate code for SecurityModule and SecurityManager

Bug: 142341141
Test: no test yet
Change-Id: Ie08c8ef7ee78e71a9c61dff6b45192ce0e1c3f1c

Merge "HID host: Support SET REPORT"

Add Page Write commands

Bug: 142060811
Test: bluetooth_test_gd

Change-Id: I270715f9b8d8872f303aea76d70d25cef407b965

Support inquiry commands and tests

Bug: 142061208
Test: bluetooth_test_gd
Change-Id: I53de39850c985f2d63cda21ad0e16686dba0b5ac

HCI: Add LE advertising and scanning interfaces

Bug: 139080884
Test: builds
Change-Id: Iea96dc023e2e59f9969b65208881a9f407d4b461

Merge "Add property check for gd shim entry"

Inquiry and page scan common module

Bug: 142061208
Test: Builds successfully
Change-Id: I0ca7afb996490fff71030bf949ea9647924fd458

Add property check for gd shim entry

Bug: 14042172
Test: Ensure boot into both stacks
Test: adb shell stop; adb shell setprop bluetooth.gd.enabled 1 ; adb shell start
Test: adb shell stop; adb shell setprop bluetooth.gd.enabled 0 ; adb shell start

Change-Id: Ied131f5444dd33201197e9f19d0f4bc88c069f8c

Rename "smp" to "security"

SMP is just name of protocol used for one aspect of security management.
Rename the namespace in order to avoid any further confusions.

Test: compliation
Bug: 142341141
Change-Id: Id505fa3cd4b653c8486db2bca5f82233e731ec51

[system][bt] fix -Wreorder-init-list again

C++20 will require members in a designated initializer to be in order
unlike C99.

Bug: 139945549
Test: mm
Change-Id: I7dc825e2cdb0777b1be01df0a41adf9b3b644ff5

Merge changes I30972458,I62dd309d

* changes:
  [system][bt] fix -Wdangling-gsl
  [system][bt] fix -Wreorder-init-list

Merge "L2CAP classic signalling handler, data pipeline, and cert test"

[system][bt] fix -Wdangling-gsl

BtAddrString() returns a std::string. It's not safe to chain a call to
c_str() as otherwise the returned std::string is a temporary, and the
expression evaluates to an immediately dangling pointer.

Bug: 139945549
Test: mm
Change-Id: I30972458abcc563b24ee0d80b289c3efd6c3e04d
Signed-off-by: Nick Desaulniers <ndesaulniers@google.com>

[system][bt] fix -Wreorder-init-list

C++20 will require members in a designated initializer to be in order
unlike C99.

Bug: 139945549
Test: mm
Change-Id: I62dd309d5749c74c18e4291e7d1cef1f3587bf41
Signed-off-by: Nick Desaulniers <ndesaulniers@google.com>

Merge "[system][bt][gd] fix -Wreorder-init-list"

[system][bt][gd] fix -Wreorder-init-list

C++20 will require members in a designated initializer to be in order
unlike C99.

Bug: 139945549
Test: mm
Change-Id: I4ed1a2c2865cfa26b45db970202340ae582ba78d
Signed-off-by: Nick Desaulniers <ndesaulniers@google.com>

L2CAP classic signalling handler, data pipeline, and cert test

Supports listening to an incoming L2CAP connection, open the channel and
transfer data.

Test: bluetooth_test_gd and cert/run_cert.sh
Bug: 138260719
Change-Id: I5a1b93dc30b43bb149c32e5f4ef5404f2d091cdb

hci_pdl: Inquiry commands

Test: Run and verify on actual device
Bug: 142061208

Change-Id: I7d99aec75691528da75a563943d0db01cb2e3920

HCI: Add GAP data to advertising packets

Test: Build
Change-Id: I7ebc7d190c4bfc5ae0635bdfbd35b7e8a147ba96

PDL: Add padding fields

Bug: 138260498
Test: bluetooth_packet_parser_test
Change-Id: I6b020b2d47565e8e724e8114a2031be71e8a22ec

PDL: Support Type Length Value in Structs

Bug: 138260498
Test: bluetooth_packet_parser_test
Change-Id: If2a83908c507f37915e9a8520a1797cf5fc704a5

PDL: Allow parent packets with no parameters

Bug: 138260498
Test: bluetooth_packet_parser_test
Change-Id: I75e7c3a5d5bc653cc5df29ec406238cf9adf8cfc

PDL: Add an error for redefinition

Bug: 138260498
Test: define a duplicate type and compile
Change-Id: I54c4016d6eabf1b969b12616d4b88e3f2b593c9e

PDL: Use fixed portion of dynamic sizes for arrays

Bug: 138260498
Test: bluetooth_packet_parser_test
Change-Id: I271f3bd2c7eef13379fe3e0c059d82e2d1ddd37a

Merge "GD HCI: add unittest for LeConnectionCallbacks"

GD HCI: add unittest for LeConnectionCallbacks

Test: run bluetooth_test_gd
Change-Id: Ibb1255a7f3736ade798825751c6063d5ed2f0455

Split L2CAP Classic and LE stacks into separate modules

* Although L2CAP LE and Classic stacks look similar, they are actually very different when diving into details
* Splitting them into two separate modules will benefit in the following ways:
 * One can start a LE only stack
 * One can turn off Classic stack entirely when not needed
 * Dependencies are more obvious, e.g. LE-CoC only depends on LE L2CAP and A2DP only depends on Classic L2CAP
* Changed test names as GTEST does not allow test name collisions
* New directory structure after the change:
  - l2cap: common headers that should be exposed to public
    - internal: common internal libraries shared between LE and classic
    - classic: classic headers that should be exposed to public
      - internal: internal libraries for classic
      - cert: certification stack for classic
   - le: LE headers that should be exposed to public
      - internal: internal libraries for LE
      - cert: certification stack for LE

Using Bash
 for file in classic_*; do git mv "$file" "classic/${file/classic_/}"; done
 for file in le_*; do git mv "$file" "le/${file/le_/}"; done
 for file in internal/le_*; do git mv "$file" "le/${file/le_/}"; done
 for file in internal/classic_*; do git mv "$file" "classic/${file/classic_/}"; done
 for file in classic/*; do if [ -f "$file" ]; then sed -i -e 's/Classic//g' "$file"; fi; done
 for file in classic/internal/*; do if [ -f "$file" ]; then sed -i -e 's/Classic//g' "$file"; fi; done
 for file in le/*; do if [ -f "$file" ]; then sed -i -e 's/Le//g' "$file"; fi; done
 for file in le/internal/*; do if [ -f "$file" ]; then sed -i -e 's/Le//g' "$file"; fi; done

Using IDE:
 replace "namespace l2cap {" with "namespace l2cap { namespace classic {" in classic/
 replace "}  // namespace l2cap" with "} }" in classic/
 replace "namespace l2cap {" with "namespace l2cap { namespace le {" in le/
 replace "}  // namespace l2cap" with "} }" in le/
 replace "l2cap/classic_" with "l2cap/classic/" in classic/
 replace "l2cap/internal/classic_" with "l2cap/classic/internal/" in classic/
 replace "l2cap/le_" with "l2cap/le/" in le/
 replace "l2cap/internal/le_" with "l2cap/le/internal/" in le/

Bug: 140938432
Test: bluetooth_test_gd
Change-Id: I29eafefc5b6be4033e5db811776230d28ca312b2

Merge "PDL: Add GetStructOffset and GetStructSize"

Merge "L2cap dynamic channel and service"

Merge changes I62125269,Icbe89252

* changes:
  PDL: Use Size for StructField sizes
  PDL: Pass the size to GenBounds

Merge "PDL: Add for_struct to GenExtractor"

Merge "Update WriteInquiryScanActivity"

Update WriteInquiryScanActivity

Test: Compiles
Bug: 142061208

Change-Id: I969e94e393812528ec4ba38a87b4055bc42c67d3

Merge "Add Read Number of Supported Iac complete"

Merge "Add Read Page Timeout Complete View"

PDL: Add GetStructOffset and GetStructSize

Structs are parsed all at once, so they don't need as much
information as packets.

Bug: 138260498
Test: bluetooth_packet_parser_test
Change-Id: I49d433332874514d2cc1c5779865437fab5533b4

PDL: Use Size for StructField sizes

Structs can have dynamic sizes, which need to be passed in.

Bug: 138260498
Test: bluetooth_packet_parser_test
Change-Id: I621252699b597e728ff1566b95babd98cf00e8d4

PDL: Pass the size to GenBounds

There are different sizes, depending on where GenBounds is called.

Bug: 138260498
Test: bluetooth_packet_parser_test
Change-Id: Icbe892529919fdad0f0b98a3b66a2b24cb25f910

PDL: Add for_struct to GenExtractor

Enable field-specific size checks when parsing structs.

Bug: 138260498
Test: bluetooth_packet_parser_test
Change-Id: I3a8d132d1277fda027d5a6df61cdfb55b5519e02

Legacy portion of btm shim layer

bug: 141963466
Test: Boots into old stack; no crash in new stack

Change-Id: I021a7f2f446a17bc4f1c1fbbd36cf1227ca6705e

L2cap dynamic channel and service

Add internal implementation for ClassicDynamicChannel
Signalling packets will be handled in next commit

Test: cert/run_cert.sh and bluetooth_test_gd
Bug: 138260719
Change-Id: I3b0fee9837034988e6490372ffbfd5ece90250f7