TEST: Fix host native test script

* Use build/soong/soong_ui.bash to get needed environment variables
  instead of getting them directly from the parent shell
* Simplify the build command use to 'soong_ui.bash' directly so that
  we can remove the call to envsetup
* When ANDROID_BUILD_TOP is not set, try finding it by looking for
  build/soong/soong_ui.bash in all parent directories until found,
  and get TOP variable from it
* Similarly, for ANDROID_HOST_OUT, use the same soong_ui.bash script to
  get it, instead of relying on the environment variable
* Also fixed a bug in test log output by prepending DIST_DIR to the
  output file path so that it can be grabbed by the server

Bug: 64687730
Test: run run_host_unit_tests script in both
        1) vanilla environment without envsetup and lunch
        2) fulled setup and lunched environemt
      to verify
        1) If nothing is setup, the script picks a default environment
2) If envirnoment is already setup, use existing environment
   instead of creating new one

Change-Id: Ife28db509fe9af55cef26c7b480196efae39d3aa

Make copyright headers consistent with Google template; remove "(C)"

Test: Comment changes only; still compiles...
Change-Id: Id699a8170112f06e4a2c9f2e0f0834d1817ace4e

Merge "GATT discovery refactor"

GATT discovery refactor

Currently, when doing service discovery, we store pending discovery
content into tBTA_GATTC_ATTR_REC structure, and rewrite it into cache,
service after service during discovery.
From now on, we save discovery result directly into pending_discovery,
and at end of discovery we swap it with cache. This remove unncecessary
intermidiate step, and simplify the discovery.

Bug: 67057055
Test: GattReadTest
Change-Id: Id09026b6fb6df04eaf0a0ebefeb6e35b3d175f62

Change Bluetooth HID Profile Name (3/6)

Make the Bluetooth HID profile name consistent with the Bluetooth HID
service name.

BluetoothInputHost → BluetoothHidDevice
BluetoothInputDevice → BluetoothHidHost
IBluetoothInputHost → IBluetoothHidDevice
IBluetoothInputDevice → IBluetoothHidHost
BluetoothProfile.INPUT_HOST → BluetoothProfile.HID_DEVICE
BluetoothProfile.INPUT_DEVICE → BluetoothProfile.HID_HOST

(Cherry-picked from commit 1878072a45cb748f396a7b97f3c14aa983d41333)
Merged-In: I6bb5ad307f8ffb78cf5972aad68665372b85f0fe
Bug: 68055651
Test: make
Change-Id: I6bb5ad307f8ffb78cf5972aad68665372b85f0fe

Merge "Use vector instead of list for keeping GATT database content"

Use vector instead of list for keeping GATT database content

We don't use any list features, but have to pay extra for it's features.
Replace with vector.

Bug: 67057055
Test: compilation
Change-Id: I4209c24b4f87780ea29e4e0707ce0a2a0d440d6b

RFCOMM: Return NULL when out of listen ports

Bug: 67846163
Test: Connect RFCOMM ports until they fail
Change-Id: Ia6cdb5340f8565db0ca5538f8d8ae9c2f25e892e

Merge "Restart legacy advertising after successfull connection"

Merge "test_vendor: Update clang-formatting"

Restart legacy advertising after successfull connection

Bug: 66139617
Test: run CTS Bluetooth LE Insecure Client Test
Change-Id: I894391f608d9ac7a625dc936cf27027fd3ab6140

Build only for current architecture by default

From now on, libbluetooth.so, and all unit tests will be compiled for
the current device platform only.

libbluetooth-types is an exception from rule, as some vendor libraries
are still compiled for 32bit.

Bug: 68028239
Test: compile and run Bluetooth
Change-Id: I51bb4da2a9e1e4e9a03c43f2825bd1dbd4a5ca58

Remove LD_LIBRARY_PATH from test/run_unit_test.sh

libbluetooth.so is now in default lib folder.

