Closed
Bug 741266
Opened 12 years ago
Closed 12 years ago
document.open with 3 or more arguments should call OpenJS, not Open
Categories
(Core :: DOM: Core & HTML, defect)
Tracking
()
RESOLVED
FIXED
mozilla14
People
(Reporter: bzbarsky, Assigned: bzbarsky)
Details
Attachments
(2 files)
248 bytes,
text/html
|
Details | |
3.70 KB,
patch
|
jst
:
review+
|
Details | Diff | Splinter Review |
Right now it seems to call Open(), which makes the behavior wrong when opening a sized window. Try the attached testcase. Changing to OpenJS here makes things work right. Since we really only support document.open() from script, that seems like a reasonable change to me.
Assignee | ||
Comment 1•12 years ago
|
||
Attachment #611339 -
Flags: review?(jst)
Assignee | ||
Updated•12 years ago
|
Assignee: nobody → bzbarsky
Whiteboard: [need review]
Updated•12 years ago
|
Attachment #611339 -
Flags: review?(jst) → review+
Comment 2•12 years ago
|
||
Comment on attachment 611339 [details] [diff] [review] document.open with 3 or more arguments should invoke the scriptable version of window.open, not the noscript one. Review of attachment 611339 [details] [diff] [review]: ----------------------------------------------------------------- ::: content/html/document/test/test_bug741266.html @@ +20,5 @@ > + > +/** Test for Bug 741266 **/ > +var w = window.open("", "", "width=100,height=100"); > +is(w.innerHeight, 100, "Popup height should be 100 when opened with window.open"); > +is(w.innerWidth, 100, "Popup widthshould be 100 when opened with window.open"); Nit: space in the message (twice)
Assignee | ||
Comment 3•12 years ago
|
||
https://hg.mozilla.org/integration/mozilla-inbound/rev/d83c1ffb7397 with that nit fixed.
Flags: in-testsuite+
Whiteboard: [need review]
Target Milestone: --- → mozilla14
Comment 4•12 years ago
|
||
backed out cause the newly added test fails on Windows https://hg.mozilla.org/integration/mozilla-inbound/rev/3be23dcd43e4 87069 ERROR TEST-UNEXPECTED-FAIL | /tests/content/html/document/test/test_bug741266.html | Popup width should be 100 when opened with window.open - got 114, expected 100 87071 ERROR TEST-UNEXPECTED-FAIL | /tests/content/html/document/test/test_bug741266.html | Popup width should be 100 when opened with document.open - got 114, expected 100
Assignee | ||
Comment 5•12 years ago
|
||
Bah. That's totally broken... I'll rework the test a bit.
Assignee | ||
Comment 6•12 years ago
|
||
With the fixed test: https://hg.mozilla.org/integration/mozilla-inbound/rev/a7ae9d421848
Comment 7•12 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/a7ae9d421848
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Updated•5 years ago
|
Component: DOM → DOM: Core & HTML
You need to log in
before you can comment on or make changes to this bug.
Description
•