Merge "Fix pairing state change in collision scenario"
am: 8fdbd954e0

* commit '8fdbd954e07f9ee9cf3b7cb3d9f8c9fee4adf1ca':
  Fix pairing state change in collision scenario

Change-Id: I5363494e95e9e37d4ee8caefa4ec11e103334e4c

Merge "Fix pairing state change in collision scenario"

Merge "Check CCB state before executing channel state machine events"
am: 671e726537

* commit '671e726537ed02791d4ae242e879c405a8ced165':
  Check CCB state before executing channel state machine events

Change-Id: Ie0a16cff81205346f08101b79866a71ac0e1fac2

Merge "Check CCB state before executing channel state machine events"

Fix pairing state change in collision scenario

Symptom:
DUT Stuck in pairing state forever with selfy stick
during repair.

Steps:
1. Pair and connect remote devices with DUT.
2. Now un pair selfy stick from DUT.
3. Keep remote in discoverable mode.
4. Start inquiry and try to pair with device.

Failure:
Pairing with selfy stick always fails after collision.

Root Cause:
As we return from btm_sec_dd_create_conn with out
moving pairing state in connection collision scenario. Pairing
state will remains idle and pairing won't proceed further.

Fix:
Move the pairing state in collision scenario so that pairing
can proceed further.

Change-Id: Ic5c446ad3902b0ce16c7223ec2d8e4a625831446

Increasing LE Properties to broadcast.
am: 234af83f69

* commit '234af83f69d70dc89a2c7ae6557b8c63f737437b':
  Increasing LE Properties to broadcast.

Change-Id: Iecc129b8dd7863fb6e27c7194b87a371ec3547a2

Increasing LE Properties to broadcast.

Use case: Remote BD_NAME is not broadcasted when BLE device
pairing is done by using NFC where we broadcasted only UUID.

steps:
Connect with BLE device by using NFC and check the remote name.

Failure: Remote device name is not available.

Root cause: BD_NAME is not broadcasted when BLE device pairing
is done.

Fix: Added property to broadcast remote BD_NAME also as part
BLE device pairing using NFC.

Change-Id: Ie6cda489aabff15a0ebbc692a16f9428729a7dba

Check CCB state before executing channel state machine events

Use Case:
When a disconnect response is being executed, a timeout may
occur and timeout event gets enqueued in BTU process. When
processing of the disconnect event completes, timout event is received
at csm and is executed. However, by this time the Channel
Control Block is already released during previous event causing
errors.

Steps:
Various connection/disconnection scenario.

Failure:
If ccb is already released while processing of
previous event, the occurance of new enqueued event causes
crash.

Root cause:
Disconnection response event has already released
ccb and set lcb for this channel to null. The occurance of
timer event after this results in crash because lcb is
dereferenced while header creation to send disc response to
peer device.

Fix:
Added check for ccb state for whether it is currently in
use or released before executing events in csm.

Change-Id: I9110e6dd5273fa162b51c8aa15bd0030567d664b

L2CAP: Handle invalid HCI packets
am: 1a17ed689f

* commit '1a17ed689f5ba836bcc06f61d59b569c846c7066':
  L2CAP: Handle invalid HCI packets

Change-Id: I36255793aaef7970ee5ae9c419c01d7a177b0cc5

L2CAP: Handle invalid HCI packets

- Handled Buffer over flow for uint16_t
- Discard invalid HCI packets from Codenomican test
  tool as data length and actual data not matching
  during reassembly

Use case:
Execute L2CAP test suit from Defensics Codenomican

Steps:
1. Pair and connect DUT to Codenomican tool
2. Execute L2CAP test suit from Defensics Codenomican

Failure:
Crash observed on DUT and Codenomican tool stuck in execution.

Root cause:
Codenomican tool sending invalid HCI packets to DUT and
there are no checks to handle buffer over flow and other invalid data
from Codenomican tool.

Change-Id: I6f93c80244fc39d607ad285185136bbbca83d7ae

