Merge changes from topic 'ble-scan-accounting'
am: 725d2dac7b  -s ours

Change-Id: Ic992092816435f0b1b24c762abd06722e32b7c58

Merge changes from topic 'ble-scan-accounting'

* changes:
  Reset ble scans in BatteryStats on start up
  Use the incoming WorkSource to notify batterystats of BLE scans

Merge "Fix a deadlock in service discovery callbacks"
am: 6fcd0081eb

Change-Id: I6e343bbc3ef5c54e372687ee989e1baa133a01c8

Merge "Fix a deadlock in service discovery callbacks"

Reset ble scans in BatteryStats on start up

Bug:22718669
Change-Id: Id31431bf35db3a52c8f1af1f210247d7ea20f6ed

Use the incoming WorkSource to notify batterystats of BLE scans

BatteryStats is going to start tracking bluetooth scanning, so
we start by passing along the WorkSource we were given (or we created
if none was specified).

This is only done for BLE scanning at the moment, and only for regular scans.
Batched scans will also need to be considered.

Bug:22718669
Change-Id: I15dd7c912320552323aaf9d713b4bb3224ffffec

Fix a deadlock in service discovery callbacks

onSearchCompleted calling gattClientGetGattDbNative directly, might
cause deadlock if the jni_workqueue is full.

Bug: 30835367
Change-Id: I05de735aa189dd2d250f2c5816d38c2ddcabb864

Merge "Fix NPE in permissionCheck"
am: 64ed2357e3

Change-Id: Ief4258ed339822801b97c9a85309608d59e98286

Merge "Fix NPE in permissionCheck"

Fix NPE in permissionCheck

permissionCheck can throw NPE when notification is received before
GATT database is discovered. Assume that permission is granted when
receiving notifications for unknown characteristic.

Bug: 30853729
Change-Id: I02aab1197b036584b2021d76f290c2c57db24a19

Merge "Add LE Secure Connection data parsing (2/4)"
am: 29101f7fd4

Change-Id: I962b635c44d2573a62936ab58bf8577ef19000cc

Merge "Add LE Secure Connection data parsing (2/4)"

Add LE Secure Connection data parsing (2/4)

Bug: 30460956
Change-Id: Ie5c3ad96e7a834c9dc0a6791cf2cd24ecaeb927e

Merge "Prevent LE scanning apps from being wrongflully downgraded to opportunistic"
am: c9db1189d8

Change-Id: I216d43f760661274145b7140efb68e5c01528073

Merge "Prevent LE scanning apps from being wrongflully downgraded to opportunistic"

Prevent LE scanning apps from being wrongflully downgraded to opportunistic

Bug: 30648104
Change-Id: I9d749a20eefdbeb126fa25a425d636205a22b2ef
(cherry picked from commit a1c7315ed05df04599e7cdc5f4bcf7d3ec8d3392)

Merge "Change URI accessed to properly update read status"
am: f337ee1fe0

Change-Id: Iad683df58a9067e275bc07c0701cce2dfb94fd83

Merge "Protect against SecurityException generated by query"
am: 369f6662fa

Change-Id: I9e379db4c8d54a7733bba873da2cb6abc73493e6

Merge "Change URI accessed to properly update read status"

Change URI accessed to properly update read status

Bug: 28970262
Change-Id: I07d9ab24d10396baa4ddbfbdfad5b1eec1b8e9d5

Merge "Protect against SecurityException generated by query"

Protect against SecurityException generated by query

Bug: 30444661
Change-Id: I1d93645f1d4475cf603cda8bb28df0b960ccb445

Merge "HFP 1.7 profile update (4/4)"
am: 1b9d2081e0

Change-Id: Ib18e452187c7d673e8ae387da99bf9e6faf43bda

Merge "HFP 1.7 profile update (4/4)"

HFP 1.7 profile update (4/4)

-> Added HF Indicator support.
-> Sent a Broadcast intent upon receiving AT + BIND and
   AT + BIEV Events with the required parameters
