am 404945ba: am d59ee6c2: Set the proper authentication level for AvOpen

* commit '404945ba3ac161f624f5420246f6c845b5c821c5':
  Set the proper authentication level for AvOpen

am d59ee6c2: Set the proper authentication level for AvOpen

* commit 'd59ee6c25920cf9db218f2d5ac46624c69c3ba09':
  Set the proper authentication level for AvOpen

Set the proper authentication level for AvOpen

During Avopen security was made none hence A2DP
connection was happening without bonding when
DUT is paired with DUT and then pairing entry
is deleted from remote and then only A2dp is
tried to be reconnected from DUT. A2dp appears
to be connected in this usecase but the device
entry is shown in available devices list instead
of paired one.
Changes made from No security to SEC Authenticate

Change-Id: I7fc39fe8acef874f0031c9b78d4269a62b94e4b0

am bd2e552d: am 8a67ac70: Don\'t open SCO for 2nd MO call if a call is active

* commit 'bd2e552d70a6fc7afaafd77366b859b3432f7c60':
  Don't open SCO for 2nd MO call if a call is active

am 2ae0e22a: am 2d0d790c: Don\'t close SCO for MO calls

* commit '2ae0e22acf4e4c710426cb76b28ba253e8a626c8':
  Don't close SCO for MO calls

am 8a67ac70: Don\'t open SCO for 2nd MO call if a call is active

* commit '8a67ac702cd4d4a4cbebd1e765800141b84e6fc0':
  Don't open SCO for 2nd MO call if a call is active

am 2d0d790c: Don\'t close SCO for MO calls

* commit '2d0d790ca98b6c46be085e1e0a3e6d16ebd8cddf':
  Don't close SCO for MO calls

am 35eb2622: am c8c082b7: SCO connection should happen after SLC.

* commit '35eb26229589fda2353ca7e0cfa5396a79d799ce':
  SCO connection should happen after SLC.

am 0899689c: am e1ded77a: Fix compile error in bta_ag_sco.c

* commit '0899689cf6cbb5344505af07b95cdae5d89e5240':
  Fix compile error in bta_ag_sco.c

Don't open SCO for 2nd MO call if a call is active

When user dials a 2nd call, do not open SCO if 1st call was
active on DUT earpiece or speaker.

Change-Id: I974f5bcf917e6972705e60cf1da195eae23f885a

am c8c082b7: SCO connection should happen after SLC.

* commit 'c8c082b730910b64dbc38a7ee89bb141be46daa6':
  SCO connection should happen after SLC.

am e1ded77a: Fix compile error in bta_ag_sco.c

* commit 'e1ded77ae6d10854dc2bd1aebdd9b53442cc4ba4':
  Fix compile error in bta_ag_sco.c

Don't close SCO for MO calls

This change removes condition to disconnect SCO
when MO call process is started.

Change-Id: I2af4ab82c097c7d9ce1bc892bdf40cc09ff6d97e

am 999e7a5c: am 5b675bc4: Create listen SCO for HS1 if HS2 disconnects during SCO xfer

* commit '999e7a5ce7ca1ff5c1f2c2f1a9b6f1a088884346':
  Create listen SCO for HS1 if HS2 disconnects during SCO xfer

am 91ab5e5e: am 902ba021: Don\'t open SCO on state change for multi party call on DUT.

* commit '91ab5e5ebfc2baa9d7193755f8c8cdb89e9a2937':
  Don't open SCO on state change for multi party call on DUT.

am e56f438b: am d288b2af: Reset sniff subrating while in SCO

* commit 'e56f438bc3938ff178ed941b730420f9560d4b7e':
  Reset sniff subrating while in SCO

am 12add1b5: am a7ce13fe: Open SCO for held call.

* commit '12add1b5c64d25879efde40cd09ccf892654cf09':
  Open SCO for held call.

am 282ceb77: am 7b0bfd32: Update call state before opening SCO.

* commit '282ceb7736057401b3e0aabbdc0e779352065ef2':
  Update call state before opening SCO.

am 82981fb8: am d345bb20: Handle multi party call states

* commit '82981fb891c719730bd7d48a4d793918cb05d2e9':
  Handle multi party call states

am d80a3f20: am c2c8a1fc: Update held call state to connected headset

* commit 'd80a3f2056477cf106e52994123305d6f9a67dd4':
  Update held call state to connected headset

am 5b675bc4: Create listen SCO for HS1 if HS2 disconnects during SCO xfer