Don\'t process pair request when controller is not ready
am: b534453b1b

* commit 'b534453b1b573995b19e9c12c846290d0789375b':
  Don't process pair request when controller is not ready

Change-Id: If2c8167b73cc3d0809912a5c88e58c3f9d49cfe0

GAP: Save pin code length for incoming legacy SAP connection
am: 0008f9e147

* commit '0008f9e14753cb2885cf9438fe0c41f29c2cfa6e':
  GAP: Save pin code length for incoming legacy SAP connection

Change-Id: Ic950cc3611b48b489b8bae057b016a6eb45708bc

Prevent crash due to second attempt to clean up eager_reader
am: 71300bf3bd

* commit '71300bf3bd58fbdfb8a440d89e10a8ae9884ad0b':
  Prevent crash due to second attempt to clean up eager_reader

Change-Id: I04bfb68a63363a05a05bd6d7152ec6f63795db19

Don't process pair request when controller is not ready

Use case: when one application is trying to pair and
other application is turning off Bluetooth.

Steps: when one application is trying to pair and
other application is turning off Bluetooth.

Failure: Bluetooth process crashed.

Root Cause: Bluetooth process crashed due to assertion,
as controller module is not ready.

Fix: Made a checks for controller module is ready or not, If
it is ready then only go for pairing.

Change-Id: I72c72f6b5545c845e79874212359854023ffad98

GAP: Save pin code length for incoming legacy SAP connection

Use case: Carkit cannot reconnect to DUT in Sim Access Profile mode

Steps:
1. Pair with a car-kit (Agree all SAP, MAP, PBAP requests)
2. Initiate disconnection on DUT
3. Initiate re-connection on the car-kit.

Failure: SAP profile reconnection failing as dut gives negetive
linkkey reply.

Root Cause: sec_flags not getting updated without pin len and stack
triggering negative link key reply even linkkey is proper for 16 digit
authentication of SAP.

Fix: Save pin length during BTM_PINCodeReply.

Change-Id: I8300dd3ff497058291143313b169190910e0a7c5

Prevent crash due to second attempt to clean up eager_reader

Null out the eager readers once they are cleaned up, so we
don't try to clean them up later if an error occurs opening
the HAL again later.

Change-Id: I07df1010a1a619cb67b306d64e274d450234a002

btif: Remove dead code from btif_storage
am: d1d0f1b514

* commit 'd1d0f1b51469e908bb94750753367de9425a075b':
  btif: Remove dead code from btif_storage

Change-Id: I42fe519b599555b6621e12fd737277b189979d01

btif: Remove dead code from btif_storage

Change-Id: I3ee4d8fd3a7697e0113d955982b21db123354a86

Add missing arguments to log messages
am: 65fc269a87

* commit '65fc269a87f330a3bbc8209b088fc08e15f871b9':
  Add missing arguments to log messages

Change-Id: Ifd55e085127c45c990fe93d628b5395dc2926f2b

Add missing arguments to log messages

Bug: 28769522
Change-Id: I37c2b8400619646a8711099cf0bd369c3cf8eb51

Add missing include for OSI_NO_INTR
am: c1e60c6317

* commit 'c1e60c6317d967eecc8c04f9eed2223d2005b03a':
  Add missing include for OSI_NO_INTR

Change-Id: I7774059590f38a712d15c2974d8ffb07954e9aba

Add missing include for OSI_NO_INTR

Fixes building bt_vendor_linux since
OSI_NO_INTR is defined in osi.h

Change-Id: Ib95a3126ed4a7b8f0bdc7db3dd2ef162825f2cbe
Signed-off-by: Stefan Stanacar <stefan.stanacar@intel.com>

Fail gracioulsy when GATT_MAX_APPS reached
am: db4ec762f0

* commit 'db4ec762f0825a944923bd3a4ff69d3147b68dfa':
  Fail gracioulsy when GATT_MAX_APPS reached

Change-Id: I6e880dfddea326a02058165fe7fc724badcb4449

