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