media: Check MPRIS Can* properties before calling the respective method

With MPRIS the player can disable support for certain control using
properties such as CanPlay, CanPause..., in this case these methods
should not be called and the code should fallback to uinput method.
This commit is contained in:
Luiz Augusto von Dentz 2013-05-10 14:02:07 +03:00 committed by Johan Hedberg
parent 666209d25c
commit 071f1e6055

View File

@ -102,6 +102,11 @@ struct media_player {
uint32_t duration;
uint8_t volume;
GTimer *timer;
bool play;
bool pause;
bool next;
bool previous;
bool control;
};
static GSList *adapters = NULL;
@ -1151,6 +1156,9 @@ static bool play(void *user_data)
DBG("");
if (!mp->play || !mp->control)
return false;
return media_player_send(mp, "Play");
}
@ -1160,6 +1168,9 @@ static bool stop(void *user_data)
DBG("");
if (!mp->control)
return false;
return media_player_send(mp, "Stop");
}
@ -1169,6 +1180,9 @@ static bool pause(void *user_data)
DBG("");
if (!mp->pause || !mp->control)
return false;
return media_player_send(mp, "Pause");
}
@ -1178,6 +1192,9 @@ static bool next(void *user_data)
DBG("");
if (!mp->next || !mp->control)
return false;
return media_player_send(mp, "Next");
}
@ -1187,6 +1204,9 @@ static bool previous(void *user_data)
DBG("");
if (!mp->previous || !mp->control)
return false;
return media_player_send(mp, "Previous");
}
@ -1512,6 +1532,21 @@ static gboolean set_repeat(struct media_player *mp, DBusMessageIter *iter)
return set_property(mp, "Repeat", value);
}
static gboolean set_flag(struct media_player *mp, DBusMessageIter *iter,
bool *var)
{
dbus_bool_t value;
if (dbus_message_iter_get_arg_type(iter) != DBUS_TYPE_BOOLEAN)
return FALSE;
dbus_message_iter_get_basic(iter, &value);
*var = value;
return TRUE;
}
static gboolean set_player_property(struct media_player *mp, const char *key,
DBusMessageIter *entry)
{
@ -1537,6 +1572,21 @@ static gboolean set_player_property(struct media_player *mp, const char *key,
if (strcasecmp(key, "LoopStatus") == 0)
return set_repeat(mp, &var);
if (strcasecmp(key, "CanPlay") == 0)
return set_flag(mp, &var, &mp->play);
if (strcasecmp(key, "CanPause") == 0)
return set_flag(mp, &var, &mp->pause);
if (strcasecmp(key, "CanGoNext") == 0)
return set_flag(mp, &var, &mp->next);
if (strcasecmp(key, "CanGoPrevious") == 0)
return set_flag(mp, &var, &mp->previous);
if (strcasecmp(key, "CanControl") == 0)
return set_flag(mp, &var, &mp->control);
DBG("%s not supported, ignoring", key);
return TRUE;