noopener feature parsing doesn't follow the spec
Reported by
bzbar...@mit.edu,
Sep 29 2016
|
||||||
Issue description
UserAgent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10.10; rv:52.0) Gecko/20100101 Firefox/52.0
Example URL:
Steps to reproduce the problem:
1. var w = window.open("", "", "noopener=1");
2. alert(w);
What is the expected behavior?
Alerts "[object Window]", per spec.
What went wrong?
Alerts undefined.
Does it occur on multiple sites: Yes
Is it a problem with a plugin? No
Did this work before? No
Does this work in other browsers? N/A
Chrome version: 55.0.2873.4 (Official Build) dev (64-bit) Channel: n/a
OS Version: OS X 10.10
Flash Version: Shockwave Flash 23.0 r0
,
Sep 30 2016
Yeah. You're right. We're using the generic parsing code here... I wonder if we should redefine it to match https://drafts.csswg.org/cssom-view/#the-features-argument-to-the-open()-method?
,
Sep 30 2016
Heh. I had to go out of my way in Gecko to implement what the spec says, fwiw... If that's not what you meant, that's slightly annoying. :( The rules at https://drafts.csswg.org/cssom-view/#the-features-argument-to-the-open()-method don't seem to match either the Gecko or the Blink implementation of normal feature parsing (which also don't match each other, fwiw). That's because they only spec it for the width/height/left/top feature names, which are int-valued, not bool-valued.... This one is clearly bool-valued, kinda (noopener=0 is treated as "true" in Blink, which is not how other bool-valued things work).
,
Sep 30 2016
,
Oct 20 2016
Actually, there is another difference between what the spec says and what Blink does, and a somewhat more serious one. Consider this:
var w = window.open("", "", "NOOPENER");
alert(w);
per spec this should alert "[object Window]", because the spec says that the specific token that triggers noopener behavior is "noopener". But Blink seems to be doing some sort of case-insensitive something....
,
Apr 10 2017
Not bindings-related. This is related to how we implement https://html.spec.whatwg.org/#apis-for-creating-and-navigating-browsing-contexts-by-name
,
Apr 13 2017
,
Apr 16 2018
This issue has been Available for over a year. If it's no longer important or seems unlikely to be fixed, please consider closing it out. If it is important, please re-triage the issue. Sorry for the inconvenience if the bug really should have been left as Available. For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
,
Apr 23 2018
|
||||||
►
Sign in to add a comment |
||||||
Comment 1 by rnimmagadda@chromium.org
, Sep 30 2016Labels: M-55 OS-Linux OS-Windows
Status: Untriaged (was: Unconfirmed)
16.8 MB
16.8 MB Download