Replace calls to std::bind by lambda expressions
am: 97980c25a1

Change-Id: Iad98f599d6bd802c42b583c6bb5d8e75da568839

No longer uses the macro DISALLOW_COPY_AND_ASSIGN
am: 83cf95e0a1

Change-Id: I6bd4c3d3bb9b12ce8a167769ce4be5dbefebe10a

Using include paths relative to vendor library dir
am: ea3ede2474

Change-Id: Ic703fda9a8c7d72f6fb563069d1948f190ac7924

Replace calls to std::bind by lambda expressions

Change-Id: I5fbe6ae6d5172e87428322084276e593a67fd7d1

No longer uses the macro DISALLOW_COPY_AND_ASSIGN

Change-Id: I3403cdfa38fabf56ffe0854064eeb0ae38eb5a4a

Using include paths relative to vendor library dir

Change-Id: I6b5dfa4098cdf2eda21d3fae60729b765532c2f9

Merge "Improve BT HAL write response to BT server reads"
am: f65e83e5b1

Change-Id: Ib48a998e19b3eada7d19897b81626f60309062b3

Merge "Improve BT HAL write response to BT server reads"

Separate the Alarm unit tests from testing the Wakelocks
am: 223cecf8b6

Change-Id: Iceb0a3e4d2f776456cad88a5276dbd9516baf175

Improve BT HAL write response to BT server reads

By polling, we ensure the socket buffer is nearly always full,
as blocking socket writes may not wake in time to avoid underflow.

Bug: 28286313
Change-Id: I056768dbd860993c41d44c33a4846d62c50db933
(cherry picked from commit 5952e40bef1afb310cd3bba7e673163d7e32d62c)

Separate the Alarm unit tests from testing the Wakelocks

* Simplify the AlarmTestHarnless unit test setup for Wakelocks.
  Now we use wakelock_set_os_callouts() to setup local unit tests and
  to keep local state whether the wakelock is acquired.
  Also, removed unused timer-related setup inside AlarmTestHarnless.

* Add explicit Wakelock unit tests

Change-Id: I08bd8d457fd48ec45a8a591bd69cb08dbaf5ee7a

Add LE Secure Connection data parsing (3/4)
am: bd60854e49

Change-Id: Ia33a911e973fe22b72a4bf4f3ce89d20b9b71fea

Add LE Secure Connection data parsing (3/4)

Bug: 30460956
Change-Id: I216142090fe99b25ef7697fceceb278b761a182b

Revert "Register for Service Changed notifications for untrusted devices"
am: 42657c2f23

Change-Id: I01ff859d5158904695965a0038b252fc98c9096d

Revert "Register for Service Changed notifications for untrusted devices"

This reverts commit 5f0e23d532f4f90d081ccc0e294f404eb5e5b688.

Instead of reusing service discovery data, gatt profile initiates
discovery by service uuid before regular service discovery is started,
and both discovery procedures interfere with each other.
As a result, link is dropped.

Bug: 30462943
Bug: 30481143
Change-Id: Ic63339f4f46eda70e575a4103974766d54a9d624
Signed-off-by: Satomi <michitsuta@casio.co.jp>

strlcpy call size fixes, memory clearing
am: 4fd9ce1b4d

Change-Id: I563fa0319297ae930efc5eb114ecf639b0a5096e

strlcpy call size fixes, memory clearing

Usecase:
1. Run PTS test cases IIA BV-03-I/IIC BV-02-I/IID BV-01-I

Failure:
Randomly errors are returned for AT+BIA from PTS.

Cause:
Command parsing advances the check beyond the string end,
which is not guaranteed to be zeroed.

Fix:
Break string parsing when the end of string is encountered
after a AT+BIA argument.

Bug: 29615016

Change-Id: Ifc77435e780fba34006372694a7da4758e4f8c36

Merge "Fix descriptor write"
am: a8be6caf45

Change-Id: I6346731e787457bbd013c958ccb306f470e9bc00

Merge "Fix descriptor write"

Fix descriptor write

This patch fixes error introduced in commit
bb920a65b959fbb593403587f89bc3fc7e831732
Writing descriptor was broken, ACTS tests were failing.

Change-Id: Ied2ff4a12df1053961677fdcb824838b7946dbaa

Add dumpsys support for LE connection parameter updates
am: 8ec8ca4a87

Change-Id: I0966798416b8a7c784f4b1e6d5af593e91006990

Add dumpsys support for LE connection parameter updates

Change-Id: Ie203eec9d8efec0d55b1b10208c64a98449044a2
(cherry picked from commit 4e5514b69b95ee1416fb0d0b5e7783bcfe1612ab)

