Use proper type tHCI_REASON btm_get_acl_disc_reason_code am: 0ef9e3edd3

Original change: https://android-review.googlesource.com/c/platform/system/bt/+/1642944

Change-Id: Ie39349b97e05ba4d12d0c291c725f6ee76c53d12

Use proper type tHCI_REASON btm_get_acl_disc_reason_code

Toward readable code

Bug: 163134718
Test: gd/cert/run
Tag: #refactor
BYPASS_LONG_LINES_REASON: Bluetooth likes 120 lines

Change-Id: I7b2adfaa55feb96c620adfa5a6cb997055f1ce10

Merge "Use whole_static_library for cxx library deps." am: 06191f0dd0

Original change: https://android-review.googlesource.com/c/platform/system/bt/+/1654111

Change-Id: I46e9d811ca25cefc855e6f7c7daef004c2bbc545

Merge "Use whole_static_library for cxx library deps."

Merge "Fix for HID disconnect bug" am: 98d8c6c190

Original change: https://android-review.googlesource.com/c/platform/system/bt/+/1642954

Change-Id: I93783ac9d210ca5021058d558da8d38127d3db0e

Merge "Fix for HID disconnect bug"

Merge changes I6b35e11d,Iacec1d37 am: f1e6b3dc73

Original change: https://android-review.googlesource.com/c/platform/system/bt/+/1653715

Change-Id: I81ba01d162c9b9829acba92dfb2a934b8d2a9d7b

Merge changes I6b35e11d,Iacec1d37

* changes:
  HCI: Send Debug packet when HCI timeout
  HCI: Add support for Debug packet

Add tBTA_HH_DEV_DSCP_INFO::ToString am: e1a5fbcb6a

Original change: https://android-review.googlesource.com/c/platform/system/bt/+/1642953

Change-Id: I851ef49d319feac4076a3dd8c3ec49bb817d49d6

Fix for HID disconnect bug

Callback to btif layer to clear state and inform about
HID host profile disconnecting a peripheral.

Bug: 170187650
Test: gd/cert/run
Tag: #refactor
BYPASS_LONG_LINES_REASON: Bluetooth likes 120 lines

Change-Id: I17cefb6e0c641ccb4517479e6b358c80e83f1ee7

Merge "Re-log btm_sec_l2cap_access_req_by_requirement" am: 6e0b0d1769

Original change: https://android-review.googlesource.com/c/platform/system/bt/+/1646051

Change-Id: I4ef9e92a016cba1fd1145f4660fcc683f0ce86a6

HCI: Send Debug packet when HCI timeout

Bug: 183061492
Test: cert/run
      atest bluetooth_test_gd
Tag: #gd-refactor
Change-Id: I6b35e11dd802138469b0024611113eb822b08637

HCI: Add support for Debug packet

Bug: 183061492
Test: cert/run
      atest bluetooth_test_gd
Tag: #gd-refactor
Change-Id: Iacec1d37a95514d23f90a26f42808d627d2c6b0f

Add tBTA_HH_DEV_DSCP_INFO::ToString

Toward loggable code

Bug: 163134718
Test: gd/cert/run
Tag: #refactor
BYPASS_LONG_LINES_REASON: Bluetooth likes 120 lines

Change-Id: Ic3cdd525b544f593634b9b233722a40f6c10b840

Merge "Re-log btm_sec_l2cap_access_req_by_requirement"

Merge "RootCanal shouldn't depend on common" am: 6913198d02

Original change: https://android-review.googlesource.com/c/platform/system/bt/+/1653108

Change-Id: Idfb8ed8ad8144c548ef5de669ac80a4d76b5dcf4

gd_acl: Ensure acl is running prior to shutdown am: b3cf7b2f47

Original change: https://android-review.googlesource.com/c/platform/system/bt/+/1650647

Change-Id: Ieb602ee9bb6c3dc52b73b01049e989005ca7d34a

Merge "RootCanal shouldn't depend on common"

Re-log btm_sec_l2cap_access_req_by_requirement

Toward loggable code

Bug: 163134718
Test: gd/cert/run
Tag: #refactor
BYPASS_LONG_LINES_REASON: Bluetooth likes 120 lines

