diff options
author | Jonas Ådahl <jadahl@gmail.com> | 2018-07-23 21:36:57 +0200 |
---|---|---|
committer | Jonas Ådahl <jadahl@gmail.com> | 2018-08-21 13:58:48 +0000 |
commit | ce3278b4b7e99137cf8628d2909e686edc16afcf (patch) | |
tree | f7037c463e0d9b1f147df1b1e23db5e4b8d370e2 | |
parent | 0d134522d83ee263cdc83ea899af59fd264bafa7 (diff) | |
download | mutter-ce3278b4b7e99137cf8628d2909e686edc16afcf.tar.gz |
renderer/native: Check calculated transform when creating view
The "backends: Move MetaOutput::crtc field into private struct"
accidentally changed the view transform calculation code to assume that
"MetaCrtc::transform" corresponds to the transform of the CRTC; so is
not the case yet; one must calculate the transform from the logical
monitor, and check whether it is supported by the CRTC using
meta_monitor_manager_is_transform_handled(). This commit restores the
old behaviour that doesn't use MetaCrtc::transform when calculating the
view transform.
Fixes: https://gitlab.gnome.org/GNOME/mutter/issues/216
-rw-r--r-- | src/backends/native/meta-renderer-native.c | 9 | ||||
-rw-r--r-- | src/backends/x11/nested/meta-renderer-x11-nested.c | 8 |
2 files changed, 13 insertions, 4 deletions
diff --git a/src/backends/native/meta-renderer-native.c b/src/backends/native/meta-renderer-native.c index 59003b397..250a6b6c1 100644 --- a/src/backends/native/meta-renderer-native.c +++ b/src/backends/native/meta-renderer-native.c @@ -2731,9 +2731,14 @@ calculate_view_transform (MetaMonitorManager *monitor_manager, MetaMonitor *main_monitor; MetaOutput *main_output; MetaCrtc *crtc; + MetaMonitorTransform crtc_transform; + main_monitor = meta_logical_monitor_get_monitors (logical_monitor)->data; main_output = meta_monitor_get_main_output (main_monitor); crtc = meta_output_get_assigned_crtc (main_output); + crtc_transform = + meta_monitor_logical_to_crtc_transform (main_monitor, + logical_monitor->transform); /* * Pick any monitor and output and check; all CRTCs of a logical monitor will @@ -2742,10 +2747,10 @@ calculate_view_transform (MetaMonitorManager *monitor_manager, if (meta_monitor_manager_is_transform_handled (monitor_manager, crtc, - crtc->transform)) + crtc_transform)) return META_MONITOR_TRANSFORM_NORMAL; else - return crtc->transform; + return crtc_transform; } static MetaRendererView * diff --git a/src/backends/x11/nested/meta-renderer-x11-nested.c b/src/backends/x11/nested/meta-renderer-x11-nested.c index 8fdf46b0b..b29b9c69e 100644 --- a/src/backends/x11/nested/meta-renderer-x11-nested.c +++ b/src/backends/x11/nested/meta-renderer-x11-nested.c @@ -51,10 +51,14 @@ calculate_view_transform (MetaMonitorManager *monitor_manager, MetaMonitor *main_monitor; MetaOutput *main_output; MetaCrtc *crtc; + MetaMonitorTransform crtc_transform; main_monitor = meta_logical_monitor_get_monitors (logical_monitor)->data; main_output = meta_monitor_get_main_output (main_monitor); crtc = meta_output_get_assigned_crtc (main_output); + crtc_transform = + meta_monitor_logical_to_crtc_transform (main_monitor, + logical_monitor->transform); /* * Pick any monitor and output and check; all CRTCs of a logical monitor will * always have the same transform assigned to them. @@ -62,10 +66,10 @@ calculate_view_transform (MetaMonitorManager *monitor_manager, if (meta_monitor_manager_is_transform_handled (monitor_manager, crtc, - crtc->transform)) + crtc_transform)) return META_MONITOR_TRANSFORM_NORMAL; else - return crtc->transform; + return crtc_transform; } static MetaRendererView * |