Sync minor nits with AOSP

No functional changes.

Change-Id: I6bab5d3b0ba4f6a543418943996b029bb3d67a6d

tests: don't use bt_os_callouts for wakelocks

Introduces alarm_set_wake_lock_paths so wake lock paths can be changed
for testing, and adds AlarmTestHarness::WakeLockHeld to test whether a
wake lock is currently held.

Bug: 25387683
Change-Id: I9a41ae8266e252a3d436f8d41ea3f9e7ecb45cdc

Revert "Revert "Enable kernel wakelocks and timers""

This reverts commit 89590b95e3f0c8f3a7c6005ee3b837cc3984c4d5.

The revert of the revert is needed to restore the original
code and prepare it for updating the wakelock mechanism.

Move btsnooz.py script from Android Wear to the BT stack project.
am: 1c50abb872

* commit '1c50abb87294b033d1604820d34ba5853d5bf253':
  Move btsnooz.py script from Android Wear to the BT stack project.

Update btsnooz log file format to log ACL and SCO headers.
am: e8c3dde6cf

* commit 'e8c3dde6cfec13e8a0b3f87397319eab4cb86ad5':
  Update btsnooz log file format to log ACL and SCO headers.

Move btsnooz.py script from Android Wear to the BT stack project.

Change-Id: I8368820b5ed9325d2c3b82885b9fe934c6bc87ca

Update btsnooz log file format to log ACL and SCO headers.

Change-Id: I72db1769197150f34ebba6fcb9c0e3db2404f342

Fix build break in system/bt
am: c637b0c828

* commit 'c637b0c8287226c90d5b573080a7a3ef80f53a46':
  Fix build break in system/bt

Add socket tx/rx accounting
am: 0620f9706d  -s ours

* commit '0620f9706d9aa06ba7f8982840eeb7ab2ad90e7a':
  Add socket tx/rx accounting

Fix build break in system/bt

This was due to libchrome uprev submitted earlier

Change-Id: Ic0f1bd80ff4563bd73754cd3b9fcd521e93616e7

Merge "Add socket tx/rx accounting"

Add socket tx/rx accounting

Records network traffic going over bluetooth sockets. This patch adds support
for L2CAP and RFCOMM.

Bug:26039657
Change-Id: Idb860b367b429fa5ef41d02684a5494a668ee8ed

Merge "service/client: implement connect and disconnect methods"
am: 5fbbf42c5f

* commit '5fbbf42c5fdf387d7a0ecab1a621161e7b382576':
  service/client: implement connect and disconnect methods

Add missing field to fake Bluetooth interface
am: 1cbff4e542

* commit '1cbff4e5423999a89072869bcba27d6e1608a2ae':
  Add missing field to fake Bluetooth interface

Add socket tx/rx accounting

Records network traffic going over bluetooth sockets. This patch adds support
for L2CAP and RFCOMM.

Bug: 26039657
Change-Id: Id30d69e4cd648f0e88ab23517e3b94f29b393995

Merge "service/client: implement connect and disconnect methods"

service/client: implement connect and disconnect methods

Change-Id: I141e8a1a917adfe7a2eb1c8adaf5e09fad9df5ac

bt: Update libchrome APIs to r369476
am: 57d7bf98ea

* commit '57d7bf98ea5f7a08c93c30f5d8a0e303d78ea432':
  bt: Update libchrome APIs to r369476

Add missing field to fake Bluetooth interface

Bug: 26686093
Change-Id: I09867ae073fb32165c64bf9d306b699c6bc843ee

bt: Update libchrome APIs to r369476

The new libchrome has been ported from Chromium and some APIs have
changed. Make necessary changes at call sites.

Change-Id: I988157d3a6e36ede6561c0fa6919bb756e8ed3dd

service: Expose Connect and Disconnect through IBluetothLowEnergy
am: 455dc8f535

* commit '455dc8f535a719af6a65a7512d90f9db878f5a58':
  service: Expose Connect and Disconnect through IBluetothLowEnergy

