[kwin] [Bug 374038] kwin_x11 crach

[Martin Gräßlin]

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

Martin Gräßlin  changed:

   What|Removed |Added

 CC||jodr...@live.com

--- Comment #10 from Martin Gräßlin  ---
*** Bug 374265 has been marked as a duplicate of this bug. ***

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

[kwin] [Bug 374038] kwin_x11 crach

[Laurent Bonnaud]

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

--- Comment #9 from Laurent Bonnaud  ---
Thank you for the quick fix!

I confirm that the bug is fixed in this neon package version:

Package: kwin-x11
Version: 4:5.8.5+p16.04+git20161228.0449-0

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

[kwin] [Bug 374038] kwin_x11 crach

[Martin Gräßlin]

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

Martin Gräßlin  changed:

   What|Removed |Added

 Status|REOPENED|RESOLVED
  Latest Commit||https://commits.kde.org/kwi
   ||n/bba18a1ba812f3a147173edc1
   ||9516f0e00e7c855
 Resolution|--- |FIXED

--- Comment #8 from Martin Gräßlin  ---
Git commit bba18a1ba812f3a147173edc19516f0e00e7c855 by Martin Gräßlin.
Committed on 27/12/2016 at 16:36.
Pushed by graesslin into branch 'master'.

Fix the osd re-placement on X11

Summary:
6383f8304711719fe22371474138efc144adc750 introduced a regression hitting
an assert on X11 when an osd window is added.

This change tries to address this by adding a GeometryUpdateBlocker
around the change. The hit assertion is:

pendingGeometryUpdate() == PendingGeometryNone ||
areGeometryUpdatesBlocked()

By using a GeometryUpdateBlocker the second condition becomes true
and we no longer hit the condition.

Test Plan:
Manual tested both on X11 and Wayland, Wayland autotest
still passes

Reviewers: #kwin, #plasma

Subscribers: plasma-devel, kwin

Tags: #kwin

Differential Revision: https://phabricator.kde.org/D3808

M  +1-0abstract_client.cpp

https://commits.kde.org/kwin/bba18a1ba812f3a147173edc19516f0e00e7c855

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

[kwin] [Bug 374038] kwin_x11 crach

[Martin Gräßlin]

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

Martin Gräßlin  changed:

   What|Removed |Added

URL||https://phabricator.kde.org
   ||/D3808
  Flags||ReviewRequest+

--- Comment #7 from Martin Gräßlin  ---
Fix at: https://phabricator.kde.org/D3808

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

[kwin] [Bug 374038] kwin_x11 crach

[Martin Gräßlin]

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

Martin Gräßlin  changed:

   What|Removed |Added

 CC||l.bonn...@laposte.net

--- Comment #6 from Martin Gräßlin  ---
*** Bug 374155 has been marked as a duplicate of this bug. ***

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

[kwin] [Bug 374038] kwin_x11 crach

[Martin Gräßlin]

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

Martin Gräßlin  changed:

   What|Removed |Added

   Priority|NOR |VHI

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

[kwin] [Bug 374038] kwin_x11 crach

[Martin Gräßlin]

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

Martin Gräßlin  changed:

   What|Removed |Added

  Component|general |core

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

[kwin] [Bug 374038] kwin_x11 crach

[Martin Gräßlin]

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

Martin Gräßlin  changed:

   What|Removed |Added

 CC||nibojc...@gmail.com

--- Comment #5 from Martin Gräßlin  ---
*** Bug 374147 has been marked as a duplicate of this bug. ***

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

[kwin] [Bug 374038] kwin_x11 crach

[Martin Gräßlin]

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

Martin Gräßlin  changed:

   What|Removed |Added

  Flags||X11+
 Ever confirmed|0   |1
 Resolution|INVALID |---
 Status|RESOLVED|REOPENED

--- Comment #4 from Martin Gräßlin  ---
I admit that I didn't test on X11. Can you try moving the new connect from
AbstractClient to ShellClient? Then on X11 we don't have this, but on Wayland
it would work nevertheless.

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

[kwin] [Bug 374038] kwin_x11 crach

[Kai Uwe Broulik]

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

--- Comment #3 from Kai Uwe Broulik  ---
Reverting 6383f8304711719fe22371474138efc144adc750 fixes (works around) the
issue

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

[kwin] [Bug 374038] kwin_x11 crach

[Kai Uwe Broulik]

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

Kai Uwe Broulik  changed:

   What|Removed |Added

 CC||k...@privat.broulik.de

--- Comment #2 from Kai Uwe Broulik  ---
I just had the same crash. I changed volume and then kwin kept crashing until I
managed to kill plasmashell.

Does this have anything to do with your "update OSD when its size changes"
patches? (or was that about the kwin internal OSD?)OSD)

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

[kwin] [Bug 374038] kwin_x11 crach

[Martin Gräßlin]

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

Martin Gräßlin  changed:

   What|Removed |Added

 Resolution|--- |INVALID
 Status|UNCONFIRMED |RESOLVED

--- Comment #1 from Martin Gräßlin  ---
You are using an unreleased version in an unreleased distribution. I'm sorry,
but there is a fair amount of chance that this setup is just broken. We don't
track such issues.

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

[kwin] [Bug 374038] kwin_x11 crach

[rооt]

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

rооt  changed:

   What|Removed |Added

Version|unspecified |git master

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