Change-Id: I39b877969a38622ec8dffed39ea8c531afab88af

Use whole_static_library for cxx library deps.

static_library has reverted to the old behavior of not being bundled
into rlibs, however whole_static_library has been added to support that
behavior.

Bug: 183182230
Test: mma
Change-Id: If0166b1347b005c03e9e1e1c2805513c0ec245d3

gd_acl: Ensure acl is running prior to shutdown

Synchronize stack shutdown

Bug: 176960731
Test: bluetooth_test_gd --gtest_filter=AclManagerLifeCycleTest.*
Test: gd/cert/run
Tag: #refactor
BYPASS_LONG_LINES_REASON: Bluetooth likes 120 lines

Change-Id: I8a2a321f9605e26da8ce340badc87e2f7208209d

GD HCI: Pipe VSEs up in shim layer am: e53576837a

Original change: https://android-review.googlesource.com/c/platform/system/bt/+/1653711

Change-Id: Ida91f1ca6a867d186c6badb3812c7b5120db590c

RootCanal shouldn't depend on common

Bug: 180413087
Test: cert/run
      atest bluetooth_test_gd
Tag: #gd-refactor
Change-Id: I743af9de1b816a485d4fb8e4c242d7beebd2eb49

GD Metrics: Check if RawAddress is null before converting am: 6ffa090b31

Original change: https://android-review.googlesource.com/c/platform/system/bt/+/1653710

Change-Id: I1092a84129bd0a0dae7ff99fb4614f4ab57181c9

GD HCI: Pipe VSEs up in shim layer

* Register known Android VSEs in shim layer
* Pipeline them through data callbacks so that they will
  be handled by btu_hcif_process_event() after beging posted
  to main thread by post_to_main_message_loop()

Tag: #gd-refactor
Bug: 183057550
Test: make, manual test BQR events
Change-Id: I849f02eec35f739f27d16803cea639c5e1ee1147

Merge "GD HCI: Support vendor specific event in GD" am: c06e2b6b87

Original change: https://android-review.googlesource.com/c/platform/system/bt/+/1652690

Change-Id: I03103934a296aac6148a2bbde4ce6b903cd4877d

GD Metrics: Check if RawAddress is null before converting

Tag: #gd-refactor
Fixes: 183670698
Bug: 183670698
Test: make
Change-Id: Ibc46813a34db08ec899145e9d196d6bf0c96d897

Merge "GD HCI: Support vendor specific event in GD"

GD HCI: Support vendor specific event in GD

* Add a handler for vendor specific command
* Add registration and unregistration APIs
* LOG_ERROR when an unknown vendor specific event is received
* Add unit tests to validate the above functions

Tag: #gd-refactor
Bug: 183057550
Test: bluetooth_test_gd
Change-Id: I00d6d892f3d60a8909352061dcae45f92d47b91d

Merge "metrics: don't crash if `address` is NULL" am: 8d9c4170bd

Original change: https://android-review.googlesource.com/c/platform/system/bt/+/1649529

Change-Id: Ie4871353997995f48dde9ff97fe2348e2a7566b2

Merge "btaa: Add the method of onActivityLogsReady into btif interface" am: 28669d1376

Original change: https://android-review.googlesource.com/c/platform/system/bt/+/1646068

Change-Id: I1bf7612dfa0ce4e1783e7f23c48a821ed7a176d3

Merge "metrics: don't crash if `address` is NULL"

Merge "btaa: Add the method of onActivityLogsReady into btif interface"

Re-log btm_sec_check_upgrade am: f6cc9dc3f2

Original change: https://android-review.googlesource.com/c/platform/system/bt/+/1646050

Change-Id: I1fbb3a843fd3198ca93fd09b3ca0994f91ca3351

Remove dependency of the hci hal on activity attribution if the flag am: 57f5c8efda

Original change: https://android-review.googlesource.com/c/platform/system/bt/+/1650824

Change-Id: Ie84fccc97cca81d810b2934aa188f1c854349bcb

Re-log btm_sec_check_upgrade

Toward loggable code

