Please push, then I'll port all of webenginepart to qCDebug in a separate commit.

Let's get this in even if there are a few issues left, it's certainly much better than the current behaviour ;)

Indeed, forget what I said, it's perfectly valid XML.

Isn't the use of single quotes for XML attribute values a little bit ... unorthodox?

Looks much better to me.

Thanks for this work, but I can't review it, I don't know anything about openssl APIs or this code.

Please add a README.md too, with the same syntax as in the other frameworks.

Yes, I already said +1 in this review request ;-)

+1 from me.

It was already documented that the arg list could be empty when called directly via DBus, so this fix is wrong. Apps have to handle empty args list correctly.

Make the behaviour explicit in the documentation.

Looks good, just some details.

Isn't a mutex needed to protect this data structure that is apparently modified by different threads?

Damn, you're right, the third alternate isn't editable.

Preserve existing shortcuts

QKeySequence supports more, just not this data table.

What if you write me down as maintainer of this code, can this then go in?

In D6466#121152, @apol wrote:

This will be problematic as soon as your application depends on any libraries.