Cert: Add name to gd_device

Bug: 145638034
Test: ./cert/run --host
Change-Id: Ib4cb22fa582dfe503c6a0e755614291a39196c4a

Cert: Use setuptools to provision GD cert tests

* Create setup.py that implements setup() from setuptools to
  provision GD cert tests in an enviornment
* Change makefile configuration such that both GD tests and the
  latest ACTS framework is packed in the same file. This helps
  prevent version mismatch between GD cert tests and ACTS framework
* Output a single bluetooth_cert_tests.zip that include above packages
  instead of two separate ZIP archives
* Running "gd/setup.py install" will install both GD tests and ACTS
  framework and it is recommended to do this in virtual environment
* Package target binaries into generic directory name "target/" so that
  GD cert tests do no need to know the name of the product being tested
* Find GD root directory via its relative with cert library directory
  so that we do not rely on os.getcwd()
* Added two more flags to gd/cert/run to allow custom test config,
  and the ability to re-use existing virtualenv
* Generate python file must be in a files sub-directory to avoid
  packing the resultant zip archive in itself

Test: source gd/cert/run --host
Bug: 150321998
Change-Id: I34b05b0ad121c8df35207f437a0326b52a4645e2

Merge "LeL2capTest add L2CAP/LE/CFC/BV-21-C"

Merge "Ignore legacy scan response event if no ADV_IND or ADV_SCAN_IND is received before"

Merge "SM: Reset fixed channel service pointer so we could continue"

Ignore legacy scan response event if no ADV_IND or ADV_SCAN_IND is received before

Test: Above Two devices start LE scan, and one device advertising
Bug: 146474336
Change-Id: I76c22705326db14df527d090151ea7bcd34615a2

Randomize the time between rotations of BLE Random Address

Bug: 150394435
Bug: 151485516
Change-Id: I342f3e34f12a200db0a5c3aab23efabac5626825

Add helper for converting between legacy and gd Address

This also fixes issue where the byte order in address is reversed

Bug: 149757450
Change-Id: I5c805feb6ca6d6780533c02545c8b548f497ba6b

Merge "Notify remote name failed for LE device"

Fix bugprone-use-after-move warnings

Bug: 150783499
Test: WITH_TIDY=1 make
Change-Id: I004a4084f2d7ab1a1f2908e930a17527d3b0d0bf

LeL2capTest add L2CAP/LE/CFC/BV-21-C

Bug: 145707677
Test: cert/run --host
Change-Id: Idb27c720510e656adffef0e805fb1fb6f3da73ff

Merge changes I5c0eea0d,I27dcb047

* changes:
  LeL2capTest: Get incoming packet stream for each channel
  LeL2cap Cert: Allow DUT initiate connection request

LeL2capTest: Get incoming packet stream for each channel

Bug: 145707677
Test: cert/run --host
Change-Id: I5c0eea0de649609e0b431ef0d30c1e2570e51ab1

Notify remote name failed for LE device

When cancel discovery for LE device in BTA_DM_DISCOVER_ACTIVE state,
there is no notification to indicate the remote name request of LE
device is cancelled and change the search state back to
BTA_DM_SEARCH_IDLE state. Call btm_inq_rmt_name_failed_cancelled()
to notify in this situation.

Bug: 146840780
Test: Manual
Change-Id: I1ed7b6ac3044f7ce19fc803f7ed5222f237fdc52

Merge "Fix bond list mismatch between Java and native"

LeL2cap Cert: Allow DUT initiate connection request

Add test cases for testing DUT to initiate COC connection request. Fix
some bugs with LE signalling manager and link.

Test: cert/run --host
Bug: 145707677
Change-Id: I27dcb047cecee219793ccbba3d735edc5fab4551

L2cap Signalling manager: Fix OnCommandReject

Checking command_just_sent_.command_code_ !=
command_reject_view.GetCode() doesn't make sense because
command_just_sent_.command_code_ is never CommandReject, but XyzRequest,
where command_reject_view.GetCode() is always CommandReject. We
shouldn't check it here.

Bug: 145707677
Test: cert/run --host
Change-Id: I424a8f715ef49372762c6df870d9c63de3b8c597

Add nullptr guard to make static analyzer happy

Change-Id: I95c4954eaea0215f2da00682e4b831eafa6ad506

Merge "LeL2cap: Add PTS test cases and allow verify from DUT"

LeL2cap: Add PTS test cases and allow verify from DUT

Bug: 145707677
Test: cert/run --host
Change-Id: I028566a37491c06bdc89864751e83844a33a894e