Bug: 163134718
Test: gd/cert/run
Tag: #refactor
BYPASS_LONG_LINES_REASON: Bluetooth likes 120 lines

Change-Id: Ib26183b2b63fcd9f877d876bb1d5d6954d82768c

Remove dependency of the hci hal on activity attribution if the flag
value indicates it is not enabled. Adds activity attribution to gd
dumpsys.

Tag: #feature
Bug: 182283555
Test: Manual
Change-Id: I4c01dcda9d5a9b6e1273285fa32c95386329cf80

btaa: Add the method of onActivityLogsReady into btif interface

The method of onActivityLogsReady will be used to pass BT activity logs
to BTAA core module to JNI.

Tag: #feature

Bug: 183215497
Bug: 170315554

Test: m

Signed-off-by: Mingguang Xu <mingguangxu@google.com>
Change-Id: I4dfeece15cb04a1b01fd5ad59da3bbceccecbf91

Merge "Keystore 2.0: Remove references to Keystore 1" am: c66d329202

Original change: https://android-review.googlesource.com/c/platform/system/bt/+/1650821

Change-Id: I103252d62202759b97769b41ce903ae44103e2e0

Remove unnecessary transport param am: ea48c4a982

Original change: https://android-review.googlesource.com/c/platform/system/bt/+/1629066

Change-Id: Ia3064244b4454249e0ffc12835b9a2a3ff830770

Add API stacl/acl/acl to set default packet type bitmask am: 64c37aeab6

Original change: https://android-review.googlesource.com/c/platform/system/bt/+/1629065

Change-Id: Ied3416cdedde63d19c855d59a58aaa9650014e61

Merge "Keystore 2.0: Remove references to Keystore 1"

Keystore 2.0: Remove references to Keystore 1

Test: N/A
Bug: 171305684
Change-Id: I4a09c24e18bef4361dc4b75e6df665a9d38689f9

Remove unnecessary transport param

Can only set packets on classic/BR_EDR

Bug: 163134718
Test: gd/cert/run
Tag: #refactor
Change-Id: Icf38654478a36671775d34c716bb86f6636bbffe

Add API stacl/acl/acl to set default packet type bitmask

Bug: 182370398
Test: gd/cert/run
Tag: #refactor
BYPASS_LONG_LINES_REASON: Bluetooth likes 120 lines

Change-Id: Idc91e1dba6784d4989adcf10c4f94b468bd535c6

Handle dual mode device with same address in bta_dm_remove_device am: e25f9378b9

Original change: https://android-review.googlesource.com/c/platform/system/bt/+/1650170

Change-Id: Ia2ce0da1020fc005eb170ff9d4343b184a8ca4b2

Handle dual mode device with same address in bta_dm_remove_device

Tag: #refactor
Bug: 183065766
Test: manual
Change-Id: Iff1b1c640b96606fa2dcdf49d30d2fa9a50ff891

Merge "[LSC] Add LOCAL_LICENSE_KINDS to system/bt" am: 34e931422b

Original change: https://android-review.googlesource.com/c/platform/system/bt/+/1649539

Change-Id: I890c698dcb192b878274e064432f8305774971e6

Merge "[LSC] Add LOCAL_LICENSE_KINDS to system/bt"

Include new hci_role file am: c4e9d02ab1

Original change: https://android-review.googlesource.com/c/platform/system/bt/+/1629206

Change-Id: If887fe7dc784a1e3ae3801da07e54930692fe2e6

Include new hci_role file

Bug: 179120287
Test: gd/cert/run
Tag: #refactor
BYPASS_LONG_LINES_REASON: Bluetooth likes 120 lines

Change-Id: I22043f815a33ee0459a4a609080651ede5a60bbe

gd: Add Unregister::acl_manager classic/le am: eef2dfe4a5

Original change: https://android-review.googlesource.com/c/platform/system/bt/+/1648789

Change-Id: I693dd7918ab50b3c8e3c4c7cc1cedcd418d520b1

[LSC] Add LOCAL_LICENSE_KINDS to system/bt

Added SPDX-license-identifier-Apache-2.0 to:
  vendor_libs/test_vendor_lib/data/Android.bp

