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

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

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

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

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

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 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

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

am b8a38cc4: build: Build net_test_btcore using GN + ninja

* commit 'b8a38cc4be1bc41135cf20b5ad344343fa9ed38c':
  build: Build net_test_btcore using GN + ninja

am e4477781: gn-build: Add missing global compiler flags

* commit 'e4477781e54bd54ff8628f5ec1ce4594324c9e9e':
  gn-build: Add missing global compiler flags

gn-build: Add tinyxml2 as a secondary source

Added tinyxml2 as a secondary source. Moved //build/secondary/testing to
//build/secondary/third_party in the process. This means that all third_party
sources (gtest, tinyxml2) should be placed in a top-level third_party directory.

BUG=21339022

Change-Id: If324ef1eb0d5061e005f62a80d23fef25e7e78a4

build: Build net_test_btcore using GN + ninja

Changes needed:
 - add gtest dependency
 - add includes to allow build on Goobuntu Trusty

Bug: 21444625
Change-Id: Ic2495258681c6b93de8d4ddb5dfff2c24248b979

gn-build: Add missing global compiler flags

This CL introduces compiler flags that enable stripping of unused symbols out of
static libraries at the linking stage. This is used by the Android build to keep
module dependencies small while building specialized executables (such as unit
tests).

Also added a preprocessor definition for the EXPORT_SYMBOL macro, recently
introduced by icoolidge@.

BUG=21339022

Change-Id: I7d3f9f64abc1f4e080196c18ccb893443ee1b64a

am 660fd9ef: Merge "Fix use-after-free while setting advertisement data."

* commit '660fd9ef8dad381066d797b39778ff2667631f1e':
  Fix use-after-free while setting advertisement data.

am 05bb84c6: Handle pairing timeout during remote name request

* commit '05bb84c66dc4813b5da13a7d2db27e80fb613293':
  Handle pairing timeout during remote name request

Handle pairing timeout during remote name request

If a remote name request is in progress and a pairing timeout is
triggered, the host needs to post the bond failure to the UI and
needs to reset the pairing cb state, otherwise the UI will be stuck
in pairing mode and scanning won't work until Bluetooth is cycled
OFF->ON.

Change-Id: I76d9a83b1db7236db51617da998b5857b0de39f0

am 8a7e179c: Reduce GATT log verbosity

* commit '8a7e179c9ebe76c0c16a71c33bf322b77e0aa439':
  Reduce GATT log verbosity

Reduce GATT log verbosity

Change-Id: I55e6ec1afdd8a13f4401809ef89ef2d80efb7f66

am 780afa36: Avoid null pointer exception for invalid L2CAP channels

* commit '780afa366705dfe53b9c69e8deb075e0bcbf31c4':
  Avoid null pointer exception for invalid L2CAP channels

am 0d6a501d: Add missing clean_up step for the OSI module.

* commit '0d6a501d0517c186f11cbee294ae2c509073de91':
  Add missing clean_up step for the OSI module.

am 0d45f25e: DO NOT MERGE Fix use-after-free while setting advertisement data.

* commit '0d45f25e4102b14d942efa530f0e6a1a6c19787a':
  DO NOT MERGE Fix use-after-free while setting advertisement data.

am f4932db3: Merge "Sniff: Use correct sniff configuration for PAN & HS" into mnc-dev

* commit 'f4932db32e7553a9d9771e0bed2f517631eaa20c':
  Sniff: Use correct sniff configuration for PAN & HS

Avoid null pointer exception for invalid L2CAP channels

Change-Id: I8ac40e9cfe6dd015f40c6f493727e123a0d247e6

resolved conflicts for merge of 1f81b64b to mnc-dev-plus-aosp

Change-Id: I7997086b1d83bc5ee06068f82ca31c290389bc44

Merge "Fix use-after-free while setting advertisement data."

Add missing clean_up step for the OSI module.

Now the OSI module's shut_down processing is split into "shut_down"
and "clean_up".

Previously, there was an ordering issue manipulating some of the
internal state during graceful shutdown/cleanup. Some of the modules
had two steps: shut_down, followed by clean_up, while other had only
shut_down step. This triggered the following assert in file alarm.c

    alarm_cancel: assertion "alarms != NULL" failed

Bug: 21406940
Change-Id: Iab1f033a69cbff646a6b0f346760ae82f8b00b8f

DO NOT MERGE Fix use-after-free while setting advertisement data.

Producer context was freeing data buffers without
performing a deep copy on the passed structure.

Also added some const correctness so we can reason
about the state of btif_adv_data_t as its data
gets associated with the advertising instance.

Bug: 21343750
Change-Id: Ifea0209e8426b61d8456c299c836ba640bd30f31

