Merge "gd: l2cap classic fixed channel test"

Merge "Remove C++ cert stack"

Remove C++ cert stack

Test: cert/run_cert_facade_only.sh
Change-Id: I91dc5569fbb0226b1b8e18ac62113803eea70c3d

Merge "Refactor of headless test to allow additional tests"

gd: l2cap classic fixed channel test

cleanup EXPECTs

Bug: 146363262
Test: bluetooth_test_gd --gtest_repeat=1000 # x86
Change-Id: Ibb9838d0c1014a187168571a375282b7686e60da

Merge "L2capTest: Migrate all remaining test cases"

Merge "L2CAP: Add test for L2CAP 2/7 Support of command information response"

Merge "gd: Fix legacy storage test config file"

L2capTest: Migrate all remaining test cases

Test: cert/run_cert_facade_only.sh
Bug: 14987419
Change-Id: Ie5bba8e9ce19cb3244a6d319e6bff20999a29c2c

Merge "gd: Clean up bluetooth_test_gd"

L2CAP: Add test for L2CAP 2/7 Support of command information response

Add test case L2CAP/COS/IEX/BV-02-C in origin test case for
L2CAP/COS/IEX/BV-02-C test_query_for_1_2_features since these two can be
done in one connection setup procedure

Add following test scripts in pts_l2cap_test:
  test_L2CAP_IEX_BV_01_C
  test_L2CAP_IEX_BV_02_C

Bug: 149468215
Test: run_cert.sh
Change-Id: Icbc727b95b18316d33bddbe2690841ac4e1cc308

gd: Clean up bluetooth_test_gd

Bug: 146363362
Test: bluetooth_test_gd
Change-Id: I0f3fb81d2b79b781918d18ea89a3ed64574a9610

L2capTest: Migrate more test cases

And fix some logic

Test: cert/run_cert_facade_only.sh
Bug: 14987419
Change-Id: Id6780f788b88eb33025a08a9396af9a254548d31

Merge "L2CAP: Add test case for TSPC_L2CAP_2_7 Can Sent Reject"

Merge "L2capTest: Migrate basic ERTM test case"

gd: Fix legacy storage test config file

Should point to appropriate target-based location

Bug: 147315979
Test: bluetooth_test_gd # x86
Test: bluetooth_test_gd # arm

Change-Id: Ic021edb63d1d9b0ee7e5947c7e53e0384d34b9e2

link manager test

Clean up EXPECTs

Bug: 146363313
Test: bluetooth_test_gd # on device
Test: bluetooth_test_gd # native

Change-Id: I2398f46380207e4e996a5cf0a4a7ea9c6add9b22

L2capTest: Migrate basic ERTM test case

And fixed the logic in control packet handler

Test: cert/run_cert_facade_only.sh
Bug: 14987419
Change-Id: Ibd7592c749350a4ac694586cff2bb37295e4c398

Refactor of headless test to allow additional tests

Bug: 147316415

Test: bt_headless --device=40:4e:36:56:f3:9b --loop=1 pairing
Test: bt_headless --device=40:4e:36:56:f3:9b --loop=1 --uuid=0x110e sdp

Change-Id: I202c7af885d86fd77fffeec5a87b56ffe18882da

Merge "Fix call order for BTM shim RegisterInquiryCallbacks"

OH MY COD! Why is gabledorsche plural?

Link to a helpful explanation on why gabeldorsche and not gabeldorsch.

Change-Id: I978ac9ba88de0be42a3dde76eafeeca59b26b50e

Fix call order for BTM shim RegisterInquiryCallbacks

This currently causes crash on bringup of GD stack

Bug: 149757450
Test: start stack with GD enabled
Change-Id: Ie86d007dd9fcbab102b7c1e95c97ae5d9989fa29

Merge changes Id1de1279,I4fb13f4b,I1f27fffa,I6f650623

* changes:
  Get rid of HciLayer shim
  Prepare to remove HCI layer shim
  Get rid of Advertising shim
  Get rid of Inquiry shim

Merge "L2capTest: Add common event handler and migrate test case"

Merge "L2CAP Cert with facade only initial commit"

L2capTest: Add common event handler and migrate test case

Migrated L2cap tests to use facade only:
test_disconnect_on_timeout
test_respond_to_echo_request
test_reject_unknown_command

Test: cert/run_cert_facade_only.sh
Bug: 14987419
Change-Id: I809a95fbf148f3784377d05e630e69539f8f93ef

L2CAP Cert with facade only initial commit

