[kwin] [Bug 448064] KWin crashes in KWin::Toplevel::isOnOutput() when (un)plugging HDMI while plasmashell was restarting

[Nate Graham]

https://bugs.kde.org/show_bug.cgi?id=448064

Nate Graham  changed:

   What|Removed |Added

 CC||vychodil.hy...@gmail.com

--- Comment #21 from Nate Graham  ---
*** Bug 455399 has been marked as a duplicate of this bug. ***

-- 
You are receiving this mail because:
You are watching all bug changes.

[kwin] [Bug 448064] KWin crashes in KWin::Toplevel::isOnOutput() when (un)plugging HDMI while plasmashell was restarting

[Nate Graham]

https://bugs.kde.org/show_bug.cgi?id=448064

Nate Graham  changed:

   What|Removed |Added

 CC||roamingan...@gmail.com

--- Comment #20 from Nate Graham  ---
*** Bug 454681 has been marked as a duplicate of this bug. ***

-- 
You are receiving this mail because:
You are watching all bug changes.

[kwin] [Bug 448064] KWin crashes in KWin::Toplevel::isOnOutput() when (un)plugging HDMI while plasmashell was restarting

[jackun]

https://bugs.kde.org/show_bug.cgi?id=448064

jackun  changed:

   What|Removed |Added

 CC|jack...@gmail.com   |

-- 
You are receiving this mail because:
You are watching all bug changes.

[kwin] [Bug 448064] KWin crashes in KWin::Toplevel::isOnOutput() when (un)plugging HDMI while plasmashell was restarting

[Nate Graham]

https://bugs.kde.org/show_bug.cgi?id=448064

Nate Graham  changed:

   What|Removed |Added

   Version Fixed In||5.24.6

-- 
You are receiving this mail because:
You are watching all bug changes.

[kwin] [Bug 448064] KWin crashes in KWin::Toplevel::isOnOutput() when (un)plugging HDMI while plasmashell was restarting

[Vlad Zahorodnii]

https://bugs.kde.org/show_bug.cgi?id=448064

Vlad Zahorodnii  changed:

   What|Removed |Added

  Latest Commit|https://invent.kde.org/plas |https://invent.kde.org/plas
   |ma/kwin/commit/4c3195270d6c |ma/kwin/commit/a8477c1cf7ac
   |8e1da8c3e2e3abe5aae75d5bf3c |bf3358c85e53b236150dd43b464
   |2   |0

--- Comment #19 from Vlad Zahorodnii  ---
Git commit a8477c1cf7acbf3358c85e53b236150dd43b4640 by Vlad Zahorodnii, on
behalf of Xaver Hugl.
Committed on 21/05/2022 at 11:08.
Pushed by merge-service into branch 'Plasma/5.24'.

toplevel: set valid output in the constructor

This makes it less easy to cause crashes and fixes some segfaults.
Related: bug 452433

(cherry picked from commit e48a5c0535f01dc380449ba8481c869ff23e5558)

Tested-by: Merge Service

Part-of: 

M  +2-1src/toplevel.cpp

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

-- 
You are receiving this mail because:
You are watching all bug changes.

[kwin] [Bug 448064] KWin crashes in KWin::Toplevel::isOnOutput() when (un)plugging HDMI while plasmashell was restarting

[Vlad Zahorodnii]

https://bugs.kde.org/show_bug.cgi?id=448064

Vlad Zahorodnii  changed:

   What|Removed |Added

  Latest Commit||https://invent.kde.org/plas
   ||ma/kwin/commit/4c3195270d6c
   ||8e1da8c3e2e3abe5aae75d5bf3c
   ||2
 Resolution|--- |FIXED
 Status|ASSIGNED|RESOLVED

--- Comment #18 from Vlad Zahorodnii  ---
Git commit 4c3195270d6c8e1da8c3e2e3abe5aae75d5bf3c2 by Vlad Zahorodnii.
Committed on 21/05/2022 at 11:03.
Pushed by merge-service into branch 'Plasma/5.24'.

Ensure that Toplevel::output() stays always in sync with geometry

Currently, if geometry updates are blocked, the Toplevel.output property
won't be updated. On the other hand, it's reasonable to use the output
property instead of manually looking up the output in window management
code, e.g. Workspace::clientArea().

In other words, using the Toplevel.output property is like walking on a
mine field, things can blow up. You can't use Toplevel.output even if it
makes perfect sense.

This change ensures that Toplevel.output property is always kept in sync
with the frame geometry. Unfortunately, this means that the output
property no longer can be updated when the frameGeometryChanged() signal
is emitted. It has to be done in moveResizeInternal() method.

(cherry picked from 510a41eeb89f51843405fa0258c852ab06d05bb8)

Part-of: 

M  +0-3src/abstract_client.cpp
M  +1-0src/events.cpp
M  +6-0src/internal_client.cpp
M  +0-17   src/toplevel.cpp
M  +2-8src/toplevel.h
M  +6-0src/unmanaged.cpp
M  +1-0src/unmanaged.h
M  +6-0src/waylandclient.cpp
M  +7-0src/x11client.cpp
M  +1-0src/x11client.h

https://invent.kde.org/plasma/kwin/commit/4c3195270d6c8e1da8c3e2e3abe5aae75d5bf3c2

-- 
You are receiving this mail because:
You are watching all bug changes.

[kwin] [Bug 448064] KWin crashes in KWin::Toplevel::isOnOutput() when (un)plugging HDMI while plasmashell was restarting

[Bug Janitor Service]

https://bugs.kde.org/show_bug.cgi?id=448064

Bug Janitor Service  changed:

   What|Removed |Added

 Status|CONFIRMED   |ASSIGNED

--- Comment #17 from Bug Janitor Service  ---
A possibly relevant merge request was started @
https://invent.kde.org/plasma/kwin/-/merge_requests/2448

-- 
You are receiving this mail because:
You are watching all bug changes.

[kwin] [Bug 448064] KWin crashes in KWin::Toplevel::isOnOutput() when (un)plugging HDMI while plasmashell was restarting

[bugzilla_noreply]

https://bugs.kde.org/show_bug.cgi?id=448064

kde@grau.net changed:

   What|Removed |Added

Summary|KWin crashes|KWin crashes in
   |inKWin::Toplevel::isOnOutpu |KWin::Toplevel::isOnOutput(
   |t() when (un)plugging HDMI  |) when (un)plugging HDMI
   |while plasmashell was   |while plasmashell was
   |restarting  |restarting
 CC||kde@grau.net

-- 
You are receiving this mail because:
You are watching all bug changes.