-> Support for sending +BIND response on indicator
   status change

Bug: 19983867
Change-Id: I2580fc771080f1bc92fc4ddd5ca7c7cb57773183
(cherry picked from commit 2ab50a41dac634a78cef60a419c3bcc848c1aa78)

Merge "Remove write type from GATT descriptor writes (2/4)"
am: ddd61d3443

Change-Id: I08ecf740a524cb511d86aa771809ccbc4afbf57b

Merge "Remove write type from GATT descriptor writes (2/4)"

Remove write type from GATT descriptor writes (2/4)

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

Change-Id: I557a55d2ba8e998bce552e40ec7031d91bc82752

Merge "Move file lookup to its own thread for beaming"
am: 001643829c

Change-Id: I6849f122ad600984b8e99a38576343d67ae86033

Merge "Move file lookup to its own thread for beaming"

Move file lookup to its own thread for beaming

Bug: 28973021
Change-Id: I40f6d32ffadd58e60dbd1e44802a629b9fab64e4
(cherry picked from commit 9b6816400cbb35df12eaf5fd59d1996d3d0dad84)

resolve merge conflicts of ebe5b95 to stage-aosp-master

Change-Id: I8c9ef03ff5e26128c9c6d69635355c4637075a45

Merge "GATT Server refactoring (3/4)"

GATT Server refactoring (3/4)

Bug: 27999121
Change-Id: I45fec0fa6d0e8e26e40e1856ba9fee6f7fbd4f7b

Merge "RemoteDevices: Optimize adding new remote devices"
am: 55e58872c3

Change-Id: I85799db7e1fd3069815254afd2d6ebb559894d5a

Merge "RemoteDevices: Optimize adding new remote devices"

RemoteDevices: Optimize adding new remote devices

RemoteDevices keeps track of the Bluetooth devices which have been
seen since the Bluetooth stack was initialized.  Each time a remote
device is discovered, it is added to a hash map.  When the size of the
hash map grows beyond a few hundred devices, the performance degrades.

This patch:
 - creates the address string once per search
  reducing the average search time with 300 devices from ~20ms to ~1ms
 - uses the hash of the address string to search
  further reducing the time with 1200 devices from ~2ms to ~1ms

HashMap<BluetoothDevice, DeviceProperties> became
 HashMap<String, DeviceProperties>

mDevice now stores the BluetoothDevice in each instance of
DeviceProperties.

TEST: Used System.nanoTime() to measure the elapsed time of
      getDeviceProperties()

Change-Id: Id44886143fb2910955294341d4938d638c72f7f9
Signed-off-by: Myles Watson <mylesgw@google.com>

Merge \"Fix bugs in play status and play pos notifications\"
am: 764eb24d86

Change-Id: I6a5119996813069494b3c7ee666025474d1d9763

Merge "Fix bugs in play status and play pos notifications"

Fix bugs in play status and play pos notifications

Fix multiple issues with status notificatios:
  * Playback state was not being sent when state wasn't sent
    accompanying a metadata change
  * Play state was being sent without an update
  * Position was being sent without a change and with the wrong
    indicator
  * Position was not being scheduled for update when playing.
  * Position was being scheduled for update when paused / stopped.

Bug: 30459664
Bug: 29400658
Bug: 28639383
Change-Id: Ic550dcab0430be45187b0267cc77875740388a2a

Merge \"Add foreground flag to AG_CALL_CHANGED broadcast intent.\"
am: df005bf244

Change-Id: I0c8d2337e82c472fe4bd07d90f61c8b39e56dd80

Merge "Add foreground flag to AG_CALL_CHANGED broadcast intent."

Merge \"MAP: Remove redundant call to sendShutdownMessage to prevent crash\"
am: 0786878b88

Change-Id: Ic62f273f84ae4b6b3c4d63ae089511b19723c1e4

Merge "MAP: Remove redundant call to sendShutdownMessage to prevent crash"