service: Expose Connect and Disconnect through IBluetothLowEnergy

Change-Id: Idcfd3fac263e61bc1bb2be97017554d552511765

Merge "Add Connect and Disconnect methods to LowEnergyClient"
am: 740bebde86

* commit '740bebde86a474a198ceecd2204bf20fd322e671':
  Add Connect and Disconnect methods to LowEnergyClient

Merge "service: add HAL for connect/disconnect"
am: 648393fe8a

* commit '648393fe8ac781cf54c7680138905b8f7737d91b':
  service: add HAL for connect/disconnect

Merge "Add Connect and Disconnect methods to LowEnergyClient"

Merge "service: add HAL for connect/disconnect"

Add Connect and Disconnect methods to LowEnergyClient

Change-Id: Ibabfd44575b7ab1c8fc7a57b844956038dd571c4

Support for A2DP Sink - Follow-up fix
am: 69a34362c6

* commit '69a34362c65533155578623d30845b04fa6285b1':
  Support for A2DP Sink - Follow-up fix

Support for A2DP Sink - Follow-up fix

Update the code so it matches the corresponding code in the internal tree.

Change-Id: I7cb143cde31e2cfca54b2172737cf262ae478186

 Support for A2DP Sink and Proper initialization of profiles
am: 6b84f291c4  -s ours

* commit '6b84f291c4edce7e4102efd8d4052a63bcb4b9dc':
  Support for A2DP Sink and Proper initialization of profiles

service: add HAL for connect/disconnect

Change-Id: I05b86af68e05433241d0206b0952d859483dee80

 Support for A2DP Sink and Proper initialization of profiles

 - Register SDP and SEP (Stream End Point) for the profile
   whose init is called. We achieve this by making registration
   api aware of profile id being called.
 - Register A2DP Src + Avrcp Target + Avrcp Controller. We need
   to support Absolute Volume as well, so Avrcp Controller is
   required here.
 - Register A2DP Sink + Avrcp Controller. In this case we do not
   support absolute volume. Support would be added in AVRCP
   controller change
 - Adapter property is updated with profile ID for which init is
   called.This is required to make changes in btService for Profile
   initialization
 - Start listening for incoming AVRCP connection if we have
   only AVRCP Controller
 - Update default peer_features based on feature mask. In case
   of incoming AVRCP connection, SDP will be done later.
   In such a scenario default peer features sent to btif should
   be based on  features-set selected during initialization.
 - Drop Control, Meta and Browse commands when TG is not up.
 - Not to call AVRCP app callbacks when mentioned service is down.
 - Close audio socket on suspend

A2DP Sink: Support for AudioTrack

 - implemetation for audiotrack to take care of audio rendering
 - support for audio focus state in bluedroid

Bluetooth: Support for Avrcp 1.3  Controller

- support for SDP registration
- support for sending vendor dependant commands
- support for abs vol
- support for receiving vendor dependant response
- serialization of connection and rc_features callback

Avrcp_Ctrl: handling of commands and events for AVRCP Controller

- support of parsing cmd and event pdus
- timeout handling

Change-Id: I1e8d49b087eff6301373e1e90e8d868f15847c34

resolve merge conflicts of 760fb7083b to master.

Change-Id: Ic5d10e4a80d53f916ec61e75d01a0e08df6114cf

Make list_foreach() more useful

- Changed |callback| return type to bool to be able to interrupt
  iteration (to find specific elements for example).
- Added new |extra| parameter for |callback| so a pointer can be passed
  in to receive output values or to pass in criteria etc.
- Also added unit tests.

Change-Id: Id1ddcbabf55292f701d0277f2a1e9ec261b9fbde

Do not send AT+CHLD=? if the 3-way call feature is not supported
am: 15df6d411e

* commit '15df6d411e871e2e1366a3599bb990ce0cb9aae1':
  Do not send AT+CHLD=? if the 3-way call feature is not supported

Do not send AT+CHLD=? if the 3-way call feature is not supported