Bug: 68860345
Bug: 151177513
Bug: 151953481

Test: m all

Exempt-From-Owner-Approval: janitorial work
Change-Id: I71d5736f416cfcc807029047f6358a6558c81ad3

gd: Add Unregister::acl_manager classic/le

Synchronize stack shutdown

Bug: 176960731
Test: bluetooth_test_gd --gtest_filter=AclManagerLifeCycleTest.*
Test: gd/cert/run
Tag: #refactor
BYPASS_LONG_LINES_REASON: Bluetooth likes 120 lines

Change-Id: Iad2b6ce5f93f3aa3c9676179d9353cfa9f679fcd

BTMLogHistory: Add HID device open/close am: 4d4697462b

Original change: https://android-review.googlesource.com/c/platform/system/bt/+/1642952

Change-Id: I3f13463bead186fdad45cc18e7592c97ae2431f7

metrics: don't crash if `address` is NULL

We check earlier if `address != nullptr` in this function; it stands to
reason that we should have the same check here, as well.

Potential use of null was caught by the static analyzer:
> system/bt/gd/common/metrics.cc:74:9: warning: Called C++ object
pointer is null [clang-analyzer-core.CallAndMessage]

Tag: #security
Bug: 181819141
Test: TreeHugger
Change-Id: I4baf5fe648c8fd841875515b024e1658b7ea20cd

BTMLogHistory: Add HID device open/close

Toward loggable code

Bug: 163134718
Test: gd/cert/run
Tag: #refactor
BYPASS_LONG_LINES_REASON: Bluetooth likes 120 lines

Change-Id: I2201febbce720a2534800e7e9c69d31b0f8b0ced

Merge "GD metrics: Add shim layers for metrics in GD 1) Add shim layer for MetricIdManager 2) Add shim layer for metric logging functions who take metric id 3) reference the 2 shim layers at btif/, main/shim and stack/" am: ed1b5fd0d7

Original change: https://android-review.googlesource.com/c/platform/system/bt/+/1627280

Change-Id: I960b2403ed0f73e6852c3720d3e163dc36cb9912

Revert "gd: Add Unregister::acl_manager classic/le" am: bcece23223

Original change: https://android-review.googlesource.com/c/platform/system/bt/+/1648788

Change-Id: I73c5cbdc2b27e3b3f4733cd4758eb30615c807ae

Merge "rusty-gd: Add controller facade service." am: 9d623f3d2f

Original change: https://android-review.googlesource.com/c/platform/system/bt/+/1647427

Change-Id: Ifec14b8b163cf310e6b45daff8e08d4b7807416f

Merge "GD metrics: Add shim layers for metrics in GD 1) Add shim layer for MetricIdManager 2) Add shim layer for metric logging functions who take metric id 3) reference the 2 shim layers at btif/, main/shim and stack/"

Revert "gd: Add Unregister::acl_manager classic/le"

This reverts commit f0436493c05fe8d70375f461bc7e158b94f74e9b.
Bug:183349791
Tag: #refactor
Test: Builds
Reason for revert: broke

Change-Id: Idb2a6845b9740a6112eabf0e40985d0a972c2ea3

Merge "rusty-gd: Add controller facade service."

Merge "Make controller prop and default commands configurable in rootcanal" am: 5f74f42d4a

Original change: https://android-review.googlesource.com/c/platform/system/bt/+/1629269

Change-Id: I35d19d73b3fbca4874c540d0ee64727ebdd62980

Merge "Make controller prop and default commands configurable in rootcanal"

gd: Add Unregister::acl_manager classic/le am: f0436493c0

Original change: https://android-review.googlesource.com/c/platform/system/bt/+/1646056

Change-Id: Ica879daee72ad6756249a52190dd183df3fa71df

Special case -INT_MAX for hex conversion am: f0ba72bb42

Original change: https://android-review.googlesource.com/c/platform/system/bt/+/1646055

Change-Id: I3e784fed297f129f62498f49cb7013b768331801

gd: Add Unregister::acl_manager classic/le

Synchronize stack shutdown