Fail gracioulsy when GATT_MAX_APPS reached

Bug: 28677905
Change-Id: I092494ab3f9e9ae4330d5d2a6f6ef010c61fe2a4

audio HAL: improve control socket error handling
am: 7a599b44bf

* commit '7a599b44bfd5ccc751445989807692bad78d9491':
  audio HAL: improve control socket error handling

Change-Id: I5988d5d0249d98746d2cf4d0be69a2043348df5a

audio HAL: improve control socket error handling

Implement retry upon control socket receive error
in a2dp_ctrl_receive() if EAGAIN or EINTR.

Implement recovery from previous socket error in
a2dp_command().

Increase socket receive timeout to account for
headsets will long activation time.

Fixed state update error in start_audio_datapath()
in case the data socket is already opened.

Added missing mutex around state updated in oput_write()
and in_read()

Made read and write code symmetric.

Bug: 28727517
Change-Id: I68d7fe63f298bbd1e387ec42fdb1277b2dcdfef5

Restart failed system calls interrupted with errno of EINTR
am: d6121a37c5

* commit 'd6121a37c579731b89348e618a823c53b938449a':
  Restart failed system calls interrupted with errno of EINTR

Change-Id: Ia6348e07b24bb22c03ed07cace2b4614a66bfcae

Restart failed system calls interrupted with errno of EINTR

In number of places we don't handle properly system calls failures
when the errno is EINTR (i.e., the system call was interrupted
by a signal). In all our use cases, the system calls should be
restarted. The handling of the following system calls (as used in the code)
has been updated/fixed:

  poll, send, recv, sendmsg, nanosleep, epoll_wait
  read - mostly (e.g., socket-like fds)
  write - mostly (e.g., socket-like fds)
  select, accept, connect

Bug: 28471477
Bug: 28658141
Change-Id: I03e6f0f67e33876780fb6d02c33eb84547ba8f95

Add sync to write the backup file to the filesystem
am: f24f5eff41

* commit 'f24f5eff41968dda6f97333c9b4c09ec4c5a8597':
  Add sync to write the backup file to the filesystem

Change-Id: Ifc375fd3443ef07def960cd702f3cf5c4f20bb3d

Add sync to write the backup file to the filesystem

Bug: 27354612
Change-Id: I5a7751b6f08831d43647391774501f211336e9eb

Build fix for feldspar-dev
am: aa39d20a60

* commit 'aa39d20a60a56a1528af0ed726e2effa90e08fb5':
  Build fix for feldspar-dev

Change-Id: I48fbf993a87aefd245e14a6e765955212017a4af

Build fix for feldspar-dev

Bug: 28485365
Change-Id: I25db2fe7577e98715eb10e105176c536fddb043a

Add config logging, make parsing stricter, and move sync
am: 72a183d00c

* commit '72a183d00c7534c80a4deb9fb41d38fd9e9455f6':
  Add config logging, make parsing stricter, and move sync

Change-Id: I45b6ee7a6956a0359ebe9ccfa9d9023096893b09

Add config logging, make parsing stricter, and move sync

The functions that parse the config file will report failure if
the config file is improperly formatted or is missing the necessary
Adapter section. Also moved sync to sync the filesystem after the
config file is saved.

Bug: 27354612
Change-Id: Ia75ff628d83bdc933c4bc16761d6c55eef19b757

Merge "btif: only save config when bonded devices change"
am: c12ef5607a

* commit 'c12ef5607a2a78201f282d551e8dfe7ee5763942':
  btif: only save config when bonded devices change

Change-Id: I33f8d9a69e60e1f10230c903b070b67c67e07acf

Merge "btif: only save config when bonded devices change"

Remove dead HID code
am: c3fe5a04a9

* commit 'c3fe5a04a926672d3b363d88e6bd67eb46ba230c':
  Remove dead HID code

Change-Id: I2b2838c8d17b80242724fd6e3617f971650807df

