[plasmashell] [Bug 381525] Crash in QQmlBinding::getPropertyData after Plasma::DataContainer::forceImmediateUpdate

[David Edmundson]

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

David Edmundson  changed:

   What|Removed |Added

 CC||nakkezsl...@gmail.com

--- Comment #18 from David Edmundson  ---
*** Bug 381876 has been marked as a duplicate of this bug. ***

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

[plasmashell] [Bug 381525] Crash in QQmlBinding::getPropertyData after Plasma::DataContainer::forceImmediateUpdate

[David Edmundson]

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

David Edmundson  changed:

   What|Removed |Added

 CC||crist...@swapps.co

--- Comment #19 from David Edmundson  ---
*** Bug 381897 has been marked as a duplicate of this bug. ***

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

[plasmashell] [Bug 381525] Crash in QQmlBinding::getPropertyData after Plasma::DataContainer::forceImmediateUpdate

[David Edmundson]

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

David Edmundson  changed:

   What|Removed |Added

 CC||frederick...@gmail.com

--- Comment #17 from David Edmundson  ---
*** Bug 381864 has been marked as a duplicate of this bug. ***

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

[plasmashell] [Bug 381525] Crash in QQmlBinding::getPropertyData after Plasma::DataContainer::forceImmediateUpdate

[Jonathan Verner]

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

--- Comment #16 from Jonathan Verner  ---
Created attachment 106392
  --> https://bugs.kde.org/attachment.cgi?id=106392=edit
New crash information added by DrKonqi

plasmashell (5.10.3) using Qt 5.9.0

- What I was doing when the application crashed:

  Interacting with the global application menu widget

-- Backtrace (Reduced):
#6  QVector::count (this=) at
/usr/include/x86_64-linux-gnu/qt5/QtCore/qvector.h:234
#7  QQmlPropertyCache::property (index=58, this=) at
../../include/QtQml/5.9.0/QtQml/private/../../../../../src/qml/qml/qqmlpropertycache_p.h:711
#8  QQmlBinding::getPropertyData (this=this@entry=0x4b7fc30,
propertyData=propertyData@entry=0x7ffcb3781548,
valueTypeData=valueTypeData@entry=0x7ffcb3781550) at qml/qqmlbinding.cpp:520
#9  0x7fb84e646c52 in GenericBinding<10>::write (this=0x4b7fc30,
result=..., isUndefined=, flags=...) at qml/qqmlbinding.cpp:246
#10 0x7fb84e6478d0 in QQmlNonbindingBinding::doUpdate (this=0x4b7fc30,
watcher=..., flags=..., scope=...) at qml/qqmlbinding.cpp:210

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

[plasmashell] [Bug 381525] Crash in QQmlBinding::getPropertyData after Plasma::DataContainer::forceImmediateUpdate

[Jonathan Verner]

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

Jonathan Verner  changed:

   What|Removed |Added

 CC||jonathan.ver...@gmail.com

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

[plasmashell] [Bug 381525] Crash in QQmlBinding::getPropertyData after Plasma::DataContainer::forceImmediateUpdate

[Christoph Feck]

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

Christoph Feck  changed:

   What|Removed |Added

 CC||ere...@polywog.org

--- Comment #15 from Christoph Feck  ---
*** Bug 381758 has been marked as a duplicate of this bug. ***

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

[plasmashell] [Bug 381525] Crash in QQmlBinding::getPropertyData after Plasma::DataContainer::forceImmediateUpdate

[Christoph Feck]

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

--- Comment #14 from Christoph Feck  ---
*** Bug 381623 has been marked as a duplicate of this bug. ***

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

[plasmashell] [Bug 381525] Crash in QQmlBinding::getPropertyData after Plasma::DataContainer::forceImmediateUpdate

[Christoph Feck]

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

Christoph Feck  changed:

   What|Removed |Added

 CC||chris.vansteenlandt@gmail.c
   ||om

--- Comment #13 from Christoph Feck  ---
*** Bug 381598 has been marked as a duplicate of this bug. ***

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

[plasmashell] [Bug 381525] Crash in QQmlBinding::getPropertyData after Plasma::DataContainer::forceImmediateUpdate

[Christoph Feck]

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

Christoph Feck  changed:

   What|Removed |Added

 CC||jodr...@live.com

--- Comment #12 from Christoph Feck  ---
*** Bug 380845 has been marked as a duplicate of this bug. ***

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

[plasmashell] [Bug 381525] Crash in QQmlBinding::getPropertyData after Plasma::DataContainer::forceImmediateUpdate

[David Edmundson]

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

