Merge "MAP: CleanUp for SDP record in MAP Service shutdown"

MAP: CleanUp for SDP record in MAP Service shutdown

Remove SDP record per MAS Instance during MAP Service
shutdown procedure.
Add more relevant debug logging.

Change-Id: I222ad97d719b9a2c45db3ab8526dd3de9c553f3d

Merge "MAP: GETMessagesLisitng include mandatory field always."

MAP: GETMessagesLisitng include mandatory field always.

Pre condition:
Atleast one SMS is present in any of the folders
with no subject.

UseCase:
1. Connect MAP connection from MCE to DUT
2. Set folder to Telecom/msg/inbox
3. Execute MessageListing with the required data

Failure:
Mandatory field "subject" is not included for few entries
in messagesListing.

Fix:
For GETMessagesLisitng  ensure blank subject string is
set to include this mandatory feild and handle possible
"null" value fetched from database.

Change-Id: Ie29a61c83b22db9008184090a2f5b17d1f34b72b

Merge "MAP: Cleanup PendingIntent in cancelUserTimeoutAlarm."

MAP: Cleanup PendingIntent in cancelUserTimeoutAlarm.

Handle cleanup for timeout Intent and pending Intent
in cancelUserTimeoutAlarm API.

Change-Id: I4f4184785675b2deffa3be8c9dd70b2673020b51

Merge "MAP: Handle possible NPE while disconnect map."

MAP: Handle possible NPE while disconnect map.

Precondition:
Verified with IOT: MecApp in MCE role

UseCase:
1. Connect MAS
2. Register Mas Instance via MNS
3. Disconnect MAS

Failure:
Bluetooth App crash sometimes when disconnect map profile.

Root Cause:
The issue is hit when MAS disconnect is finished before MNS unregister.
In this case NullpointerException is happening because of clearing
BluetoothMapContentObserver already as a part of cleanup.

Fix:
Handle Null pointer exception while last
MAS instance disconnect is handled before MNS
unregistration.

Change-Id: I4e72038a5ddc6c3ac8c515def57cb4f98fe87675

Merge "MAP: Add MNS SDP Search during setNotificationRegistration."

MAP: Add MNS SDP Search during setNotificationRegistration.

Precondition:
Verified with remote MCE device that starts MNS server
only before issuing setNotification Registration to MSE.

Use Case:
1. Discover and Pair with MSE from MCE.
2. Connect to MSE on Mas insance 0.
3. Enable Notification Registration for MAS Instance 0

Failure:
MNS connection happens over RFCOMM
and MAS Instance is connected over L2CAP.

Root Cause:
Current design performs MNS SDP search only at the time
of successful MAS Instance Connect. MNS connect implementation
tries connection over insecure RFCOMM channel based on UUID
when MNS Server record is not available.

Fix:
Add implementation to check and perform MNS SDP search
if this info is not available when MCE requests
setNotificationRegistration. This fix is required to
handle MCE implementation that start MNS Sever and
expose SDP only before sending setNotificationRegistration.

Change-Id: If1e8d33dcac078f04d13b8bacf8e1d1513c119e1

Merge "MAP: Declare and fetch only 'INTERESTED_MESSAGE_TYPE_CLAUSE' MMS."

MAP: Declare and fetch only 'INTERESTED_MESSAGE_TYPE_CLAUSE' MMS.

Precondition:

Messaging App Settings configured to receive delivery reports for MMS.

Use Case:
Follow standard steps to perform messagesListing type MMS from MCE.
1. Connect to MSE on Mas instance 0.
2. Set Path to root.
3. Set Path to telecom.
4. Set Path to msg.
5. Set Path to inbox.
6. GET messagesListing type MMS.

Failure:
Extra entries like delivery reports that are stored in DB and
which are not user interested types are also included
in MSE messageListing for MMS type.

Fix:
OMA-TS-MMS-ENC MMS Standard defines many types in X-Mms-Message-Type.
Extra entries like delivery reports which is stored in DB are not
required to be shown to user, so these should be filtered, and
only  m-send-req (128), m-retrieve-conf (132), m-notification-ind (130)
types should be fetched and shown to user in messageListing response.
This patch removes these unecessary message types from MMS Messaging Listing.

Change-Id: I1cb5d6e58477a32a89693f555a268e0855267b9a

Merge "MAP: Synchronize notifcationReg reply and MNS connect action."

MAP: Synchronize notifcationReg reply and MNS connect action.

Use Case:
1. Connect to carkit
2. Disconnect connection with carkit.
3. Reconnect to carkit