Bug: 176960731
Test: bluetooth_test_gd --gtest_filter=AclManagerLifeCycleTest.*
Test: gd/cert/run
Tag: #refactor
BYPASS_LONG_LINES_REASON: Bluetooth likes 120 lines

Change-Id: Ic7a41bbcb01dd61c5db7f4326daa8dd6585cbe86

Special case -INT_MAX for hex conversion

Bug: 183171310
Test: bluetooth_test_gd --gtest_filter=StringsTest*
Tag: #refactor
BYPASS_LONG_LINES_REASON: Bluetooth likes 120 lines

Change-Id: I4150f4f6dc3445e4853bea51c68eee26e891e45a

Merge changes Ic1f5b589,I34e05959 am: 3ea5a36745

Original change: https://android-review.googlesource.com/c/platform/system/bt/+/1646072

Change-Id: I32dedbc0b37ce6bfa06eacfd6f742b109511bf37

Merge changes Ic1f5b589,I34e05959

* changes:
  PDL: Add more tests for C++
  GD: Include init_flags where used

Cert: OOB Test, verify different data. am: d06d953590

Original change: https://android-review.googlesource.com/c/platform/system/bt/+/1604953

Change-Id: I802f11f085ac3f04e867cae7c2767e0f87f4b3a3

Cert: OOB Test, verify different data.

This test ensures that the data fetched is always different.

Bug: 178007935
Test: cert/run SecurityTest:test_make_sure_oob_data_different
Tag: #gd-refactor
Change-Id: I1847eebdcead9e13cb4bc2d8eb86cdb63bcaf978

rusty-gd: Add controller facade service.

The ControllerTest pass with this change.

ControllerExports is currently not being updated as a result of
set_local_name. That will be handled as a follow up.

Bug: 171749953
Tag: #gd-refactor
Test: gd/cert/run --rhost ControllerTest

Change-Id: I1ceca00bac74451e915518288108ba149750dc6f

Merge "IBluetoothManager: Update createBondOutOfBand interface" am: bf1156d1c1

Original change: https://android-review.googlesource.com/c/platform/system/bt/+/1559439

Change-Id: I74b3a1e3e8208933e1e920b40091bf3b38e9e519

PDL: Add more tests for C++

Bug: 182216573
Test: atest bluetooth_test_gd
Tag: #gd-refactor
Change-Id: Ic1f5b5897451fcf31eb5b1b0b64426eb5ad2b1f7

Merge "IBluetoothManager: Update createBondOutOfBand interface"

GD: Include init_flags where used

Bug: 182586624
Tag: #gd-refactor
Test: cert/run
Change-Id: I34e05959b0167740f4e6c426c5f36c54d8d65c6e

GD metrics: Add shim layers for metrics in GD
1) Add shim layer for MetricIdManager
2) Add shim layer for metric logging functions who take metric id
3) reference the 2 shim layers at btif/, main/shim and stack/

Bug: 181819141
Test: atest bluetooth_test_gd
Manual test by printing log at legacy stack and gd stack to make sure
function calls, parameters and callbacks are fowarded correctly

Change-Id: Ic5ee1ef95a7e4d7683de328ac78894986ba5011e

Merge "Marking bluetooth_test_gd as not unit tests" am: 4ac7b735a2

Original change: https://android-review.googlesource.com/c/platform/system/bt/+/1624644

Change-Id: Iad8d903eb99963297e19982b5d0ee7ace31f78c3

Merge "Marking bluetooth_test_gd as not unit tests"

GD metrics: add logging definitions to GD am: 4676225d0f

Original change: https://android-review.googlesource.com/c/platform/system/bt/+/1638300

Change-Id: If17e107875050ae1274ca840543569dd1b01eeb0

GD metrics: add logging definitions to GD

Test: atest bluetooth_test_gd
Bug: 181819141

Change-Id: I1a49859c6ca93e196e95f6e230c15af094fae57b

Use proper types tHCI_ROLE am: 5b1000785f

Original change: https://android-review.googlesource.com/c/platform/system/bt/+/1629205

Change-Id: Iaa91062b8de2873ff48e26aab4e90aa24ee48865

Break out tHCI_ROLE type into separate file am: 1c1ddde827