The HF shall not issue the AT+CHLD=? test command in case
either the HF or the AG does not support the Three-way calling feature.
Currently HF sends AT+CHLD=? to AG device when SLC sequence on going.
It affects to SCO statemachine so that after going out of range and
coming back in range while active call alives, SCO state goes bad and
never comes back. This is out of specification of HFP and it should be
fixed by checking peer and local 3way call capability check.

Bug: 25703926
Change-Id: I66adac2345c6fb0df6741fdbfa67d9483fc38a00

 Support for A2DP Sink and Proper initialization of profiles

 - Register SDP and SEP (Stream End Point) for the profile
   whose init is called. We achieve this by making registration
   api aware of profile id being called.
 - Register A2DP Src + Avrcp Target + Avrcp Controller. We need
   to support Absolute Volume as well, so Avrcp Controller is
   required here.
 - Register A2DP Sink + Avrcp Controller. In this case we do not
   support absolute volume. Support would be added in AVRCP
   controller change
 - Adapter property is updated with profile ID for which init is
   called.This is required to make changes in btService for Profile
   initialization
 - Start listening for incoming AVRCP connection if we have
   only AVRCP Controller
 - Update default peer_features based on feature mask. In case
   of incoming AVRCP connection, SDP will be done later.
   In such a scenario default peer features sent to btif should
   be based on  features-set selected during initialization.
 - Drop Control, Meta and Browse commands when TG is not up.
 - Not to call AVRCP app callbacks when mentioned service is down.
 - Close audio socket on suspend

A2DP Sink: Support for AudioTrack

 - implemetation for audiotrack to take care of audio rendering
 - support for audio focus state in bluedroid

Bluetooth: Support for Avrcp 1.3  Controller

- support for SDP registration
- support for sending vendor dependant commands
- support for abs vol
- support for receiving vendor dependant response
- serialization of connection and rc_features callback

Avrcp_Ctrl: handling of commands and events for AVRCP Controller

- support of parsing cmd and event pdus
- timeout handling

Change-Id: I1e8d49b087eff6301373e1e90e8d868f15847c34

service: Implement IBinder.dump()
am: dff1aa9fe7

* commit 'dff1aa9fe7d305a94e0de0e9d70ca22504fabcf0':
  service: Implement IBinder.dump()

service: Implement IBinder.dump()

Add dump() support to BluetoothBinderServer and change IBluetooth
implementation to allow the system to call it correctly.

Bug: 26095165
Change-Id: I32ca78e37484b5d748b8caf4c293aafcf01dfbff

Merge "service: Add RegisterTestClient"
am: f6e78042f7

* commit 'f6e78042f78ba5bd804a823193fef9206612858c':
  service: Add RegisterTestClient

Merge "service: Add RegisterTestClient"

service: add scan support to bluetooth-cli
am: ba197a21d8

* commit 'ba197a21d810b0ef814ed9c23dd3b290613751c7':
  service: add scan support to bluetooth-cli

service: Add LE scan result support to Binder API
am: 45480a0c51

* commit '45480a0c515da83d0b9d3488beea07a00b19aea1':
  service: Add LE scan result support to Binder API

service: Add LE scan support to Binder API
am: 6dc92022a0

* commit '6dc92022a02423778975e96363fcae787d47374a':
  service: Add LE scan support to Binder API

service: Add RegisterTestClient

Extract registration code into separate helper, that'll be used in
tests that require more than one client.

Bug: 25744656
Change-Id: Ib3ceac8bba02cf1eb83f4265849ec423ad588e42

service: add scan support to bluetooth-cli

Adde the start-le-scan and stop-le-scan commands to
bluetooth-cli for testing.

Bug: 25744656
Change-Id: I02d42fcca071eebefd82f027cd9b89af27116a00

service: Add LE scan result support to Binder API

Added Binder API documentation, bindings, and logic for BLE device
scan result.

Bug: 25744656
Change-Id: I5130a6b47438e266fab4a7b52b5372909824bc81

service: Add LE scan support to Binder API