Merge "Save reversed BR/EDR link key derived from LE LTK"

Merge "L2capTest ICS 2/27"

L2capTest ICS 2/27

Test: cert/run --host
Bug: 144770889
Change-Id: Ib1207d79448be85ddc2dffb2699e709f29ff5e06

Merge "BluetoothMetricid: Add metric allocator api in java"

SM: Ensure we cannot cancel a pairing handler twice.

The callback for a ClassicPairingHandler is a OnceCallback,
so we only want Cancel to call it Once.

Bug:
Test: bluetooth_test_gd
Change-Id: I8c79b93fcce87d3f5197401ac25a5b96d1255dd7

SM: Reset fixed channel service pointer so we could continue

Bug:
Test: bluetooth_test_gd
Change-Id: I326ee8f8206d139fadb4585a47832e2a0416dd4a

Merge "LeL2capTest: Add COC PTS test"

Fix bond list mismatch between Java and native

When a paired device is diconnected while authenticating, do not report
bond state change to Java since the stack does not remove the device in
this situation.

Bug: 142842685
Test: Manual
Change-Id: I308320b0c10387d1cce27f709c8b2348c14dc423

HFP: fix collision for the outgoing and incoming connection

Since the HF incoming connection and the HF outgoing connection use the
same HF control block (p_scb) at the same time, the case will cause the
btif_hf_cb flag going wrong due to collision. Then it will trigger a
crash due to a check failure when executing BTA_AG_OPEN_EVT. The patch
will ignore the outgoing connection when the connection
collision is happening.

Bug: 143128949
Test: manual

Change-Id: Id5495bd3ecb1ed9e03de5bd74a61ab102604eaae

LeL2capTest: Add COC PTS test

Bug: 145707677
Test: cert/run --host
Change-Id: I8ddee9be99deb535dea97bbcc6cb33c83ab55266

BluetoothMetricid: Add metric allocator api in java

Bug: 142563377
Test: Manual atest AdapterServiceTest

Change-Id: I46d1c1d5bf0c8beea1f46b1024cbd5ceb9c0b562
Merged-In: I46d1c1d5bf0c8beea1f46b1024cbd5ceb9c0b562

LeL2capTest initial commit

Test: cert/run --host
Change-Id: I7e1aea927a63fa4b84774ef213c7724bd1db2202

Merge "PDL: Always compare size to base struct"

Merge "Clear LE cache before starting le scan or the report is skipped"

PacketParser: ToString() method for PacketViews

This is a helper method that should make debugging easier.

Bug: 150669615
Test: added
Change-Id: I15aceaa6a5b7ef628bbc98d9a1ef37950ebc22a2

L2capLe: Expose dynamic channel manager API

Test: bluetooth_test_gd
Change-Id: I4c17b257bcfc517d249db30ab6f29f4b9f4df468

Fix L2capTest:test_continuation_flag

Test: cert/run --host
Change-Id: I8bab6d3d6f49ca5c27991e6d140b5e2710e25569

PDL: Always compare size to base struct

Fixes: 146523920
Test: bluetooth_test_gd
Change-Id: I08f5023578ab171a99291eee5444d470370e272a

Merge "L2CAP: Add test for L2CAP 2/2 Support of configuration process"

Merge "btsnoop: Fix btsnoop_net runs unexpected"

Merge "A2DP: SBC encoding counter errata"

Merge "A2DP: AAC encoding counter errata"

btsnoop: Fix btsnoop_net runs unexpected

By default, btsnoop_net was disabled using "BT_NET_DEBUG" macro.
But, it enabled actually and manages socket connection for sending
btsnoop logs. The reason was, TRUE/FALSE macro were not defined.
This fix including "bt_types.h" to work macro properly.

Test: Turn on btsnoop logging and observe btsnoop_net runs or not.

Signed-off-by: KimJeongYeon <jeongyeon.kim@samsung.com>

A2DP: SBC encoding counter errata

When the encoder interval is 20 milliseconds per tick for {44.1 kHz /
16 bits per sample / Stereo}, there were around three microseconds shift
every tick, but would cause one SBC frame mismatched after 20 seconds.
Here try to adjust the timeline to increase the accuracy by following
methods:

1. add the remainder back to the timeline when converting the time
   period to data size.
2. change the timestamp to be presented by values in 1/10 microseconds.

Bug: 149546181
Test: manually
Change-Id: Ic42b2f331628c44aa927c7b9e35562be1fa750c7

A2DP: AAC encoding counter errata

