From c4ab885616843c513770e1933a607acdbc6caacd Mon Sep 17 00:00:00 2001
From: David Edmundson <kde@davidedmundson.co.uk>
Date: Thu, 25 Apr 2019 00:45:06 +0100
Subject: [PATCH] Initialise XdgOutput properties on creation

Summary:
initWaylandOutput can be called before or after all properites are set.
If it's called after, we need to still have xdgOuput set correctly.

Test Plan:
Simplified Aleix's patch that avoids a no-op setScale call
Now it works.

Reviewers: #kwin, apol

Reviewed By: apol

Subscribers: kwin

Tags: #kwin

Differential Revision: https://phabricator.kde.org/D20800
---
 abstract_output.cpp | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/abstract_output.cpp b/abstract_output.cpp
index e2f1598e25..eaba345c76 100644
--- a/abstract_output.cpp
+++ b/abstract_output.cpp
@@ -171,6 +171,9 @@ void AbstractOutput::createXdgOutput()
         return;
     }
     m_xdgOutput = waylandServer()->xdgOutputManager()->createXdgOutput(m_waylandOutput, m_waylandOutput);
+    m_xdgOutput->setLogicalSize(pixelSize() / scale());
+    m_xdgOutput->setLogicalPosition(m_globalPos);
+    m_xdgOutput->done();
 }
 
 void AbstractOutput::initWaylandOutput()