Added Binder API documentation, bindings, and logic for BLE device
scans. Also added the start-le-scan and stop-le-scan commands to
bluetooth-cli for testing.

Bug: 25744656
Change-Id: I08c9d23ddd78cff1702495cf6d431cf5b9a7b894

Merge "service: Add OnScanResult Delegate event."
am: c18c15db15

* commit 'c18c15db150bf315226b2f4c1768605fdc983426':
  service: Add OnScanResult Delegate event.

Merge "service: Add Start|StopScan API to LowEnergyClient"
am: 7effa54c8f

* commit '7effa54c8fbe4cbf18a7d3fde99053b58e93f929':
  service: Add Start|StopScan API to LowEnergyClient

Merge "service: Parametrize LowEnergyClient with Adapter for tests"
am: a1bcabda3c

* commit 'a1bcabda3c80f0a95b49506554788d23636eb0a9':
  service: Parametrize LowEnergyClient with Adapter for tests

Merge "service: Add OnScanResult Delegate event."

Offload config save functionality to BTIF thread
am: 32b3a8a408

* commit '32b3a8a408b23aa28324ad9f56654c98fe2ac914':
  Offload config save functionality to BTIF thread

Merge "service: Add Start|StopScan API to LowEnergyClient"

Merge "service: Parametrize LowEnergyClient with Adapter for tests"

Offload config save functionality to BTIF thread

Offload config save functionality to btif thread from
timer thread as timer callback thread is critical in
a2dp playback case.

If the timer callback thread is busy in config save due
to IO operations, it may lead to a2dp audio choppy.

Fix to avoid the "bt_config.conf" file corruption from
the file system. This will avoid losing the paired
information in some corner case, such as abrupt power
off and on. This patch will ensure bt_config is saved to
NVRAM.

Bug: 24875861
CRs-Fixed: 953993
Change-Id: I893e9afefa89cbab6e7ddd8835ca77d3e316874c

Correctly verify return value of BTM_CancelInquiry
am: 4fc9732527

* commit '4fc97325278c6ce3c78b14ec26f32b85a15e784e':
  Correctly verify return value of BTM_CancelInquiry

Correctly verify return value of BTM_CancelInquiry

Change-Id: Iefb38023bc2b3e390eec14516f954e36f34ed5b4
Signed-off-by: Yan Laijun <yan.laijun@gmail.com>

service: Add OnScanResult Delegate event.

Added the LowEnergyClient::Delegate class and implemented scan result
reporting via the OnScanResult Delegate event.

Bug: 25744656
Change-Id: I050118c4f6761a7a32fa3ec8d9a1b41126a4c3ae

service: Add Start|StopScan API to LowEnergyClient

Added the StartScan and StopScan methods to LowEnergyClient and basic unit
tests. This currently only supports regular scans with no batch scan or
hw/sw filters.

Bug: 25744656
Change-Id: Iddf6e897377f90a5eef81ef36696b1d7074a7dab

service/hal: Add per-client Scan interface
am: 3f6aa07299

* commit '3f6aa07299fd9db634274fd22821b349f18b5e0f':
  service/hal: Add per-client Scan interface

service: Parametrize LowEnergyClient with Adapter for tests

Bug: 25744656
Change-Id: I4eee8098c264c2135c2bb4f02a2319ba0dfec978

service/hal: Add per-client Scan interface

Added a new per-client scan function to hal::BluetoothGattInterface.
The intention here is to push most of the per-client reference counting,
scan settings and filter coalescence below the HAL. This CL does this
first inside the Bluetooth daemon's HAL wrappers in a way that
represents what the future HAL scan API might look like.

This implements a basic reference counting scheme to share the global
controller scan session among different clients.

Bug: 25744656
Change-Id: I20c5cfc291be70d72576ebee014cc13544d5a299

service: use client_id instead of client_if
am: 6d596c0729

* commit '6d596c072995a876414b55a0471ef13146053b9c':
  service: use client_id instead of client_if

service: use client_id instead of client_if

Change-Id: Id045c81f748c99ab0525ce8869ef9cdd0207712e

