Review Request: Fixes for screen hotplugging

[Anthony Bryant]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://reviewboard.kde.org/r/4451/
-----------------------------------------------------------

(Updated 2010-06-24 15:39:08.285730)


Review request for Plasma.


Summary (updated)
-------

Aaron's patch almost fixed the problem, but there's still the issue that the view is getting created on startup even if its screen does not exist.
I've added a few checks in createWaitingDesktops() to make sure a view is only created if it's for a containment with a valid screen and desktop.


Diffs (updated)
-----

  /trunk/KDE/kdebase/workspace/plasma/desktop/shell/plasmaapp.cpp 1142286 

Diff: http://reviewboard.kde.org/r/4451/diff


Testing (updated)
-------

Started plasma with and without an external screen and tried adding and removing it a few times, with and without per virtual desktop views.


Thanks,

Anthony

-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://mail.kde.org/pipermail/plasma-devel/attachments/20100624/b5a88f7c/attachment.htm