From 6e9217d3372b8d4fa737e485d361d09a85be64f0 Mon Sep 17 00:00:00 2001 From: Matt Ranostay Date: Wed, 27 Mar 2019 18:25:47 -0700 Subject: binding: bluetooth: allow multiple paths for org.bluez.MediaPlayer1 interfaces Previously org.bluez.MediaPlayer1 paths could only resolve to player0 but certain phones request another instance for another application (e.g. player1). This patchset changes the default media player path to last connected, and which in theory should be from the media application in use on the phone. From limited experiments not in use interfaces will be reaped shortly after going idle. Bug-AGL: SPEC-2281 Change-Id: Id4bdc89cd395d5cd3ac043394761a95b306e5c6d Signed-off-by: Matt Ranostay --- README.md | 5 +++-- binding/bluetooth-api.c | 10 +++++++--- binding/bluetooth-api.h | 4 ++-- 3 files changed, 12 insertions(+), 7 deletions(-) diff --git a/README.md b/README.md index fa77efc..5e80f7e 100644 --- a/README.md +++ b/README.md @@ -289,8 +289,9 @@ Playing audio reporting event (not all fields will be passed in every event): "genre": "Dance & DJ/General" }, "position": 5600, - "status": "playing" - "connected": false + "status": "playing", + "connected": true, + "player": "player0" } diff --git a/binding/bluetooth-api.c b/binding/bluetooth-api.c index 4c1165c..f849301 100644 --- a/binding/bluetooth-api.c +++ b/binding/bluetooth-api.c @@ -356,13 +356,14 @@ static void bluez_devices_signal_callback( event = ns->media_event; } json_object_object_add(jresp, "properties", jobj); - } else if (is_mediaplayer1_interface(path) && - g_str_has_suffix(path, BLUEZ_DEFAULT_PLAYER)) { - + } else if (is_mediaplayer1_interface(path)) { + gchar *player = find_index(path, 5); json_object_object_add(jresp, "connected", json_object_new_boolean(TRUE)); json_object_object_add(jresp, "type", json_object_new_string("playback")); + json_object_object_add(jresp, "player", + json_object_new_string(player)); mediaplayer1_set_path(ns, path); event = ns->media_event; } else { @@ -390,10 +391,13 @@ static void bluez_devices_signal_callback( event = ns->media_event; } else if (is_mediaplayer1_interface(path)) { + gchar *player = find_index(path, 5); json_object_object_add(jresp, "connected", json_object_new_boolean(FALSE)); json_object_object_add(jresp, "type", json_object_new_string("playback")); + json_object_object_add(jresp, "player", + json_object_new_string(player)); event = ns->media_event; /* adapter removal */ } else if (split_length(path) == 4) { diff --git a/binding/bluetooth-api.h b/binding/bluetooth-api.h index 6b24d13..a0b807b 100644 --- a/binding/bluetooth-api.h +++ b/binding/bluetooth-api.h @@ -116,9 +116,9 @@ static inline gboolean is_mediaplayer1_interface(const char *path) if (split_length(path) != 6) return FALSE; - // TODO: allow mutiple players per device + // Check for 'playerX' suffix, not always player0 data = find_index(path, 5); - ret = !g_strcmp0(data, BLUEZ_DEFAULT_PLAYER); + ret = !strncmp(data, BLUEZ_DEFAULT_PLAYER, sizeof(BLUEZ_DEFAULT_PLAYER) - 1); g_free(data); return ret; -- cgit 1.2.3-korg