Bug: 67853426
Test: run tests using run_unit_test.sh
Change-Id: Ib3982d9600717593b817bda3553f53a629eb4afc

test_vendor: Update clang-formatting

Test: build
Change-Id: I40313b5df0dbed20e9d4afd4a7eac0083c644d7c

Fix out-of-bound array access in jv handle err log

Since the port_handle is checked invalid in bta_jv_rfc_port_to_cb
the error print of bta_jv_cb.port_cb[port_handle - 1].handle is
inappropriate

Test: mm -j8
Change-Id: I0562706512bb993d9a65676d11274d21fa9e70ef

Linux: bluetooth.default.so -> libbluetooth.so

Also rename the build goal for both the library.

Bug: 67853426
Test: run Bluetooth
Change-Id: Ic5bd27f10ade0b068057049dd98c09e082cafee1

Clarify naming for characteristic value handle

We always reffer to characteristic by it's value handle.

Bug: 67057055
Test: compilation
Change-Id: I41a5d910ed471800c996b44dccfdbab3d7c3bdda

Merge "Test: Rename performance_test to net_test_performance"

Test: Rename performance_test to net_test_performance

Bug: 67059793
Test: make
Change-Id: I6303d4c367e465c8404538d904eabca80b509a16

hw/bluetooth.default.so -> libbluetooth.so (1/2)

Also rename the build goal for both the library and headers.

Bug: 67853426
Test: run Bluetooth
Merged-In: Iaffeebc1d3f2761e13f6d66d4e855dc342603054
Change-Id: Iaffeebc1d3f2761e13f6d66d4e855dc342603054
(cherry picked from commit 842ff8ed7f5b00d5f6da9d975f37e65c41bb8ebc)

Revert "Add -fno-exceptions to default build config"

This reverts commit d4b1d66dbe28818df3408a920ef37a3e0afca0aa.

Change Ie53b82e5d670f44a2460880c32dd9102d1caad7d
is a better fix. This flag won't disable exceptions in pre-compiled STL library.

Bug: 67963594
Test: build
Change-Id: I8c8cf0ebeb79bae697f0074de2d2e39f8e8f8703

Merge "Fix failing ConfigTest"

Fix failing ConfigTest

When running in cloud, different version of STL library is used. It
throws exceptions and make tests fail.

Bug: 67963594
Test: ConfigTest
Change-Id: Ie53b82e5d670f44a2460880c32dd9102d1caad7d

Run clang-format on headers

Bug: 67853426
Test: compilation
Change-Id: I37dcac01e78fafb4ef0c7e97578d04605c46e88e

Move Bluetooth headers to system/bt (1/3)

Bug: 67853426
Test: run Bluetooth
Change-Id: I75748d29a35d10d377d39e0441116dbfd04c82e9

Merge "Remove tabs from build/Android.bp"

Merge "HID: Fix the behavior of virtual_cable_unplug"

Merge "Do not set random address when public address is in use"

Remove tabs from build/Android.bp

Test: make
Change-Id: I02dc1cf5a5e6c0ed2df60fcc3fb79edbb026cf8c

Merge "Add -fno-exceptions to default build config"

Merge "Make system/bt build with the VNDK."

Make system/bt build with the VNDK.

When the vndk is enabled (BOARD_VNDK_VERSION=current),
vendor libraries must only link against vendor variants
in the build system. This also means that they receive
restricted sets of headers and that automatic global
headers are restricted.

Test: build system/bt with BOARD_VNDK_VERSION=current
Bug: 37342627
Change-Id: Ie1507e9b5bdb626fa0049faadd225c2c1f451661
Merged-In: Ie1507e9b5bdb626fa0049faadd225c2c1f451661
(cherry picked from commit 41401a5916f750f29af61bae35ca99f7e851333f)

Allocate tSDBP_DISCOVERY_DB instead of bytes