Original change: https://android-review.googlesource.com/c/platform/system/bt/+/1629204

Change-Id: Ib79ff239fda650189859183d6a4722fe7f446c99

IBluetoothManager: Update createBondOutOfBand interface

Add plumbing for both p192 and p256 data types.

Ignore-AOSP-First: Conflicts with CL previously merged internal
Bug: 178007935
Test: atest net_test_btif
Tag: #refactor
Change-Id: I6b1424c22495821bfbbf56830f3848e6c03b74d4
Merged-In: I6b1424c22495821bfbbf56830f3848e6c03b74d4

gd_acl: No need to read remote version twice am: 505a752435

Original change: https://android-review.googlesource.com/c/platform/system/bt/+/1632994

Change-Id: Ib7ad3976f63abaa8a522c48d927ed79d5bf2a2bb

gd_acl: On classic connect issue read supported features am: 0fb0d8c33f

Original change: https://android-review.googlesource.com/c/platform/system/bt/+/1632993

Change-Id: Id6a93b24b69c725ddcf631f77ddf46eb0c599070

Convert bluetooth::shim::BtmStatus from 16 to 8 bits am: 9d9c9e1599

Original change: https://android-review.googlesource.com/c/platform/system/bt/+/1629203

Change-Id: Ib082a5c1260492ffad0963d94f08a6779e0131cc

Properly log btm_read_failed_contact_counter_complete am: 4169b42315

Original change: https://android-review.googlesource.com/c/platform/system/bt/+/1629202

Change-Id: Id6161fc000314372dffe6c111615ca921eb890f2

Properly declare tACL_CONN::tPolicy::Role() am: c7b63f3984

Original change: https://android-review.googlesource.com/c/platform/system/bt/+/1629201

Change-Id: Ic3789f24220ac5a5cde71dfcb2a072a67d238c18

Add stack/include/hcidefs::hci_role helpers and aliases am: 462f3c181c

Original change: https://android-review.googlesource.com/c/platform/system/bt/+/1629200

Change-Id: Ib88b74aed5af735f70034589c307513c22d73a87

Const-ify stack/acl/btm_acl::packet types defs am: 9e550caf1d

Original change: https://android-review.googlesource.com/c/platform/system/bt/+/1629199

Change-Id: I415f7f6d1d181d8bb49f2c9e38923df3b9705b53

tBTM_AUTH_COMPLETE_CALLBACK Specialize result to HCI_STATUS am: 4600433399

Original change: https://android-review.googlesource.com/c/platform/system/bt/+/1628904

Change-Id: Ie70644611bdca8d7ca935332685e05bdbfdf6b65

Move callback definitions to stack/include/security_client_callbacks am: a10232648d

Original change: https://android-review.googlesource.com/c/platform/system/bt/+/1628903

Change-Id: Ifff75fbcc6bd0e9a906ea8b054897a38d57ad334

Properly type tBTA_AV_STATUS am: 403162ff12

Original change: https://android-review.googlesource.com/c/platform/system/bt/+/1629519

Change-Id: I9eab9118d91cf3348e8f6d05d9c8a0262a9c85a4

Remove useless bta_dm_authentication_complete_cback return value am: 45dd8c98b3

Original change: https://android-review.googlesource.com/c/platform/system/bt/+/1628902

Change-Id: I2bb2485bdcb4492191e688abba0e5a5572dc7ee3

Use proper types tHCI_ROLE

Bug: 179120287
Test: gd/cert/run
Tag: #refactor
BYPASS_LONG_LINES_REASON: Bluetooth likes 120 lines

Change-Id: I6fd932fa39b4d7ea8b88db0877806a636a90deed

Break out tHCI_ROLE type into separate file

Used throughout stack

Bug: 179120287
Test: gd/cert/run
Tag: #refactor
BYPASS_LONG_LINES_REASON: Bluetooth likes 120 lines

Change-Id: I6a4cb56561d8afbb6715e7598903b0fc6428a950

gd_acl: No need to read remote version twice

Bug: 166280067
Test: Manual verification via wireshark
Tag: #refactor

Change-Id: I2296b99a2bbcfb3adb9585e6541fc21d726fd9eb