Merge "Sniff: Use correct sniff configuration for PAN & HS" into mnc-dev

Sniff: Use correct sniff configuration for PAN & HS

Change-Id: I95c302dd46cdcc63058c9cb3de17fdfd6ffe8d2e

Set default symbol visibility to 'hidden'.

Bug: 20427982
Change-Id: I003469d6dfce681a38baa71b5709236a1a4a2189

Fix use-after-free while setting advertisement data.

Producer context was freeing data buffers without
performing a deep copy on the passed structure.

Also added some const correctness so we can reason
about the state of btif_adv_data_t as its data
gets associated with the advertising instance.

Bug: 21343750

Change-Id: Ifea0209e8426b61d8456c299c836ba640bd30f31

am 6c56b871: Merge "A2DP audio HAL: implement get_presentation_position()" into mnc-dev

* commit '6c56b87166fc7893204c4dd468d873eb6cb64c03':
  A2DP audio HAL: implement get_presentation_position()

Merge "A2DP audio HAL: implement get_presentation_position()" into mnc-dev

Merge commit '82f066fa' into manualmerge

Change-Id: If90cefa8c28b1784d2403533d4d349e5dd4dfa91

am 67a784b9: Make secure connections mode configurable

* commit '67a784b9f9fc73816b9dccd18fcb3863888b444a':
  Make secure connections mode configurable

Merge "Increase the min LE connection interval to 11.25ms" into mnc-dev

Make secure connections mode configurable

Change-Id: Id6101abc1fc339147c8b55023c7426ec1c807ee6

Increase the min LE connection interval to 11.25ms

To reduce audio choppiness, increase the min LE connection interval
to 11.25ms from 7.5ms

Bug: 18776956
Change-Id: I8404ebe2f9562d555a2131ae30dd63b74bb362a4

A2DP audio HAL: implement get_presentation_position()

Bug: 21199150.
Change-Id: If8e95645b636be82a32420cad1ca6a3993e14f1c

am bb6836b8: build: Build net_test_osi using GN + ninja

* commit 'bb6836b8e3145522c119e9478d5c7e7df75c3890':
  build: Build net_test_osi using GN + ninja

am cdc6f3a3: build: Add build arguments for libhardware and core dirs

* commit 'cdc6f3a3efb18bdbed6dad93d9537c18d15af717':
  build: Add build arguments for libhardware and core dirs

am 5c57faaa: build: Add gtest as a secondary source

* commit '5c57faaaca6b6f10647b46cf291546501a1aa837':
  build: Add gtest as a secondary source

am 9ca3fb82: build: Add missing copyright headers to new GN build files

* commit '9ca3fb824639c2f15c104ca99be872e0e55035cb':
  build: Add missing copyright headers to new GN build files

build: Build net_test_osi using GN + ninja

This CL adds support to build the net_test_osi target using GN + ninja.
The BUILD.gn file now pulls in gtest as a dependency. Missing libc
includes have been added to sources that prevented compilation on
Goobuntu (14.04).

The osi/include/log.h header has been modified to conditionally call the
Android log utilities vs plain-old printf on non-Android builds.

BUG=21339022

Change-Id: If3c356360c56c63b3bf551b41dc8d3269d8e7e09

build: Add build arguments for libhardware and core dirs

This patch adds GN arguments to add libhardware and core library include
paths in a generic manner.

Example args.gn file:

    libhw_include_path = "../../libhardware/include"
    core_include_path = "../../core/include"

Also introduced in this CL is the "OS_GENERIC" preprocessor define, that
will allow us to conditionally compile Android-specific vs generic code
until we shed the Android-specific dependencies.

BUG=21339022

Change-Id: I42b1cc501738653a6e886a0365c9cbef2e9a525e

build: Add gtest as a secondary source

As a temporary solution, this CL introduces gtest as a secondary build
source. For now, gtest can be included in testonly targets as a direct
GN target dependency, by placing the gtest source under "testing/gtest"
at the root of the tree.

Moving forward, we can use solutions like gclient to pull in external
dependencies such as gtest, or we can depend on it being available on
the system.

BUG=21339022,21340407

Change-Id: I80ab3a81a53d7c29d54857a8d0cd9677fed77549

build: Add missing copyright headers to new GN build files

BUG=21339022

Change-Id: I946fad7bf8116054f9081e064748793e79a6f85a

am dea4f9b4: Pairing is prompted with a wrong bd-address

* commit 'dea4f9b42fd373d8a46565d933ec49a6583a9117':
  Pairing is prompted with a wrong bd-address

Pairing is prompted with a wrong bd-address

Slave initiated encryption was not being notified on the correct
address and causes the key to be lost on slave device triggering
new pairing once in a while.