resolve merge conflicts of d19e078 to stage-aosp-master

Change-Id: I901ce1a7cafc2f526d8a7e515e9fe8ffdd20f33e

Use standard types, consistent ifdef style everywhere

Remove the typedefs in stack/include/bt_types.h

Use standard types everywhere.
Use standard style for #if statements:
 - #if (VAR_NAME == TRUE)
 - #if (VAR_NAME1 == TRUE && VAR_NAME2 == TRUE)
Use __func__ instead of __FUNCTION__
Fix some debug statements to use __func__

Update script to be less disruptive to aligned assignment blocks.

Change-Id: I8f8f068e6c26ce74fd3b3707e1e31fd0b919cdd0

Fix APR crashes in read rssi callback
am: 7294d958dc

Change-Id: I5e7daa5962e92c8fb2c6adf683fe616f0af294d1

Fix APR crashes in read rssi callback

When read RSSI timeouts, callback is called with NULL result. This was
causing crash.

Bug: 30562545
Change-Id: I05b7eb2808c58655525ed5c7019ae175181c837a

Implements some additional commands
am: 03c2c7c22a

Change-Id: I5a0f2011966daad1679a4f196996552db40f5e56

Implements some additional commands

- Added/Improved responses to certain opcodes on bt_vendor.cc
- Added handlers for a couple of commands used in L

Change-Id: I85c02553057e4ea3a3e51795c2e1a2bb679fd5e1

HFP 1.7 profile update (1/4)
am: 4de9bbcde7

Change-Id: I52433802594e44702152b57f00f7d5511f4741ac

HFP 1.7 profile update (1/4)

-> Added AG support for HFP 1.7 version upgrade which
   adds new type of indicator called HF(headset) indicators.
-> Added support for two new AT commands AT + BIND and AT + BIEV
   which have been appended to Hands-Free SLC sequence.
-> Added support to propagate the above commands and their data to
   the upper layers.

Bug: 19983867
Change-Id: I93d5b2af949f9fb99507a954e623cd0927ddc976
(cherry picked from commit e5bed64926d6ae45501617052f90932287da135b)

Rmove write type from GATT descriptor writes (4/4)
am: bb920a65b9

Change-Id: I9301e43850d5e97256f3eca0a47047c47757c08b

Rmove write type from GATT descriptor writes (4/4)

According to the Bluetooth Core specification v4.2, Vol 3, Part G,
section 4.12.3: "The Attribute Protocol WRITE REQUEST is used
for this sub-procedure".

Change-Id: I60bb0b1489b417af96975265a023c356ace293bd

Merge changes from topic 'beautiful-gatt-server'
am: 3cf12a6cfc

Change-Id: I542982eb13daa23c2ee75c74c3a0a04a94449de0

Merge changes from topic 'beautiful-gatt-server'

* changes:
  GATT Server refactoring (4/4)
  Add C++ Parcelable types for Service, Characteristic, Descriptor

Merge "Fix build when SMP_DEBUG=TRUE plus logging improvements"
am: 980147c889

Change-Id: I97ffc1ef3c2a953f486829b57da9f558c2b3ecd5

Merge "Fix build when SMP_DEBUG=TRUE plus logging improvements"

Relax some of the timing parameters in the Alarm unit tests
am: 9713b9476c

Change-Id: Ibb7599098fd10e2b005622488160ffdce10aea3f

GATT Server refactoring (4/4)

Right now in order to add gatt service, we need to add it piece by
piece - each characteristic and descriptor separately. This causes lots
of concurrency errors in higher layers. Instead accept whole service
definition and start whole service at once.

Also, use handles to identify GATT server attributes. It's already done
on clinet side.

Bug: 27999121
Change-Id: I42c796809423d116fe6755ef8b1c40da270427a5

Add C++ Parcelable types for Service, Characteristic, Descriptor

This types will be used in upcoming GATT Client/Server code. They will
match ones you can found in frameworks/base.

Change-Id: I6080310a726bcbcd0cd36970712928b188f7bd24

Fix build when SMP_DEBUG=TRUE plus logging improvements

Bug: 30460956
Change-Id: I2611501d1961b104c8e66b1a3fca414936dc321f

Relax some of the timing parameters in the Alarm unit tests

In certain environments (e.g., virtual machines), the alarm timeouts
can late by tens of ms (40ms have been observed in some cases).
Relaxing some of the unit tests timing parameters, so we can
continue testing for correctness as covered by the AlarmTest.

Bug: 28907913
Change-Id: I211c36a689774c5cdf427039e4c9184cf4147604