Remove dead HID code

This patch removes dead code in HID related to setting scan parameters
on remote device.

Change-Id: I96e0036b65d2ebb3b099d817683ed3d5a11b6bb7

Rename all *.cpp files to *.cc to follow new style
am: e6d1202990

* commit 'e6d1202990e0ede65db4470b4eaa20fc4b1f791e':
  Rename all *.cpp files to *.cc to follow new style

Change-Id: I6075f783e4bc7d708aa83f07d3be8b21f7a5401c

btif: only save config when bonded devices change

Bug: 27354612

Change-Id: I15f816c85d3ab1991abf076f7c1ae2914d773d54

Rename all *.cpp files to *.cc to follow new style

Change-Id: Ia79ee09348e67ff4a1712ee11a1e8480180a2623

Rewrite btif_gatts_register_app to new style
am: d71bc1732f

* commit 'd71bc1732fc0ed095a95dd625ff4080a0a1b44f3':
  Rewrite btif_gatts_register_app to new style

Change-Id: Id387cb8e9ba5ef05f55e96e5957bddfa5f0e91c4

Rewrite btif_gatts_open to new style
am: 3c79a3b7ca

* commit '3c79a3b7caf12d9b182afc2da071be3a5615a0dd':
  Rewrite btif_gatts_open to new style

Change-Id: I0be50ae5dc763554665f901a236fbebcd5fdb0c2

Rewrite btif_gatts_close to new style
am: b333ae2e69

* commit 'b333ae2e69d652b42a7c02500f35ec781df4178c':
  Rewrite btif_gatts_close to new style

Change-Id: I9dd7bcda53f618828d00c6d4c70e6fb10100ffb5

Rewrite btif_gatts_add_included_service to new style
am: 0cd448fe8f

* commit '0cd448fe8f7bf57bb1530d0216b2e18e322e3b3c':
  Rewrite btif_gatts_add_included_service to new style

Change-Id: I676dd5a6b9b10db3eddce2ffacad20f25ec07597

Rewrite btif_gatts_start_service to new style
am: b14f76be6e

* commit 'b14f76be6e859a796d64d083b78e6816f415de20':
  Rewrite btif_gatts_start_service to new style

Change-Id: I95917a0243aac10250d740d819d29d5174e3c7a6

Rewrite btif_gatts_stop_service to new style
am: 7ce14cc65e

* commit '7ce14cc65ea3796da3e12811e7bc07b529a2f9fa':
  Rewrite btif_gatts_stop_service to new style

Change-Id: Ib39f38544c5fb34d090c4e73562db52511b01720

Rewrite btif_gatts_send_indication to new style
am: 858cf9468b

* commit '858cf9468bb402ddf806087461aed5f4ea865f6b':
  Rewrite btif_gatts_send_indication to new style

Change-Id: Iac5c2a93f5c52f805e204c5dc0a473eb12ffe1ea

Rewrite btif_gatts_send_response to new style
am: 757109e314

* commit '757109e31404689f1769522637425bcc11edc074':
  Rewrite btif_gatts_send_response to new style

Change-Id: I1fe226e89603e826f72e1595f2c3af26579cb747

Use unnamed namespace in btif_gatt_client
am: 44be0690a1

* commit '44be0690a1d15d1ffc97e4c7560d8aee5bb86062':
  Use unnamed namespace in btif_gatt_client

Change-Id: I3323ea4938ea4f82fadaf7923161ab345d57a662

Use unordered_set to keep p_dev_cb
am: ce7d46a263

* commit 'ce7d46a263d8e0e7877b62b99b046c7e08ee75f8':
  Use unordered_set to keep p_dev_cb

Change-Id: I6e93bf2f0d215c9208a7b4978098f9dcb19674fd

Remove unused structures from btif_gatt_client
am: d635ab3102

* commit 'd635ab31023396ed2a33634303c4aa5d074f890c':
  Remove unused structures from btif_gatt_client