Test: cert/run_cert_facade_only.sh
Bug: 149874197
Change-Id: I2ec34c19ee609ac8ef4f4da4cc8a9d66b6977543

Merge changes from topic "bt-avrcp-controller-cover-art"

* changes:
  Find and capture cover art BIP PSM and send it to Java/JNI
  Add cover artwork image handles to requested attributes objects
  Find cover art supported feature and send up to JNI
  Indicate support for controller cover art feature in SDP record

Add LeAclManagerTest, SimpleSecurityTest to dist

Test: m dist bluetooth_stack_with_facade
Change-Id: Icb8922b2432c9e02b9a231614051d4b9573ce3b2

Merge "Swap Cert and DUT index"

Get rid of HciLayer shim

Bug: 149757450
Change-Id: Id1de12794f3378301442dfab01e3857a5d1c9459

Prepare to remove HCI layer shim

Bug: 149757450
Change-Id: I4fb13f4b7d9a80540ce683eb13bf0e6da39b9fe6

Get rid of Advertising shim

Bug: 149757450
Change-Id: I1f27fffa5aca3bed420c9bfe4e1bb404460f3ffc

Get rid of Inquiry shim

Bug: 149757450
Change-Id: I6f650623e7f89ed8c75e0f4ddadc218c7f3c8b3c

Get rid of Name shim

Bug: 149757450
Change-Id: Ie273f88a1fa27a8fa6f748abcf864b080591cdaa

Merge "HCI: Remove unused ClassicSecurityManager API"

Swap Cert and DUT index

Cert is 0, DUT is 1

Test: cert/run_cert_facade_only.sh
Change-Id: I76231b8cbdb03e2f3a998c82eac859363a9da0d8

Merge "Get rid of NameDB shim"

Merge "SM: Add JustWorks cert tests"

Merge "AVRCP Controller manage active device"

Merge changes from topic "gd_sm_ui_wiring"

* changes:
  SM: Add UI Callbacks to CPH
  Security UI Callbacks wiring

Merge "Remove redundant BTA_AR_DEBUG define"

Get rid of NameDB shim

Bug: 149757450
Change-Id: I7d34028f71d333b8b12a8f8a3d164df365d66de5

Get rid of Scanning shim

Bug: 149757450
Change-Id: I8f41bb6d79324bab6dbe40ea5223f42431c57638

Merge "L2CAP: Add test scripts for pts test"

Merge "L2CAP: Update remote tx_window during channel configuration"

Remove redundant BTA_AR_DEBUG define

Bug: none
Test: compile
Change-Id: Id63e2f9a8881c16b2b48eeef2b941f7c0cde628b

SM: Add JustWorks cert tests

Add some classic JustWorks use cases as cert tests.

Bug: 145638034
Test: ./cert/run_cert_facade_only.sh
Change-Id: I56a1ba5fb42a97e8b87e5f8d87722a93bba0f08f

HCI: Remove unused ClassicSecurityManager API

Bug: 146363467
Test: bluetooth_test_gd
Change-Id: Ib8994f858161ce0ba5143cef49da755f5f5599df

SM: Add UI Callbacks to CPH

Test: bluetooth_test_gd
Change-Id: I8b7b587f720a897e416ed3abea0fde6ad6788338

Security UI Callbacks wiring

Test: bluetooth_test_gd
Change-Id: I713e4efd923359a36fc943362d59ae619dd515ca

Merge "Fix crash at unexpected code after root inflammation"

Get rid of Storage shim

Bug: 149757450
Change-Id: I90130121c5fdf23ec8cacf19fedd4b3e154351a5

Get rid of Connectability shim

Bug: 149757450
Change-Id: Idef691e97109e2634fc05fc7bddfeca72b967d11

Get rid of Page shim

Bug: 149757450
Change-Id: I9ce812831a0d8ff62772ea5b98ab4a7e646b162a

Get rid of Security shim

Bug: 149757450
Change-Id: Id7137d64059cf5d15cd94419255c4b1f051bee60

smp_generate_stk: reduce log level for key type information

Change-Id: I7499a3f507914db9cb9c062f4ad9cfccdb51f06b

Merge "gd: Unregister events at the hci layer"

Merge "Save bonding and pairing state to security record"

Get rid of Discoverability shim

Bug: 149757450
Change-Id: Ibe6cccc0971d4a723ba5489332ca7933e7609a34

Get rid of the redundant shim interfaces