Merge "service: Rename variable holding advertisement settings"
am: 8b3e3ed3d9

* commit '8b3e3ed3d9f9bc12c3ffeb380ca68ed3b5852ef6':
  service: Rename variable holding advertisement settings

Merge "service: Rename variable holding advertisement settings"

service: Rename variable holding advertisement settings

There will be more variables with settings. Make sure they
all have unique meaningful names.

Change-Id: I4b23fd3fa5a16f9132801b9141e4f8ed0be33be7

Merge "Additional headsets blacklisted for absolute volume"
am: 7632bee5a5

* commit '7632bee5a5a5ee501efe188f36a05dd830842072':
  Additional headsets blacklisted for absolute volume

Merge "Additional headsets blacklisted for absolute volume"

service: Make Adapter mockable.
am: 0a0d393213

* commit '0a0d3932136995a722cc125ef920679113d8ce0f':
  service: Make Adapter mockable.

service: Make Adapter mockable.

Turned the Adapter class into an abstract interface so that we
can inject a mock Adapter in tests where there is an Adapter dependency
but where we don't care about side-effects/stack-calls created from the
Adapter.

Bug: 25744656

Change-Id: Ibf7dd9a4cc1008cbb36e60f7b87cfcb4ef2bd5d8

Additional headsets blacklisted for absolute volume

Bug: 26070064
Change-Id: Ida1faec964982a4630f42ab378fe5b6dd6e21c16

Assert on memory allocation failure
am: c02acb7761

* commit 'c02acb7761d2539ab83c6eff22b7de5b5da51b78':
  Assert on memory allocation failure

Assert on memory allocation failure

This CL is a pre-cursor to cleaning up repeated memory allocation code
full of null-pointer checks. In the vast majority of cases in the
Bluetooth stack, a failed allocation is not recoverable and results in
undefined behaviour. By asserting, we can ensure we have a shot at
catching and fixing any issues not caused by OOM conditions.

Bug: 26494317
Change-Id: I53dff98c596068211934a1808f90de2d4484f952

Merge "service: Add global scan support getters"
am: 9f41ecbcb4

* commit '9f41ecbcb4a89ce64dbf1f046501ed78931b6db3':
  service: Add global scan support getters

Merge "service: Add global scan support getters"

Implement OOB pairing for LE devices using TK.
am: 1a5bb5f303

* commit '1a5bb5f30385af44d11ffba10e0f5af79f0ae422':
  Implement OOB pairing for LE devices using TK.

Implement OOB pairing for LE devices using TK.

This patch implements OOB pairing for LE devices using TK. Patches
for other pairing methods, and other transports will follow.

Bug: 22932952
Change-Id: Iad3c0c035de3b5a62ef24d3e3b655773fa03d5c1

Add script to change data types to native C types
am: 86b76c6241

* commit '86b76c6241ec2b0af06ab93300cfc51a5865021d':
  Add script to change data types to native C types

Add script to change data types to native C types

Please see source code for usage information and application.

Bug: 22948224
Change-Id: Ia2dbd618ddcdf13abf3c63d4649147f400e00cfe

A2DP task media alarm non-null protection
am: e04dbb3707

* commit 'e04dbb3707806be965f5669d3213263c0178ef26':
  A2DP task media alarm non-null protection

A2DP task media alarm non-null protection

Prevent the alarm from being started multiple times.

Bug: 26277592
Change-Id: I764f93bab5490bf5f3942a1beb706457197d3ec1

net_test_bluetooth: fix inconsistency in set/get name test
am: 19cd152d2f

* commit '19cd152d2fcf441b069672adbed517ec920f419c':
  net_test_bluetooth: fix inconsistency in set/get name test

net_test_bluetooth: fix inconsistency in set/get name test

Fixed an issue where the set/get name test would fail every so often
due to the fact that the old name property would become invalid after
the properties array was freed.

Bug: 25793348
Change-Id: I4513219da2fb947b3b199f25a61c308b9fced8b6