* commit '5b675bc41986a159023c51b561e81939f0632f95':
  Create listen SCO for HS1 if HS2 disconnects during SCO xfer

am 902ba021: Don\'t open SCO on state change for multi party call on DUT.

* commit '902ba02141c5cea3320ec1746c0fe0fad80122bc':
  Don't open SCO on state change for multi party call on DUT.

am d288b2af: Reset sniff subrating while in SCO

* commit 'd288b2af6bf5ebb41554f223adf2e7f536d2f1e5':
  Reset sniff subrating while in SCO

am a7ce13fe: Open SCO for held call.

* commit 'a7ce13feaa2a84ff329f2bbd4ab964f0f6d54f0a':
  Open SCO for held call.

am 7b0bfd32: Update call state before opening SCO.

* commit '7b0bfd32833a8da08069b60e6568aace90778b62':
  Update call state before opening SCO.

am d345bb20: Handle multi party call states

* commit 'd345bb208fe35923b60bca6af76b0a6e023fc56f':
  Handle multi party call states

am c2c8a1fc: Update held call state to connected headset

* commit 'c2c8a1fce4a577fc1a14a96fe96cab3043db4140':
  Update held call state to connected headset

SCO connection should happen after SLC.

This change makes sure that SCO audio
connection request is not processed until
HFP SLC connection is done.

Change-Id: Ia2fcad49882deaaf049ae190d90442e5ca7ad71b

Fix compile error in bta_ag_sco.c

Change-Id: I8839ba732560a31a5e8a2351ea4bfb16999cf5ea

Create listen SCO for HS1 if HS2 disconnects during SCO xfer

During SCO transfer process from HS1 to HS2, firstly active SCO for
HS1 is disconnected. However, if suddenly HS2 disconnects even
before SCO transfer is complete, listen SCO for HS1 is not created.
This causes further incoming SCO connections from HS1 to be rejected.
The change creates listen SCO for HS1 and moves SCO state properly
to LISTEN.

Change-Id: I0993c1ba1c24b3b7e9c243d179a913d7a0c40446

Don't open SCO on state change for multi party call on DUT.

When multi party calls(1 active, 1 held) are ongoing on phone
speaker and any of these call drops, do not open SCO for
other call since user expects call to still remain on phone
speaker.

Change-Id: I9079c809ac665fc1c6e890302e37d23a12f8d4f4

Reset sniff subrating while in SCO

Sniff subrating should not be used when link is
in SCO connection. Since the subrating value is
higher, the link would take more time to process
ACL data while in SCO connection causing unwanted
results like delay in SCO disconnects.

Change-Id: I7b964faa397624b65e0158209b8743efe13a6075

Open SCO for held call.

Use case:
1. Connect to headset/carkit
2. Make an outgoing call from AG
3. Turn off BT on AG
4. Hold the call from AG
5. Turn on BT on AG (No SCO link is established here)
6. Unhold call from AG (still no SCO link is established)

Failure:
No SCO connection established when held call is there
during headset connection.

Root cause:
SCO is not initiated when SLC is done during held call.

Fix:
AG opens SCO connection when HF connected while in call and
the call was in held state.

Change-Id: I63d7655d26ddbf68cbf94c745df745cffa917b5e

Update call state before opening SCO.

This change makes sure to update the current
call state before opening SCO connection when
incoming call is answered. Some car kits are
strict in checking this sequence and go in
bad state if not done this way.

Change-Id: Ie744dc26c02d897eaf016ee73022bfc2a0db067e

Handle multi party call states

UseCase:
1. Set up Bluetooth connection between phone and remote device.
2. Disconnect BT link by turning off remote device.
3. Start Mo call from phone.
4. Receive MT call from far end.
5. Turn on remote device.
6. BT connection gets established successfully, but audio
doesn't get transferred to headset.

Failure:
Call audio not routed to BT HS. SCO not established.

Root cause:
When a HS connects while in an active call and 2nd call
(incoming/outgoing/alerting) setup ongoing, incorrect post SLC
call setup updates might cause HS to get confused as phone had
already updated correct call states in CIND response during SLC.
This may cause HS to behave abnormally like delayed response to
BCS as seen with jabra wave +.

Fix:
Avoid sending incorrect call updates post SLC since we already
have sent correct updates during SLC as reponse to CIND.

Change-Id: I53e57baf7cca782af6f374a043667971c66a956d

