commit | 7b97d4d4a45d25a4ac80f3b59a40c115f0a03af4 | [log] [tgz] |
---|---|---|
author | Pavlin Radoslavov <pavlin@google.com> | Sun May 27 20:06:20 2018 -0700 |
committer | Pavlin Radoslavov <pavlin@google.com> | Tue May 29 11:12:40 2018 -0700 |
tree | 8b86139948b6f28f62829c785a8f2a718d4206dc | |
parent | d51883c1f464179bbb66efb04777fc57767ea4c1 [diff] |
Explicitly stop streaming when switching streaming to another device * Explicitly call btif_av_stream_stop() inside btif_a2dp_source_end_session_delayed() when ending the session. It should be called regardless whether we are using software or hardware based encoding. * Removed extra btif_av_stream_stop() call inside BtifAvSource::SetActivePeer(), because it is not needed anymore. It is replaced by the call inside btif_a2dp_source_end_session_delayed() * Added "peer_address" argument to btif_av_stream_stop() so it can be used to stop the stream to a specific peer. * Fix an issue inside bta_av_chk_2nd_start() that was triggering extra start request to some headsets - the extra start request was blocking follow-up legitimate requests. * Simplify the logic inside bta_av_do_start() so it is easier to identify issues when start streaming. The actual processing logic itself is not changed. * Add extra log messages or update existing ones to help debug similar issues in the future. Bug: 79909084 Test: Manual: streaming A2DP, connect a new device, change active device Change-Id: Id583aa44aec7082dc5f94532e6b41a7ea33813a1
Just build AOSP - Fluoride is there by default.
Instructions for Ubuntu, tested on 14.04 with Clang 3.5.0 and 16.10 with Clang 3.8.0
mkdir ~/fluoride cd ~/fluoride git clone https://android.googlesource.com/platform/system/bt
Install dependencies (require sudo access):
cd ~/fluoride/bt build/install_deps.sh
Then fetch third party dependencies:
cd ~/fluoride/bt mkdir third_party cd third_party git clone https://github.com/google/googletest.git git clone https://android.googlesource.com/platform/external/aac git clone https://android.googlesource.com/platform/external/libchrome git clone https://android.googlesource.com/platform/external/libldac git clone https://android.googlesource.com/platform/external/modp_b64 git clone https://android.googlesource.com/platform/external/tinyxml2
And third party dependencies of third party dependencies:
cd fluoride/bt/third_party/libchrome/base/third_party mkdir valgrind cd valgrind curl https://chromium.googlesource.com/chromium/src/base/+/master/third_party/valgrind/valgrind.h?format=TEXT | base64 -d > valgrind.h curl https://chromium.googlesource.com/chromium/src/base/+/master/third_party/valgrind/memcheck.h?format=TEXT | base64 -d > memcheck.h
NOTE: If system/bt is checked out under AOSP, then create symbolic links instead of downloading sources
cd system/bt mkdir third_party cd third_party ln -s ../../../external/aac aac ln -s ../../../external/libchrome libchrome ln -s ../../../external/libldac libldac ln -s ../../../external/modp_b64 modp_b64 ln -s ../../../external/tinyxml2 tinyxml2 ln -s ../../../external/googletest googletest
cd ~/fluoride/bt gn gen out/Default
cd ~/fluoride/bt ninja -C out/Default all
This will build all targets (the shared library, executables, tests, etc) and put them in out/Default. To build an individual target, replace “all” with the target of your choice, e.g. ninja -C out/Default net_test_osi
.
cd ~/fluoride/bt/out/Default LD_LIBRARY_PATH=./ ./bluetoothtbd -create-ipc-socket=fluoride
Follows the Chromium project Eclipse Setup Instructions until “Optional: Building inside Eclipse” section (don't do that section, we will set it up differently)
Generate Eclipse settings:
cd system/bt gn gen --ide=eclipse out/Default
In Eclipse, do File->Import->C/C++->C/C++ Project Settings, choose the XML location under system/bt/out/Default
Right click on the project. Go to Preferences->C/C++ Build->Builder Settings. Uncheck “Use default build command”, but instead using “ninja -C out/Default”
Goto Behaviour tab, change clean command to “-t clean”