Fine with me.
If https://repology.org/project/telepathy-qt/versions is correct Fedora and OpenSUSE still don't have 0.9.8 though.

218a5229348f7d131da7e91f25894231469654f6 did the same.

Remove accidentally introduced empty line

In D25098#557440, @aacid wrote:

Have you checked introducing DontSavePainterState doesn't cause any graphical issues? (your test plan doesn't say so)

In D25095#557350, @pino wrote:

Strictly speaking, we just need the includes from KWindowSystem, as the library is not actually used (and when linking with as-needed KWindowSystem will not be required).

Not sure if that makes it worth to depend on KWindowSystem, or if it's maybe better to copy the header over.

Oh, I was looking for something else but I didn't found anything. Thanks!
Included fixx11.h

Nice! I don't want to discourage you, but while QtScript is still there it has been deprecated. Perhaps this still makes sense as a intermediate step, but at some point this should be ported to the QJS* classes from Qt5Qml (e.g. like done for kio https://phabricator.kde.org/R241:4ad1902278f0fe20db3e10db40e922f015943964).

In D24413#542042, @wbauer wrote:

In D24413#542035, @ognarb wrote:

Since the method is only one line long, why not using lambda instead here?

Because I couldn't get it to compile successfully, I got this error:
In lambda function passing 'const EngineController' as 'this' argument discards qualifiers

Do you have commit access or should I push this for you?

I don't think so and Frameworks also need 3.5 since quite some time.

Used the ReadWrite flag to simplify