When the encoder interval is 23 milliseconds per tick for {44.1 kHz /
16 bits per sample / Stereo}, 3 microseconds shift every tick would
cause one AAC frame mismatched after 180 seconds. Here try to adjust the
timeline to increase the accuracy by following methods:

1. add the remainder back to the timeline when converting the time
   period to data size.
2. change the timestamp to be presented by values in 1/10 microseconds.

Bug: 149546181
Test: manually
Change-Id: I16742042ba64a9355c39c0248be9208567312a9f

Merge "Limit fuzzed L2CAP packets to 64K"

Limit fuzzed L2CAP packets to 64K

The maximum ACL packet size is 64K, so fuzzing beyond that for L2CAP
is not worthwhile.

Tested with packets smaller than the limit to verify that the
failure only occurs with packets beyond the limit.

Bug: 146869530
Test: bluetooth_gd_fuzz_test \
      clusterfuzz-testcase-minimized-bluetooth_gd_fuzz_test-5672164365697024
Change-Id: I1361fb4904d6c1a55caad63bf65c67bef0207362

L2CAP: Add test for L2CAP 2/2 Support of configuration process

* Add handle for configuration request with invalid mtu option.
  Negotiate with kDefaultClassicMtu.
* Add handle for configuration response with result unacceptable
  parameters. Negotiate with no options.

* Add test case in l2cap_test:
    test_continuation_flag [L2CAP/COS/CFD/BV-01-C]
    test_configuration_negotiation_with_reject
      [L2CAP/COS/CFD/BV-02-C]
    test_send_requested_options [L2CAP/COS/CFD/BV-03-C]
    test_configuration_negotiation_with_unsupported_paremeter
      [L2CAP/COS/CFD/BV-11-C]
    test_unknow_option_response [L2CAP/COS/CFD/BV-12-C]

Bug: 149962814
Test: run_cert_facade_only.sh
Change-Id: I48a026cc203af2aa9067908b8bc419bc14e509ce

Use GD style logging through whole stack

This make it easier to copy code. Also if we ever decide to expose
legacy code to GD, having same logging format will make things easier.

Test: compilation
Change-Id: I21ac96e1be1baf4c2633372e9be38b9f4cf74199
Merged-In: I21ac96e1be1baf4c2633372e9be38b9f4cf74199

Merge changes I44f2e73a,I1755c0c4,Iee69245a,Ie7f4e084

* changes:
  Cert: make test setup more robust to binary failures
  Cert: use "git checkout" instead of "git co"
  Cert: Add required host libraries to test ZIP
  Cert: Handle None filter function in FilteringStream

Cert: make test setup more robust to binary failures

* Add checks to see if a backing process or root-canal is alive
  immediately after launching them to avoid making the whole
  test stuck on a response from these dead processes
* Add methods to disable verity on test devices and push test files
  to required directories on the Android device

Bug: 150321998
Test: source gd/cert/set_up_and_run_device_cert.sh
Change-Id: I44f2e73aa88e7924c7e594faad020f8199c76994

Cert: use "git checkout" instead of "git co"

* Also serach for dalvik instead of out dir

Test: source cert/set_up_and_run_device_cert.sh
Change-Id: I1755c0c46ef79ad6b92be438f2380f3be54e542c

Cert: Add required host libraries to test ZIP

* So that this ZIP archive contains all files we need to run a test
  in a separate environment

Bug: 150321998
Test: source cert/run --host
Change-Id: Iee69245a94adadbb551b0b6bbcdd04375787bb81

Cert: Handle None filter function in FilteringStream

Test: source cert/run --host
Change-Id: Ie7f4e084494f0dad6b5b22ec75fe0e2933b361c4

Merge "HCI: Add packet definitions from 5.2 spec part 2"

Merge "L2capTest: Remove unused classic facade"

HCI: Add packet definitions from 5.2 spec part 2

Bug: 150669615
Test: compilation
Change-Id: I3cf18b118b94fd158d5086cffcbcaef8b1a60336

Merge "BluetoothMetrics: Implement and test metric_id_allocator and lru"

L2capTest: Remove unused classic facade

Test: cert/run --host
Change-Id: Id60428b7491123de02b8dff17d28c7e2f6a6d94d

Merge stage-aosp-master to aosp-master - DO NOT MERGE

Change-Id: I7a18fc2613aeb4a0a8b6d5e81b1985bfaa117540

OS Queue and Reactor: Minor fixes

* When deleting EnqueueBuffer, unregister if needed
* Add a mutex to Reactor::WaitForUnregisteredReactable to prevent from
potential race
* EnqueueBuffer: Use an atomic_bool to synchronize Register/Unregister

