glvideoflip: implement GstVideoDirection interface

It implements now this interface with its video-direction
property. Values are changed to GstVideoOrientationMethod but they have
the same value than the originals.

https://bugzilla.gnome.org/show_bug.cgi?id=768687
This commit is contained in:
Xabier Rodriguez Calvar 2016-07-26 19:55:13 +02:00 committed by Sebastian Dröge
parent 15da533580
commit 8ada38e8f4
2 changed files with 63 additions and 41 deletions

View file

@ -46,6 +46,7 @@ enum
{ {
PROP_0, PROP_0,
PROP_METHOD, PROP_METHOD,
PROP_VIDEO_DIRECTION
}; };
static GstStaticPadTemplate _sink_template = GST_STATIC_PAD_TEMPLATE ("sink", static GstStaticPadTemplate _sink_template = GST_STATIC_PAD_TEMPLATE ("sink",
@ -98,11 +99,6 @@ gst_video_flip_method_get_type (void)
return video_flip_method_type; return video_flip_method_type;
} }
#define gst_gl_video_flip_parent_class parent_class
G_DEFINE_TYPE_WITH_CODE (GstGLVideoFlip, gst_gl_video_flip,
GST_TYPE_BIN, GST_DEBUG_CATEGORY_INIT (GST_CAT_DEFAULT,
"glvideoflip", 0, "glvideoflip element"););
static void gst_gl_video_flip_finalize (GObject * object); static void gst_gl_video_flip_finalize (GObject * object);
static void gst_gl_video_flip_set_property (GObject * object, guint prop_id, static void gst_gl_video_flip_set_property (GObject * object, guint prop_id,
const GValue * value, GParamSpec * pspec); const GValue * value, GParamSpec * pspec);
@ -114,6 +110,24 @@ static GstPadProbeReturn _input_sink_probe (GstPad * pad,
static GstPadProbeReturn _trans_src_probe (GstPad * pad, GstPadProbeInfo * info, static GstPadProbeReturn _trans_src_probe (GstPad * pad, GstPadProbeInfo * info,
gpointer user_data); gpointer user_data);
static void
gst_gl_video_flip_video_direction_interface_init (GstVideoDirectionInterface
* iface);
#define gst_gl_video_flip_parent_class parent_class
G_DEFINE_TYPE_WITH_CODE (GstGLVideoFlip, gst_gl_video_flip,
GST_TYPE_BIN, GST_DEBUG_CATEGORY_INIT (GST_CAT_DEFAULT,
"glvideoflip", 0, "glvideoflip element");
G_IMPLEMENT_INTERFACE (GST_TYPE_VIDEO_DIRECTION,
gst_gl_video_flip_video_direction_interface_init););
static void
gst_gl_video_flip_video_direction_interface_init (GstVideoDirectionInterface
* iface)
{
/* We implement the video-direction property */
}
static void static void
gst_gl_video_flip_class_init (GstGLVideoFlipClass * klass) gst_gl_video_flip_class_init (GstGLVideoFlipClass * klass)
{ {
@ -128,10 +142,13 @@ gst_gl_video_flip_class_init (GstGLVideoFlipClass * klass)
gobject_class->get_property = gst_gl_video_flip_get_property; gobject_class->get_property = gst_gl_video_flip_get_property;
g_object_class_install_property (gobject_class, PROP_METHOD, g_object_class_install_property (gobject_class, PROP_METHOD,
g_param_spec_enum ("method", "method", "method", g_param_spec_enum ("method", "method",
"method (deprecated, use video-direction instead)",
GST_TYPE_GL_VIDEO_FLIP_METHOD, DEFAULT_METHOD, GST_TYPE_GL_VIDEO_FLIP_METHOD, DEFAULT_METHOD,
GST_PARAM_CONTROLLABLE | G_PARAM_READWRITE | G_PARAM_CONSTRUCT | GST_PARAM_CONTROLLABLE | G_PARAM_READWRITE | G_PARAM_CONSTRUCT |
G_PARAM_STATIC_STRINGS)); G_PARAM_STATIC_STRINGS));
g_object_class_override_property (gobject_class, PROP_VIDEO_DIRECTION,
"video-direction");
gst_element_class_add_static_pad_template (element_class, &_src_template); gst_element_class_add_static_pad_template (element_class, &_src_template);
gst_element_class_add_static_pad_template (element_class, &_sink_template); gst_element_class_add_static_pad_template (element_class, &_sink_template);
@ -233,10 +250,10 @@ _transform_caps (GstGLVideoFlip * vf, GstPadDirection direction, GstCaps * caps)
gst_structure_get_int (structure, "height", &height)) { gst_structure_get_int (structure, "height", &height)) {
switch (vf->active_method) { switch (vf->active_method) {
case GST_GL_VIDEO_FLIP_METHOD_90R: case GST_VIDEO_ORIENTATION_90R:
case GST_GL_VIDEO_FLIP_METHOD_90L: case GST_VIDEO_ORIENTATION_90L:
case GST_GL_VIDEO_FLIP_METHOD_FLIP_UL_LR: case GST_VIDEO_ORIENTATION_UL_LR:
case GST_GL_VIDEO_FLIP_METHOD_FLIP_UR_LL: case GST_VIDEO_ORIENTATION_UR_LL:
gst_structure_set (structure, "width", G_TYPE_INT, height, gst_structure_set (structure, "width", G_TYPE_INT, height,
"height", G_TYPE_INT, width, NULL); "height", G_TYPE_INT, width, NULL);
if (gst_structure_get_fraction (structure, "pixel-aspect-ratio", if (gst_structure_get_fraction (structure, "pixel-aspect-ratio",
@ -251,10 +268,10 @@ _transform_caps (GstGLVideoFlip * vf, GstPadDirection direction, GstCaps * caps)
} }
} }
break; break;
case GST_GL_VIDEO_FLIP_METHOD_IDENTITY: case GST_VIDEO_ORIENTATION_IDENTITY:
case GST_GL_VIDEO_FLIP_METHOD_180: case GST_VIDEO_ORIENTATION_180:
case GST_GL_VIDEO_FLIP_METHOD_FLIP_HORIZ: case GST_VIDEO_ORIENTATION_HORIZ:
case GST_GL_VIDEO_FLIP_METHOD_FLIP_VERT: case GST_VIDEO_ORIENTATION_VERT:
break; break;
default: default:
g_assert_not_reached (); g_assert_not_reached ();
@ -268,7 +285,7 @@ _transform_caps (GstGLVideoFlip * vf, GstPadDirection direction, GstCaps * caps)
/* with object lock */ /* with object lock */
static void static void
_set_active_method (GstGLVideoFlip * vf, GstGLVideoFlipMethod method, _set_active_method (GstGLVideoFlip * vf, GstVideoOrientationMethod method,
GstCaps * caps) GstCaps * caps)
{ {
gfloat rot_z = 0., scale_x = 1.0, scale_y = 1.0; gfloat rot_z = 0., scale_x = 1.0, scale_y = 1.0;
@ -276,34 +293,34 @@ _set_active_method (GstGLVideoFlip * vf, GstGLVideoFlipMethod method,
GstPad *srcpad; GstPad *srcpad;
switch (method) { switch (method) {
case GST_GL_VIDEO_FLIP_METHOD_IDENTITY: case GST_VIDEO_ORIENTATION_IDENTITY:
break; break;
case GST_GL_VIDEO_FLIP_METHOD_90R: case GST_VIDEO_ORIENTATION_90R:
scale_x *= vf->aspect; scale_x *= vf->aspect;
scale_y *= 1. / vf->aspect; scale_y *= 1. / vf->aspect;
rot_z = 90.; rot_z = 90.;
break; break;
case GST_GL_VIDEO_FLIP_METHOD_180: case GST_VIDEO_ORIENTATION_180:
rot_z = 180.; rot_z = 180.;
break; break;
case GST_GL_VIDEO_FLIP_METHOD_90L: case GST_VIDEO_ORIENTATION_90L:
scale_x *= vf->aspect; scale_x *= vf->aspect;
scale_y *= 1. / vf->aspect; scale_y *= 1. / vf->aspect;
rot_z = 270.; rot_z = 270.;
break; break;
case GST_GL_VIDEO_FLIP_METHOD_FLIP_HORIZ: case GST_VIDEO_ORIENTATION_HORIZ:
scale_x *= -1.; scale_x *= -1.;
break; break;
case GST_GL_VIDEO_FLIP_METHOD_FLIP_UR_LL: case GST_VIDEO_ORIENTATION_UR_LL:
scale_x *= -vf->aspect; scale_x *= -vf->aspect;
scale_y *= 1. / vf->aspect; scale_y *= 1. / vf->aspect;
rot_z = 90.; rot_z = 90.;
break; break;
case GST_GL_VIDEO_FLIP_METHOD_FLIP_VERT: case GST_VIDEO_ORIENTATION_VERT:
scale_x *= -1.; scale_x *= -1.;
rot_z = 180.; rot_z = 180.;
break; break;
case GST_GL_VIDEO_FLIP_METHOD_FLIP_UL_LR: case GST_VIDEO_ORIENTATION_UL_LR:
scale_x *= -vf->aspect; scale_x *= -vf->aspect;
scale_y *= 1. / vf->aspect; scale_y *= 1. / vf->aspect;
rot_z = 270.; rot_z = 270.;
@ -331,10 +348,17 @@ _set_active_method (GstGLVideoFlip * vf, GstGLVideoFlipMethod method,
} }
static void static void
gst_gl_video_flip_set_method (GstGLVideoFlip * vf, GstGLVideoFlipMethod method, gst_gl_video_flip_set_method (GstGLVideoFlip * vf,
gboolean from_tag) GstVideoOrientationMethod method, gboolean from_tag)
{ {
GST_OBJECT_LOCK (vf); GST_OBJECT_LOCK (vf);
if (method == GST_VIDEO_ORIENTATION_CUSTOM) {
GST_WARNING_OBJECT (vf, "unsupported custom orientation");
GST_OBJECT_UNLOCK (vf);
return;
}
/* Store updated method */ /* Store updated method */
if (from_tag) if (from_tag)
vf->tag_method = method; vf->tag_method = method;
@ -342,7 +366,7 @@ gst_gl_video_flip_set_method (GstGLVideoFlip * vf, GstGLVideoFlipMethod method,
vf->method = method; vf->method = method;
/* Get the new method */ /* Get the new method */
if (vf->method == GST_GL_VIDEO_FLIP_METHOD_AUTO) if (vf->method == GST_VIDEO_ORIENTATION_AUTO)
method = vf->tag_method; method = vf->tag_method;
else else
method = vf->method; method = vf->method;
@ -367,6 +391,7 @@ gst_gl_video_flip_set_property (GObject * object, guint prop_id,
switch (prop_id) { switch (prop_id) {
case PROP_METHOD: case PROP_METHOD:
case PROP_VIDEO_DIRECTION:
gst_gl_video_flip_set_method (vf, g_value_get_enum (value), FALSE); gst_gl_video_flip_set_method (vf, g_value_get_enum (value), FALSE);
break; break;
default: default:
@ -383,6 +408,7 @@ gst_gl_video_flip_get_property (GObject * object, guint prop_id,
switch (prop_id) { switch (prop_id) {
case PROP_METHOD: case PROP_METHOD:
case PROP_VIDEO_DIRECTION:
g_value_set_enum (value, vf->method); g_value_set_enum (value, vf->method);
break; break;
default: default:
@ -409,29 +435,25 @@ _input_sink_probe (GstPad * pad, GstPadProbeInfo * info, gpointer user_data)
if (gst_tag_list_get_string (taglist, "image-orientation", if (gst_tag_list_get_string (taglist, "image-orientation",
&orientation)) { &orientation)) {
if (!g_strcmp0 ("rotate-0", orientation)) if (!g_strcmp0 ("rotate-0", orientation))
gst_gl_video_flip_set_method (vf, GST_GL_VIDEO_FLIP_METHOD_IDENTITY, gst_gl_video_flip_set_method (vf, GST_VIDEO_ORIENTATION_IDENTITY,
TRUE); TRUE);
else if (!g_strcmp0 ("rotate-90", orientation)) else if (!g_strcmp0 ("rotate-90", orientation))
gst_gl_video_flip_set_method (vf, GST_GL_VIDEO_FLIP_METHOD_90R, gst_gl_video_flip_set_method (vf, GST_VIDEO_ORIENTATION_90R, TRUE);
TRUE);
else if (!g_strcmp0 ("rotate-180", orientation)) else if (!g_strcmp0 ("rotate-180", orientation))
gst_gl_video_flip_set_method (vf, GST_GL_VIDEO_FLIP_METHOD_180, gst_gl_video_flip_set_method (vf, GST_VIDEO_ORIENTATION_180, TRUE);
TRUE);
else if (!g_strcmp0 ("rotate-270", orientation)) else if (!g_strcmp0 ("rotate-270", orientation))
gst_gl_video_flip_set_method (vf, GST_GL_VIDEO_FLIP_METHOD_90L, gst_gl_video_flip_set_method (vf, GST_VIDEO_ORIENTATION_90L, TRUE);
TRUE);
else if (!g_strcmp0 ("flip-rotate-0", orientation)) else if (!g_strcmp0 ("flip-rotate-0", orientation))
gst_gl_video_flip_set_method (vf, gst_gl_video_flip_set_method (vf,
GST_GL_VIDEO_FLIP_METHOD_FLIP_HORIZ, TRUE); GST_VIDEO_ORIENTATION_HORIZ, TRUE);
else if (!g_strcmp0 ("flip-rotate-90", orientation)) else if (!g_strcmp0 ("flip-rotate-90", orientation))
gst_gl_video_flip_set_method (vf, gst_gl_video_flip_set_method (vf,
GST_GL_VIDEO_FLIP_METHOD_FLIP_UR_LL, TRUE); GST_VIDEO_ORIENTATION_UR_LL, TRUE);
else if (!g_strcmp0 ("flip-rotate-180", orientation)) else if (!g_strcmp0 ("flip-rotate-180", orientation))
gst_gl_video_flip_set_method (vf, gst_gl_video_flip_set_method (vf, GST_VIDEO_ORIENTATION_VERT, TRUE);
GST_GL_VIDEO_FLIP_METHOD_FLIP_VERT, TRUE);
else if (!g_strcmp0 ("flip-rotate-270", orientation)) else if (!g_strcmp0 ("flip-rotate-270", orientation))
gst_gl_video_flip_set_method (vf, gst_gl_video_flip_set_method (vf,
GST_GL_VIDEO_FLIP_METHOD_FLIP_UL_LR, TRUE); GST_VIDEO_ORIENTATION_UL_LR, TRUE);
g_free (orientation); g_free (orientation);
} }

View file

@ -78,9 +78,9 @@ struct _GstGLVideoFlip
GstCaps *input_caps; GstCaps *input_caps;
/* properties */ /* properties */
GstGLVideoFlipMethod method; GstVideoOrientationMethod method;
GstGLVideoFlipMethod tag_method; GstVideoOrientationMethod tag_method;
GstGLVideoFlipMethod active_method; GstVideoOrientationMethod active_method;
gfloat aspect; gfloat aspect;
}; };