blob: 783c42ad7056406f081ed81e4934abe11bb63e26 [file] [log] [blame]
Fix pause/play
The current player state is now notified via the state-changed signal,
and in the GTK UI it was only used to keep track of the desired state.
This is a backport of upstream commit 738479c7a0.
Upstream-Status: Backport
Signed-off-by: Jussi Kukkonen <jussi.kukkonen@intel.com>
---
gtk/gtk-play.c | 8 ++++++--
lib/gst/player/gstplayer.c | 12 ------------
lib/gst/player/gstplayer.h | 2 --
3 files changed, 6 insertions(+), 16 deletions(-)
diff --git a/gtk/gtk-play.c b/gtk/gtk-play.c
index 6e7a098..e2b605a 100644
--- a/gtk/gtk-play.c
+++ b/gtk/gtk-play.c
@@ -34,6 +34,7 @@ typedef struct
GtkWidget *info_bar;
GtkWidget *volume_button;
gulong seekbar_value_changed_signal_id;
+ gboolean playing;
} GtkPlay;
/* Compat stubs */
@@ -118,12 +119,13 @@ play_pause_clicked_cb (GtkButton * button, GtkPlay * play)
{
GtkWidget *image;
- if (gst_player_is_playing (play->player)) {
+ if (play->playing) {
gst_player_pause (play->player);
image =
gtk_image_new_from_icon_name ("media-playback-start",
GTK_ICON_SIZE_BUTTON);
gtk_button_set_image (GTK_BUTTON (play->play_pause_button), image);
+ play->playing = FALSE;
} else {
gchar *title;
@@ -136,6 +138,7 @@ play_pause_clicked_cb (GtkButton * button, GtkPlay * play)
title = gst_player_get_uri (play->player);
set_title (play, title);
g_free (title);
+ play->playing = TRUE;
}
}
@@ -335,7 +338,7 @@ video_dimensions_changed_cb (GstPlayer * unused, gint width, gint height,
static void
eos_cb (GstPlayer * unused, GtkPlay * play)
{
- if (gst_player_is_playing (play->player)) {
+ if (play->playing) {
GList *next = NULL;
gchar *uri;
@@ -452,6 +455,7 @@ main (gint argc, gchar ** argv)
}
play.player = gst_player_new ();
+ play.playing = TRUE;
g_object_set (play.player, "dispatch-to-main-context", TRUE, NULL);
diff --git a/lib/gst/player/gstplayer.c b/lib/gst/player/gstplayer.c
index 069b284..78e7ba1 100644
--- a/lib/gst/player/gstplayer.c
+++ b/lib/gst/player/gstplayer.c
@@ -1422,18 +1422,6 @@ gst_player_set_uri (GstPlayer * self, const gchar * val)
g_object_set (self, "uri", val, NULL);
}
-gboolean
-gst_player_is_playing (GstPlayer * self)
-{
- gboolean val;
-
- g_return_val_if_fail (GST_IS_PLAYER (self), FALSE);
-
- g_object_get (self, "is-playing", &val, NULL);
-
- return val;
-}
-
GstClockTime
gst_player_get_position (GstPlayer * self)
{
diff --git a/lib/gst/player/gstplayer.h b/lib/gst/player/gstplayer.h
index 6933dd7..35fb5bb 100644
--- a/lib/gst/player/gstplayer.h
+++ b/lib/gst/player/gstplayer.h
@@ -93,8 +93,6 @@ gchar * gst_player_get_uri (GstPlayer * player);
void gst_player_set_uri (GstPlayer * player,
const gchar * uri);
-gboolean gst_player_is_playing (GstPlayer * player);
-
GstClockTime gst_player_get_position (GstPlayer * player);
GstClockTime gst_player_get_duration (GstPlayer * player);
--
2.1.4