RTP: Workaround drawing issues on resolution change

This commit is contained in:
Marvin W 2022-01-27 22:08:35 +01:00
parent 5089e9cad7
commit 0c524cdc4a
No known key found for this signature in database
GPG key ID: 072E9235DB996F2A

View file

@ -7,7 +7,7 @@ public class Dino.Plugins.Rtp.VideoWidget : Gtk.Bin, Dino.Plugins.VideoCallWidge
private static uint last_id = 0; private static uint last_id = 0;
public uint id { get; private set; } public uint id { get; private set; }
public Gst.Element element { get; private set; } public Gst.Base.Sink sink { get; private set; }
public Gtk.Widget widget { get; private set; } public Gtk.Widget widget { get; private set; }
public Plugin plugin { get; private set; } public Plugin plugin { get; private set; }
@ -20,18 +20,22 @@ public class Dino.Plugins.Rtp.VideoWidget : Gtk.Bin, Dino.Plugins.VideoCallWidge
private Gst.Element? connected_device_element; private Gst.Element? connected_device_element;
private Stream? connected_stream; private Stream? connected_stream;
private Gst.Element prepare; private Gst.Element prepare;
private Gst.Caps last_input_caps;
private Gst.Caps last_caps;
public VideoWidget(Plugin plugin) { public VideoWidget(Plugin plugin) {
this.plugin = plugin; this.plugin = plugin;
id = last_id++; id = last_id++;
element = Gst.ElementFactory.make("gtksink", @"video_widget_$id"); sink = Gst.ElementFactory.make("gtksink", @"video_widget_$id") as Gst.Base.Sink;
if (element != null) { if (sink != null) {
Gtk.Widget widget; Gtk.Widget widget;
element.@get("widget", out widget); sink.@get("widget", out widget);
element.@set("async", false); sink.@set("async", false);
element.@set("sync", true); sink.@set("sync", true);
sink.@set("ignore-alpha", false);
this.widget = widget; this.widget = widget;
this.widget.draw.connect(fix_caps_issues);
add(widget); add(widget);
widget.visible = true; widget.visible = true;
} else { } else {
@ -42,76 +46,135 @@ public class Dino.Plugins.Rtp.VideoWidget : Gtk.Bin, Dino.Plugins.VideoCallWidge
public void input_caps_changed(GLib.Object pad, ParamSpec spec) { public void input_caps_changed(GLib.Object pad, ParamSpec spec) {
Gst.Caps? caps = ((Gst.Pad)pad).caps; Gst.Caps? caps = ((Gst.Pad)pad).caps;
if (caps == null) return; if (caps == null) {
warning("Input: No caps");
return;
}
int width, height; int width, height;
caps.get_structure(0).get_int("width", out width); caps.get_structure(0).get_int("width", out width);
caps.get_structure(0).get_int("height", out height); caps.get_structure(0).get_int("height", out height);
debug("Input resolution changed: %ix%i", width, height);
resolution_changed(width, height); resolution_changed(width, height);
last_input_caps = caps;
}
public void processed_input_caps_changed(GLib.Object pad, ParamSpec spec) {
Gst.Caps? caps = (pad as Gst.Pad).caps;
if (caps == null) {
warning("Processed input: No caps");
return;
}
int width, height;
caps.get_structure(0).get_int("width", out width);
caps.get_structure(0).get_int("height", out height);
debug("Processed resolution changed: %ix%i", width, height);
sink.set_caps(caps);
last_caps = caps;
} }
public void after_size_allocate(Gtk.Allocation allocation) { public void after_size_allocate(Gtk.Allocation allocation) {
if (prepare != null) { if (prepare != null) {
Gst.Element crop = ((Gst.Bin)prepare).get_by_name(@"video_widget_$(id)_crop"); Gst.Element crop = ((Gst.Bin)prepare).get_by_name(@"video_widget_$(id)_crop");
if (crop != null) { if (crop != null) {
int output_width = allocation.width;
int output_height = allocation.height;
int target_num, target_den;
if (last_input_caps != null) {
int input_width, input_height;
last_input_caps.get_structure(0).get_int("width", out input_width);
last_input_caps.get_structure(0).get_int("height", out input_height);
double target_ratio = 3.0/2.0;
double ratio = (double)(output_width*input_height)/(double)(input_width*output_height);
if (ratio > target_ratio) {
target_num = (int)((double)input_width * target_ratio);
target_den = input_height;
sink.@set("force-aspect-ratio", true);
} else if (ratio < 1.0/target_ratio) {
target_num = input_width;
target_den = (int)((double)input_height * target_ratio);;
sink.@set("force-aspect-ratio", true);
} else {
target_num = output_width;
target_den = output_height;
sink.@set("force-aspect-ratio", false);
}
} else {
target_num = output_width;
target_den = output_height;
sink.@set("force-aspect-ratio", false);
}
Value ratio = Value(typeof(Gst.Fraction)); Value ratio = Value(typeof(Gst.Fraction));
#if VALA_0_52 #if VALA_0_52
Gst.Value.set_fraction(ref ratio, allocation.width, allocation.height); Gst.Value.set_fraction(ref ratio, target_num, target_den);
#else #else
gst_value_set_fraction(ref ratio, allocation.width, allocation.height); gst_value_set_fraction(ref ratio, target_num, target_den);
#endif #endif
crop.set_property("aspect-ratio", ratio); crop.set_property("aspect-ratio", ratio);
} }
} }
} }
public bool fix_caps_issues() {
// FIXME: Detect if draw would fail and do something better
if (last_caps != null) {
Gst.Caps? temp = last_caps.copy();
temp.set_simple("width", typeof(int), 1, "height", typeof(int), 1, null);
sink.set_caps(temp);
sink.set_caps(last_caps);
}
return false;
}
public void display_stream(Xmpp.Xep.JingleRtp.Stream stream, Xmpp.Jid jid) { public void display_stream(Xmpp.Xep.JingleRtp.Stream stream, Xmpp.Jid jid) {
if (element == null) return; if (sink == null) return;
detach(); detach();
if (stream.media != "video") return; if (stream.media != "video") return;
connected_stream = stream as Stream; connected_stream = stream as Stream;
if (connected_stream == null) return; if (connected_stream == null) return;
plugin.pause(); plugin.pause();
pipe.add(element); pipe.add(sink);
prepare = Gst.parse_bin_from_description(@"aspectratiocrop aspect-ratio=4/3 name=video_widget_$(id)_crop ! videoconvert name=video_widget_$(id)_convert", true); prepare = Gst.parse_bin_from_description(@"aspectratiocrop aspect-ratio=4/3 name=video_widget_$(id)_crop ! videoconvert name=video_widget_$(id)_convert", true);
prepare.name = @"video_widget_$(id)_prepare"; prepare.name = @"video_widget_$(id)_prepare";
prepare.get_static_pad("sink").notify["caps"].connect(input_caps_changed); prepare.get_static_pad("sink").notify["caps"].connect(input_caps_changed);
prepare.get_static_pad("src").notify["caps"].connect(processed_input_caps_changed);
pipe.add(prepare); pipe.add(prepare);
connected_stream.add_output(prepare); connected_stream.add_output(prepare);
prepare.link(element); prepare.link(sink);
element.set_locked_state(false); sink.set_locked_state(false);
plugin.unpause(); plugin.unpause();
attached = true; attached = true;
} }
public void display_device(MediaDevice media_device) { public void display_device(MediaDevice media_device) {
if (element == null) return; if (sink == null) return;
detach(); detach();
connected_device = media_device as Device; connected_device = media_device as Device;
if (connected_device == null) return; if (connected_device == null) return;
plugin.pause(); plugin.pause();
pipe.add(element); pipe.add(sink);
prepare = Gst.parse_bin_from_description(@"aspectratiocrop aspect-ratio=4/3 name=video_widget_$(id)_crop ! videoflip method=horizontal-flip name=video_widget_$(id)_flip ! videoconvert name=video_widget_$(id)_convert", true); prepare = Gst.parse_bin_from_description(@"aspectratiocrop aspect-ratio=4/3 name=video_widget_$(id)_crop ! videoflip method=horizontal-flip name=video_widget_$(id)_flip ! videoconvert name=video_widget_$(id)_convert", true);
prepare.name = @"video_widget_$(id)_prepare"; prepare.name = @"video_widget_$(id)_prepare";
prepare.get_static_pad("sink").notify["caps"].connect(input_caps_changed); prepare.get_static_pad("sink").notify["caps"].connect(input_caps_changed);
pipe.add(prepare); pipe.add(prepare);
connected_device_element = connected_device.link_source(); connected_device_element = connected_device.link_source();
connected_device_element.link(prepare); connected_device_element.link(prepare);
prepare.link(element); prepare.link(sink);
element.set_locked_state(false); sink.set_locked_state(false);
plugin.unpause(); plugin.unpause();
attached = true; attached = true;
} }
public void detach() { public void detach() {
if (element == null) return; if (sink == null) return;
if (attached) { if (attached) {
if (connected_stream != null) { if (connected_stream != null) {
connected_stream.remove_output(prepare); connected_stream.remove_output(prepare);
connected_stream = null; connected_stream = null;
} }
if (connected_device != null) { if (connected_device != null) {
connected_device_element.unlink(element); connected_device_element.unlink(sink);
connected_device_element = null; connected_device_element = null;
connected_device.unlink(); connected_device.unlink();
connected_device = null; connected_device = null;
@ -120,9 +183,9 @@ public class Dino.Plugins.Rtp.VideoWidget : Gtk.Bin, Dino.Plugins.VideoCallWidge
prepare.set_state(Gst.State.NULL); prepare.set_state(Gst.State.NULL);
pipe.remove(prepare); pipe.remove(prepare);
prepare = null; prepare = null;
element.set_locked_state(true); sink.set_locked_state(true);
element.set_state(Gst.State.NULL); sink.set_state(Gst.State.NULL);
pipe.remove(element); pipe.remove(sink);
attached = false; attached = false;
} }
} }
@ -130,6 +193,6 @@ public class Dino.Plugins.Rtp.VideoWidget : Gtk.Bin, Dino.Plugins.VideoCallWidge
public override void dispose() { public override void dispose() {
detach(); detach();
widget = null; widget = null;
element = null; sink = null;
} }
} }