1421137 - Unexpected NS_ERROR_MALFORMED_API on document.open().

Status: RESOLVED WORKSFORME

(Core :: DOM: Core & HTML, defect, P3)

Description

[Dave Gomboc]

User Agent: Mozilla/5.0 (X11; Linux x86_64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/62.0.3202.94 Safari/537.36

Steps to reproduce:

Received error reports from browers indicating that NS_ERROR_MALFORMED_API sometimes occurs when document.open() is called from Firefox releases 56 and 57.




Actual results:

Under one or more unidentified conditions, JS code executing in Firefox 57, 56 (and very rarely, 52) throw an exception indicating that NS_ERROR_MALFORMED_API occurs at the point where document.open() is called.  Such an error occurs only a small fraction of the time.



Expected results:

It was expected that an NS_ERROR_MALFORMED_API error would never be issued as a result of calling document.open().  https://developer.mozilla.org/en-US/docs/Web/API/Document/open does not indicate that this is even possible.

Naturally, this Mozilla-specific behaviour is not observed with other browsers, ever.

However, in addition, no other browser indicates any alternative kind of error when executing the same compiled code.

Comment 1

[Kohei Yoshino [:kohei]]

Please provide a testcase or URL of the broken site.

Comment 2

[Dave Gomboc]

Within the past day or so, here are a few of the web pages for which error reports were relatively prominent:

http://beauty.pclady.com.cn/menskin/
http://beauty.pclady.com.cn/129/1299718.html
https://nowapoczta.wp.pl/d/#/mails/?label=152 (this appears to require login information that I do not have)
https://poczta.o2.pl/d/#/mails/?label=1       (same for this one)
https://poczta.o2.pl/wylogowano

Comment 3

[Hsin-Yi Tsai (she/her) [:hsinyi]]

(In reply to Dave Gomboc from comment #2)
> Within the past day or so, here are a few of the web pages for which error
> reports were relatively prominent:
> 
> http://beauty.pclady.com.cn/menskin/
> http://beauty.pclady.com.cn/129/1299718.html
> https://nowapoczta.wp.pl/d/#/mails/?label=152 (this appears to require login
> information that I do not have)
> https://poczta.o2.pl/d/#/mails/?label=1       (same for this one)
> https://poczta.o2.pl/wylogowano

Did you mean to say "NS_ERROR_MALFORMED_URI"?

Comment 4

[Andrew Overholt [:overholt]]

Assuming Hsin-Yi is correct, there are ~85 returns of such an error: https://searchfox.org/mozilla-central/search?q=return+NS_ERROR_MALFORMED_URI&path=

Boris, without other context here, do you have any thoughts about where to look further?

Comment 5

[Boris Zbarsky [:bzbarsky]]

Looks like a duplicate of bug 1368005 at first glance: calling document.open() on a srcdoc document will throw this exception right now.  Per spec it's ... not clear what it should do; if following the spec steps directly you get into a state the spec expects to be unreachable (a non-srcdoc document with an "about:srcdoc" url), which is rather undesirable.

Comment 6

[Dave Gomboc]

(In reply to Hsin-Yi Tsai [:hsinyi] from comment #3)
> (In reply to Dave Gomboc from comment #2)
> > Within the past day or so, here are a few of the web pages for which error
> > reports were relatively prominent:
> > 
> > http://beauty.pclady.com.cn/menskin/
> > http://beauty.pclady.com.cn/129/1299718.html
> > https://nowapoczta.wp.pl/d/#/mails/?label=152 (this appears to require login
> > information that I do not have)
> > https://poczta.o2.pl/d/#/mails/?label=1       (same for this one)
> > https://poczta.o2.pl/wylogowano
> 
> Did you mean to say "NS_ERROR_MALFORMED_URI"?

Yes, I did mean NS_ERROR_MALFORMED_URI.  Sorry for the delay in response (conference, then sick).

Comment 7

[Virginia (please send need info request to Brindusa Tot)]

Hi,

Since URLs of broken sites have been provided I'm removing the testcase-wanted tag.

Regards,

Comment 8

[Boris Zbarsky [:bzbarsky]]

I expect this is fixed now that bug 1368005 is fixed. Dave, can you confirm?

Comment 9

[Scott Burnham]

Attachment: same problem

Thanks from blacgengledpo1980.

Comment 10

[BugBot [:suhaib / :marco/ :calixte]]

Redirect a needinfo that is pending on an inactive user to the triage owner.
:mccr8, since the bug has recent activity, could you have a look please?

For more information, please visit BugBot documentation.

Comment 11

[Andrew McCreight [:mccr8]]

Let's just assume this was fixed 3 years ago as Boris expected.