https://bugs.kde.org/show_bug.cgi?id=369228
Bhushan Shah <bhus...@gmail.com> changed:
What |Removed |Added
----------------------------------------------------------------------------
Latest Commit| |http://commits.kde.org/plas
| |ma-workspace/5cfb957c87994d
| |f3cb9c8aa081ad42e67038a7d8
Resolution|--- |FIXED
Status|UNCONFIRMED |RESOLVED
--- Comment #2 from Bhushan Shah <bhus...@gmail.com> ---
Git commit 5cfb957c87994df3cb9c8aa081ad42e67038a7d8 by Bhushan Shah.
Committed on 07/10/2016 at 02:52.
Pushed by bshah into branch 'Plasma/5.8'.
[shell] Fix crash when moving panel between two monitors
Summary:
When moving panel between two screens there is time when panel is on
edge of screen and screenToFollow is null, at this point if any of the
resize or position operation gets triggered, it will attempt to get
screenToFollow's size or geometry and will result in crash as for
sometime screenToFollow is null.
This is quite a timing related bug, if you manage to move panel in time
there would be no crash otherwise there are 3-4 different crashes are
possible.
To solve this we avoid setting null values for screenToFollow, this
should not happen ideally, but setting breakpoint on setScreenToFollow
shows that null value is set from QML and should be investigated.
Test Plan:
moved panel around like crazy, can't reproduce crash on
QScreen::size or QScreen::geometry, without patch it would crash 100%
Reviewers: mart, #plasma, davidedmundson
Reviewed By: #plasma, davidedmundson
Subscribers: plasma-devel
Tags: #plasma
Differential Revision: https://phabricator.kde.org/D2970
M +4 -0 shell/panelview.cpp
http://commits.kde.org/plasma-workspace/5cfb957c87994df3cb9c8aa081ad42e67038a7d8
--
You are receiving this mail because:
You are watching all bug changes.
