Bug 397471

Summary: Greasemonkey stoped working
Product: [Applications] Falkon Reporter: The_assassin <private2.6.3>
Component: extensionsAssignee: David Rosca <nowrep>
Status: RESOLVED NOT A BUG    
Severity: normal CC: private2.6.3
Priority: NOR    
Version First Reported In: unspecified   
Target Milestone: ---   
Platform: Other   
OS: Linux   
Latest Commit: Version Fixed/Implemented In:
Sentry Crash Report:

Description The_assassin 2018-08-15 08:12:06 UTC
User scripts are no longer working to me with QtWebEngine 5.11.1. Please somebody confirm or reject that!

NOTE: QtWebEngine 5.11 only, everything is fine with lower versions.
Comment 1 The_assassin 2018-08-16 10:00:30 UTC
Ooops, forgot that GreaseMonkey is a plugin...., moving there.....

In addition, after fixing that (if you plan), adding support of '@resource' metadata value would be nice.
Comment 2 David Rosca 2018-08-16 11:21:23 UTC
Works fine here, can you please test with some simple userscript that just calls alert() on all pages?
Also is there anything in JS console and/or terminal output?
Comment 3 The_assassin 2018-08-16 12:25:56 UTC
I did "play" a lot with this before i post the issue, so i already did that. Just there is no "connection" to the script (on window or console). I tried with the system Qt as well as with precompiled Qt from the installer, so I thought the problem is in Qt version, but curently I'm confused....
I also tested the embeded "speed-dial" script, and it returns log, so it works fine.
I only did no try the newest Falkon source compiled with older Qt to see if it will works.
All other plugins are working....
Comment 4 The_assassin 2018-08-16 13:03:24 UTC
Well, I did catch it!
The "@match" metadata entry couse the issue. Please take a look, or just close the issue, thanks!
Comment 5 David Rosca 2018-08-16 13:43:38 UTC
Can you please elaborate?
Comment 6 The_assassin 2018-08-16 14:43:55 UTC
I have some scripts for a long time who are including both entries: 
// @include       *.example.org/*
// @match       *example.com/*
and they are was working so far. I don't know why i have both entries, it probably is wrong.
So for the tests, i just used one of them as a template and just changed it to:
// @include       *
// @match       *
and the scripts continued not working until i deleted the @match entry (this still works on Qt 5.10).
However, looks like having both entries is wrong.
Sorry for the inconvenience!