--- Comment #11 from David Edmundson  ---
Upstream bug: https://bugreports.qt.io/browse/QTBUG-61681

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

[plasmashell] [Bug 381525] Crash in QQmlBinding::getPropertyData after Plasma::DataContainer::forceImmediateUpdate

[David Edmundson]

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

--- Comment #10 from David Edmundson  ---
To answer the ineviatible question: Can we make ConfigView use a shared engine
to work round it.

The theme thing isn't really an issue. 

We can't easily change and keep compatibility. 

Changing the base class of ConfigView is an API break, and just making a new
context internally would break  existing use of ConfigView::rootContext() which
PanelConfig uses.

I don't see a simple solution for a workaround.

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

[plasmashell] [Bug 381525] Crash in QQmlBinding::getPropertyData after Plasma::DataContainer::forceImmediateUpdate

[Nikola Schnelle]

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

Nikola Schnelle  changed:

   What|Removed |Added

 CC||n.schne...@gmail.com

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

[plasmashell] [Bug 381525] Crash in QQmlBinding::getPropertyData after Plasma::DataContainer::forceImmediateUpdate

[David Edmundson]

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

--- Comment #9 from David Edmundson  ---
Confirmed, it's this change:

https://codereview.qt-project.org/#/c/192037/3/src/qml/jsruntime/qv4qobjectwrapper.cpp

remove those two and we don't crash.

Debug shows it's the same QQmlData between the two engines; which I think is
right; but it either shouldn't be setting the prop cache when one gets deleted,
or it should be re-instantiating it (changing QQmlData::get(*m_target, false to
true) where it crashes.

I'll make a reproducible simple Qt case tomorrow.

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

[plasmashell] [Bug 381525] Crash in QQmlBinding::getPropertyData after Plasma::DataContainer::forceImmediateUpdate

[Kai Uwe Broulik]

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

Kai Uwe Broulik  changed:

   What|Removed |Added

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

--- Comment #8 from Kai Uwe Broulik  ---
I think it's so the config dialog can use desktop style QtQuick Controls
whereas the rest of Plasma does not.

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

[plasmashell] [Bug 381525] Crash in QQmlBinding::getPropertyData after Plasma::DataContainer::forceImmediateUpdate

[David Edmundson]

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

--- Comment #7 from David Edmundson  ---
That matches your idea about that link:

The AppletConfig does create a new QQmlEngine, for a reason that I don't
remember.
AppletInterface (which is "Plasmoid" from QML terms) is shared in both.

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

[plasmashell] [Bug 381525] Crash in QQmlBinding::getPropertyData after Plasma::DataContainer::forceImmediateUpdate

[David Edmundson]

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

--- Comment #6 from David Edmundson  ---
Interesting notes:

I could reproduce the version of this crash in analog-clock, I put in some mad
GDB, and found it was updating:

Plasmoid.toolTipMainText: ...
The crash in network manager show connectionTooltipIconChanged, 

which is used in a binding on Plasmoid.toolTip

That's too much of a coincidence.

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

[plasmashell] [Bug 381525] Crash in QQmlBinding::getPropertyData after Plasma::DataContainer::forceImmediateUpdate

[Christoph Feck]

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

--- Comment #5 from Christoph Feck  ---
LOL, of course I mean data->propertyCache. Too many bug triaging...

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

[plasmashell] [Bug 381525] Crash in QQmlBinding::getPropertyData after Plasma::DataContainer::forceImmediateUpdate

[Christoph Feck]

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

--- Comment #4 from Christoph Feck  ---
Could be caused by commit for https://bugreports.qt.io/browse/QTBUG-57633
because it adds code to clear data->propertyCrash.

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

[plasmashell] [Bug 381525] Crash in QQmlBinding::getPropertyData after Plasma::DataContainer::forceImmediateUpdate

[David Edmundson]

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

David Edmundson  changed:

   What|Removed |Added

 CC||herb...@graeber-clan.de

--- Comment #3 from David Edmundson  ---
*** Bug 381654 has been marked as a duplicate of this bug. ***

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

[plasmashell] [Bug 381525] Crash in QQmlBinding::getPropertyData after Plasma::DataContainer::forceImmediateUpdate

[David Edmundson]

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

David Edmundson  changed:

   What|Removed |Added

 Ever confirmed|0   |1
 Status|UNCONFIRMED |CONFIRMED
Summary|plasma crashes after|Crash in
   |editing the properties of a |QQmlBinding::getPropertyDat
   |widget "Clock dials"|a  after
   ||Plasma::DataContainer::forc
   ||eImmediateUpdate

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