Test: build
Change-Id: Ib7a33cf7cb89715da781b4e93d0bfe3f74490cea

Remove hw_module_t dependency in Bluetooth - cleanup

Bug: 67853426
Test: run Bluetooth

Linux build fix

Test: compile using ninja

Add -fno-exceptions to default build config

* C++ exceptions are expensive and are not used in our code
* We need to disable exception to achieve desired behavior for
  std::stoi, which we don't want it to crash when failing to parse an
  incoming integer

Test: make, net_test_osi
Change-Id: Idc363ff1f25f085c75df3c473ee4d2a210af3b17

Move include->internal_include (1/2)

Change-Id: If351c7d51a380c33ec9bae3b3f240ed53639f3c9

Merge "Fix failing ConfigTest"

Merge changes I4b90b6ee,I37c5a138,I6ff0c0dc

* changes:
  Inline bta_gattc_start_disc_* where possible
  bta_gattc_explore_srvc minor style fixes
  Remove bta_gattc_incl_srvc_disc_cmpl

Merge "Remove hw_module_t dependency in Bluetooth (1/3)"

Fix failing ConfigTest

Specifying base as 0 let stoi autodetect it. Default is 10.

Bug: 67595284
Test: ConfigTest
Change-Id: I09a996edb6fde81de7c59e3aaa4a33134fe40891

Remove hw_module_t dependency in Bluetooth (1/3)

Bug: 67853426
Test: run Bluetooth
Merged-In: Ia3808552137d1f770f2c0305aaa01181f383d064
Change-Id: Ia3808552137d1f770f2c0305aaa01181f383d064

Inline bta_gattc_start_disc_* where possible

Bug: 67057055
Test: compilation
Change-Id: I4b90b6eede46ffb777180caaab7cb073f898ddba

Use DIST_DIR from environment for XML log output

Bug: 64687730
Test: DIST_DIR=`pwd` ./test/run_host_unit_tests.py -j40 --enable_xml
Change-Id: Ib81703ec76d65700068563c557c090b1f197e650

bta_gattc_explore_srvc minor style fixes

Bug: 67057055
Test: compilation
Change-Id: I37c5a138ccddcb1ddd09e3efb5dfcdaeb5bdad04

Remove bta_gattc_incl_srvc_disc_cmpl

Bug: 67057055
Test: compilation
Change-Id: I6ff0c0dc203232f9fa2b43ccbacb015106d2b6d1

Merge "Add script to run host native unit tests"

Merge "hfp: change initial codec from MSBC_T1 to CVSD"

Do not set random address when public address is in use

Test: compilation
Change-Id: Ie9832b258aa85a9a4086c1608a32aa6f4049853b

Add script to run host native unit tests

* Add run_host_unit_tests.py, which does
  1. Run "mmma -jN system/bt"
  2. Run all host native unit tests configured in HOST_TESTS
  3. Output structured XML GTest log output

Bug: 64687730
Test: ./test/run_host_unit_tests.py -j40
Change-Id: I389c67a2e0f60639d1bbfa20375364964b184233

HID: Fix the behavior of virtual_cable_unplug

The Bluetooth HID profile itself should not unbond other device; the
Bluetooth HID host and device should only disconnect the HID profile
when sending or receiving virtual_cable_unplug command.

Bug: 67866553
Test: Test with SL4A.
Change-Id: Ie378a7292309d2b11f53fcd82bca1cf42e64b0c0

config refactor - use strings

Use strings instead of char* arrays. This simplify memory management,
and makes the code easier to follow.

Bug: 67595284
Test: manual
Change-Id: If5dca2a16038360c361f2809601dcac056cb8769

persistent configuration refactor

Use std::list instead of list_t, use std::string instead of const char*

Bug: 67595284
Test: manual
Change-Id: I65f576977b547b09dc9aebea49d4e399e5712ec6

hfp: change initial codec from MSBC_T1 to CVSD