It was agred to expose gd types into the system/bt/main folder.
All the interfaces already use pimpl pattern to make them testable.
Thus there is no need for the I* shim interfaces.

Test: updated tests
Bug: 149757450
Change-Id: I9662f9ab48c61b764e4df082da1a73483cb0b2b9

Save bonding and pairing state to security record

Bug: 142341141
Test: bluetooth_test_gd

Change-Id: I226d9f2f74e12d911fbb8e1f3d5924db7763bdaf

Canonicalize header include files

Bug: 147316415
Test: Compiles and Builds
Change-Id: I00316dba9cce8d727fc718250237e80ff87cdd00

Do not redefine logging if already defined

This patch lets us include gd code in system/bt/main. Currently if we
were to do that, we would get errors about logs being redefined.

Test: compilation
Change-Id: Ifc2c9eb913c311a50763a26f864dd91872c273cc

Merge "gd: Add shim access to name db"

Modify service_type constants to not collide with constants in GD

Test: compilation
Change-Id: Ieb9540a3c5ca390a1e04d20ae378d1ab06b33707

L2CAP: Add test case for TSPC_L2CAP_2_7 Can Sent Reject

Add following cert test in simple_l2cap_test:
  test_send_rej [L2CAP/ERM/BV-16-C]

Add following test case in pts_l2cap_test:
  test_L2CAP_ERM_BV_16_C

Bug: 149658425
Test: run_cert.sh
Change-Id: I4f160883d47d2294f88d98177813ec587a8c2e76

L2CAP: Add test scripts for pts test

Add following PTS test scripts:
  L2CAP/EXF/BV-01-C
  L2CAP/CMC/BV-01-C
  L2CAP/CMC/BV-02-C
  L2CAP/ERM/BV-01-C
  L2CAP/ERM/BV-02-C
  L2CAP/ERM/BV-03-C
  L2CAP/ERM/BV-08-C
  L2CAP/ERM/BV-09-C
  L2CAP/ERM/BV-10-C
  L2CAP/ERM/BV-11-C
  L2CAP/ERM/BV-18-C
  L2CAP/ERM/BV-19-C
  L2CAP/ERM/BV-20-C

Add cert tests:
  test_initiated_configurtion_ertm [L2CAP/CMC/BV-01-C]
  test_receive_i_frames [L2CAP/ERM/BV-02-C]
  test_acknowledging_received_i_frames [L2CAP/ERM/BV-03-C]
  test_transmit_s_frame_rr_with_poll_bit_set [L2CAP/ERM/BV-08-C]
  test_transmit_s_frame_rr_with_final_bit_set [L2CAP/ERM/BV-09-C]
  test_receive_s_frame_rr_final_bit_set [L2CAP/ERM/BV-18-C]
  test_receive_i_frame_final_bit_set [L2CAP/ERM/BV-19-C]
  test_recieve_rnr [L2CAP/ERM/BV-20-C]

Bug: 145784332
Test: run_cert.sh
Change-Id: If5b25880f731dbfbb0447a6edaccbbf3d8829d11

L2CAP: Update remote tx_window during channel configuration

* Update tx_window as transmit window in configuration request from
  remote.
* Fix logic of checking valid f bit, f bit should be valid only if:
  1. tx_state is in XMIT and f not set
  2. tx_state is in WAIT_F and f set to POLL_RESPONSE

* Add follwoing test cases in simple_l2cap_cert:
  test_resume_transmitting_when_received_rr
  test_resume_transmitting_when_acknowledge_previously_sent
  test_respond_to_rej
  test_handle_duplicate_srej
  test_handle_Receipt_rej_and_rr_with_f_set
  test_handle_rej_and_i_frame_with_f_set

* Add following test case in pts_l2cap_test:
  test_L2CAP_ERM_BV_05_C
  test_L2CAP_ERM_BV_06_C
  test_L2CAP_ERM_BV_13_C
  test_L2CAP_ERM_BI_03_C
  test_L2CAP_ERM_BI_04_C
  test_L2CAP_ERM_BI_05_C

Bug: 149182548
Test: run_cert.sh
Change-Id: I0828e570017b954980f23912937e36524814a0f6

gd: Unregister events at the hci layer

bt_headless with the pairing option fails

Bug:149524116
Test: blueooth_test_gd
Test: bt_headless
Change-Id: I65d9f928c6ff63e5ddde8380cae847d180876fac

Merge "PDL: Serialize nested packets in Python"

Merge changes Iea352d4b,Ic0737465

