Closed Bug 417067 Opened 12 years ago Closed 12 years ago

[FIX]Stop treating _new as a synonym for _blank

Categories

(Core :: DOM: Navigation, defect)

defect
Not set

Tracking

()

RESOLVED FIXED

People

(Reporter: bzbarsky, Assigned: bzbarsky)

Details

Attachments

(1 file)

Attached patch Like soSplinter Review
Apparently no other UA does this, and it's not clear why we ever added it to start with.  See the thread at <http://lists.w3.org/Archives/Public/public-html/2008Feb/0048.html>.

I think we should just ditch this.
Attachment #302876 - Flags: superreview?(jst)
Attachment #302876 - Flags: review?(jst)
Comment on attachment 302876 [details] [diff] [review]
Like so

r+sr=jst.

I wouldn't be surprised if this breaks some oddball sites out there though. I wouldn't expect it to be common at all, but I wouldn't be surprised if we'll see it happen.
Attachment #302876 - Flags: superreview?(jst)
Attachment #302876 - Flags: superreview+
Attachment #302876 - Flags: review?(jst)
Attachment #302876 - Flags: review+
Comment on attachment 302876 [details] [diff] [review]
Like so

I think we should take this... it's compat with other UAs, and definitely worth it in those terms, in my opinion.
Attachment #302876 - Flags: approval1.9?
Comment on attachment 302876 [details] [diff] [review]
Like so

a=beltzner
Attachment #302876 - Flags: approval1.9? → approval1.9+
Checked in.  It'd be nice to have a mochitest here too...
Status: NEW → RESOLVED
Closed: 12 years ago
Flags: in-testsuite?
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.