MAP: Remove redundant call to sendShutdownMessage to prevent crash

Usecase:
Perform BT off/on repeatedly.

Issue:
Bluetooth crashes.

Fix:
Remove redundant call to sendShutdownMessage() from stop API to prevent
MapServiceMessageHandler becoming null and being dereferenced.

Bug: 29149190
Change-Id: I739664975043c4a1895fd2113869c87cdb704338

Merge \"PAN: Avoid reconfiguration of bt-pan interface in NAP role\"
am: aa83abd905

Change-Id: I084e393e11a893744a6b05b9a9ed3227b0df121a

Merge "PAN: Avoid reconfiguration of bt-pan interface in NAP role"

PAN: Avoid reconfiguration of bt-pan interface in NAP role

Issue:
Browsing is not working with remote PANU Device(s).

RCA:
DUT is in NAP role, bt-pan interface is configured for NAP role,
when 1st remote PANU is connected. bt-pan interface is being
reconfigured, whenever 2nd PANU is connected, which leads bt-pan
interface to go in bad state. BT is able to write Ethernet packets on
bt-pan interface(TUN driver) after reconfiguration, but it's
unable to read Ethernet packets from bt-pan interface(TUN driver).

Fix:
If bt-pan interface is already configured in NAP role,
then do not reconfigure bt-pan, when 2nd PANU is connected.

BUG: 29992925
Change-Id: Ib6e0cb91d573649af45ffe25c7095a4bd74a5022

Merge \"Fix AVRCP crash, position and state tracking\"
am: 5b8a0db93b

Change-Id: Ic553fbdb7e2bbb2356b0525e8d54a2a59945d27e

Merge "Fix AVRCP crash, position and state tracking"

Fix AVRCP crash, position and state tracking

When the MediaController goes away (media stops) a NPE would cause a
crash.

Initialize the position so that an initial notification gets sent
when position is requested, send it when the music is paused, and
send play state whenever the app sends us a play state update.

Bug: 30039744
Bug: 29416450
Change-Id: I44397b3dc40d328fa75a2c15875fad298a5090dd

Add foreground flag to AG_CALL_CHANGED broadcast intent.

To fix the issue that the watch is still ringing after answer an
incoming call on phone. It is caused by the ACTION_CALL_CHANGED
broadcast reception processing delayed.

Bug:29931814

Change-Id: I8d8f297c3a13dfe65674eca94724a50d82618f5e
Signed-off-by: m00230717 <m00230717@notesmail.huawei.com>

Merge \"Check for call/call setup state before creating SCO\"
am: 598902d8e2

Change-Id: I74d86b349b86efb7c82f114aa72be4a4af55ce0a

Merge "Check for call/call setup state before creating SCO"

Check for call/call setup state before creating SCO

Use case:
1. Run HFP AG PTS test cases.
or
1. Pair and connect phone with Sony xba-bt75 Bluetooth headset.
2. Make incoming call and accept it.

Failure:
PTS test cases fail. There will be no call audio on
Sony xba-bt75 headset.

Root cause:
Telephony asking to create SCO before incoming call
is accepted.

Fix:
If there is no active/held call, outgoing call setup
in progress, we should not create SCO. Telephony will
ask to create SCO for incoming call before the call
becomes active, this creates IOT issues and fail below
PTS test cases.
TC_AG_ACC_BV_08_I
TC_AG_ACC_BV_10_I
TC_AG_ACC_BV_11_I
TC_AG_ACC_BV_12_I
TC_AG_ACC_BV_13_I
TC_AG_ACC_BV_14_I

Bug: 29417803
Change-Id: I0c1002b31da65cbc0b0e74b590468d37ea51d065

Merge \"Refactor playback position tracking\"
am: 0d83e2969a

Change-Id: I02e4595765448e2b475b7192d8468eb588a774c5

Merge "Refactor playback position tracking"

Refactor playback position tracking