Failure:
MNS Failure during Reconnection.
User will in some timing conditions not get notification
of new messages and then turns OFF/ON the carkit.
MAS is connected fine but for MNS car sends BAD REQUEST.

Root Cause:
This isssue is reported only with one particular carkit in MCE role,
where in MNS Connect request issued before acknowleding
the setNotificationReg Request.

Fix:
Synchronize handleRegistration method in handler thread to ensure
setNotificationReg response from MSE is sent before requesting
a MNS Connect to MCE.

Change-Id: I30559c643af7733754fce551b97e31543a0cb160

Merge "MAP: Synchronize mMsgList to avoid extra MessageDeleted MNS event"

MAP: Synchronize mMsgList to avoid extra MessageDeleted MNS event

Precondition:
There are no messages in Messaging App.

Use Case:
1. Connect to MSE on Mas insance 0,
   Register and Enable Event Notifications from any MCE.
2. Set Path to root.
3. Set Path to telecom.
4. Set Path to msg.
5. Set Path to outbox.
6. Compose an SMS and add the recipient.
7. Click PushMessage option.

Failure:
Map Server sends a unexpected MessageDeleted EventReport Notification
even though PushMessage received from MCE is processed Successfully.

Root Cause:
Issue happens only for one corner case, where onChange() gets triggered
while processing a local initiated pushMsg before the entry is appended
to database, as local MsgListSms contains an extra entry for newly added
msg from MCE.

Fix:
Synchronize contentObserver mMsgList SMS/MMS to update list with
local initiated changes before fetching complete database list and
handling onChange() to avoid extra MessageDeleted MNS event.

Change-Id: Id4d5dad4d2b4b1e17f40218c11d13e39c9b885a0

Merge "Send error for if VR is initiated during call."

Send error for if VR is initiated during call.

Use case:
Connect to BT headset.
Make a call. Let the call audio be on BT headset.
Initiate voice recognition(VR) from BT headset.

Failure:
AG is not sending error for VR when initiated during call.
Headset might go into bad state if AG don't send error.

Rootcause:
AG is not sending error for VR when initiated during call.

Fix:
When VR is activated during call, send error response to headset.

Change-Id: Ib12d464fc6a6948ddf22250a65a9e5bfeb4a8fb1

Merge "Fix register for notification logic"

Fix register for notification logic

Bug: 27790582
Change-Id: I118071b907ec16faad401ba6583ca395d08d6d6e

Merge "Use handles to identify GATT attributes (3/4)"

Use handles to identify GATT attributes (3/4)

Bug: 27778668
Change-Id: Id6ec1229998c9fc2ecd4d0a908405a7148703bec

Merge changes I38c8ec64,Iac544226

* changes:
  Make sure wakelock is held before releaseing it
  Do not delete/re-create wakelock on acquisition/release

Make sure wakelock is held before releaseing it

Bug: 27748109
Change-Id: I38c8ec640f5def7c53348d2dd96a0a2492ff00cc

Do not delete/re-create wakelock on acquisition/release

Also provide additional status for wakelock acquisition/release errors.

Bug: 27721443
Change-Id: Iac544226fed4a73a50bd954e74d96edd4bed93d2

Merge "Add scan results to scan stats"

Add scan results to scan stats

Ex:
GATT Client Map
  Entries: 1

  no.nordicsemi.android.mcp
  LE scans (started/stopped)         : 3 / 3
  Scan time in ms (min/max/avg/total): 3050 / 6901 / 4815 / 14447
  Total number of results            : 338
  Last 3 scans                       :
    1970/01/01 08:20:06 - 6901ms 0 results
    1970/01/01 08:20:13 - 4496ms 171 results
    1970/01/01 08:20:39 - 3050ms 167 results

Bug: 27294154
Change-Id: I32fd6da035f2c37302a105733ac6a9817547b7a3

Merge "Change how services are passed up to the stack"

Change how services are passed up to the stack

Bug: 27455533
Change-Id: I4c8218fb4c7437ab4597071ad9b580a797e34339

Merge "Print bluetooth state as a string"

Print bluetooth state as a string

Bug: 27294154
Change-Id: I992f720d94be1f0762424ebad36ec1b6af58600b

Merge "Add enabled time to dump output"

Add enabled time to dump output

Ex:
Bluetooth Status
  enabled: true
  state: 12
  address: F8:CF:C5:CE:F5:69
  name: AOSP on Shamu
  time since enabled: 00:00:32.999

Bug: 27294154
Change-Id: Ie65964411e2095434a082f3d988133190c1f2f59

Merge "Move ScanStats to its own file"

Move ScanStats to its own file

ScanStats was growing too large and had many components so
it was moved into its own file.