4.11.2 Audio Connection Setup by HF For all HF initiated audio connection
establishments for which both sides support the Codec Negotiation feature,
the HF shall trigger the AG to establish a Codec Connection. This is
necessary because only the AG knows about the codec selection and settings
of the network.

In the event we do not support codec negotiation we should make our initial
codec CVSD

Bug: 66324788
Change-Id: Ib68129ea1399c54224109e3ca7ba12784973d68f

Merge "Fix net_test_bluetooth"

Remove casts in bta/ and stack/btm

Make pointers to const structs const pointers.
Pass complete unions instead of casting smaller objects.

Test: build
Change-Id: I7d6b61183554be70ac7f58f4133a76ec68e35408

Fix net_test_bluetooth

* net_test_bluetooth should try to load Bluetooth library from
  system/lib/hw or system/lib64/hw if it cannot find it from default
  path
* Also replaced legacy logging macro with libbase logging statements

Bug: 67059247
Test: net_test_bluetooth
Change-Id: I6c45f66aedf675397f377ca56203bec9708324e1

Merge "bta_gattc_cache: prepare for type unification"

Merge "bta: Remove unsafe casts in BTA_AG"

Merge "stack: Remove casts and spell local_keys correctly"

Merge "btif: Use key types to avoid casting"

bta_gattc_cache: prepare for type unification

Bug: 67057055
Test: compilation
Change-Id: I46eb07a8f118b61551b3adc404ec306ad6a14b9c

stack: Remove casts and spell local_keys correctly

Test: build
Change-Id: Ide6ec9528e1f8efca9850eab53aa2bdb88ba363d

btif: Use key types to avoid casting

Test: build
Change-Id: If816c480219552dc8833b094a6228817883b774d

bta: Remove unsafe casts in BTA_AG

Test: build
Change-Id: I19659173498972f838ccc59dcac00e0f26417340

Minor style fixes in bta_gattc_act.cc

Bug: 67057055
Test: compilation test
Change-Id: I862f2da834cf6f5a024ee7321229c337980f3638

gatt_utils minor style fixes

Test: compilation
Change-Id: I76f3f26b5140b7b60508756b1e841b09aeeb5e7d

gatt_act_discovery indentation fix

Bug: 67057055
Test: compilation test
Change-Id: I22640f47b5faf6e1d8718dec003b3cc83b51c514

BTA GATT cache cleanup bta_gattc_discover_pri_service

Bug: 67057055
Test: compilation test
Change-Id: I23464a61282d60400ae36debc820c66a4f1a1e1a

Fix crash in l2cble_start_conn_update

Bug: 67774334
Test: compilation
Change-Id: I42ec986c6c5f3fe81f507596bf1df05dcf3ec0ba

HID: Refactor bta_dm_pm_ssr to be more readable

* Reuse values obtained from tables
* Continue early when Bluetooth addresses do not match, so that the
  rest of the method can save some indentation

Test: make, HID host test
Change-Id: I7b2d4ec35a167c93da25a112e73c8d37531714a5

BTA GATT cache cleanup bta_gattc_srvc_in_list

Bug: 67057055
Test: compilation test
Change-Id: I22e752c1d270128ff0ae4f633d4f3eccf4cb725e

BTA GATT cache cleanup add_*_to_list

Bug: 67057055
Test: compilation test
Change-Id: If70cfc4c024434ffe77e53410b5de8f70af6b33e

BTA GATT cache cleanup bta_gattc_disc_res_cback

Bug: 67057055
Test: compilation test
Change-Id: Iccfffe5de654070058ce3f4af3f48fa0975b7cf3

make Uuid::kBase into local constexpr

This fixes a bug where From128BitBE is used before kBase is initialized

Bug: 67671467
Test: Share a picture using Bluetooth
Change-Id: Ifb5b5f90dbca9646f117b9ff5d01439407424597

BTA GATT cache cleanup

