<html> <head> <base href="https://bugzilla.gnome.org/" /> </head> <body> <div> <a class="bz_bug_link bz_status_NEW " title="NEW - Epiphany application mode on Wayland broken with WebKit 2.13.92" href="https://bugzilla.gnome.org/show_bug.cgi?id=771561#c10">Comment # 10</a> on <a class="bz_bug_link bz_status_NEW " title="NEW - Epiphany application mode on Wayland broken with WebKit 2.13.92" href="https://bugzilla.gnome.org/show_bug.cgi?id=771561">bug 771561</a> from <a href="page.cgi?id=describeuser.html&login=ofourdan%40redhat.com" title="Olivier Fourdan <ofourdan@redhat.com>"> Olivier Fourdan</a> <pre>Created <a href="attachment.cgi?id=335846" name="attach_335846" title="Sample program">attachment 335846</a> <a href="attachment.cgi?id=335846&action=edit" title="Sample program">[details]</a> Sample program (In reply to Sjoerd Simons from <a href="show_bug.cgi?id=771561#c9">comment #9</a>) > CSD != shadows. Not sure what you mean by that sentence... With CSD, the shadows are drawn by the client, i.e. they are within the toplevel GdkWindow size whereas with SSD the shadows are drawn by the window manager, ie outside of the GdkWindow. That's precisely why you need _GTK_FRAME_EXTENTS for CSD on X11. > If your claim is correct (which i don't think it is, but i could be wrong) > other code in gdkwindow-wayland is obviously wrong as > gdk_wayland_window_get_window_geometry already removed the margins for the > geometry, so calling _set_geometry_hints with that geometry will decrease > with the margins twice.. > > clamping make sense to prevent protocol errors, but from what i can tell the > problem here isn't the application given dumb values... To demonstrate why we need to remove the shadow sizes, please consider the attached sample program. What this code does is trivially setting the min height of its window to 1 pixel usign the GtkWindow API (gtk_window_set_geometry_hints()). Now, this needs to work in all 3 following cases: 1. X11 without CSD (i.e. with SSD) 2. X11 with CSD 3. Wayland Which means that the geometry value passed to X11 must match what the program sets as-is, whereas the values given to CSD must add up the header bar and the shadow values. This program works as expected in all 3 cases, ie the window *content* (ie not counting the shadows and hederbar) minimum is 1 in X11 with SSD, X11 with CSD and Wayland with CSD. Applying your patch results in the min height being too large in Wayland as seen in WAYLAND_DEBUG=1 logs: Without your patch: [3747632.054] -> <a href="mailto:zxdg_toplevel_v6@28.set_min_size">zxdg_toplevel_v6@28.set_min_size</a>(248, 37) [3747632.062] -> <a href="mailto:zxdg_toplevel_v6@28.set_max_size">zxdg_toplevel_v6@28.set_max_size</a>(248, 248) [3747632.070] -> <a href="mailto:zxdg_surface_v6@27.set_window_geometry">zxdg_surface_v6@27.set_window_geometry</a>(26, 23, 248, 239) With your patch: [3782500.453] -> <a href="mailto:zxdg_toplevel_v6@28.set_min_size">zxdg_toplevel_v6@28.set_min_size</a>(300, 89) [3782500.462] -> <a href="mailto:zxdg_toplevel_v6@28.set_max_size">zxdg_toplevel_v6@28.set_max_size</a>(300, 300) [3782500.472] -> <a href="mailto:zxdg_surface_v6@27.set_window_geometry">zxdg_surface_v6@27.set_window_geometry</a>(26, 23, 248, 248) And resizing to the min gives: [4270822.924] <a href="mailto:zxdg_toplevel_v6@28.configure">zxdg_toplevel_v6@28.configure</a>(248, 37, array) And 37 is the height of the gtkheaderbar, matching the min height (without your patch) whereas your patch gives a min height of 89.</pre> </div> <hr> You are receiving this mail because: <ul> <li>You are on the CC list for the bug.</li> </ul> </body> </html>