GM 3.0 and 3.0 beta1 ignoring @noframes

[Yonezpt]

After ~4 hours of trying to understand why my script started throwing errors in the past couple of days I finally discovered that the problem lied with the new Greasemonkey version and beta as well; The addon is ignoring the @noframes rule.

I don't know yet if this was by design or if it was overlooked and I don't see it mentioned in the changelog, but all scripts will now run in iframes despite having @noframes declared.

Sample script:

// ==UserScript==
// @name        Test
// @namespace   Test
// @version     1
// @run-at      document-start
// @match       https://www.youtube.com/*
// @grant       none
// @noframes
// ==/UserScript==
if (self === top) {
    console.info('I am not an iframe');
} else if (self !== top) {
    console.info('I am an iframe');
}

Open https://www.youtube.com/watch?v=8pRJn77UeOM, for example, and observe the console.

The 3 iframe related strings show that the script is running inside iframes.

[janekptacijarabaci]

Confirmed.

script.noframes - undefined

https://github.com/greasemonkey/greasemonkey/blob/master/modules/ipcscript.js#L16

From:

  this.namespace = aScript.namespace;

To:

  this.namespace = aScript.namespace;
  this.noframes = aScript.noframes;

And maybe add:

https://github.com/greasemonkey/greasemonkey/blob/master/modules/ipcscript.js#L73

From:

      'namespace': this.namespace,

To:

      'namespace': this.namespace,
      'noframes': this.noframes,

[arantius]

This fix is now included in version 3.1beta1. Please browse to Greasemonkey's AMO page and open "Development Channel" near the bottom to install the beta version. Can you confirm the fix? Thank you.

[Yonezpt]

I can confirm, the issue is fixed on the version you mentioned. Thanks for the quick response.