This patch splits bta_gattc_add_attr_to_cache into two functions:
bta_gattc_add_descr_to_cache and bta_gattc_add_incl_srvc_to_cache

Bug: 67057055
Test: compilation test
Change-Id: I92702495e8631197e56c2cb0ab126b59cf7e90e4

Fix logs in btif_hh

Fixed two logs which are incosistent with the actual error

Test: not needed.
Change-Id: I8937797148c8c71ef91f9fba5799192bf52692d8

Use LOG() macros in BTA GATT logging

Test: compilation test
Change-Id: If7676af5bf915df57ab6e876d2f27500ebf5c07d

Merge "Linux: silence gnu-variable-sized-type-not-at-end warnings"

Merge "HCI: Restart 5 seconds after an HCI timeout"

Linux: silence gnu-variable-sized-type-not-at-end warnings

Test: compile on linux using ninja
Change-Id: If3fed35017e4aa8cfc2740b49eb4ff547b4e7e5f

Change DI record default company ID to Google

Test: manual
Change-Id: I181e9716dc2c9e48272dcc7f1a00841b4ed5c8dc

Linux build fix - include what you use

sys/uio.h should be included for writev

Change-Id: I1f66391d15109366356af09c825afa1216fbbf34

Fix linking error during stack startup

Change-Id: I0ac207eada1ddf2181a4c2826ede74310ce119d1

Manually convert to unary where coccinelle could not

Test: build
Change-Id: I7b5ea758b94bf225e8776f64256d644c3e96b198

Use unary operator instead of == bool

Automatically generated with coccinelle:

@@
expression e1;
@@

-e1 == false
+!e1

@@
expression e1;
@@

-false == e1
+!e1

@@
expression e1;
@@

-e1 == true
+e1

@@
expression e1;
@@

-true == e1
+e1

Test: build
Change-Id: Ic7df0dc43b550594855e457466b6bccd8f3443a3

Merge "Remove the build team from OWNERS in system/bt"

Merge "Remove -Wno-typedef-redefinition"

HCI: Restart 5 seconds after an HCI timeout

Bug: 67209266
Test: build
Change-Id: I69a4a17382742327268a80dbc5c5be49059ecb3c

Remove the build team from OWNERS in system/bt

Test: build
Change-Id: I17f1fa9c0b5cdf4dc871c54f5731371f6828255c

Remove -Wno-typedef-redefinition

Test: build
Change-Id: I42cc5ad8f77061bdce8906f7e454c8120f4e71b2

Use -Werror in system/bt/vendor_libs/linux/interface/

Bug: 66996870
Test: build with WITH_TIDY=1
Change-Id: I2eca0a5fff52c83a86b060fd274c5a4cd1c53664

Merge "Decrese the default verbosity for A2DP codec info logs"

Merge "Added extra A2DP logging when there is an error"

Merge "Fix net_test_bluetooth"

Decrese the default verbosity for A2DP codec info logs

Bug: 64159791
Test: Manual
Change-Id: Ida10f5d2aa6643df136a41891d7775e175c06570
(cherry picked from commit b4ae75f4524fd5bf55d015f860f1deaee0d81f43)

Added extra A2DP logging when there is an error

Bug: 64159791
Test: Manual
Change-Id: Ic1001892b7abc9784a21a183d141df51b1b92fc0
(cherry picked from commit 6c6d2f6a14847ee75c72b1366449c45ab66feb00)

Fix net_test_bluetooth

Test: net_test_bluetooth
Change-Id: I8c84f0db3ce0e4f6efbd6d7d3cdc794be8fb7faa

btm: Return the result in btm_pm_compare_modes

Test: build
Bug: 67383347
Change-Id: Id4407c1a73592674f0d86b1be1152abc088908b5

Use std::list in GATT related code

Bug: 67057055
Test: sl4a Gatt* tests
Change-Id: I8201ebdad5ba4c3d5d0a2fd3d0fe9dc900b51d60