* changes:
  GRPC: Use server_stream_call.cancelled() instead of hidden API
  RootCanal HAL: Fixed race condition when tearing down rootcanal HAL

PDL: Serialize nested packets in Python

Bug: 148096951
Test: ./cert/run_cert_facade_only.sh
Change-Id: I2c54bb5d35dec9a9b5101165d2d1ccb22b116ef4

Fix crash at unexpected code after root inflammation

* Clean up the hci/startup timers after the receiving of root
  inflammation packet so they will not be scheduled again and
  expire before the abort_timer.
* Abort Bluetooth when HAL is dead with correct reason.

Bug: 144592765
Test: Manual
Change-Id: I1458defeebaac7951aa8c7f52decf5b6ca3ab4fc

gd: Add shim access to name db

Provides replacement for remote name request db
for legacy shim side.

Bug: 149524116
Test: bluetooth_test_gd

Change-Id: I69c8c65c01935ae371debedcef2d921cea7beb4a

Merge "Use mobly.asserts"

Merge "gd: Authenticate only if necessary"

Merge "gd: Add link layer collision hci error code"

Merge "gd: Add a remote name database"

Merge "gd: Ensure all events are handled for new classic pairing manager"

Merge "gd: Expand and log more acl events"

Merge "Set proper scan type before first discovery"

gd: Add a remote name database

Simple module to cache remote names.  Used by shim layer.

Bug: 149524116
Test: bluetooth_test_gd

Change-Id: I63e94e0ff1ea04c3ec35d502ac1de001a11f7b98

gd: Authenticate only if necessary

If a device are pairing but not bonding then
authenticate may get called twice, once when
the channel is set up and again when the
pairing manager is launched.

Bug: 149524116
Test: bluetooth_test_gd

Change-Id: Icddecd549030ade41e5679d46580881b56833d9e

gd: Expand and log more acl events

Bug: 146086425
Test: blueooth_test_gd

Change-Id: I8dc506eea0bbcc3a9310d758311f369a31993bfd

gd: Ensure all events are handled for new classic pairing manager

a new classic pairing manager never processes the initiating event.

Bug: 146086425
Test: bluetooth_test_gd

Change-Id: I3471bc5849315456d7429747e894cf80eda8b8cb

Merge "Update time stamp when new inq_db"

Merge "Remove some unused inquiry code and callback function"

Use mobly.asserts

Test: cert/run_cert_facade_only.sh
Change-Id: I3f19bba39953eca9c013aab8813610c3fbfb5f83

Get rid of UI callbacks from SecurityManagerListener

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

Merge "HCI: Set default event mask"

Merge "Cert: Move DUT assignment to common code"

Update time stamp when new inq_db

Test: Simulate a sitution with many LE devices around and start le scan
Bug: 146474336
Change-Id: I3a0f87786d0c7b13e4d85942ac0387fc334c26d4

Set proper scan type before first discovery

Set scan type when start first discover after bt enabled.
And replace adv evt hex code to define code

Test: Enable Bluetooth and then start discovery.
Bug: 146474336
Change-Id: I63c8cda5343de15381c83da4ea3afde6b75aaf6a

Remove some unused inquiry code and callback function

Test: manually
Bug: 146474336
Change-Id: I6d3a0d9066ef6a4fbc14f938db6f120147340804

Merge "Don't guess transport during pairing"

Security UI interface cleanup

We currently have two separate UI interfaces for LE and Classic, this
patch merges them into single one.

Bug: 142341141
Test: tests updated
Change-Id: I5fa347a1d4d50d85bb051782ca27f38f3d146239

Merge "Pass UI Into Security Manager"

GRPC: Use server_stream_call.cancelled() instead of hidden API

* Use server_stream_call.cancelled() to check if a stream
  is cancelled to avoid calling hidden APIs

Test: run cert tests multiple times
Bug: 148147248
Change-Id: Iea352d4b428b9b57b911576a44f7a91842613fc7

RootCanal HAL: Fixed race condition when tearing down rootcanal HAL

* One race condition exist when incoming_packet_callback_ is being
  reset while an incoming packet is being processed. This is fixed
  by adding a dedicated mutex incoming_packet_callback_mutex_
* Another race condition happen when HciHalHostRootcanal::Stop()
  tries to close the rootcanal FD while incomding_packet_callback
  is trying to read from it

Test: run cert tests multiple times
Bug: 148147248
Change-Id: Ic07374651c5ec38e90e7ddc545ca4fcd4a84cd04