+ /* We need to attach a painted client before attempting to set the display mode,
+ * otherwise the nested server will not have rendered anything, so won't be focused,
+ * so the host server won't apply any set configuration
+ */
I don't see why we need to attach a painted client *before* attempting to set the display mode. AIUI the session display configuration gets set with or without focus - and will be (re)applied whenever the session (re)gains focus.
I.e. does this change in the test work around bug in this branch? Or am I missing something?
+ /* We need to attach a painted client before attempting to set the display mode,
+ * otherwise the nested server will not have rendered anything, so won't be focused,
+ * so the host server won't apply any set configuration
+ */
I don't see why we need to attach a painted client *before* attempting to set the display mode. AIUI the session display configuration gets set with or without focus - and will be (re)applied whenever the session (re)gains focus.
I.e. does this change in the test work around bug in this branch? Or am I missing something?