After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 736279 - Fix size of mutter in nested mode.
Fix size of mutter in nested mode.
Status: RESOLVED FIXED
Product: mutter
Classification: Core
Component: general
unspecified
Other All
: Normal normal
: ---
Assigned To: mutter-maint
mutter-maint
Depends on:
Blocks:
 
 
Reported: 2014-09-08 18:35 UTC by Owen Taylor
Modified: 2014-09-08 18:43 UTC
See Also:
GNOME target: ---
GNOME version: ---

Attachments
MetaBackendX11: Don't XResizeWindow behind clutter's back (1.81 KB, patch)
2014-09-08 18:35 UTC, Owen Taylor 		committed Details | Review
Moving setting of window-scaling-factor to meta_clutter_init() (2.02 KB, patch)
2014-09-08 18:35 UTC, Owen Taylor 		committed Details | Review

Description Owen Taylor 2014-09-08 18:35:21 UTC 
Here's a couple of patches that should make mutter reliably come up
at the right window size in nested mode - this was previously not working,
especially on hi-dpi systems.
Comment 1 Owen Taylor 2014-09-08 18:35:23 UTC 
Created attachment 285669 [details] [review]
MetaBackendX11: Don't XResizeWindow behind clutter's back

In the case of a nested Wayland compositor inside an X session,
Clutter is managing the toplevel window size, so don't call
XResizeWindow on it - this will confuse Clutter and get the size
and the hints out of sync on the toplevel window.
Comment 2 Owen Taylor 2014-09-08 18:35:26 UTC 
Created attachment 285670 [details] [review]
Moving setting of window-scaling-factor to meta_clutter_init()

Setting the scaling factor immediately after calling clutter_init()
avoids creating the stage at one size, then later resizing it to
a different size.
Comment 3 Jasper St. Pierre (not reading bugmail) 2014-09-08 18:36:34 UTC 
Review of attachment 285669 [details] [review]:

I'm unhappy with this, but sure.
Comment 4 Jasper St. Pierre (not reading bugmail) 2014-09-08 18:37:26 UTC 
Review of attachment 285670 [details] [review]:

Yes.
Comment 5 Owen Taylor 2014-09-08 18:43:41 UTC 
Attachment 285669 [details] pushed as 602dd7f - MetaBackendX11: Don't XResizeWindow behind clutter's back
Attachment 285670 [details] pushed as 33689ec - Moving setting of window-scaling-factor to meta_clutter_init()