Bug 450053

Summary: Chromium(on ozone wayland) opens in maximized state and size, but does not show restore icon when using system borders
Product: [Plasma] kwin Reporter: Samuel Reddy <samuelsumukhreddy>
Component: wayland-genericAssignee: KWin default assignee <kwin-bugs-null>
Status: RESOLVED FIXED    
Severity: normal CC: behzad.a_ir, nate
Priority: NOR    
Version: 5.24.0   
Target Milestone: ---   
Platform: Other   
OS: Linux   
Latest Commit: https://invent.kde.org/plasma/kwin/commit/a43496bcc4795cbe462cd72198e62b0a8923f589 Version Fixed In: 5.24.1
Sentry Crash Report:
Attachments: Video of this issue
This is the correct attachment

Description Samuel Reddy 2022-02-11 23:39:58 UTC 
SUMMARY:
When using Chromium(on ozone wayland) with system borders, if chromium is closed maximized, then chromium is reopened, it opens in a maximized state and size, but does not show restore icon. If you click the maximize icon, it restores the window.


STEPS TO REPRODUCE
1. enable system borders in chromium
2. make chromium maximized
3. close chromium
4. reopen chromium
5. chromium in maximized state and size but kwin does not show restore icon

OBSERVED RESULT:
Chromium shows maximized icon, but when you click it, it restores the window.

EXPECTED RESULT:
Chromium should show the restore icon instead of the maximize icon.

SOFTWARE/OS VERSIONS:
Operating System: Arch Linux
KDE Plasma Version: 5.24.0
KDE Frameworks Version: 5.90.0
Qt Version: 5.15.2
Kernel Version: 5.16.8-arch1-1 (64-bit)
Graphics Platform: Wayland
Processors: 8 × AMD Ryzen 5 3450U with Radeon Vega Mobile Gfx
Memory: 5.7 GiB of RAM
Graphics Processor: AMD Radeon Vega 8 Graphics

ADDITIONAL INFORMATION:
I attached a video of the problem
Comment 1 Samuel Reddy 2022-02-11 23:42:24 UTC 
Created attachment 146609 [details]
Video of this issue
Comment 2 Samuel Reddy 2022-02-11 23:47:31 UTC 
Created attachment 146610 [details]
This is the correct attachment

The original one was the wrong one.
Comment 3 Behzad A 2022-02-12 20:09:59 UTC 
Can't reproduce with git-master and Chrome-Dev Ver 100.X
Comment 4 Behzad A 2022-02-12 20:16:37 UTC 
OK, this issue occurs only with chrome stable, then is issue is related to chromium
Comment 5 Vlad Zahorodnii 2022-02-14 10:05:13 UTC 
It's a kwin bug.
Comment 6 Bug Janitor Service 2022-02-14 10:08:15 UTC 
A possibly relevant merge request was started @ https://invent.kde.org/plasma/kwin/-/merge_requests/2010
Comment 7 Vlad Zahorodnii 2022-02-14 10:18:05 UTC 
On a bit unrelated note: chromium still seems not to work as expected when restoring from maximize mode. chromium needs to ack the configure event and repaint its windows, but it appears like it still doesn't do that sometimes. it's worth checking if this issue is already reported.
Comment 8 Vlad Zahorodnii 2022-02-14 13:56:51 UTC 
Git commit c76855520cf35f1d927d829b2528a5a7eb0091ab by Vlad Zahorodnii.
Committed on 14/02/2022 at 13:41.
Pushed by vladz into branch 'master'.

wayland: Fix maximize mode propagation to decoration

If the window is initially maximized, there won't be any current
decoration when XdgToplevelClient changes the maximize mode, we need to
use m_nextDecoration.

M  +2    -2    src/xdgshellclient.cpp

https://invent.kde.org/plasma/kwin/commit/c76855520cf35f1d927d829b2528a5a7eb0091ab
Comment 9 Vlad Zahorodnii 2022-02-14 13:59:47 UTC 
Git commit a43496bcc4795cbe462cd72198e62b0a8923f589 by Vlad Zahorodnii.
Committed on 14/02/2022 at 13:59.
Pushed by vladz into branch 'Plasma/5.24'.

wayland: Fix maximize mode propagation to decoration

If the window is initially maximized, there won't be any current
decoration when XdgToplevelClient changes the maximize mode, we need to
use m_nextDecoration.


(cherry picked from commit c76855520cf35f1d927d829b2528a5a7eb0091ab)

M  +2    -2    src/xdgshellclient.cpp

https://invent.kde.org/plasma/kwin/commit/a43496bcc4795cbe462cd72198e62b0a8923f589