GitLab

The g_memdup() function is replaced by a safer version in newer versions
of GLib.

since http RFC state that the header names should be processed case in-sensitive, broadway should not rely on the actual case. E.g. the go-language libraries tend to rewrite the header, which cause problems with e.g. Caddy

Fixes #3406

With the removal of `gdk_seat_grab` we do not need to keep the
definitions of `GdkGrabStatus` and `GdkGrabOwnership` public.

Move those definitions to become internal only.

surface->x/y (and various x,y arguments) should be in the parent
coordinates, so treat it as such. We also keep track of the root coords
as these are needed for popup positioning.

Also, drop the isTemp property server side and the weird initial
placement at (100, 100) in the daemon. We now fully control window
placement from the client instead. If this is not we want we should do
a serious design for that but until then lets do the simplest thing.

We now send very minimal diff operations.

This goes back to a very naive diff, but that reuses nodes from
previous frames using the node id. This will be a bettter base
to start from.

This allows generic transforms nodes to work.

When sending render nodes from the client to the daemon we add an id,
and whenever we're about to re-send the entire tree node we instead
send the old id. We track all the nodes for the previous frame
of the surface this way.

Having the id on the daemon side will allow us do to much better deltas.

The new type propagation for g_object_ref() is raising a compiler
warning for an assignment with a different type.

Als fixes indentation/tabs and removes some unused code.

Reusing pre-created nodes is a lot faster both in terms of
dom modifications and of transfer sizes.

This doesn't really change anything, but it is in preparation for diffing
the trees.

Instead of using the old buffer code, just use textures to define the
window contents.

-Wint-conversion is important because it checks casts from ints to
pointers.

-Wdiscarded-qualifiers is important to catch cases where we don't
strings when we should.

The previous greater than comparisons would never trigger the 2nd case.

http://www.viva64.com/en/b/0383/

Peter Wu authored Jul 23, 2015 and Matthias Clasen committed Jul 26, 2015

Call chain:
 - input_data_cb
   - broadway_server_read_all_input_nonblocking (input)
     - broadway_input_free (input)
       (now input is invalid)
     attempt to use input->active -> use-after-free

Make broadway_server_read_all_input_nonblocking return a boolean, TRUE
if the input was valid, FALSE otherwise. This allows input_data_cb to
detect whether the input was gone or not.

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

Krzesimir Nowak authored Jun 30, 2015 and Matthias Clasen committed Jul 01, 2015

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

This is required if /dev/shm is not mounted on your system, which can
happen for instance in certain container configurations.

Use the new --cert and --key parameters to broadwayd to pass paths to
cert and key files.

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

Remove checks for NULL before g_free() and g_clear_object().
Merge check for NULL, freeing of pointer and its setting
to NULL by g_clear_pointer().

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

Mostly missing declarations and unused functions.

Domenico Tortorella authored Aug 14, 2014 and Matthias Clasen committed Aug 14, 2014

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

We need to use gio/gunixsocketaddress.h and do so
only if HAVE_GIO_UNIX is defined.