Bug: 27294154
Change-Id: Ic20c9e74d5d8b074f7e966625fb1bffab6c94f2d

Merge "Prevent javacrash when TELEPHONY_SERVICE is not found"
am: 8367ebbea6

* commit '8367ebbea6f2298736457345cefe28882ef2e60a':
  Prevent javacrash when TELEPHONY_SERVICE is not found

Merge "Prevent javacrash when TELEPHONY_SERVICE is not found"

Prevent javacrash when TELEPHONY_SERVICE is not found

The API context.getSystemService can return NULL
A Javacrash can happend when trying to obtain the TELEPHONY_SERVICE
and getting null instead.

The patch warn about the value of the instance of the service returned.
It also log and prevent its use in function startListenForPhoneState and
stopListenForPhoneState.

If the TELEPHONY_SERVICE is not available, the internal variables wont
change.

Change-Id: I9e4abc88a58f7f037bbc4f3f7c1cf1b44328ce38
Signed-off-by: franck Lenormand <franckx.lenormand@intel.com>
Signed-off-by: Zhiquan Liu <zhiquan.liu@intel.com>

Merge "MAP: Support messageListing PARAMETER_MASK_ALL_ENABLED by default."
am: 6dfb1505ad

* commit '6dfb1505ade8b7543f1ffaaf28d80f4bcac45ab8':
  MAP: Support messageListing PARAMETER_MASK_ALL_ENABLED by default.

Merge "MAP: Support messageListing PARAMETER_MASK_ALL_ENABLED by default."

MAP: Support messageListing PARAMETER_MASK_ALL_ENABLED by default.

Support application parameter mask to enable and send across
all optional and mandatory feilds in GET messagesListing response
instead of only mandatory feilds to handle IOT Specific behavior
that donot include parameter mask attribute in messageListing request.

Some carkits in MCE role show read messages as unread when
"read" feild (optional) is not included in messagesListing
response by default.

Change-Id: I1fc65d85c35573e21d0c9d5bfdefd58de2dcf3ae

Merge "Bluetooth: Fix to avoid PAN JNI crash due to ENABLE_TIMEOUT"
am: 954f845ccc

* commit '954f845cccb897b47fc22753ffca3da3748ff8ed':
  Bluetooth: Fix to avoid PAN JNI crash due to ENABLE_TIMEOUT

Merge "Bluetooth: Fix to avoid PAN JNI crash due to ENABLE_TIMEOUT"

Bluetooth: Fix to avoid PAN JNI crash due to ENABLE_TIMEOUT

A case, where BT ON/OFF is running in the loop(stress). In one
of the iteration, BT ON gets failed due to ENABLE_TIMEOUT.
But BTIF for PAN had already been initialized. SSR cleanup is
triggered due to BT on failure. So PAN JNI callback obj gets deleted,
PAN control_state_callback is triggered just after PAN JNI cleanup.
BT gets crashed. Checking for NULL to avoid crash in any unknown
scenario.

Change-Id: Ie95470348f11be6c9eccc1e4ac3869d6789f3f6c

Merge "Remove unused callbacks"
am: 975faa165d

* commit '975faa165d52a87faa00b56d06fccd9b2a0da865':
  Remove unused callbacks

Merge "Remove unused callbacks"

Remove unused callbacks

Bug: 27455533
Change-Id: I743b8ba7187a397f088ac720f239e368dd84a7a2

Merge "Stop accessing statemachine after cleanup"
am: 0ef82f788d

* commit '0ef82f788d6362b21a06792d3189a7a57ff9be38':
  Stop accessing statemachine after cleanup

Merge "Stop accessing statemachine after cleanup"

Stop accessing statemachine after cleanup

Usecase:
-pair and connect DUT with Remote
-Start A2DP streaming
-Initiate disconnection from DUT

Failure:
Bluetoth crashed after initiating disconnect from DUT

Rootcause:
Accessing state machine context after cleanup.

Fix:
Bluetooth: Added null check to stop accessing state machine after cleanup

Change-Id: I85ca55aaa0b7604125e45a62e32ae02eca605e43

Merge "Get whole GATT db, instead of one by one element."
am: 34ab8dd613

* commit '34ab8dd6136e674715d2732d0b610cb36008a720':
  Get whole GATT db, instead of one by one element.

Merge "Get whole GATT db, instead of one by one element."

Get whole GATT db, instead of one by one element.

This patch uses new get_gatt_db method instead of current set of
callback to obtain GATT db.t

Bug: 27455533
Change-Id: I52e12253dd850ce8de35297c941638189cbbed1e

Merge "Proper handling of BTRH command in HFP client"
am: e8915edfdc

