Don't create() windows that aren't native, upon receiving a palette change event.

Authored by dfaure on Sep 9 2019, 8:47 PM.


Test Plan

run kmail or falkon, then run kcmshell5 fonts, change font size

ASSERT: "!d->offscreenWindow->handle()" in file /d/qt/5/kde/qtdeclarative/src/quickwidgets/qquickwidget.cpp, line 1014

Apparently that QQuickWindow should not be create()d so calling winId()
unconditionally here is a bad idea, skip non-native windows.

Diff Detail

R135 Integration for Qt applications in Plasma
No Linters Available
No Unit Test Coverage
Build Status
Buildable 16289
Build 16307: arc lint + arc unit
dfaure created this revision.Sep 9 2019, 8:47 PM
Restricted Application added a project: Plasma. · View Herald TranscriptSep 9 2019, 8:47 PM
Restricted Application added a subscriber: plasma-devel. · View Herald Transcript
dfaure requested review of this revision.Sep 9 2019, 8:47 PM
davidedmundson accepted this revision.Sep 13 2019, 12:05 PM

Makes sense as we install the colour scheme when a window's surface is created.

This revision is now accepted and ready to land.Sep 13 2019, 12:05 PM
dfaure closed this revision.Sep 13 2019, 12:51 PM