Proper handling of AV connection collision
am: 09d91c0ceb

Change-Id: I7ace6b2005e544c8821271a9d575ee49f883ada1

Proper handling of AV connection collision

Use case:
1. Pair to Remote
2. Turn OFF/ON BT on DUT
3. After DUT's BT is turned ON, Remote would reconnect to DUT

Failure:
  Bluetooth settings UI continues to display Connecting when
  BT was Turned OFF and ON.

Root cause:
- This is a connection collision case where remote is not responding
  to DUT initiated SDP attribute fetch request for AV. AV holds outgoing
  connection for sometime and meanwhile remote starts AV connection
  from its end. Then DUT disconnects the SDP channel after a second
  without waiting to fetch requested info, and this confuses DUT's AV
  state machine. The outgoing connection could not proceed as incoming
  AV L2CAP connects, because DUT's AV state machine is not equipped
  with handling this scenario. On incoming connection timer expiry, it
  does not start AV media channel as it finds AV Open has not been done
  from upper layer after incoming L2CAP got connected.

- When the incoming path 2s timer expires, the DUT checks whether Open
  has been called after incoming state is set. This is not the case here,
  because Open was triggered earlier as as part of the outgoing
  connection. As a result, A2DP cannot connect at all.

Fix:
- Properly set collision flags for handling AV connection collision.
- Update BTA AV handle even when BTIF AV state machine is yet to be built.

Bug: 30362987
Change-Id: I02a3adb62479b0f762bc792a5727d06e11eaaa52

Allow the Bluetooth test library to be used as a vendor library
am: 6bfe283db7

Change-Id: I0be79f4e37637a8af166a8b307c4b664bc777036

Allow the Bluetooth test library to be used as a vendor library

Change-Id: I479fd72482399f878ef045271c7b493c94aa5ab7

btif: Fix logic bug within register_application
am: 696996c115

Change-Id: Id2c2d4d2c171dfa9d8389a901e1f0f627628fa8e

btif: Fix logic bug within register_application

We were checking for the SINK bit twice in the same condition,
and would incorrectly report a SINK only mask as a DUAL device,
and would never hit the logic path to report a device as SINK.

Bug: 29763285
Change-Id: Ibc39db6396c09ddcd63896302dabf72841304cf2

Don\'t use move in FOR_EACH_* macro
am: e84c4bdb5f

Change-Id: Ibda5234458e95627b16fcc94edb4331b2c8f960a

Don't use move in FOR_EACH_* macro

FOR_EACH_* macros are resolved to for loop. Using std::move inside it
will cause already moved object to be used when more than one observer
is registered. This patch fixes that.

Change-Id: Iaaa314f5eeed4094d1e0fc951d3664fb1514ba15

Make parameter set S4 default for eSCO
am: 3edefef0ce

Change-Id: I2beae8e74d45d29ca8868b9baa62ef3c818ed1de

Make parameter set S4 default for eSCO

Bug: 29615817
Change-Id: I5256b31a746433f10c2e7de1629952f668c99d16
(cherry picked from commit af5b8248593c80acd8b1afb695eafe5801b44f7d)

Process Gatt read multiple response properly
am: 089cd116de

Change-Id: Ieff85e5040d7fa3db24a7946a05d4958480ddd08

Process Gatt read multiple response properly

The Gatt server multi response queue is made NULL after each Gatt
server response is sent by BT stack in response to the remote Gatt
client's request. But the multi response queue is not initialized
again to process subsequent read multiple Gatt server responses.
Hence, the BT stack does not send Gatt server responses for the
subsequent read multiple requests from remote device which inturn
leads to disconnection. This change would process read multiple
responses by initializing the queue when it is null.

Some of Bluetooth certification test cases involve testing
Gatt server's capability to process and send read multiple
response.

Test case: TC_GAR_SR_BV_05_C

This change also cleans up unnecessary initialization and freeing
of multi rsp queue. This change makes sure that the stack initializes
multi rsp queue only when it is necessary(ie only when the Gatt
multiple read response is being processed).

Bug: 29011255

Change-Id: Ia3e2e1569ee16cac9c518f95501945257b9fb7fd

Fix linux build, missing strlcpy
am: 3a2dc880ce

Change-Id: I306f667701519fed87b97138e0b47aab0cf2cc3d

Update libchrome BUILD.gn
am: 3ad44520b8

Change-Id: If6e3c284c52fe8db1cbb61c7a41ec0024b5a696a

Fix linux build, missing strlcpy

Compilation was failing with "missing strlcpy", this patch fixes that

Change-Id: Ic35d4bf4bed065fce27250a2d63d2fff606f6a3e

