GitLab

This closes bug #592363.

This reverts commit bffa9237.

This reverts commit b9cc53b4.

Add another instance of DisplayInfo for the new window which will show the
extended video stream, and its getters and setters in the class hierarchy.

This patch is traversal to all the videooutput subsystem.

It adds a new parameter to the size_changed() signal. This new parameter
indicates the video output mode of the current frame that changed its size.

With this new parameter the UI would know which window resize in case the
output is other window.

call-window.cpp: remove direct use of libnotify by asking the plugin (if it's there, and if it knows)

That makes ekiga fully runtime-independent of libnotify's presence.

Added some properties-api to the Ekiga::Service base class, with reference implementation answering nothing is available

Given this results:

$ git blame --line-porcelain lib/engine/components/common-videooutput/videooutput-manager-common.cpp | sed -n 's/^author //p' | sort | uniq -c | sort -rn
    281 Matthias Schneider
    118 Víctor Manuel Jáquez Leal
     21 Julien Puydt
      5 Damien Sandras

$ git blame --line-porcelain lib/engine/components/x-videooutput/videooutput-manager-x.cpp | sed -n 's/^author //p' | sort | uniq -c | sort -rn
    270 Matthias Schneider
    254 Víctor Manuel Jáquez Leal
     39 Julien Puydt
      1 Damien Sandras

$ git blame --line-porcelain lib/engine/components/x-videooutput/videooutput-manager-x.h | sed -n 's/^author //p' | sort | uniq -c | sort -rn
     93 Matthias Schneider
     20 Víctor Manuel Jáquez Leal
     15 Julien Puydt
      1 Damien Sandras

And by request of my client, I propose to add my client the copyright
assignation, and myself in the authoring of these files.

https://bugzilla.gnome.org/show_bug.cgi?id=683456

With this patch the user can choose to display the "extended video" through
the view menu.

The menu item is enabled only when the on_videoutput_device_opened_cb()
callback has the extended stream (ext_stream) as true.

When the opened video output device doesn't contain an extended stream, and
the view is set on that stream, the configuration is reset to the local view.

Also, the validation of the values for the key
/apps/ekiga/general/user_interface/video_display/video_view is more generic,
using enum symbols rather than numbers.

This patch contains the logic when the extended remote video stream changes
its size, hence the UI window should be adjusted.

Also contains the case that if the stream adds or removes the extended stream,
the UI setup should be readjusted too.

Finally this patch handles the frame display of the extended remote stream if
the UI requests it.

No UI changes yet though.

This patch is traversal to all the videooutput subsystem.

It adds a new parameter to the on_device_opened() signal. This new parameter
indicates if there is an extended stream in the connection.

With it, the UI can enable the option to display the extended stream in the
video widget.

This patch adds a boolean variable (ext_stream_active). It tells if there is
an extended remote video stream in the connection. It is stored in the frame
info (FrameInfo) structure.

FrameInfo is the structure that is going to be used for each frame that
arrives. So it will tell to the user interface if connection contains an
extended video.

The patch specifies the value of this new attribute for current_frame and
last_frame instances of FrameInfo.

This patch adds the X resources required to display the extended stream in
case the user selects that view.

The resources are a XWindow and a Display instances.

The patch contains the logic to handle these two new instances: construction,
destruction, and also reception of the frame from the byte array and the
display synchronization.

Since there is no a way in the user interface to enable the extended stream
view, this patch doesn't modify the user experience.

TODO: add this bits to dx-videooutput too. Since I don't have any experience,
nor access to DirectX, I prefer to avoid its modification.

Make more readable the method using a local boolean variable that
represents no synchronization demand by any video stream.

This patch adds the extended remote video stream enum in the video output
mode.

Also it adds the switch labels where the enum is handled in order to avoid
compiler's complains, but no logic is added in those cases, in other words it
is just the skeleton. In the following commits that logic will be added.