readability fix: No assigns in if conditionals am: f33b6f434f am: 769d8fa541 am: aee1e33601
am: d47e06f3d6

Change-Id: Iadd79c52b6bf9478e7dc70b3e063176cec37be9e

readability fix: No assigns in if conditionals am: f33b6f434f am: 769d8fa541
am: aee1e33601

Change-Id: I2201dcc51a56b9f2ed90d6d5abcf218dd61bbb1f

readability fix: No assigns in if conditionals am: f33b6f434f
am: 769d8fa541

Change-Id: I4207f71bf0e482ce1dec299c804d0569e79ff79f

readability fix: No assigns in if conditionals
am: f33b6f434f

Change-Id: I11c5af715b2ad86f33491fae7a5bd530f07a2e96

readability fix: No assigns in if conditionals

Coccinelle-assisted:
@@
variable i;
expression E;
statement S1, S2;
@@

+ i = E;
  if (
(
-   (i = E)
+   i
    != ...
|
-   (i = E)
+   i
    == ...
|
-   (i = E)
+   i
    < ...
|
-   (i = E)
+   i
    > ...
|
-   (i = E)
+   i
    <= ...
|
-   (i = E)
+   i
    >= ...
|
-   (i = E)
+   i
)
    ) S1 else S2

for file in $(find . -name "*.cc"); do
  spatch --sp no-if-assigns.cocci --in-place $file
done

clang-format --style=file -i bta/**/*.cc

Test: mma -j37 and basic sanity testing on angler, sailfish

Change-Id: I41a2964afac347c24e13869b6c172e321e646091

Merge "Fix an inappropriate NULL-pointer check in handle_rc_passthrough_cmd" am: 9f0cc0803c am: 43e7d69946 am: 807d696d03
am: a49234f772

Change-Id: I491915e3074e66b5efc9999bb758f232c89961ec

Merge "Fix an inappropriate NULL-pointer check in handle_rc_passthrough_cmd" am: 9f0cc0803c am: 43e7d69946
am: 807d696d03

Change-Id: Iffc5422f3daae61051fa46fbaab38b5ef5da8c25

Merge "Fix an inappropriate NULL-pointer check in handle_rc_passthrough_cmd" am: 9f0cc0803c
am: 43e7d69946

Change-Id: I74c1098f11c0c1a6e3e48f5c7d56da650100ede2

Merge "Fix an inappropriate NULL-pointer check in handle_rc_passthrough_cmd"
am: 9f0cc0803c

Change-Id: I3532ac2520edb3f5845ad36549bde71a566aaba0

Merge "Fix an inappropriate NULL-pointer check in handle_rc_passthrough_cmd"

Merge "Fix x86_64 compilation on Ubuntu 14.04" am: a4338e32c9 am: 3e7fe75f48 am: 237f83cacc
am: b04ccfba1d

Change-Id: I44d91111ccd58e650e56c830334589da82137ec4

Merge "Fix x86_64 compilation on Ubuntu 14.04" am: a4338e32c9 am: 3e7fe75f48
am: 237f83cacc

Change-Id: Ic4f08a3703fdf746162cffc93ac04f6e49e38531

Merge "Fix x86_64 compilation on Ubuntu 14.04" am: a4338e32c9
am: 3e7fe75f48

Change-Id: Ib50f669253b4b159b0511bda5093ee3143b6a7ba

Merge "Fix x86_64 compilation on Ubuntu 14.04"
am: a4338e32c9

Change-Id: Ibba0d7622b417595b7b0234263c675ff57f47c8d

Merge "Fix x86_64 compilation on Ubuntu 14.04"

Merge changes I9ebb32f0,I00393e2b am: a9bfdf980f am: 6a1e9995a9 am: 74e4e4b303
am: 9955212a62

Change-Id: I99073cebf28559e62626b6dfb72783b88157d866

Fix formatting after removing defines am: 9008888d24 am: 7413f94c6f am: 6ff3e65656
am: a83a584c4c

Change-Id: Icb52eed25563768b68dc9fb8482b5e4f03ab78a2

Remove BTA_GATT_INCLUDED am: 9979121f08 am: f6dc423c99 am: 813880af5e
am: 1de8e7a891

Change-Id: I89a94345cc9dc063308b239fcaaf0e630eb4b459

Merge changes I9ebb32f0,I00393e2b am: a9bfdf980f am: 6a1e9995a9
am: 74e4e4b303