Update libchrome BUILD.gn

Update BUILD.gn to match libchrome from top of AOSP.

Change-Id: I1d8638d67f816479e58af627ab5bd00649b72697

Add OnConnectionStateChanged to IBluetoothGattServerCallback
am: 7932727cbe

Change-Id: I9edcca7deb2484d8b64c8f63872b4b08c9f4ecc6

Add OnConnectionStateChanged to IBluetoothGattServerCallback

Currently native GattServer clients can't tell wether someone connected
or disconnnected to the device. This patch fixed that, by exposing
OnConnectionStateChangedCallback

Bug: 29835693
Change-Id: Ia7ef592f4031d021ab1b45808126ca57f9348d0e

Fix the implementation of macro BTM_IS_PUBLIC_BDA()
am: 42e055353c

Change-Id: I9273b46c7e11da04f93cd38383de697dcf3dc750

Fix the implementation of macro BTM_IS_PUBLIC_BDA()

Bug: 28705665
Change-Id: Id714a799c649ff85cf8bab9d8370626606fa7f2c

Disable auto-pairing for hands-free devices
am: 43b12942ca

Change-Id: I1ca91fdcf1be9be6e1048123b79bf2eb4f44a60f

Disable auto-pairing for hands-free devices

Fixes: 30004157
Change-Id: Id170856fbee5ea345f02f55d3a6e3376d846c2de
(cherry picked from commit e8b81851c33095fe3b3ac78329073309be5421e1)

Add callbacks to GATT read/write operation
am: 24ea892320

Change-Id: I49b30d782f1260f63a75a4006c61b9ecc0f23d76

Add callbacks to GATT read/write operation

There is no need for read/write callback to be a global event. It should
execute local callback instead.

Change-Id: Id7e915af9d30092d2f754eddc7c3aed4970eeb2f

Add logs for AVRCP commands that originate from a remote device
am: da34e5b65e

Change-Id: Ia29f28b3bb71da2b1f1beec72dad059fa9febf3f

Add logs for AVRCP commands that originate from a remote device

Change-Id: I02b213913685d831bfe00e1938baef953428e048

Merge \"SMP: Delay authorization complete.\"
am: 8afa82d0c3

Change-Id: Iade6d0379d44b26c6b2b401985c09e16916915d4

Merge "SMP: Delay authorization complete."

Fix google-explicit-constructor warnings in system/bt.
am: 3398730eac

Change-Id: I54f2a53f6a2a6bf3ceb524966d98e2c1c80a58a3

Fix google-explicit-constructor warnings in system/bt.

* Declare conversion constructors as 'explicit'.
* Add NOLINT to implicit converters.

Bug: 28341362
Change-Id: I81fecf19c0ee2e64dd6778b3dea78e04f111bfb4
Test: build with clang-tidy

Log RSSI when A2DP buffers are flushed
am: 741c1b4cc9

Change-Id: I4dc09a22922650dbeef7e3a254cc1f571419113c

A2DP media task: Flush full TX queue on congestion
am: de3505fc6c

Change-Id: I82ed74baec551c7e3c1535eddb223c21a471d8c6

Log RSSI when A2DP buffers are flushed

To add another datapoint and potentially narrow down *why* we had to
flush the buffers, add RSSI logging to keep track of signal strength
during audio drop-out events.

Change-Id: Ie52f56290951130165a9f375a898a9bacf459d55

A2DP media task: Flush full TX queue on congestion

Currently, our TX queue does not allow for quick recovery of minor
congestion. As we only clear as much room in the queue as is needed to
fix the news frames needed for a given timer tick, any minor congestion
on the TX path can result in single frames to be dropped over an
excessively long amount of time.

From the "just rip the band-aid off" school of thought, drop the whole
queue instead to cause a single drop-out on congestion and catch up to
the live stream immediately, hoping that the stream recovers...

Bug: 29601962
Change-Id: I482c3cfef6d8b51143e54733eb2da5f70663636f
(cherry picked from commit d881c4b46776fe03a5861a3021bbe739b1f3cc58)

SMP: Delay authorization complete.

Delay authorization complete during Bond Pending state in SMP.
Instead of automatically advancing the Bond Pending state, create
a 500ms window to allow the slave to stop pairing by sending over
the Pairing Failed command during the Bond Pending state.

BUG: 28475887
Change-Id: Ie59ff23128b5d523aa13c4db599be97396097cc3

Add BT_HCI_TIMEOUT and BT_CONFIG_SOURCE log events.
am: e909ff50ff

Change-Id: Iafd1759f79364e80101344f5023726feb146ff53