Use PlaybackState directly to track the position through time.  Update
the state when necessary (track changes, etc.)

Remove complex logic and centralize sending of Play Position
Notificaiton response and scheduling.

Update dumpsys and add current package session.

Bug: 29416450
Bug: 29606822

Change-Id: Ie82b48d6d7f9e1a05b64835731433d05a3a3a41a

Merge "Import translations. DO NOT MERGE" into stage-aosp-master

Import translations. DO NOT MERGE

Auto-generated-cl: translation import

Bug: 29390361
Change-Id: Idcc9b8fe3f43c8d81a5e11bd6e42f2096406775d

Merge \"Fix misc-macro-parentheses warnings\"
am: 89902a185b

Change-Id: Id01ad59201fd97150ecb1ecc539255f64a021246

Merge "Fix misc-macro-parentheses warnings"

Merge \"Fix work profile app not being able to create bond\"
am: 61ca891c48

Change-Id: I38ddc0e7e26b70c8041672af5fddd7c3ade601a8

Merge "Fix work profile app not being able to create bond"

Fix misc-macro-parentheses warnings

Add parentheses around macro arguments used beside operators.

Bug: 28705665
Change-Id: I6ef4764d3565b7ce6eb18570a6bf8c98b6b3631c

Use the live AdapterService in ProfileService.
am: d4152d61ac

Change-Id: I4755e0a99783caf68b4c6a84f9507d6764a87b87

Use the live AdapterService in ProfileService.

Previously, if AdapterService was null while ProfileService is
created, the ProfileService would never work because it would
store a stale reference to the AdapterService.

This change ensures that ProfileService accesses the current
AdapterService at any given time.

Bug: 29519304
Bug: 29610505

Change-Id: I25efbf4a58932f519ac9f0b89c4112a442c994eb
(cherry picked from commit b55c6a1f4ce60ad552e9c9a6d06fab6781a4dadb)

Close output stream before getting response
am: e026ad5004

Change-Id: I55fc6fee16f4e25817fc6cbad37e83052a41a7dd

Close output stream before getting response

If the output stream is still open after finishing the sending of
a file, a CONTINUE response code is received instead of an OK code.

Bug: 29583227
Change-Id: I1b2515e8fe5eec208bbbe1df17f96cc20c5ea884

Merge commit '303ef0bd1343806adba9b00e39ca05b7c9e50a4f' into manual_merge_303ef0b

Change-Id: I9cfbac32c1a9758860e856bd0cbfa4f9985d6690

Merge "Prevent energy data overflow" into nyc-dev

Fix how file size is calculated if not provided
am: 120ba1db4d

Change-Id: I891ecc0ba9c66d37ac785df4412a0ab69390026b

Fix how file size is calculated if not provided

Currently if there is no file size provided, InputStream.available()
is used to guess how many bytes are available. According to the
java documentation, available() should not be used to calculate
file sizes, as it isn't guarenteed to return the total number of
bytes in the stream. This is fixed by reading the stream to
calculate the length, then resetting the stream.

Bug: 29334784
Change-Id: Ic851c46d053157e4d5404352d76f9ff87a509607

Prevent energy data overflow

- Stores energy data as longs
- Monitors for overflow and logs + discards sample

Bug:27598905
Change-Id: I442c0f63adb5197a45e7982e68f1260c856ed351

Merge \\"Vectors as parameters to GATT (2/3)\\" am: d3b3006b32
am: f35b6cf04e

Change-Id: I0403260b56219dfdb6c5ac122b327c2fd9d9daf6

Merge \"Vectors as parameters to GATT (2/3)\"
am: d3b3006b32

Change-Id: Idfd8188d8414e210ef45217a78b0118bc2e02aec

Merge "Vectors as parameters to GATT (2/3)"

Merge \\"Vectors as parameters to GATT write/indicate (2/3)\\" am: 6ff73f0db0
am: 5eef5be110

Change-Id: Ib5ec0f3a39a7e85bc333bd5c8b7096a4906ff467