Change-Id: Ie635eae8cc0200d6d0f824da6c0b0624e508cfb9

Rewrite bta_scan_results_cb to new style
am: 92519cc7f1

* commit '92519cc7f18867e9e1897f63f212c8e0a221bdf7':
  Rewrite bta_scan_results_cb to new style

Change-Id: Id862aec6bf50b30206f7068fe0910952ca267aaf

Rewrite multi adv callbacks to new style
am: 845b49d255

* commit '845b49d2559bae6e8084672c6b7020f2820aea9e':
  Rewrite multi adv callbacks to new style

Change-Id: Ibb571b3834166abce6b0940d3e718d0e3b1a5caf

Rewrite batch scan related functions to new style
am: 5f848b4d24

* commit '5f848b4d24f596d56da335ee07dd976b3101cfd2':
  Rewrite batch scan related functions to new style

Change-Id: I1b783c927eff45c0255403a7a79701725044a4bc

Rewrite bta_track_adv_event_cb to new style
am: 4ed8924e9d

* commit '4ed8924e9d820df37f03230dece6fd68eb1fc4df':
  Rewrite bta_track_adv_event_cb to new style

Change-Id: I18f0d83f99e1daf5e2713770523edbbc0ac6de37

Rewrite btm_read_rssi_cb to new style
am: c171ac407c

* commit 'c171ac407cfc6d30a11af6b12ead39688eb897f7':
  Rewrite btm_read_rssi_cb to new style

Change-Id: I08c5364b5112ade6a660a0d36bfeec45a5bf8655

Rewrite scan filter related callbacks to new style
am: da5b59b5bf

* commit 'da5b59b5bf9cb34574c16ba784c1484055fcb1e3':
  Rewrite scan filter related callbacks to new style

Change-Id: Ifeb575fd0310c1d57ec60f50a449ff667f7ee896

Rewrite btif_gattc_scan_filter_add_remove to new style
am: 904aefe1a9

* commit '904aefe1a9c2808a7460ebb9896391acdaa39f4e':
  Rewrite btif_gattc_scan_filter_add_remove to new style

Change-Id: Ied31fc699158e732404a4b0784b84c47310af96f

Rewrite register/unregister app to new style
am: c8b5185a5d

* commit 'c8b5185a5d4c4c9c15f3ec5df4acd37c1d46f1f1':
  Rewrite register/unregister app to new style

Change-Id: Iefb9ce04cf30bcf1615f3bd3c744856d1f65c83c

Rewrite scan start/stop to new style
am: 955ff4e88e

* commit '955ff4e88ef7391632e19d1b0234cae40b6f42c1':
  Rewrite scan start/stop to new style

Change-Id: I19b1cc577701849d2d86b8d54f21b791d869ba17

Rewrite btif_gattc_open to new style
am: 26e51cb185

* commit '26e51cb18509e7e04755b4fa4250c60f105ab9d2':
  Rewrite btif_gattc_open to new style

Change-Id: I3545e59f8913cd69703e311d327d49f58636b87b

Rewrite btif_gattc_close to new style
am: b454953bb6

* commit 'b454953bb6620cbafba86351b91d58b47dda107d':
  Rewrite btif_gattc_close to new style

Change-Id: If0be0f67d14230a2ee0886ad7f8d033a211657bc

Rewrite btif_gattc_search_service to new style
am: d05e50e7df

* commit 'd05e50e7df462c928041bf565821d77fb493bc28':
  Rewrite btif_gattc_search_service to new style

Change-Id: I2fda5393fc407799f2e3b025fcd8f3a5609d7ed9

Rewrite btif_gattc_execute_write to new style
am: a684a52231

* commit 'a684a52231bf0422d991fb16e4dcfc88e4584680':
  Rewrite btif_gattc_execute_write to new style

Change-Id: Ib01994744d50b6b2bf737ad95f0d2fb4f7a78776

Rewrite btif_gattc_write_char to new style
am: 39a3361288

