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

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

Details

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

Repository
R135 Integration for Qt applications in Plasma
Branch
Plasma/5.16
Lint
No Linters Available
Unit
No Unit Test Coverage
Build Status
Buildable 16289
Build 16307: arc lint + arc unit
dfaure created this revision.Mon, Sep 9, 8:47 PM
Restricted Application added a project: Plasma. · View Herald TranscriptMon, Sep 9, 8:47 PM
Restricted Application added a subscriber: plasma-devel. · View Herald Transcript
dfaure requested review of this revision.Mon, Sep 9, 8:47 PM
davidedmundson accepted this revision.Fri, Sep 13, 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.Fri, Sep 13, 12:05 PM
dfaure closed this revision.Fri, Sep 13, 12:51 PM