Update held call state to connected headset

In a three way call scenario, when an active
call is dropped by remote, the single held call
state is not updated to the connected headset.
This change will make sure that it is updated
properly.

Change-Id: I558602e791279d510edb23989b125cd8f7a7ebde

am 1839e76a: am a954124f: Add support for MITM for BluetoothSockets (4/4)

* commit '1839e76a6313e9979ef027521916797fdf95348b':
  Add support for MITM for BluetoothSockets (4/4)

am a954124f: Add support for MITM for BluetoothSockets (4/4)

* commit 'a954124fc5429f259e47258af64e159c621cc6e0':
  Add support for MITM for BluetoothSockets (4/4)

Add support for MITM for BluetoothSockets (4/4)

This change adds an option to enforce Man-in-the-middle protection
for the authentication process. This feature is needed for the Sim
Access Profile.

Change-Id: I2eda507ffdcb4a194434adedf207e1e9951b80c3
Signed-off-by: Casper Bonde <c.bonde@samsung.com>

am 2abcbc3a: am 56a3be0d: Replace malloc/calloc/free with osi_malloc/osi_calloc/osi_free

* commit '2abcbc3ab3788c77fbe82463698295f3206d56aa':
  Replace malloc/calloc/free with osi_malloc/osi_calloc/osi_free

am 61ea384f: am 1b0f1077: SAP: Change to use new SDP Api (1/4)

* commit '61ea384f18fbf2699c41e534e4c04db5ac4a75f1':
  SAP: Change to use new SDP Api (1/4)

am 56a3be0d: Replace malloc/calloc/free with osi_malloc/osi_calloc/osi_free

* commit '56a3be0d6eaa1bc264736e399fb32d3094f0b513':
  Replace malloc/calloc/free with osi_malloc/osi_calloc/osi_free

am 1b0f1077: SAP: Change to use new SDP Api (1/4)

* commit '1b0f107768ba1a0f93abf68851bacbe650f27eff':
  SAP: Change to use new SDP Api (1/4)

Replace malloc/calloc/free with osi_malloc/osi_calloc/osi_free

There were several leftover places that were still using
malloc/calloc/free .
Those are replaced to use osi_malloc/osi_calloc/osi_free like
majority of the rest of the code.

Note: There are few remaining places that are still using
malloc/calloc/free:
 * Some of the unit tests
 * audio_a2dp_hw/audio_a2dp_hw.c
   It is used as part of the audio.a2dp.default.so library,
   and the bluetooth.default.so library
   Its usage of malloc/calloc/free should be refactored
   independently.

Change-Id: Iafbed996e5f1ae8eb1343fb2acfadf32e515e419

SAP: Change to use new SDP Api (1/4)

Added support for Sim Access Profile (SAP) in the SDPManager.
To make it easier to test, both SDP record creation and seach
is added.

Change-Id: Idae480fb64224e0e5dec7fb07ff4efb9312a5461
Signed-off-by: Casper Bonde <c.bonde@samsung.com>

am 6f3daa4a: am 5feb0af8: Fix memory corruption due to BLE multi-adv macro

* commit '6f3daa4a7f7172a06d96f240cdb79d912faaf4d8':
  Fix memory corruption due to BLE multi-adv macro

am b0d12262: am 87072892: HID: Handle closing of uhid driver in case failure

* commit 'b0d12262ccf88d86b558f664c747e8c899f52c49':
  HID: Handle closing of uhid driver in case failure

am 55196545: am 6ba8fda6: Add proper checks for PAN & BNEP in BD stack

* commit '551965451fa43a1e503a6a51c84b162aea13e245':
  Add proper checks for PAN & BNEP in BD stack

am c4eb0c5e: am e11f228d: Reset pairing state to idle only for the current device

* commit 'c4eb0c5e29f0ea4708dd8f6a834c9f9c24aaa3c6':
  Reset pairing state to idle only for the current device

am 190b2447: am a9130e07: Moved the OSI alarm_shutdown() operations to alarm_cleanup()

* commit '190b24470159b053c5c4cb405b8ae17bad30f791':
  Moved the OSI alarm_shutdown() operations to alarm_cleanup()

am 9a5a3717: am b65ec7bd: Disable the OSI allocation tracker.

* commit '9a5a37174f84708130be8531bd0cd251305365b5':
  Disable the OSI allocation tracker.

am f32fc7b8: am a250280f: Check transport before subscribing for service changes