Change-Id: I5441a9e0bcf8ba45199483ccd02f207892ec296e

am c39c7615: Fix problem where multi-adv instances cannot be re-used

* commit 'c39c7615b7b2eb267725df73395e16dd78a8ef8f':
  Fix problem where multi-adv instances cannot be re-used

Fix problem where multi-adv instances cannot be re-used

After using all multi-adv instances and stopping them, the instance ID
for each instance is incorrectly reset, causing the instance to be not
usable anymore.

Bug: 21404808
Change-Id: Ie3a3f07bc2204c7268567cb56248cd3eb7751742

am 04b5e929: Recitfy condition when removing device from IRK resolving list

* commit '04b5e9294be52782500f0be01fd10d43bb330ce0':
  Recitfy condition when removing device from IRK resolving list

Recitfy condition when removing device from IRK resolving list

This patch corrects the conditional logic before sending HCI
command to remove the IRK list entry from the IRK resolving
list based on the max size of resolving list

Change-Id: I9adc933f0988ae907b3e218ba63814d106784787

am a9c5301d: Fix build after cross-key issues broke non privacy enabled builds

* commit 'a9c5301d8dcddedc46f6a713039b60b3cb0e2bf6':
  Fix build after cross-key issues broke non privacy enabled builds

am 0baef63c: Fix device/address types and IRK value handling

* commit '0baef63c70dba86c4cdb2744a2ab6c3f713ba353':
  Fix device/address types and IRK value handling

am c0a4342c: Fixed cross key for LE when paired over BR-EDR

* commit 'c0a4342caa940e4d214d748c09583e7c8bff7c74':
  Fixed cross key for LE when paired over BR-EDR

am d0aa8e53: Fix SMP pairing request issue on LE

* commit 'd0aa8e53aa7ac1137a47570e08c2b963aaa49141':
  Fix SMP pairing request issue on LE

am 6fc9526b: Unpair the other transport on a device paired

* commit '6fc9526bba383fdb33c0536ed050489aa2215554':
  Unpair the other transport on a device paired

Merge "Resolved conflicts & merged commit '877123f5' into mnc-dev-plus-aosp" into mnc-dev-plus-aosp

Resolved conflicts & merged commit '877123f5' into mnc-dev-plus-aosp

Change-Id: Ie2961d5dc80c6f7e755ba184bd93f01aaffe44a2

am 09182829: Merge "Initial attempt at a GN/Ninja-based build system"

* commit '0918282997ee6a2b24c49e8758da959d004eaadb':
  Initial attempt at a GN/Ninja-based build system

Merge "Initial attempt at a GN/Ninja-based build system"

Static address is seen on scanner after unpairing

Fix for bonded devices that cannot be removed from resolving list when
scanning, advertising or when a connection is being initiated.
Suspend all active adv, scanning or connection initiation upon
completion of any resolving list operation.

Original author: Chaojing Sun <cjsun@broadcom.com>
Change-Id: Id104dace2944b1878eaf65b93aba3d01e477191d

Fix for chip capability version check

Fixes the issue of extended scan command being sent out
for non-capable chipsets

Bug: 20916419
Change-Id: Ib5d848f49c34a3fc9c053a7488e12fb0c252587c

Fix build after cross-key issues broke non privacy enabled builds

Change-Id: Idef7239b07115be9290d7cca6610c603ffb3afa5

Fix device/address types and IRK value handling

Handles the device and address types based on BD address
and provides the IRK values as available.

Change-Id: I351e0aea7f5e55caefa405b34261c75e78306552

Fixed cross key for LE when paired over BR-EDR

SMP state machine was resending security request and waiting for a response
when paired over BR-EDR. The state machine state timed out due to this and
SMP failed while doing cross key pairing. Thus, user was asked for pairing
again.

Original author: Priti Aghera <paghera@broadcom.com>
Change-Id: I568f936bb7bee21fb78e8454469ecad3445f026f

Fix SMP pairing request issue on LE

The cause of the failure:
- the master device sent the three keys to slave, but slave only receive
two and got the link drop before the third key is received.
- the slave device treats it as pairing failure due to the key missing.
SMP should wait for all keys been sent to controller before dropping the link.

Implement L2CAP fix channel tx_complete callback for this reason,
and it has been applied on the channel 6 (SMP).
Channel 7 (BR_SMP channel) was not keeping track of total_tx_unacked
number and closed the link too early. Added check in the
smp_br_key_distribution() and the ACL data tracking there before
posting SMP_BR_AUTH_CMPL_EVT.

Original author: Chaojing Sun <cjsun@broadcom.com>

Change-Id: If48a4c5e28b1d177f14ff089e8dfa3ace41eba83