Change-Id: I3a011f6c98fa1dbda00152d2567db9c68f7db0ed

Fix formatting after removing defines am: 9008888d24 am: 7413f94c6f
am: 6ff3e65656

Change-Id: Ic0562ab869a421a54215f74378c6c9253908e45f

Remove BTA_GATT_INCLUDED am: 9979121f08 am: f6dc423c99
am: 813880af5e

Change-Id: I4dbc17498fd279a157293bad3fa952aa9dca9257

Merge changes I9ebb32f0,I00393e2b am: a9bfdf980f
am: 6a1e9995a9

Change-Id: Ibc5ea1e948a790a2dcd419a7185a415af84ff2a4

Fix formatting after removing defines am: 9008888d24
am: 7413f94c6f

Change-Id: I78f6253c2c3697cd12e21eb6f30f79b6570570e5

Remove BTA_GATT_INCLUDED am: 9979121f08
am: f6dc423c99

Change-Id: I1277d2d00f4c477bff7656ebc232ccbba4d3c953

Merge changes I9ebb32f0,I00393e2b
am: a9bfdf980f

Change-Id: Ia3988047f5869db610602e821a504d6038359079

Fix formatting after removing defines
am: 9008888d24

Change-Id: I1eb41520b1434fd99d6459384e7ed1e7b0050984

Remove BTA_GATT_INCLUDED
am: 9979121f08

Change-Id: Ie0ad2741c407eabbfbf1a26100a73dc115aa9b93

Merge changes I9ebb32f0,I00393e2b

* changes:
  Fix formatting after removing defines
  Remove BTA_GATT_INCLUDED

Fix a unit test failure related to SBC maxbitpool testing am: 29a0f792cd am: f8044ce96a am: 2696899797
am: 099362ebfa

Change-Id: I3469621921207877531816887ed9f627e5eb9a12

Fix a unit test failure related to SBC maxbitpool testing am: 29a0f792cd am: f8044ce96a
am: 2696899797

Change-Id: I7a1cd29ef55ce6407090e2d12db78100248c2dc4

Fix a unit test failure related to SBC maxbitpool testing am: 29a0f792cd
am: f8044ce96a

Change-Id: I786d2306e850d6ba7fbbdfcc0ee4d28d2b8227f1

Fix a unit test failure related to SBC maxbitpool testing
am: 29a0f792cd

Change-Id: I0df5b12f7c4866c9c40adad9a14ab70d637b8e2b

Fix x86_64 compilation on Ubuntu 14.04

* Add necessary includes in bt_trace.h
* Add toolchain specific configs
* Add libc++ import for clang
* Add libc++abi include for clang
* Add ipc_handler_linux.cc in service/BUILD.gn
* Use shared_mutex when _LIBCPP_VERSION < 3500
* Updated the build instructions

Test: ninja -C out/Default
Change-Id: I362ebf33f62910be817339cc6bdc62830f57db30

Fix an inappropriate NULL-pointer check in handle_rc_passthrough_cmd

This fixes a potential NULL-pointer access in case p_remote_cmd is NULL

Test: mm -j 8
Change-Id: I06ae9afb9ae161296828e06b2c00d921189bee5c

Fix a unit test failure related to SBC maxbitpool testing

The unit test failure was introduced / overlooked by an earlier CL.

Test: running "test/run_unit_tests.sh net_test_stack"
Change-Id: I40824a903a0dbef3bdab965ef5c390a8939cc5ee

Remove SMP_INCLUDED define am: 2e8e9f4213 am: 8a94420f7d am: 17178729e0
am: 2a3cd5cc34

Change-Id: I580c0999b216cd26ee983baf1a3ce4a3977ad949

Remove BLE_INCLUDED define am: 84baa7f16e am: a4b8fc803b am: da39c46cca
am: c32dbc0bdb

Change-Id: Ie00143e8f86e9a5216d932834cb8d976b0e65310

Remove SMP_INCLUDED define am: 2e8e9f4213 am: 8a94420f7d
am: 17178729e0

Change-Id: I75c4155a39cc7ae335e99149599f1927e58cfb9a

Remove BLE_INCLUDED define am: 84baa7f16e am: a4b8fc803b
am: da39c46cca

Change-Id: I4508a91f639fc2dd6411392a56e5003f393fc7c5

