rnhmjoj/qutebrowser
1
0
Fork 0
Code Issues Pull Requests Releases Wiki Activity

Greasemonkey: fix scoping function decleration.

Browse Source 
Apparently making a function like `function foo() {};` in block scope is
illegal. It should be named via assignment.

Switched to an IIFE anyway because it doesn't need a name.
This commit is contained in:
Jimmy 2018-04-25 16:51:35 +12:00
parent c7a9792b67
commit 19242b6cb2
1 changed files with 2 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
qutebrowser/javascript/greasemonkey_wrapper.js
View File

@ -197,12 +197,11 @@
// with `with` scope because then it would get assigned to the // with `with` scope because then it would get assigned to the
// proxy and we would get the same problem, so we have to make yet // proxy and we would get the same problem, so we have to make yet
// another nested scope. // another nested scope.
function qute_gm_window_scope() { // why can't this be anonymous? (function() {
let window = qute_gm_window_proxy; let window = qute_gm_window_proxy;
// ====== The actual user script source ====== // // ====== The actual user script source ====== //
{{ scriptSource }} {{ scriptSource }}
// ====== End User Script ====== // // ====== End User Script ====== //
}; })();
qute_gm_window_scope();
}; };
})(); })();