* commit 'e8915edfdc2d0c28d1dec94ae848566f6f7de471':
  Proper handling of BTRH command in HFP client

Merge "Proper handling of BTRH command in HFP client"

Proper handling of BTRH command in HFP client

Usecase:
1. Enable hf client role and connect to AG.
2. Run PTS test case RHH/BV-04.

Failure:
PTS test case fails since the test expects the call to be active.

Rootcause:
Hf client is not sending AT+BTRH for accepting held call

Fix:
Remove unnecessary check against of action policy flag while
executing BTRH command

Change-Id: Iebd5e02f3bf70cc39288219340cd47ebad3d8349

Merge "Add more statistics to scan logs"
am: 939ea0884b

* commit '939ea0884bbfddba164b43ab5e2340a015448d02':
  Add more statistics to scan logs

Merge "Fix warnings in packages/apps/Bluetooth"
am: 53dd65a5dd

* commit '53dd65a5dd14f6133103daab3d8a0ca6b35329d5':
  Fix warnings in packages/apps/Bluetooth

Merge "Add more statistics to scan logs"

Add more statistics to scan logs

Added length of connection time to connections. Also added scan type
(opportunistic/background) to last X scans.

Example of new print:
GATT Client Map
  Entries: 2

  no.nordicsemi.android.mcp (Registered)
  LE scans (started/stopped)         : 2 / 2
  Scan time in ms (min/max/avg/total): 1604 / 7274 / 4439 / 8878
  Last 2 scans                       :
    1970/02/19 21:30:20 - 1604ms
    1970/02/19 23:59:00 - 7274ms
  Application ID                     : 6
  UUID                               : 0c452531-a57e-44df-9ac6-e2b780d2f715
  Connections: 1
    6: FF:08:11:53:44:A1 10008ms

  com.example.apanicke.bletest (Registered)
  LE scans (started/stopped)         : 2 / 2
  Scan time in ms (min/max/avg/total): 2946 / 9996 / 6471 / 12942
  Last 2 scans                       :
    1970/02/19 21:30:07 - 2946ms Opp
    1970/02/19 21:30:17 - 9996ms Opp
  Application ID                     : 5
  UUID                               : f25c1bdf-559f-43a6-8348-40ca92975b33
  Connections: 0

Bug: 27294154
Change-Id: I7460fb1568f20a1ce13b01d101517852cf9aec4a

Merge "Fix warnings in packages/apps/Bluetooth"

Fix warnings in packages/apps/Bluetooth

Bug: 27301103
Change-Id: I2f2698a2fb3f7ca7031a803e43a398f59bd32cdb

Merge "Protect ScanStats from race conditions"
am: 3e1e2f6e9e

* commit '3e1e2f6e9ecd3604c63e96ca535a10bebf87d6e2':
  Protect ScanStats from race conditions

Merge "Protect ScanStats from race conditions"

Protect ScanStats from race conditions

Bug: 27294154
Change-Id: Icc0b0a6cfb93fca4a9a2aee49b94c0be1d62527f

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

Import translations. DO NOT MERGE

Auto-generated-cl: translation import

Bug: 27293916
Change-Id: Ie02519e03cdb8bead5cfca8e424de0184482580d

Merge "Fix logic for repeated record scan calls"
am: 3f5e211e2b

* commit '3f5e211e2bb4320db5a9f8af3fb29623983ff72e':
  Fix logic for repeated record scan calls

Merge "Fix logic for repeated record scan calls"

Fix logic for repeated record scan calls

If recordStartScan or recordStopScan is called multiple times,
there are issues where logged values are incorrect.

Bug: 27294154
Change-Id: Ie049e917ae26fec4c50874ed703e404146c15044

Merge "Mark background and opportunistic scans in scan statistics"
am: fcdb8bd4a3

* commit 'fcdb8bd4a3aebeecb89bd733dd678263e45e892b':
  Mark background and opportunistic scans in scan statistics

Merge "Mark background and opportunistic scans in scan statistics"

Mark background and opportunistic scans in scan statistics

Bug: 27294154
Change-Id: I71d8f388aa83da581958d2169ca9a5eb6088a367

Merge "Fix the argument processing when dumping Bluetooth state"
am: ac9407b0c3

* commit 'ac9407b0c3657b945d4371bc1e827a8dd5958720':
  Fix the argument processing when dumping Bluetooth state

Merge "Fix the argument processing when dumping Bluetooth state"

Fix the argument processing when dumping Bluetooth state

Apparently, when dump() is invoked via bugreport for
Bluetooth's AdapterServe, the optional arguments contain "-a".
The existing logic for checking the arguments for "--proto"
and return the result in Protobuf format failed to detect that,
and suppressed any output.