* commit '39a33612889a562da5c919f14df24707ba3e6007':
  Rewrite btif_gattc_write_char to new style

Change-Id: I8bbead7e38bb8755663ce48b924244814dbe3a28

Rewrite btif_gattc_reg_for_notification to new style
am: ca9d889000

* commit 'ca9d889000b087ddb9bedaa716176be986be158f':
  Rewrite btif_gattc_reg_for_notification to new style

Change-Id: I171687aa0f7398317f4c34aae624c7192de610ab

Rewrite btif_gattc_listen to new style
am: 144998f689

* commit '144998f68990205ba711e701be6be8325dd6e820':
  Rewrite btif_gattc_listen to new style

Change-Id: I151ca8a21d468adcd0180373337c4774ccd84506

Rewrite btif_gattc_set_adv_data to new style
am: 1ce0819a83

* commit '1ce0819a83f47f3bec3c16c46befd1e7faddf0ff':
  Rewrite btif_gattc_set_adv_data to new style

Change-Id: I45c99c023d75a7af08b54db0a650c73981b43fa0

Rewrite read RSSI and set MTU to new style
am: c9fe4b6f62

* commit 'c9fe4b6f62c95b195fa99c4fbb75863e547929bf':
  Rewrite read RSSI and set MTU to new style

Change-Id: Iea321bd4e8aa6462564a323797cded2e10c4ef9c

Rewrite btif_gattc_conn_parameter_update to new style
am: 7cb29a3802

* commit '7cb29a38027f9cb24fc30eefb389f848cfc17082':
  Rewrite btif_gattc_conn_parameter_update to new style

Change-Id: Ifad5e8babdc6eff78f0c532aa5a924062ce3a310

Rewrite btif_gattc_scan_filter_param_setup to new style
am: 1a9e2ac4da

* commit '1a9e2ac4dab5a74139613f4ff5dde1b979522f0e':
  Rewrite btif_gattc_scan_filter_param_setup to new style

Change-Id: I054ccfefa0e12d55305dba0e0c25c745fcb54bb4

Rewrite scan filter related methods to new style
am: 71d4e33a08

* commit '71d4e33a08a5eaf458c4cd3de1ca65a542be81e3':
  Rewrite scan filter related methods to new style

Change-Id: I9da69f4300b888b0cc4f21c855f64df3fa8f60f9

Rewrite btif_gattc_multi_adv_enable to new style
am: 3a70173f66

* commit '3a70173f66b9ca3c5ad6260ca963180c479c5d0d':
  Rewrite btif_gattc_multi_adv_enable to new style

Change-Id: I2b27b9a8691b8db10026c6d6f82e194f1d6fe38a

Rewrite btif_gattc_multi_adv_update to new style
am: 5dfe8ba500

* commit '5dfe8ba500ad0594c787a0b905369f36de7da38b':
  Rewrite btif_gattc_multi_adv_update to new style

Change-Id: I37dd5cd7a3605ee621352228ad73d6cd2033b002

Rewrite multi adv set data to new style.
am: ed8aefe613

* commit 'ed8aefe6134359cc70b127a23bac3a6d4c24065e':
  Rewrite multi adv set data to new style.

Change-Id: Iea27f1f0c783a2f74bf487e265358fe5c97fd338

Rewrite btif_gattc_multi_adv_disable to new style
am: 772fc669dc

* commit '772fc669dcf1221df03377e5c16507f2f7caa91b':
  Rewrite btif_gattc_multi_adv_disable to new style

Change-Id: I82f7092afc59bf24e21ebccb2ecef95636741ac7

Rewrite btif batch scan methods to new style.
am: 2af2129df7

* commit '2af2129df720baee246d4ff9a0f1a02884da856e':
  Rewrite btif batch scan methods to new style.

Change-Id: I9280ad6c5b4417c0c0262ea668a04f93d1cf56e6

Rewrite btif_get_gatt_db to new style
am: 61a98132c3