Fix bug where a bonded device could enter BONDING and BONDED states again.
am: 0b7fc58908

* commit '0b7fc589089ce4a96daa668b9d293ce01982424d':
  Fix bug where a bonded device could enter BONDING and BONDED states again.

Fix bug where a bonded device could enter BONDING and BONDED states again.

The following sequence of events was observed:
- start bonding with device A
- bond state for A goes from 10 -> 11 -> 12
- everyone's happy
- start bonding with device B
- bond state for B goes from 10 -> 11
- bond state for A goes from 12 -> 11 -> 12

The bond state for A should not have been changed in the last step
since it was not participating in any bonding procedure at the time.

The above sequence can be reproduced if a device D bonds with
A and takes on the slave role and then D bonds with B and switches
to a master role. When D performs the role switch, it receives an
updated link key from A.

Since the link key update procedure is tied in with the pairing flow,
we see spurious bond state changes. This CL checks the pairing control
block to see if D is, in fact, pairing with A and if not, it skips the
bond state updates.

Bug: 25870383
Change-Id: Ic6ff548dbe4e960c965bdc9ef5c50a263b9b3b22

Bluetooth: Remove std=c++11
am: 32579680eb

* commit '32579680eb9c52fa58b3eb7e3fd8a7de8f264983':
  Bluetooth: Remove std=c++11

Bluetooth: Remove std=c++11

It's the default now.

Change-Id: Ice463e6e74d56bffd00a80824721dbe57241ea06

Merge "Bluetooth: Silence unused-parameter warning"
am: 9a7624c052

* commit '9a7624c05228aafd3933eec73baf5a64d3b76f63':
  Bluetooth: Silence unused-parameter warning

Merge "Bluetooth: Silence unused-parameter warning"

Bluetooth: Silence unused-parameter warning

Silence these for now. Reduce build noise.

Change-Id: I787c31f60c8d24e3dd2bae8d1acaa7592b7e29ba

Merge "Fix absolute volume reporting if AVRC_ADV_CTRL_INCLUDED is false"
am: c0231ad1f9

* commit 'c0231ad1f96ac76cf31db0afde550bfc10b6ac7a':
  Fix absolute volume reporting if AVRC_ADV_CTRL_INCLUDED is false

Merge "Fix absolute volume reporting if AVRC_ADV_CTRL_INCLUDED is false"

Merge "Fix crash in HFP client\'s +COPS parsing code."
am: 83702c9ac1

* commit '83702c9ac1c825161e1163cbfb238215a076abd5':
  Fix crash in HFP client's +COPS parsing code.

Only send first 16 characters of operator name in +COPS.
am: a459df848e

* commit 'a459df848e03b0f2fb3f15d54404b48b94cff560':
  Only send first 16 characters of operator name in +COPS.

Merge "Fix crash in HFP client's +COPS parsing code."

Fix absolute volume reporting if AVRC_ADV_CTRL_INCLUDED is false

Without this change, setting AVRC_ADV_CTRL_INCLUDED to false would
result in the native code reporting the remote device supports
absolute volume even though we don't.

Bug: 26070064
Change-Id: I5b88fa0adb89983de28a216898a1d8957fa454a1

Fix crash in HFP client's +COPS parsing code.

If the Audio Gateway sends a malformed +COPS message (an operator
name > 16 characters) then the %n in sscanf format specifier is
ignored and sscanf will not assign a value to the appropriate
argument.

In such a case, the existing code will perform pointer arithmetic
using an uninitialized stack variable as an offset which may result
in pointing to an invalid memory address. When that memory is
subsequently dereferenced, we observe a crash.

This change ensures that the stack does not crash even if an invalid
+COPS message is sent from the Audio Gateway.

Bug: 24871011
Change-Id: I9bb42c75bcd90487831fc6950c571c87098559e7

Only send first 16 characters of operator name in +COPS.

According to the HFP spec, the operator name is required to be at
most 16 characters long.

Bug: 24871011
Change-Id: I9987de57a327348fc2203b9502e9df446a87793f