Bug 6696 – REGRESSION: JavaScript window.opener set incorrectly

• Status: VERIFIED
• Resolution: FIXED
• Component: DOM

[Julien KRIVACSY]

This bug happen only in WebKit and not in Safari (417.8). When you open a new window using Javascript [window.open(...)], the new window doesn't like to interpret window.opener as the opener window but as itself. We can't send values from one window to another.

You can check this at http://www.emprint.fr/webkit/test.html

[Alexey Proskuryakov]

Confirmed; setting priority to P1 (regression).

[Joost de Valk (AlthA)]

Adding regression keyword and to summary.

[David Kilzer (:ddkilzer)]

I'm pretty sure this bug is what's causing Yahoo! Mail's "insert address" feature to stop working.

1. Log in to Yahoo! Mail using Safari+WebKit.
2. Click on "Compose" button.
3. Click on "To:" link.
4. Select an address to add (check a checkbox in the "To" column).
5. Click on "Insert Checked Contacts" button.

Expected: The address should be added to the "To:" text box.

Actual: Nothing happens.

[David Kilzer (:ddkilzer)]

Created attachment 5897 [details]
Target window contents (click on other test file)

[David Kilzer (:ddkilzer)]

Created attachment 5898 [details]
Test file (click here first)

[David Kilzer (:ddkilzer)]

Test fails: WebKit-SVN-r11976.dmg Tue Jan 10 4:35:03 GMT 2006
Test works: WebKit-CVS-2006-01-07 09:37:19 GMT.dmg

Regression was checked in between revisions r11922 and r11976, inclusive.

[David Kilzer (:ddkilzer)]

After further testing, it appears that in ToT WebKit that this is true for the pop-up window, which it shouldn't be:

window.opener == window

[Darin Adler]

This was a regression from the Frame change, bug 6450. I have a fix.

[Darin Adler]

Created attachment 5924 [details]
patch that fixes the name collision that created this problem

[Julien KRIVACSY]

Thank you :) It's ok in the new release.

[David Kilzer (:ddkilzer)]

Marking verified per Comment #10.

[Eric Seidel (no email)]

Removing Regression keyword from bugs already fixed.