Add BT_HCI_TIMEOUT and BT_CONFIG_SOURCE log events.

These events are used to track HCI timeout and configuration file
usage respectively.

Change-Id: I5c8b6ef20b61f03b9ccb74f3b1c9784e4653b770

Simplify GATT characteristic read memory handling
am: 3c5451da9d

Change-Id: I988d7eefcb9a0856ef61496610eac26b29aefa7d

Simplify GATT characteristic read memory handling

Pass read result in a fixed sized array instead of a pointer.
This eliminates the need for any special memory management in btif.

Change-Id: I2a33d74b12f3059050855c9fc90fff785cc985ba

Add Subaru and Nissan car kits to auto pair blacklist
am: d13870072e

Change-Id: I753967a57a3e460638ab904ad987beda96399975

Match socket buffer sizes between audio HAL and BT server am: 3d02215979
am: 753ef725b9

Change-Id: Ibb2fd3f2ac5b07c61f03f5fd8b649f22ca48b688

Match socket buffer sizes between audio HAL and BT server
am: 3d02215979

Change-Id: Id4e09877ebba3a3096bde0a59e8c4c5dc7a26051

Add Subaru and Nissan car kits to auto pair blacklist

Bug: 29157087
Change-Id: If227963b939a9b0ed55e6d900364c71932d3cfe8

Match socket buffer sizes between audio HAL and BT server

Adjust our mixer buffer size to be one quarter that of
the socket buffer size for quadruple buffering.

Increase socket buffer size from 20*512 to 28*512 to
smooth out variability in data draw from AudioFlinger.

Bug: 28286313
Change-Id: I8a9ca9e1f4639a0724cfe126acc670c2058cb0fb
(cherry picked from commit e97d975cce213b8c6e4f69bc3fc75411a1743647)

A2DP media task: Flush full TX queue on congestion
am: d881c4b467

Change-Id: I4e15610487705635ece8fb6bd727c320dfaacd7a

A2DP media task: Flush full TX queue on congestion

Currently, our TX queue does not allow for quick recovery of minor
congestion. As we only clear as much room in the queue as is needed to
fix the news frames needed for a given timer tick, any minor congestion
on the TX path can result in single frames to be dropped over an
excessively long amount of time.

From the "just rip the band-aid off" school of thought, drop the whole
queue instead to cause a single drop-out on congestion and catch up to
the live stream immediately, hoping that the stream recovers...

Bug: 29601962
Change-Id: I482c3cfef6d8b51143e54733eb2da5f70663636f

Get rid of unused BTA_DYNAMIC_MEMORY am: 25984b0d22
am: 88eb96fd30

Change-Id: Idcbb1232a2fbd1b253ad9d5daeea990ff21dea3c

Get rid of unused BTA_DYNAMIC_MEMORY
am: 25984b0d22

Change-Id: I182b5b707f6537c8c50cb5cac794ecedebc45b4d

resolve merge conflicts of 6b142ed to nyc-dev-plus-aosp

Change-Id: I222ff056f7ba91903a60f1ef6958b6e4ec6f7f79

Use bta_gattc_cb directly, not through aliases am: a05734c69c
am: de22f90d28

Change-Id: I210d6bdb64a073a61c82420bb995cec88d1e2ccf

Use bta_gattc_cb directly, not through aliases
am: a05734c69c

Change-Id: Iefcdc6e3ea4504e859464e778e6ca88222fc379b

Remove log spam in btif_gattc_update_properties
am: ffdb33b5fc  -s ours

Change-Id: I4ec69857999b0d7c89596a02386f796f50388ba7

Get rid of unused BTA_DYNAMIC_MEMORY

Change-Id: Idc7fcddfecac9ab7d9800b496e2380ae277f357f

Use bta_gattc_cb directly, not through aliases

Right now, bta_gattc_cb is not used directly, instead it's passed as
argument to methods that can use it directly.

Change-Id: I54fef4b8b1e11b3289e7961f70bea187173e9804

Merge "Remove SBC frame scrambling" into nyc-dev

Remove SBC frame scrambling

Bug: 29601962
Change-Id: I30ce09f1c7550a1be9fd1c8ed70745f9aa0cdb44

Remove log spam in btif_gattc_update_properties

Bug: 29601962
Change-Id: I4a86b786609b0203a72b750a1c4a38c82fcc5f97

resolve merge conflicts of b7f64bc to stage-aosp-master
am: 649e13fbfb

Change-Id: I74fb93559793619a604f7ab442b0274ed3e3fd27

resolve merge conflicts of b7f64bc to stage-aosp-master

Change-Id: I8359cd1895d6a0e49e834a8f45823865617a207d