Unpair the other transport on a device paired

When unpair is initiated for LE or BR-EDR transport for a device,
close the link for other transport and unpair the device on
both transports, since both transports on a device are paired
due to cross key.

Original author: Priti Aghera <paghera@broadcom.com>
Change-Id: I13d4757eb056c4bd4c7cc0699ec61a28852702c5

Static address is seen on scanner after unpairing

Fix for bonded devices that cannot be removed from resolving list when
scanning, advertising or when a connection is being initiated.
Suspend all active adv, scanning or connection initiation upon
completion of any resolving list operation.

Original author: Chaojing Sun <cjsun@broadcom.com>
Change-Id: Id104dace2944b1878eaf65b93aba3d01e477191d

Fix for chip capability version check

Fixes the issue of extended scan command being sent out
for non-capable chipsets

Bug: 20916419
Change-Id: Ib5d848f49c34a3fc9c053a7488e12fb0c252587c

am 643aea51: Fix allocation for LE scan filter parameters

* commit '643aea51738e1a472f29354f02b7d941df488a4a':
  Fix allocation for LE scan filter parameters

am ecce764b: Fix for premature BLE RPA timeout

* commit 'ecce764b6cc35d15b591ffac48d1dc044f32fa6e':
  Fix for premature BLE RPA timeout

am 7cb76b42: Fix GATT connection failure after BT reset

* commit '7cb76b4277d603e5abfe7f9222a45ebdf7a715e2':
  Fix GATT connection failure after BT reset

am 6e872cb5: Use correct structure to read LE connection handle

* commit '6e872cb5f5329d7f747091ba759f832bab4b45ec':
  Use correct structure to read LE connection handle

Fix allocation for LE scan filter parameters

This change allocates memory for LE scan filter parameters
to copy filter params set by btif inquiry process initiated
using Settings UI

Change-Id: I56ed33b80af3f26ddc8a83edb807f88048503fa7

Fix for premature BLE RPA timeout

RPA timeout expires prematurely when multiple FindMe server
instances start and stop advertisiment continuously. During start
adv TIMER3 is started using oneshot timer and during stop adv
same TIMER0 is stopped, so the TIMER3 is still running and expires
prematurely. This fix stops TIMER3 during disable adv and when RPA
is sent to controller.

Change-Id: I336f84043e5e0e8a833cc6080f8bcc0c1665893e

Fix GATT connection failure after BT reset

Handles the address type incorrect value storage issue
and removes usage of DMT supported flag which was based
on legacy code

Bug: 20912064
Change-Id: I5bd20dc9cd0997ac3dbd4b346cc0513f1809f241

Use correct structure to read LE connection handle

This patch corrects the retrieval of the connection handle
after the gatt write operation has succeeded successfully.

Change-Id: Iac79a64d6c626c2349b6f1a3744ea49a521a45f4

am 0ef6b347: Build the shared library with --whole-archive

* commit '0ef6b347b8cf1a9e83e6b680a0ee025da24f00b6':
  Build the shared library with --whole-archive

am 7bd4038e: Fix for IRK and IR key write issue

* commit '7bd4038ea9a2f1953dc98fa67aec413b6247c584':
  Fix for IRK and IR key write issue

am 60361470: Re-enabled enhanced inquiry scan

* commit '603614704658ce7081a61a4f8929a1a979b2b449':
  Re-enabled enhanced inquiry scan

am 1536a0e3: Fix format-security warning found by clang/llvm.

* commit '1536a0e3d6a8b149644dfe3f7238ae5c6f6e853a':
  Fix format-security warning found by clang/llvm.

Build the shared library with --whole-archive

For stack static libraries, use LOCAL_WHOLE_STATIC_LIBRARIES
to ensure they get --whole-archive applied to them.

This means module symbols in static libraries won't be
removed by the linker and dlsym will find them.

This patch also removes the code hacks we needed to
trick the linker into including the module symbols in
the final shared library.

Change-Id: I2463d0e6fb38f1e75c8293179cf9d4ca33eda84e

Fix for IRK and IR key write issue

Fixes the incorrect flipping of values
for IRK and IR keys in bt_config.conf file

Change-Id: Ia4f9c274b6a3f7594ec31d8bd89b27d1ae54ac78

Re-enabled enhanced inquiry scan

A merge/re-factoring error caused enhanced inquiry scan to be disabled,
severely slowing down inquiry and device name resolution.

Change-Id: Id7bc1687a1e1bbbbc8b8b42cc184ce628b6d1b9c

Fix format-security warning found by clang/llvm.

BUG: 21299815
Change-Id: I3dd7e92444a9b439c210f7e6ce677307a38113ab