* commit 'f32fc7b886cfdafef892d4175f4e85046ad4b42e':
  Check transport before subscribing for service changes

am 5feb0af8: Fix memory corruption due to BLE multi-adv macro

* commit '5feb0af81d30dbcc33ea2ffb10714a58dd0cef09':
  Fix memory corruption due to BLE multi-adv macro

Fix memory corruption due to BLE multi-adv macro

INST_ID_IDX_MAX macro was not protected with parenthesis, resulting
in incorrect calculation of the size needed to allocate memory for
the client_if map.

Change-Id: Ib3591f4f9ac7288f2191e9078a71aab431ca7130

am 87072892: HID: Handle closing of uhid driver in case failure

* commit '87072892dc006f4c037aae0a346eda6c18e6b8ba':
  HID: Handle closing of uhid driver in case failure

am 6ba8fda6: Add proper checks for PAN & BNEP in BD stack

* commit '6ba8fda60a9137a9f53f7e14edd87184610744fc':
  Add proper checks for PAN & BNEP in BD stack

am e11f228d: Reset pairing state to idle only for the current device

* commit 'e11f228d1a18486cd00c36487812a763487a236f':
  Reset pairing state to idle only for the current device

am a9130e07: Moved the OSI alarm_shutdown() operations to alarm_cleanup()

* commit 'a9130e07646835a8550611bed32138ed79385a38':
  Moved the OSI alarm_shutdown() operations to alarm_cleanup()

am b65ec7bd: Disable the OSI allocation tracker.

* commit 'b65ec7bdb30115ce0d39cf7ee1e355b61fc2a360':
  Disable the OSI allocation tracker.

am a250280f: Check transport before subscribing for service changes

* commit 'a250280feff8c8fa28a2e33881a2cdf73540f753':
  Check transport before subscribing for service changes

HID: Handle closing of uhid driver in case failure

This patch handles closure of the uhid driver in case of following:
- Due to concurrency of disconnection from a conencted device
  and an incoming connection request from another device, the error
  code for the ACL disconnection becomes HCI_ERR_HOST_REJECT_SECURITY
  in the stack.
- On reception of disconnection with that error codes, the uhid driver
  was not closed from the btif layer leading to issues in reconnection
  with same device that got disconnected.

Change-Id: I13c83757051850cc631aa1c24a036a2e1a4d0087

Add proper checks for PAN & BNEP in BD stack

This patch fixes issues for PAN & BNEP in BD stack identified
by static analysis tool.

Change-Id: I29417dae982abb5cef11379f8d03baad11ffde8b

Reset pairing state to idle only for the current device

Reset the pair state during auth complete for the BD_ADDR of the pairing
device only. The authentication complete event may come for profile level
authetication also. In such cases, if device already in paired state,
there is no need to reset the pairing state.

Change-Id: Id0ef9127895177527a6d4559ad873aa1c62a56bb

am 74385ecc: am e35d6ccc: gn-build: Build net_test_device using GN + ninja

* commit '74385eccb4d2f6185cc791b72ff16784661dc3ad':
  gn-build: Build net_test_device using GN + ninja

am e35d6ccc: gn-build: Build net_test_device using GN + ninja

* commit 'e35d6ccc58c52279eae36078ba3368d240f3ab1f':
  gn-build: Build net_test_device using GN + ninja

gn-build: Build net_test_device using GN + ninja

This CL adds gtest dependency to device/BUILD.gn to enable building
net_test_device using GN + ninja.

Bug: 21444879
Change-Id: I3760511d884f7b3562f86abcec96f679704ef2f4

am 426459b5: am 6c014aca: gn-build: Get bluetooth.default.so to compile

* commit '426459b5ae224d9affe441c6b91701b921b4fb1d':
  gn-build: Get bluetooth.default.so to compile

am 6c014aca: gn-build: Get bluetooth.default.so to compile

* commit '6c014acaabf783f2f07b66695ee8bf510a2fff88':
  gn-build: Get bluetooth.default.so to compile

am bdfefe0a: am 303b6ff1: gn-build: Get //bta:bta to compile

* commit 'bdfefe0a140a759317502c5e598e1018947f628e':
  gn-build: Get //bta:bta to compile

am 303b6ff1: gn-build: Get //bta:bta to compile

* commit '303b6ff16e1d257a491d2ae4fac2cd1b78850f67':
  gn-build: Get //bta:bta to compile