Merge \"Vectors as parameters to GATT write/indicate (2/3)\"
am: 6ff73f0db0

Change-Id: Icbe4d10addaa4ef47d94b34ae7eed15fb621307e

Vectors as parameters to GATT (2/3)

Bug: 29005882
Bug: 28485365
Change-Id: I3c1a53ed2766b6b7b8d914da3d19875dd67ddd7d

Merge "Vectors as parameters to GATT write/indicate (2/3)"

Vectors as parameters to GATT write/indicate (2/3)

Bug: 29005882
Bug: 28485365
Change-Id: Iac09cb6dc66cf48e8c52030de44d27afe24b0b99

Merge \\"Fix sender name string to match MAP spec\\" am: 6b9c9d4458
am: a2823f7e34

Change-Id: I76946cbaa6d1e726beb6c7c6a6e44ca485e3ea94

Merge \"Fix sender name string to match MAP spec\"
am: 6b9c9d4458

Change-Id: I7d528b876ebaf7ed0e048cdd72eb3521c1f1a309

Merge "Fix sender name string to match MAP spec"

Fix sender name string to match MAP spec

In section 3.1.6 of the Bluetooth MAP spec version
1.2.2, the field name should be "sender_name" instead
of "senderName".

Bug: 26904410
Change-Id: I0dacaf50b0830239d2c2e7e787dc7998f3ed6723

Import translations. DO NOT MERGE
am: 720b9fd1a3  -s ours

Change-Id: Ifc431fcf953313840ba1fc5cccd5c161c4e68d22

Import translations. DO NOT MERGE

Auto-generated-cl: translation import

Bug: 29390361
Change-Id: I4ab9b1d07af08a87f41da8e95072fb018837a5bb

Merge \\"Fix null callback in updateCurrentMediaControllers\\" am: f533643e2b
am: ed7ceb136f

Change-Id: I9eba1bbc3e7828f8d6d6f58bba7bfabf1b103cde

Merge \"Fix null callback in updateCurrentMediaControllers\"
am: f533643e2b

Change-Id: I29edb9f667095e0089f864aaabd5ef0b9090f26e

Merge "Fix null callback in updateCurrentMediaControllers"

Fix null callback in updateCurrentMediaControllers

Change-Id: I0f134ce147b8a7396c87900fb4770606d53dd765

Merge \\"AVRCP: Handle multiple MediaController instances\\" am: d6d2993587
am: 9350175ee7

Change-Id: I13fa1202796805ff6cc55a804ebbcd97b9d29e42

Merge \"AVRCP: Handle multiple MediaController instances\"
am: d6d2993587

Change-Id: I3c041a36caaf4ca5ef9d12ee0cc389acab1bfce5

Merge "AVRCP: Handle multiple MediaController instances"

AVRCP: Handle multiple MediaController instances

Find a MediaController with metadata to show when there are more than
one.

Display more information about the MediaControllers so that we can tell
which packages are being used.

Bug: 29169493
Change-Id: Ifecad6eb44d3461c03685b6650d469721752dfdd

Merge \\"Prevent crash on invalid URIs, remove ACTION_OPEN\\" am: b85456f9bd
am: 88f03e1262  -s ours

Change-Id: Id222bb90fdf2a2e7965c37319c53d85811a5680b

Merge \"Prevent crash on invalid URIs, remove ACTION_OPEN\"
am: b85456f9bd

Change-Id: I0dff47e550e1d836a88ef4ec8a297307cd098abc

Merge "Prevent crash on invalid URIs, remove ACTION_OPEN"

Prevent crash on invalid URIs, remove ACTION_OPEN

Previously if an invalid URI for a notification is received
with ACTION_OPEN, Bluetooth crashed. This fixes this.

It also removes ACTION_OPEN as it is internal-only and unused.

Bug: 28940687
Change-Id: I54e17b7180e770bb8a5cf48c3d4b2a35657cb67f