Now any unrecognized options are ignored, and the default
(text) output is returned.

Bug: 27293885
Change-Id: I0f0d94507ebf6fed7628d49cd1de2a1b2801e6ad

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

Merge "Add a missing null pointer check for AdapterService"
am: 2871b49c82

* commit '2871b49c827a9cb994e4ba82af5a20a026cf8f71':
  Add a missing null pointer check for AdapterService

Merge "Add a missing null pointer check for AdapterService"

Add a missing null pointer check for AdapterService

Add a missing null pointer check for AdapterService
inside class AdapterServiceBinder when receiving
a Binder request to dump Bluetooth state.

Bug: 27343461
Change-Id: I62e67120f6eb6edb5a93b19a9cca5c09862d6506

Merge "Add timestamp to last scans"
am: 781af1e9b1

* commit '781af1e9b11912f706a7ca00f8b20427fbfe996a':
  Add timestamp to last scans

Merge "Add timestamp to last scans"

Add timestamp to last scans

Ex:
  no.nordicsemi.android.mcp (Registered)
  LE scans (started/stopped)       : 5 / 4
  Scan time in ms (min/max/avg)    : 97 / 1476 / 897
  Last 4 scans (timestamp - duration):
    1970/02/13 21:10:19 - 1476ms
    1970/02/13 21:10:21 - 97ms
    1970/02/13 21:10:23 - 929ms
    1970/02/13 21:10:25 - 531ms
  Application ID                   : 5
  UUID                             : f77437a1-5e86-4e2d-8223-6193a095c7f5
  Current scan duration in ms      : 1452
  Connections: 0

Bug: 27294154
Change-Id: I66ba6b4bc2d45ced7d6414db007394a16a30956e

Import translations. DO NOT MERGE

Auto-generated-cl: translation import

Bug: 27293916
Change-Id: I7a6f09c5a34f78bbbbfaa472ab041f55e3015561

Merge "Fix warnings in packages/apps/Bluetooth"
am: d34003325c

* commit 'd34003325c6f2b1618a0356e81f8e9644593c2b8':
  Fix warnings in packages/apps/Bluetooth

Merge "Fix warnings in packages/apps/Bluetooth"

Fix warnings in packages/apps/Bluetooth

Bug: 27301103
Change-Id: I5eec4610f64af4191f64e946122623b6fe222b92

Merge "Clean up gatt dump output"
am: afdc8ec399

* commit 'afdc8ec399bc0d4ab118a83e3cacec41263e2d15':
  Clean up gatt dump output

Merge "Clean up gatt dump output"

Clean up gatt dump output

Change-Id: Ia411bb3446b657b06a1748daa14628da79d59ca0

Merge "Output the Protobuf result in Base64 string"
am: f481ad9463

* commit 'f481ad94638e3db693a6ada090afec659cd2ec7c':
  Output the Protobuf result in Base64 string

Merge "Output the Protobuf result in Base64 string"

Output the Protobuf result in Base64 string

Outputing in binary format does not work, because
the reader assumes the output is a string.

Change-Id: If24797264e82c649c21f9b984f04ec70adbcec94

Merge "Add Java Protobuf dump and ScanEvent Protobuf"
am: 03a1b4fc09

* commit '03a1b4fc094ab32ccf0c48d299c3cee1348bab3f':
  Add Java Protobuf dump and ScanEvent Protobuf

Merge "Add Java Protobuf dump and ScanEvent Protobuf"

Add Java Protobuf dump and ScanEvent Protobuf

Add Protobuf dumping for the Java layer. Also add ScanEvent Protobuf
to already existing scan event logging.

Change-Id: Ib05973c8d98df3bead84b22b3773c42effb25c87

Merge "Improve BT OPP throughput"
am: ceba109434

* commit 'ceba109434fca8bf56b051edd888244adf54687d':
  Improve BT OPP throughput

Merge "Improve BT OPP throughput"

Improve BT OPP throughput

Progress Bar updation during OPP takes around 40ms for each update.
This causes a significant delay in OPP file transfer, which eventually
will decrease the throughput.
To improve the throughput, invoke the progressBar updation only if there
is a change in the percentage.

Change-Id: I64a0b47093698959026d2b97f296c709a7d92703
Signed-off-by: Deepak A Metri <deepak.a.metri@intel.com>
Signed-off-by: Cedric Bondier <cedric.bondier@intel.com>
Signed-off-by: Zhiquan Liu <zhiquan.liu@intel.com>

Merge commit 'f54b7cfd127ce744d1957947a631a92a242748f8' into HEAD