am 99d68906: am d30195cb: gn-build: Get btif to compile as its own module

* commit '99d68906fa80be84821d0fba6472f04f7f9bff10':
  gn-build: Get btif to compile as its own module

am d30195cb: gn-build: Get btif to compile as its own module

* commit 'd30195cbf09735da593e186c4c99cce6232b072e':
  gn-build: Get btif to compile as its own module

gn-build: Get bluetooth.default.so to compile

Now that most of the modules here are compiled in their own static libraries,
there is no need to include their sources from main/BUILD.gn. This CL cleans
this up and adds all the missing dependencies for the //main:bluetooth.default
shared library target. Missing includes and non-Android compile errors have
always been fixed.

BUG=21339022

Change-Id: Ibcc693d5ff17aa9c446db777d4b9fbb706f8c7dd

gn-build: Get //bta:bta to compile

This CL gets the //bta:bta target to compile using GN + ninja.

BUG=21339022

Change-Id: I98e7ab732515b638f4f0c356a62f218ec223662a

gn-build: Get btif to compile as its own module

This CL adds support to build //btif:btif as its own target as a static library
by introducing a new module-level BUILD.gn file, adding missing includes, and
conditionally compiling code that is Android-specific (depends on libcutils).
This removes/breaks some functionality on non-Android builds since the
corresponding features are missing until we have a more unified replacement.

BUG=21339022

Change-Id: Ibb0372da45f256c55fa894fdda3d70319ea008c9

Moved the OSI alarm_shutdown() operations to alarm_cleanup()

Also, removed the OSI module's start_up and shut_down steps,
because now they are no-op.

Bug: 21558791
Change-Id: I24259b327f399af57c37937111158baa9704f644

am eae1f0c0: am 17cd1274: gn-build: Get //udrv:udrv to compile

* commit 'eae1f0c0d39a7a937711442ce236bc436ac328b9':
  gn-build: Get //udrv:udrv to compile

am 45a98e2e: am 6b5811e2: gn-build: Get //embrdv/sbc to compile

* commit '45a98e2e682bd4a60188926c54ca6463ad781410':
  gn-build: Get //embrdv/sbc to compile

am 3524b93e: am 24253fd1: gn-build: Compile audio.a2dp.default.so

* commit '3524b93e113a4df8e388f43ada2be57c4549aa71':
  gn-build: Compile audio.a2dp.default.so

am d5f795e9: am 499282bc: gn-build: Get //utils:utils to compile

* commit 'd5f795e92c0d8876494c9790dc2ec59ecd36e5ae':
  gn-build: Get //utils:utils to compile

am 17cd1274: gn-build: Get //udrv:udrv to compile

* commit '17cd127445af1bd8a89bb1ea419e415392f1eee3':
  gn-build: Get //udrv:udrv to compile

am 6b5811e2: gn-build: Get //embrdv/sbc to compile

* commit '6b5811e2b4c6369738ad2d58a7d4cf4740d9a298':
  gn-build: Get //embrdv/sbc to compile

am 24253fd1: gn-build: Compile audio.a2dp.default.so

* commit '24253fd1f10e34e820d2b8bd6c0c78d342dd1591':
  gn-build: Compile audio.a2dp.default.so

am 499282bc: gn-build: Get //utils:utils to compile

* commit '499282bc6a165ef3aa76bbef4722a833a2aea675':
  gn-build: Get //utils:utils to compile

am 1ceba264: am 6850e336: gn-build: Get //stack:stack to compile

* commit '1ceba264ffdfe1ff19915db387f3b94758252f3c':
  gn-build: Get //stack:stack to compile

am b7a246f1: am a2647d0f: gn-build: Get HCI and net_test_hci to compile

* commit 'b7a246f189b336e623ed88f3ede05df1fabc64e7':
  gn-build: Get HCI and net_test_hci to compile

am b434fbb7: am 2c494e15: gn-build: Clean up GN files

* commit 'b434fbb7b739d62c29c0f6943f61b2884a6735c6':
  gn-build: Clean up GN files

am ab511940: am bbb12526: gn-build: Add tinyxml2 as a secondary source

* commit 'ab511940631d194180eacaae66c65cade4d899ad':
  gn-build: Add tinyxml2 as a secondary source

am 6850e336: gn-build: Get //stack:stack to compile

* commit '6850e3364800487eb4c7b5f9ab5a6e366a9a11eb':
  gn-build: Get //stack:stack to compile

