1278917 - Execution of atomic javascript function gots interrupted by an onload callback eventhandler

Status: UNCONFIRMED

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

Description

[jsrosman]

Attachment: race.htm

User Agent: Mozilla/5.0 (Windows NT 6.1; WOW64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/51.0.2704.79 Safari/537.36

Steps to reproduce:

- place attached file race.htm on website and open it in browser
- you can also try this link https://jsfiddle.net/34oubfeq/ 


Actual results:

In Firefox the webpage shows: ERROR: RACE Condition!!!!!


Expected results:

2 alerts with "OK" should appear and Webpage should show "OK"

Comment 1

[Olli Pettay [:smaug][[e-mail]]]

This is a dup of the bug that we spin event loop during alert

Comment 2

[jsrosman]

It also happens when we do a sync XHR in stead of an alert (See commented code in the example)

Comment 3

[dkwakkel]

Having same issue with chrome on linux:
User Agent: Mozilla/5.0 (X11; Linux x86_64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/50.0.2661.37 Safari/537.36

Comment 4

[Andrew Overholt [:overholt]]

(In reply to Olli Pettay [:smaug] (high review load, please consider other reviewers) from comment #1)
> This is a dup of the bug that we spin event loop during alert

Bug 1173716?

Comment 5

[dkwakkel]

Sorry, wrong browser used for lookup of user agent. I had same issue with Firefox on linux (so with chrome it is working fine): Mozilla/5.0 (X11; Ubuntu; Linux x86_64; rv:45.0) Gecko/20100101 Firefox/45.0

Comment 6

[jsrosman]

Attachment: Race condition without usage of alert()

Comment 7

[jsrosman]

(In reply to Andrew Overholt [:overholt] from comment #4)
> (In reply to Olli Pettay [:smaug] (high review load, please consider other
> reviewers) from comment #1)
> > This is a dup of the bug that we spin event loop during alert
> 
> Bug 1173716?

This is not a duplicate but I think it is related. I have attached a new reproduction scenario (race_case2.htm) where we load 2 iframes while doing a synchroneous XmlHttpRequest. Same issue in FF. This can also be reproduced with window.open().

Comment 8

[Manikumar Nunna]

Hi All,

Please fix this. This is a fundamental break in the functionality.

Comment 9

[Jonathan Watt [:jwatt]]

Mass bug change to replace various 'parity' whiteboard flags with the new canonical keywords. (See bug 1443764 comment 13.)

Comment 12

[Richard Cheng]

Hello, can I try to solve this issue?

Comment 13

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

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

For more information, please visit BugBot documentation.

Comment 14

[Edgar Chen [:edgar]]

There are some details in https://bugzilla.mozilla.org/show_bug.cgi?id=1173716#c20.

Comment 16

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

[domcore-bugbash-triaged] Doing domcore random bug triage - this is still valid.

Comment 18

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

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

For more information, please visit BugBot documentation.

Comment 19

[Masayuki Nakano [:masayuki] (he/him)(JST, +0900)]

It was requested by the spam comment self.