Bug: 150174451
Test: cert/run --device
Change-Id: I7abcc2f763f8f0575dc0c67141571904c9a20136

Merge "Cert: Unzip all cert files into the same virtual environment" am: d03c48a520

Change-Id: Iaeded4e112ad91e69ac8c040490256193bd66741

Merge "Cert: Unzip all cert files into the same virtual environment"

HCI: Add packet definitions from 5.2 spec am: 2800f01798

Change-Id: Ib944a6ac4c9274bd5c9db18d6fa00d8b88ea5eab

Cert: Unzip all cert files into the same virtual environment

* All Python import must root from system/bt/gd
* Include all .py files in system/bt/gd in bluetooth_cert_test.zip
* At runtime, unzip all cert files into the same bluetooth_cert_test_env
  folder so that generated files and developer written files are
  overlayed together
* This helps avoid duplicate module problem at runtime

Bug: 150321998
Test: source cert/run --host
Change-Id: I7a8b1fe9fbccae1b1b4e8bf31a759a1c1eb7475c

HCI: Add packet definitions from 5.2 spec

Bug: 150669615
Test: compilation
Change-Id: I7d7bbfa93fd0f3df3c4df35881ab53f3a6ea9c51

Fix L2CAP Cert and AclManagerFacade am: 5b735ba9d4

Change-Id: I73665cb46e8a8844fcb7b4a39957bfc5dc72aef4

Fix on device L2capTest am: 6d0c1f93a5

Change-Id: Ibf0a696bd3544372909383f880d24e430ecbec3a

HCI: Synchronize ACL Queue am: b708a906f3

Change-Id: I83d72c5e88f455d780deb030b6cf45641f07eaf4

Fix L2CAP Cert and AclManagerFacade

* Store RetransmissionAndFlowControlMode from remote config request
* Send correct config request from cert
* SendAclPacket: Hold lock until packet has been dequeued to avoid
  double RegisterEnqueue

Test: cert/run --device
Bug: 150174451
Change-Id: I2c31f336a19d4a3e1e05efb56c20b9e4dbdd98a7

Fix on device L2capTest

* Synchronize between Queue callback context and module handler context
* Synchronize between Stop() and Queue callback context
* Fix DataPipelineManager channel registration logic with
  SignallingManager

Bug: 150174451
Test: cert/run --device
Change-Id: If58dcb5b0cc400f9c9ed1bbc6ec2c01de52789b2

HCI: Synchronize ACL Queue

* Synchronize Queue registration with Module Handler and Stop()
* Synchronize UnregisterCompletedAclPacketsCallback() as this is invoked
  during Stop()
* Add a timeout in AclManagerFacade to prevent from deadlock

Bug: 150174451
Test: cert/run --device
Change-Id: I3a476af54aaa57e25a7fd51089b44954ae574dc6

Minor improvements to Module, Reactor, StackManager am: 25ab0d1206

Change-Id: Iadc2bd7a1826cae5c808cd8e3c65b12beb4cedee

Fix NeighborTest am: 7ff79a3e50

Change-Id: Id4d0978e1cfd5587f120c6f7615874fec5463dcc

Minor improvements to Module, Reactor, StackManager

Add some logs. Fix the timeout in StackManager because some systems are
slow.

Test: bluetooth_test_gd
Change-Id: Idd3f0b14fd66dde250eadc84c1d5cad10664154d

Fix NeighborTest

Test: cert/run --host
Change-Id: I2c7296817e95cc2711360f310515afb1d4f7d909

BluetoothMetrics: Implement and test metric_id_allocator and lru

Test: run atest on unittests
Bug: 142563377

Change-Id: I05739258479a67d47550a6cb1db1c6f8c1ff3e98

Merge "Cert: Do not use ANDROID_ build environment variables in test" am: e9c391e7b8

Change-Id: I861b9ad8f674d45feb1071444fcb4458cae41365

Merge "Cert: Do not use ANDROID_ build environment variables in test"

Merge changes I099e53e6,Icab0e0d7,Ic8b73158,I989b35eb,Ie3bf7c90, ... am: cc80ee299b

Change-Id: I72adfd873b9794c41ac5f3c3ef21c8a892617df4

Merge changes I099e53e6,Icab0e0d7,Ic8b73158,I989b35eb,Ie3bf7c90, ...