am a2647d0f: gn-build: Get HCI and net_test_hci to compile

* commit 'a2647d0f5b0078d95d547a96548462b910bd453c':
  gn-build: Get HCI and net_test_hci to compile

am 2c494e15: gn-build: Clean up GN files

* commit '2c494e15583364de89ab12f417c19dc2d61efea6':
  gn-build: Clean up GN files

am bbb12526: gn-build: Add tinyxml2 as a secondary source

* commit 'bbb12526d4810511cb7c79e17fdf75b0884d8c3d':
  gn-build: Add tinyxml2 as a secondary source

gn-build: Get //udrv:udrv to compile

This patch gets the //udrv:udrv target to compile using GN + ninja as its own
build target. Since this module only contains a single file, keeping it as a
"source_set".

Change-Id: I98c3b78b374f4102b2e6e565841eaa474bef9056

gn-build: Get //embrdv/sbc to compile

This CL builds the SBC codec that lives in embdrv/sbc to compile as a static
library. Unlike the Android build that creates two separate libraries for the
encoder and the decoder, the GN build creates a single static library.

BUG=21339022

Change-Id: I9f12e63339c850d26f49a3756526a185ca7fec8d

gn-build: Compile audio.a2dp.default.so

This patch adds the build file to get the audio.a2dp.default shared module to
compile using GN + ninja.

BUG=21339022

Change-Id: I2c39ee49ca01f023c73b3fa675f6d9d2dc80e133

gn-build: Get //utils:utils to compile

This CL conditionally removes the dependency on libcutils to get the
//utils:utils static library target to compile in GN. This technically breaks
the functionality on non-Android platforms until we come up with a common way to
make the logic work.

BUG=21339022

Change-Id: I47cd8f7991942dbe43a577cef1f95626560f7f72

Disable the OSI allocation tracker.

Temporary disable the allocation tracker initialization
(even for BLUEDROID_DEBUG) when initializing Bluetooth.
This is a short-term workaround solution for several issues related to
the usage of the allocation tracker.

* Inconsistent usage of osi_malloc()/osi_calloc() and osi_free()
  - Within some of the Bluetooth-related unit tests we have two copies
    of the same libosi library: one copy statically linked against the
    unit test, and another dlopen() at runtime as part of bluetooth.default.so
    Each of those copy has its own static variables.
  - For the dlopen() copy we do call allocation_tracker_init(), while for the
    static copy within the unit test we don't call allocation_tracker_init()
  - Occasionally, there is a  memory allocation via osi_calloc()
    within the dlopen()-ed library, and then it is osi_free()-ed
    within the statically linked library.
 Such (mis)usage creates issues in two ways: (1) free()-ing incorrect
 pointer, and (2) the osi hash_map() in the dlopen()-ed library still considers
 the osi-tracked memory as allocated.
 NOTE: (1) could trigger random crashes, while (2) is the reason the unit
 test triggers an assert

* Avoid potential issues that could result from the fact that calling
  allocation_tracker_init() is not mandatory; i.e., the same
  issue described above could be triggered if osi_malloc()/osi_callod()
  was called before the call to allocation_tracker_init()

* There is still code that uses malloc(3)/calloc(3) and free(3) instead
  of osi_malloc()/osi_calloc() and osi_free()

Also, add missing pthread_mutex_lock()/pthread_mutex_unlock() guards,
and fix the allocation_tracker_uninit() implementation so it works
properly even if void allocation_tracker_init() wasn't called.

Bug: 21561735
Change-Id: Ic83d6cd40af1189c4ee9c1dbfd0ad8e4666e1502

gn-build: Get //stack:stack to compile

This CL adds a missing include to get the //stack:stack target to compile.

BUG=21339022

Change-Id: Ie41816514c97ddcb1e4f1f0b9481364a563d8b9b

gn-build: Get HCI and net_test_hci to compile

This CL fixes the HCI module so that the HCI static library and the net_test_hci
unit test can be compiled using GN + ninja.

BUG=21339022,21444689

Change-Id: If3d8649b25297880e5e674a2026badb1b0e01fee

gn-build: Clean up GN files

Removed unnecessary library and include flags from build files. To make the
initial build outputs more similar to Android, this CL changes the usage of the
GN "source_set" idiom to "static_library". This comes at a performance loss
while keeping the builds similar until we get the whole stack to compile.

BUG=21339022

Change-Id: I46f70b3b64b2537634a1dbb517a5168566230efb