<table><tr><td style="">broulik created this revision.<br />broulik added a reviewer: Plasma.<br />broulik set the repository for this revision to rKSCREENLOCKER KScreenLocker.<br />Restricted Application added a project: Plasma.<br />Restricted Application added a subscriber: plasma-devel.</td><a style="text-decoration: none; padding: 4px 8px; margin: 0 8px 8px; float: right; color: #464C5C; font-weight: bold; border-radius: 3px; background-color: #F7F7F9; background-image: linear-gradient(to bottom,#fff,#f1f0f1); display: inline-block; border: 1px solid rgba(71,87,120,.2);" href="https://phabricator.kde.org/D2557" rel="noreferrer">View Revision</a></tr></table><br /><div><strong>REVISION SUMMARY</strong><div><p>When setting the view geometries, the code tries to ensure the wallpaper is loaded at the correct size already by completing initialization of the view with correct arguments. This codepath is never executed, however, as the property contains the rootObject instead of the actually expected QmlObjectSharedEngine.</p>

<p>Moreover, accessing rootObject() while will force completion right away.</p>

<p>This patch fixes the flickering of the wallpaper plugin which reloads the image because its source size changed.</p></div></div><br /><div><strong>TEST PLAN</strong><div><p>Locked my screen, no more flickering one second after the lockscreen showed up.</p>

<p>Still flickers in test mode as it first sets the size the view geometry (which equals the screen) but then only does a "show" in this case and so the window is maximized and the resulting anchors.fill changes the size of the wallpaper again.</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>rKSCREENLOCKER KScreenLocker</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D2557" rel="noreferrer">https://phabricator.kde.org/D2557</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>greeter/greeterapp.cpp</div></div></div><br /><div><strong>EMAIL PREFERENCES</strong><div><a href="https://phabricator.kde.org/settings/panel/emailpreferences/" rel="noreferrer">https://phabricator.kde.org/settings/panel/emailpreferences/</a></div></div><br /><div><strong>To: </strong>broulik, Plasma<br /><strong>Cc: </strong>plasma-devel, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas<br /></div>