* changes:
  Assorted cleanup
  Simplify disconnect for cert channels
  Simplify sending information requests
  Add information response matchers
  Move dut & cert channels into return result
  Use multi-match in a few places where it makes sense
  Add function to send s_frames & clean up sending
  Rename L2capMatchers.InformationFrame & SupervisoryFrame
  Distingiush between validated and unvalidated channels open
  Simplify channel open, so it can fit on one line
  Migrate i frame sending into CertL2cap
  Start migrating to channel abstractions
  Make ertm & fcs separable, reduce redundant code
  Add function to simplify sending control packets

Cert: Do not use ANDROID_ build environment variables in test

* Various environment variables should come from the build artifacts
  instead of the running environment.
* Hence getting rid of varioud usage of ANDROID_* variables in test
  files and figure them out through build artifacts
* Generate environment_provider.py at build time to provide
  build specific variables to the test at run time

Test: source cert/run
Bug: 150321998
Change-Id: I783c3784ad6018cb807a46701675fa5307e39175

Queue: Wait for unregistration if on different thread am: 4813536675

Change-Id: Iee6c21432077bcd345723efdbe59432fbac88db6

Assorted cleanup

Hide methods that are only used internally, remove unused ones, etc.

Test: cert/run --host --test_filter=L2capTest
Change-Id: I099e53e6b9f1d229fdc1ee14b30d66e27c2c7a09

Simplify disconnect for cert channels

Test: cert/run --host --test_filter=L2capTest
Change-Id: Icab0e0d7e356ae228c3125e452c93ad057be6074

Simplify sending information requests

Test: cert/run --host --test_filter=L2capTest
Change-Id: Ic8b731582ad2087922c18c06ad40e0319ae80b99

Add information response matchers

Test: cert/run --host --test_filter=L2capTest
Change-Id: I989b35eb6ed05f1d845d332e238a44eb136f966a

Move dut & cert channels into return result

Simplfies access in tests (no self. prefix) and
is also less like global variables.

Test: cert/run --host --test_filter=L2capTest
Change-Id: Ie3bf7c9039a6018f16a587dab542b3d8f88781cc

Use multi-match in a few places where it makes sense

Test: cert/run --host --test_filter=L2capTest
Change-Id: I8fa955ef15c36d8b3c91c9a7d97289741d08fa96

Add function to send s_frames & clean up sending

Test: cert/run --host --test_filter=L2capTest
Change-Id: I3bc7d2f0489a8a99febbc7bf746f355a2c77223d

Rename L2capMatchers.InformationFrame & SupervisoryFrame

Use IFrame and SFrame, to match the sending functions on channel
Test: cert/run --host --test_filter=L2capTest

Change-Id: I4d75e59b570a32ad2d8005a1365156025b2f4416

Distingiush between validated and unvalidated channels open

Since the common case is a validated channel, push the response
validation into _open_channel so we don't need to repeat it
in the test function.

Create _open_unvalidated_channel, so tests that need to do
custom validation off the common path can do so.

Test: cert/run --host --test_filter=L2capTest
Change-Id: Ic4c7d74109c23486e0862b7eca38f62cb8469228

Fix RepeatingAlarmTest am: b0e3826ac4

Change-Id: Ib3a6ea5594602feffbc4590d21c55b4dc82c2e22

Queue: Wait for unregistration if on different thread

Before Queue object is deleted, we must make sure that the enqueue or
dequeue callback isn't executing. If on same thread, callbacks are
synchronized. If on different thread, we must wait for unregistration to
synchronize callbacks.

Test: bluetooth_test_gd
Bug: 150174451
Change-Id: Id3c980aa0bf7bd9fa10c33c5cca3751df38f7d97

Fix RepeatingAlarmTest

We may not be cancel the alarm within 1 millisecond for some machine.
Make it a bit longer.

Test: bluetooth_test_gd
Change-Id: Ia0eae89b8022d41281db55c152dbb748332f2dee

Simplify channel open, so it can fit on one line

Test: cert/run --host --test_filter=L2capTest
Change-Id: I377a313008465b725991433158e5c828c8568cf3

Migrate i frame sending into CertL2cap

Test: cert/run --host --test_filter=L2capTest
Change-Id: I4af9e1576741b77d8cd24aa4852ee868660be688

Merge "Start CertL2cap, an abstraction to make L2cap testing simpler" am: a9017fae2b

Change-Id: I3390b0353ab77967be2eb854f007fa4f6d41e052

Merge "Start CertL2cap, an abstraction to make L2cap testing simpler"

Merge "SecurityModule: Restructure L2CAP connections" am: 5f7aca01ef

Change-Id: I2b931ac4509299b3a048565edd8471ab81b16034