* commit '61a98132c39d3181a0ea23988ac36fbfef8194b4':
  Rewrite btif_get_gatt_db to new style

Change-Id: I0431c64890aa823f679c33fe98260b3c3d2450e2

Rewrite btif_gattc_set_scan_parameters to new style
am: 4c634eeafc

* commit '4c634eeafc79e68b40ff1ade0d96ba7b1c34f85f':
  Rewrite btif_gattc_set_scan_parameters to new style

Change-Id: I51ea085022cba4ff8c9aa221d6cb523a1b6cb5bc

Rewrite GATT read operations to use new style
am: 61a112e488

* commit '61a112e488a7686e5cc9680434c42895ef9db878':
  Rewrite GATT read operations to use new style

Change-Id: I981bfd27a7010da48b204f22f7854055665497e4

Use MessageLoop in bt_jni_workqueue thread
am: 140645f6b8

* commit '140645f6b8d87464c08f082bd14740eecab23f71':
  Use MessageLoop in bt_jni_workqueue thread

Change-Id: I0d60049b62c4295bea9f2fa5f12bcbee81a5c962

Rewrite btif_gatts_register_app to new style

Bug: 28485365
Change-Id: I5b9bd5a807492b254a5486c80dfb56769e95943f

Rewrite btif_gatts_open to new style

Bug: 28485365
Change-Id: I20fdb5ffe6d52db291e9862f7e24fb43e060950f

Rewrite btif_gatts_close to new style

Bug: 28485365
Change-Id: Ib8d7ed81cf958ab3dd5403c46409cbd506ac3dc1

Rewrite btif_gatts_add_included_service to new style

Bug: 28485365
Change-Id: I4465493532578942045b979e542608972c5fa2ec

Rewrite btif_gatts_start_service to new style

Bug: 28485365
Change-Id: I2c8b02ce42fd2bde3a4c448f6fb19ef8e3eb169a

Rewrite btif_gatts_stop_service to new style

Bug: 28485365
Change-Id: I79aaaa57139dca373fdf7c0424353cd5de0cb48f

Rewrite btif_gatts_send_indication to new style

Bug: 28485365
Change-Id: I1d3221be272e53245e09077dad9d291829350b45

Rewrite btif_gatts_send_response to new style

Bug: 28485365
Change-Id: Id0dbfe28b0f77f542cd6961c041e70f90d0aa78b

Use unnamed namespace in btif_gatt_client

Instead of marking each function as const, we can enclose them in
namespace, which will have same effect.
This patch also format rest of the file to follow cc style guide.

Change-Id: I5e07d80558de06851c8b5f696d8a239fd4de73e2

Use unordered_set to keep p_dev_cb

Bug: 28485365
Change-Id: I5c6c45153c04b1ac9215addf436c7731f376dd15

Remove unused structures from btif_gatt_client

Bug: 28485365
Change-Id: If1a4c0d721966255038381b3a0f045910fa923dd

Rewrite bta_scan_results_cb to new style

Bug: 28485365
Change-Id: I1029053c9417e6224e365c42aaeba09759e85ff2

Rewrite multi adv callbacks to new style

Bug: 28485365
Change-Id: I94e8378515543135b51d0d27c988f6dc2ffb6a70

Rewrite batch scan related functions to new style

Bug: 28485365
Change-Id: Id82aa6abb0cf9f809567ee8f728ccd8256d43096

Rewrite bta_track_adv_event_cb to new style

Bug: 28485365
Change-Id: Iebf02592cb279a4aca48c9593923b67ddeabf428

Rewrite btm_read_rssi_cb to new style

Bug: 28485365
Change-Id: Iec6114714d0201556bee27a8311356168c7afcac

Rewrite scan filter related callbacks to new style

Bug: 28485365
Change-Id: Icd5c3529ed2938e095dacc53df716dcd0e69a56d

Rewrite btif_gattc_scan_filter_add_remove to new style

Bug: 28485365
Change-Id: I618644b08ecffbb8d22186a7dc70189fb8e50a53