Review Request 126491: Set root context properties on the right context
David Edmundson
david at davidedmundson.co.uk
Sun Dec 27 22:13:06 UTC 2015
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/126491/
-----------------------------------------------------------
(Updated Dec. 27, 2015, 10:13 p.m.)
Status
------
This change has been marked as submitted.
Review request for Plasma.
Changes
-------
Submitted with commit c692bc53a8c5cf5802084085b9aed11203998d38 by David Edmundson to branch Plasma/5.5.
Bugs: 355885, 356545 and 356916
https://bugs.kde.org/show_bug.cgi?id=355885
https://bugs.kde.org/show_bug.cgi?id=356545
https://bugs.kde.org/show_bug.cgi?id=356916
Repository: plasma-workspace
Description
-------
engine()->rootContext() will give a different result to
rootContext() when using a shared engine.
One being in the right context, the other being the context of the
shared engine. Using the latter means two panels end up sharing the same
object which leads to some interesting results and QML crashes in
QV4::QObjectWrapper::wrap when one gets deleted.
I've tested against 356545, but I think it's the cause of a lot of the
mystery QML bugs we couldn't figure out.
BUG: 356545
BUG: 355885
BUG: 356916
Diffs
-----
shell/containmentconfigview.cpp bf2b9287acb8188d63068b5389b3d55bfbdbf76b
shell/panelconfigview.cpp 29a50bc03d8750e2370c5c2b5cfe761a031c43e9
shell/panelview.cpp 893def0ced7297271d03dd27f40f8f6599a2354f
shell/shellcorona.cpp 17ce61b57f71332b9b4029d44301462ef90d3dfd
Diff: https://git.reviewboard.kde.org/r/126491/diff/
Testing
-------
Thanks,
David Edmundson
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20151227/b8f1830e/attachment.html>
More information about the Plasma-devel
mailing list