Remove SMP_INCLUDED define am: 2e8e9f4213
am: 8a94420f7d

Change-Id: I3cf3108a03429d3c58650a5d9c7cc53e42d7626c

Remove BLE_INCLUDED define am: 84baa7f16e
am: a4b8fc803b

Change-Id: I290f827e4a7cdc1576b2d0abdc004a2e4ed350a9

Remove SMP_INCLUDED define
am: 2e8e9f4213

Change-Id: I367644b2861f510a54c04159314b25a8f37dfd17

Remove BLE_INCLUDED define
am: 84baa7f16e

Change-Id: I5fd29a1f8c9f7d88e04c689fbd3514cfcb546af4

Fix formatting after removing defines

clang-format -i --style=file bta/*/* btif/*/* include/*

Test: mma -j32
Change-Id: I9ebb32f0cc5bd24a7cb2ae25699999aab5036b13

Remove BTA_GATT_INCLUDED

Test: Connect to a BLE Keyboard
Change-Id: I00393e2bf09fb65533e953896563e69aab5a22dc

Remove SMP_INCLUDED define

Test: Connect to a BLE Keyboard
Change-Id: I785692cb61f4ca5ccccab7424aa03f50c54523c9

Remove BLE_INCLUDED define

Test: Connect to a BLE Keyboard
Change-Id: I5f8f4017c90c3c404004632fd10e6c2b93bd7783

Merge "Audio pops due to queue not flushed when focus taken off" am: 4ef712926d am: cb28a5aa8e am: ce3efe9c62
am: 61eb3303e8

Change-Id: I21fe6782a618d0244853aaa459a0e0fd7a7c778b

Merge "Audio pops due to queue not flushed when focus taken off" am: 4ef712926d am: cb28a5aa8e
am: ce3efe9c62

Change-Id: I3e805fa8e8889ab353d7ba11d08ce1d0c3919a91

Merge "Audio pops due to queue not flushed when focus taken off" am: 4ef712926d
am: cb28a5aa8e

Change-Id: I6648c4869610f37257cdd0cc7477ea8cff2f3cb5

Merge "Audio pops due to queue not flushed when focus taken off"
am: 4ef712926d

Change-Id: I8d330883afc4c593ab615250be435ca71c95b199

Merge "Report proper advertiser status on registration error" am: 1f28e7bdf1 am: 84c99c6ca0 am: 86c0e36be0
am: 2ad350d9dc

Change-Id: Iacb780c49c92e5af4aaba5c3b3b0fd819d5cbe84

Merge "Audio pops due to queue not flushed when focus taken off"

Merge "Report proper advertiser status on registration error" am: 1f28e7bdf1 am: 84c99c6ca0
am: 86c0e36be0

Change-Id: I5b6a6ab7ca5a74771bd018098a45652ad2af4283

Merge "Report proper advertiser status on registration error" am: 1f28e7bdf1
am: 84c99c6ca0

Change-Id: I1c036dfef83184a596f7e5260dbec0cd77e824c0

Merge "Report proper advertiser status on registration error"
am: 1f28e7bdf1

Change-Id: I3e87e5c14339f54abe763051e7e62bdb1e26507c

Merge "Report proper advertiser status on registration error"

btif: DRY up command functions am: 1ee1d33e3a am: df56cc2c63 am: 3d8a763419
am: aefa497bb3

Change-Id: Ica15fecd936e8b1185dbb3e567e57d839aa558dd

btif: DRY up command functions am: 1ee1d33e3a am: df56cc2c63
am: 3d8a763419

Change-Id: Ib32742f7f768bb7db2c669c0492267a6bbab587f

btif: DRY up command functions am: 1ee1d33e3a
am: df56cc2c63

Change-Id: I03685e2393cbc7ef3914f3c75a36570f9c26a9c9

btif: DRY up command functions
am: 1ee1d33e3a

Change-Id: I6d900fa8bc9bf99772c22311d99f01bf3a03ae63

btif: DRY up command functions

also:
clang-format -i --style=file btif/src/btif_rc.cc

Test: sl4a AvrcpController tests
Change-Id: I95f7e6922c096e2460ecdeafad162515f5600d97

Audio pops due to queue not flushed when focus taken off

Test: (Manual) The stutter after music is resumed is gone

Bug: b/32296397

Change-Id: Id188c56a196c2ece13510488e6bcfb717bcc610c

Report proper advertiser status on registration error

When advertiser is not started because too many advertisers are
registered, return proper error code.

Bug: 30622771
Test: sl4a ConcurrentBleAdvertisingTest
Change-Id: Ie8a94b964170edbbf1c56f39b03ad98f745a4861

Remove some usages of UINT_TO_PTR macro am: 58d57a46cb am: d0f6018f0f am: ba3f39aba3
am: 6cffa58d6e

Change-Id: I33db02df8006d2dda54669873f469a64a6bbb1ba

Remove some usages of UINT_TO_PTR macro am: 58d57a46cb am: d0f6018f0f
am: ba3f39aba3

Change-Id: Ib66d4dfa83f0edab48c55d9e6de0369186821e0a

Remove some usages of UINT_TO_PTR macro am: 58d57a46cb
am: d0f6018f0f

Change-Id: I0f560bce461f8631996e5575299f60655511ebed

Remove some usages of UINT_TO_PTR macro
am: 58d57a46cb

Change-Id: Ib7b119d61954ea75fbf0e9a195c7d2b9ca2b2d8a

Remove some usages of UINT_TO_PTR macro

Bug: 32995283
Test: Sanity test with various devices
Change-Id: I20a0b1571c044170ba5a8792d8e990a6790cbce2

Constant Cleanup: Remove BT_TRACE_VERBOSE am: a1e7e55610 am: a17f209cc4 am: 1a67d47acf
am: d3b637b05e

Change-Id: I071727470853b15e067212da58d0af01f5098eb6

Constant Cleanup: Remove BT_TRACE_VERBOSE am: a1e7e55610 am: a17f209cc4
am: 1a67d47acf

Change-Id: I07f4811bd3a057dd60bdd16a103e67ffae0f911b

Constant Cleanup: Remove BT_TRACE_VERBOSE am: a1e7e55610
am: a17f209cc4

Change-Id: Idbf47810a5dce10340782de3181ec3eafe314a03

Constant Cleanup: Remove BT_TRACE_VERBOSE
am: a1e7e55610

Change-Id: I9fd413cc608ea9e7f838314685324c0c9e8d5a52

Constant Cleanup: Remove BT_TRACE_VERBOSE

BT_TRACE_VERBOSE turned on constants-as-strings in
logs in many parts of the stack, which is preferable
to obscure numbers.

Test: m -j37, check logspam is not worse than normal on a device
Change-Id: I1d24f1c1337290a8a2bcd9a0d937c9e0f1f1c611

Move startAdvertising() logic into native code (2/3) am: a75087effc am: 6ce255a1fb am: 21abbfc3e9
am: ecffef42b0

Change-Id: Ie9920ca22b7c8c9a21c56c967006fd4987d9da56

Move startAdvertising() logic into native code (2/3) am: a75087effc am: 6ce255a1fb
am: 21abbfc3e9

Change-Id: Iab2b59bccb2fc58ced8c2ea589e9f6bf68acc4f2

Move startAdvertising() logic into native code (2/3) am: a75087effc
am: 6ce255a1fb

Change-Id: I49ab29caee690b676b54d8e4f305680b1047b40d

Move startAdvertising() logic into native code (2/3)
am: a75087effc

Change-Id: Id4350dc5b6ba474106026eb1d781c341ceee8c78

Move startAdvertising() logic into native code (2/3)

AdvertiseManager.startAdvertising() was implemented in Java layer for
historical reasons. Now that we have nice native callbacks and tests,
it can be moved into native code, where it will be shared between
Android daemon and bluetoothtbd.

Bug: 30622771
Test: native tests added to cover startAdvertising
Change-Id: Idf5d3c53b6ad21d8c7e239b5b150b88e073bb709

Merge "Do not reject valid SBC codec parameters" am: 54ee0d8916 am: ddbac25620 am: 20e708590b
am: 48a88c89de

Change-Id: I913b07410bffe86b740fb9afeda61e2b9f4aec70

Merge "Do not reject valid SBC codec parameters" am: 54ee0d8916 am: ddbac25620
am: 20e708590b

Change-Id: Ic397a124af42299027134b80639a64a2ab545c68

Merge "Do not reject valid SBC codec parameters" am: 54ee0d8916
am: ddbac25620

Change-Id: I427c3838eaeefff281635d40eb7aef12aa5837ed

Merge "Do not reject valid SBC codec parameters"
am: 54ee0d8916

Change-Id: I41e3215f9f385d60bc7a183011fccc4f51a6cca3

Merge "Do not reject valid SBC codec parameters"

Do not reject valid SBC codec parameters

Fixes logic in A2DP_CodecInfoMatchesCapabilitySbc that could otherwise
reject valid SBC codec parameters, leading to A2DP being disconnected.

Example:
Tesla Model S - min bitpool = 2; max bitpool = 60
Android - min bitpool = 2; max bitpool = 53

Bug: 32688022
Test: Added test to test_a2dp_is_source_codec_supported to cover this bug
Change-Id: If02bab1c47f926f82a636184cf3cbd1edbbe223a

Add BUILD.gn for tinyxml am: cc51b4f8c2 am: bf85dc4ab2 am: 2329fe8fc2
am: 8e07842cd2

Change-Id: I3f58157c09d8d3b672f692c886a59cfbfcc4c988

Add BUILD.gn for tinyxml am: cc51b4f8c2 am: bf85dc4ab2
am: 2329fe8fc2

Change-Id: I82a14a1596f97b831efb3abd67c46efc7d2f14e4

Add BUILD.gn for tinyxml am: cc51b4f8c2
am: bf85dc4ab2

Change-Id: Ia536410a14df3c89f351fc25747f02891e377e41

Add BUILD.gn for tinyxml
am: cc51b4f8c2

Change-Id: Ic27a1bb7c558fcc91bffb75e726c36547f1b5dda

Add BUILD.gn for tinyxml

Change-Id: I82d2bb25470d46516e0ab6114bf85cb93f9023ac

Revert "Bluetooth: Fix the issue that peer A2DP SNK device can not auto-reconnect." am: 475f8e0e65 am: 1d6f0568b3 am: dd882c7201
am: cf6da7d3cd

Change-Id: I5cd2bca7785ea329b2e240b7befe5da0456f1eff

Revert "Bluetooth: Fix the issue that peer A2DP SNK device can not auto-reconnect." am: 475f8e0e65 am: 1d6f0568b3
am: dd882c7201

Change-Id: Id6dacce5f84cc405902cdac8d6b27cb2e2c217ed

Revert "Bluetooth: Fix the issue that peer A2DP SNK device can not auto-reconnect." am: 475f8e0e65
am: 1d6f0568b3

Change-Id: I7c9ad51dd6b758c5df72802b4c32dcdf029f0911

Revert "Bluetooth: Fix the issue that peer A2DP SNK device can not auto-reconnect."
am: 475f8e0e65

Change-Id: I9169f8f868fd91a340a189f677fdc5afe0f4c539

Revert "Bluetooth: Fix the issue that peer A2DP SNK device can not auto-reconnect."

This reverts commit 5bd9ffbe6fbf22b7072bc1a63e3152a916276ddf.

Despite the original patch intention, it actually causes various remote
devices (tested Android Auto, Logitech Speakerphone, Tesla Model S and
many more) to no connect A2DP (only HFP) in various re-connect
scenarios.

This is due to the fact that the stream end point type is not correctly
determined yet in some of these scenarios causing the AVDT stream to be
closed prematurely and thus the A2DP connection to fail.

Test: manual; various devices
Fixes: 32688022

Limit the use of AVRC_ADV_CTRL_INCLUDED define am: 365fbfb547 am: 5150b5f548 am: 3b49f01010
am: 0add2f3b8c

Change-Id: I6aa1c4ea95d8e12e99339bd3c0b8a18d1d9dbad4

Limit the use of AVRC_ADV_CTRL_INCLUDED define am: 365fbfb547 am: 5150b5f548
am: 3b49f01010

Change-Id: I49396efabbd35be9aa48121eeea02a9d144a6a36

Limit the use of AVRC_ADV_CTRL_INCLUDED define am: 365fbfb547
am: 5150b5f548

Change-Id: I695342f41f2172f18ee6502ba357e9b615b09b6d

Limit the use of AVRC_ADV_CTRL_INCLUDED define
am: 365fbfb547

Change-Id: I13c2a5acf6f85b1866892e98875b7ab7e44528b1

Limit the use of AVRC_ADV_CTRL_INCLUDED define

Use AVRC_ADV_CTRL_INCLUDED just to control features instead of
controlling features and inclusion of functions.

Change-Id: Ifff0d7006c3d8df1d57c79ace19ee60487c2f0a5