453969 - Intermittent test_bug382113.html | Child got load event - got false, expected true

Reporter

Description

•

16 years ago

http://tinderbox.mozilla.org/showlog.cgi?log=SeaMonkey/1220691024.1220695552.1067.gz#err0 (SeaMonkey) Win2k3 comm-central dep unit test on 2008/09/06 01:50:24 [ *** 3136 INFO Running /tests/content/base/test/test_bug382113.html... NEXT ERROR *** 3137 ERROR TEST-UNEXPECTED-FAIL | /tests/content/base/test/test_bug382113.html | Child got load event - got false, expected true *** 3138 ERROR TEST-UNEXPECTED-FAIL | /tests/content/base/test/test_bug382113.html | Object got load event - got false, expected true *** 3140 INFO Running /tests/content/base/test/test_bug383430.html... ] NB: 1st time I see this, so filing as Unconfirmed for now.

Serge Gautherie (:sgautherie)

Reporter

Comment 1

•

16 years ago

http://tinderbox.mozilla.org/showlog.cgi?log=SeaMonkey/1223155511.1223159473.29335.gz Win2k3 comm-central dep unit test on 2008/10/04 14:25:11

Blocks: 438871

Status: UNCONFIRMED → NEW

Ever confirmed: true

Summary: "ERROR TEST-UNEXPECTED-FAIL | .../test_bug382113.html | Child got load event - got false, expected true" → [SeaMonkey, Windows] "ERROR TEST-UNEXPECTED-FAIL | .../test_bug382113.html | Child got load event - got false, expected true"

Serge Gautherie (:sgautherie)

Reporter

Comment 2

•

16 years ago

http://tinderbox.mozilla.org/showlog.cgi?log=SeaMonkey/1225632626.1225637359.17878.gz Win2k3 comm-central dep unit test on 2008/11/02 05:30:26

Serge Gautherie (:sgautherie)

Reporter

Comment 3

•

16 years ago

http://tinderbox.mozilla.org/showlog.cgi?log=SeaMonkey/1227386398.1227390660.27581.gz Win2k3 comm-central dep unit test on 2008/11/22 12:39:58

Serge Gautherie (:sgautherie)

Reporter

Comment 4

•

16 years ago

Firefox 3.1 too: http://tinderbox.mozilla.org/showlog.cgi?log=Firefox3.1/1230398438.1230402573.21602.gz WINNT 5.2 mozilla-1.9.1 unit test on 2008/12/27 09:20:38

Flags: wanted1.9.1?

Summary: [SeaMonkey, Windows] "ERROR TEST-UNEXPECTED-FAIL | .../test_bug382113.html | Child got load event - got false, expected true" → [Windows] "ERROR TEST-UNEXPECTED-FAIL | .../test_bug382113.html | Child got load event - got false, expected true"

Mike Beltzner [:beltzner, not reading bugmail]

Updated

•

16 years ago

Whiteboard: [orange]

Serge Gautherie (:sgautherie)

Reporter

Comment 5

•

16 years ago

http://tinderbox.mozilla.org/showlog.cgi?log=SeaMonkey/1238877579.1238882604.26256.gz WINNT 5.2 comm-central unit test on 2009/04/04 13:39:39

David Dahl :ddahl

Comment 6

•

15 years ago

http://tinderbox.mozilla.org/showlog.cgi?log=Firefox-Unittest/1255718450.1255720922.13556.gz Linux mozilla-central test mochitests [testfailed] Started 11:40, finished 12:23

Serge Gautherie (:sgautherie)

Reporter

Updated

•

15 years ago

OS: Windows Server 2003 → All

Summary: [Windows] "ERROR TEST-UNEXPECTED-FAIL | .../test_bug382113.html | Child got load event - got false, expected true" → mochitest-plain: intermittent "test_bug382113.html | Child got load event - got false, expected true"

Paul O'Shannessy [:zpao] (not bugmail, email directly)

Comment 7

•

15 years ago

http://tinderbox.mozilla.org/showlog.cgi?log=Firefox-Unittest/1255975857.1255977532.24899.gz Linux mozilla-central test opt mochitests on 2009/10/19 11:10:57

Phil Ringnalda (:philor)

Comment 9

•

15 years ago

http://tinderbox.mozilla.org/showlog.cgi?log=Firefox-Unittest/1256655436.1256655972.1395.gz WINNT 5.2 mozilla-central test opt mochitests-1/5 on 2009/10/27 07:57:16

Shawn Wilsher :sdwilsh

Comment 10

•

15 years ago

WINNT 5.2 mozilla-central opt test mochitests-1/5 [testfailed] Started 12:33, finished 12:41 http://tinderbox.mozilla.org/showlog.cgi?log=Firefox-Unittest/1259872413.1259872841.29053.gz&fulltext=1

Bernd

Comment 11

•

15 years ago

http://tinderbox.mozilla.org/showlog.cgi?log=Firefox/1263039407.1263039954.15922.gz Linux mozilla-central opt test mochitests-1/5 on 2010/01/09 04:16:47

Benjamin Smedberg

Comment 12

•

15 years ago

http://tinderbox.mozilla.org/showlog.cgi?log=Firefox/1263302665.1263303248.3524.gz

dwitte@gmail.com

Comment 13

•

15 years ago

http://tinderbox.mozilla.org/showlog.cgi?log=Firefox3.6/1266942996.1266946441.5869.gz

Phil Ringnalda (:philor)

Comment 14

•

15 years ago

http://tinderbox.mozilla.org/showlog.cgi?log=Firefox/1267370165.1267370942.17069.gz Linux mozilla-central opt test mochitests-1/5 on 2010/02/28 07:16:05 s: moz2-linux-slave26

Phil Ringnalda (:philor)

Comment 15

•

15 years ago

http://tinderbox.mozilla.org/showlog.cgi?log=Firefox/1270136228.1270136687.17688.gz Linux mozilla-central opt test mochitests-1/5 on 2010/04/01 08:37:08 s: moz2-linux-slave28

Comment hidden (Legacy TBPL/Treeherder Robot)

philor https://tbpl.mozilla.org/php/getParsedLog.php?id=7720371&tree=Mozilla-Inbound Rev3 WINNT 5.1 mozilla-inbound opt test mochitests-1/5 on 2011-12-02 22:15:57 10523 ERROR TEST-UNEXPECTED-FAIL | /tests/content/base/test/test_bug382113.html | Child got load event - got false, expected true 10524 ERROR TEST-UNEXPECTED-FAIL | /tests/content/base/test/test_bug382113.html | Object got load event - got false, expected true 41057 ERROR TEST-UNEXPECTED-FAIL | /tests/content/base/test/test_xhr_progressevents.html | lengthComputable while running data for without length, as xml for document[0] - got true, expected false 41058 ERROR TEST-UNEXPECTED-FAIL | /tests/content/base/test/test_xhr_progressevents.html | event type while running data for without length, as xml for document[1] - got load, expected progress 41059 ERROR TEST-UNEXPECTED-FAIL | /tests/content/base/test/test_xhr_progressevents.html | response should be null while running data for without length, as xml for document[1] - got [object XMLDocument], expected null 41060 ERROR TEST-UNEXPECTED-FAIL | /tests/content/base/test/test_xhr_progressevents.html | event.loaded increased while running data for without length, as xml for document[1] 41062 ERROR TEST-UNEXPECTED-FAIL | /tests/content/base/test/test_xhr_progressevents.html | lengthComputable while running data for without length, as xml for document[1] - got true, expected false 41063 ERROR TEST-UNEXPECTED-FAIL | /tests/content/base/test/test_xhr_progressevents.html | event type while running data for without length, as xml for document[1] - got loadend, expected progress 41064 ERROR TEST-UNEXPECTED-FAIL | /tests/content/base/test/test_xhr_progressevents.html | response should be null while running data for without length, as xml for document[1] - got [object XMLDocument], expected null 41065 ERROR TEST-UNEXPECTED-FAIL | /tests/content/base/test/test_xhr_progressevents.html | event.loaded increased while running data for without length, as xml for document[1] 41067 ERROR TEST-UNEXPECTED-FAIL | /tests/content/base/test/test_xhr_progressevents.html | lengthComputable while running data for without length, as xml for document[1] - got true, expected false 41068 ERROR TEST-UNEXPECTED-FAIL | /tests/content/base/test/test_xhr_progressevents.html | Test timed out.

Comment hidden (Legacy TBPL/Treeherder Robot)

Serge Gautherie (:sgautherie)

Reporter

Updated

•

13 years ago

Flags: wanted1.9.1?

Comment hidden (Legacy TBPL/Treeherder Robot)

Nobody; OK to take it and work on it

Updated

•

12 years ago

Keywords: intermittent-failure

Comment hidden (Legacy TBPL/Treeherder Robot)

Nobody; OK to take it and work on it

Updated

•

12 years ago

Whiteboard: [orange]

Comment hidden (Legacy TBPL/Treeherder Robot)

Ed Morley [:emorley]

Updated

•

12 years ago

Summary: mochitest-plain: intermittent "test_bug382113.html | Child got load event - got false, expected true" → Intermittent test_bug382113.html | Child got load event - got false, expected true

Comment hidden (Legacy TBPL/Treeherder Robot)

Ed Morley [:emorley]

Comment 193

•

11 years ago

Closing inactive keywords:intermittent-failure bugs where the TBPLbot has previously commented and the test isn't marked as disabled; filter on orange-cleanup-201401.

Status: NEW → RESOLVED

Closed: 11 years ago

Resolution: --- → WORKSFORME

Comment hidden (Legacy TBPL/Treeherder Robot)

Wes Kocher (:KWierso) (Not reading bugmail; email directly if needed)

Updated

•

11 years ago

Status: RESOLVED → REOPENED

Resolution: WORKSFORME → ---

Comment hidden (Legacy TBPL/Treeherder Robot)

Serge Gautherie (:sgautherie)

Reporter

Comment 210

•

10 years ago

(After 6 years,) I suggest to disable this test, until someone can try and diagnose/fix it.

Status: REOPENED → NEW

Flags: needinfo?(bzbarsky)

Comment hidden (Legacy TBPL/Treeherder Robot)

Boris Zbarsky [:bzbarsky]

Assignee

Comment 213

•

10 years ago

Uh... This should so not be failing. All this test is testing is that we get onload in the child frame before we get it in the parent! Why did it start failing so much in the last week?

Flags: needinfo?(bzbarsky) → needinfo?(bugzillamozillaorg_serge_20140323)

Serge Gautherie (:sgautherie)

Reporter

Comment 214

•

10 years ago

(In reply to Boris Zbarsky [:bz] from comment #213) > Why did it start failing so much in the last week? This is indeed the currently interesting clue/question. (Though the answer might be the same as why it was failing 6 years ago...) I don't know: that would be the diagnose (todo) part. Application regression? (badly) Failing/Unclean previous test? Test harness improvement (at detecting)? ...

Flags: needinfo?(bugzillamozillaorg_serge_20140323)

Boris Zbarsky [:bzbarsky]

Assignee

Comment 215

•

10 years ago

Presumably the first of those.... can we get something like a regression range?

Serge Gautherie (:sgautherie)

Reporter

Updated

•

10 years ago

URL: http://mxr.mozilla.org/mozilla-centra...

Keywords: qawanted

Whiteboard: [qawanted: do comment 215]

Comment hidden (Legacy TBPL/Treeherder Robot)

Joshua Mitchell (Inactive)

Comment 329

•

10 years ago

clearing outdated keywords - please re-add / update if needed

Keywords: qawanted

Comment hidden (Legacy TBPL/Treeherder Robot)

Serge Gautherie (:sgautherie)

Reporter

Comment 352

•

10 years ago

(In reply to Joshua Mitchell [:Joshua_M] from comment #329) > clearing outdated keywords My 2014-08-26 request was not outdated, I think.

Keywords: qawanted

Comment hidden (Legacy TBPL/Treeherder Robot)

Ryan VanderMeulen [:RyanVM]

Comment 367

•

10 years ago

(In reply to Serge Gautherie (:sgautherie) from comment #352) > My 2014-08-26 request was not outdated, I think. Not that it didn't any good anyway. I'm going to attempt to bisect this on Try, but it'll be a PITA given that OSX 10.8 is chronically backlogged and Win8 x32 doesn't even run in production anymore.

Comment hidden (Legacy TBPL/Treeherder Robot)

Ryan VanderMeulen [:RyanVM]

Comment 377

•

10 years ago

I just ran a set of 60 OSX 10.8 debug mochitest-1 runs reverted to revision dac8b4a0bd7c, which per comment 204 should have been at the front end of the spike. The result? No failures. I'm afraid that at this point we have little to no chance of accurately pinpointing the cause of the spike via Try bisection. Boris, I'm afraid that given the frequency of this failure and the lack of activity for the past 2 months, I'm going to have little choice but to just disable the test at this point.

Flags: needinfo?(bzbarsky)

Boris Zbarsky [:bzbarsky]

Assignee

Comment 378

•

10 years ago

So I don't get this. How is it possible that this fails so often on tbpl in general but not on try? Is this failing on particular types of slaves? Something else? This test failing is REALLY REALLY BAD. Like web compat fail, breaks sites bad. The fact that we're not getting reports of that in the real world and can't reproduce this failure on try makes me very suspicious....

Flags: needinfo?(bzbarsky) → needinfo?(ryanvm)

Ryan VanderMeulen [:RyanVM]

Comment 379

•

10 years ago

Comment 377 references reverting to a changeset from August when the spike started, not the current situation in the ~200 reports in this bug since. My point is that if you're waiting on QA to pinpoint what caused this to spike, this bug is going to continue going nowhere as it has for months at this point.

Flags: needinfo?(ryanvm)

Boris Zbarsky [:bzbarsky]

Assignee

Comment 380

•

10 years ago

Oh, I guess this is only about <object>, not <iframe>. That would reduce the web compat impact significantly... John, are you able/willing to take a look here? Did something change in object loading that would cause this to happen? Something happening off an event that doesn't block onload in the parent document?

Flags: needinfo?(jschoenick)

Comment hidden (Legacy TBPL/Treeherder Robot)

Ryan VanderMeulen [:RyanVM]

Comment 395

•

10 years ago

Disabled on OSX and Windows debug. https://hg.mozilla.org/integration/mozilla-inbound/rev/b1f8b6f4541b

Whiteboard: [qawanted: do comment 215] → [test disabled on OSX and Windows debug][leave open]

Ryan VanderMeulen [:RyanVM]

Comment 396

•

10 years ago

https://hg.mozilla.org/mozilla-central/rev/b1f8b6f4541b And a bustage follow-up, because '==' and '=' aren't the same thing. Who knew. https://hg.mozilla.org/mozilla-central/rev/3694deb38dd4

Comment hidden (Legacy TBPL/Treeherder Robot)

Johnny Stenback (:jst)

Updated

•

10 years ago

Flags: needinfo?(jst)

Comment hidden (Legacy TBPL/Treeherder Robot)

Ryan VanderMeulen [:RyanVM]

Comment 632

•

10 years ago

I think it's time to admit defeat and disable the test outright.

Flags: needinfo?(bzbarsky)

Comment hidden (Legacy TBPL/Treeherder Robot)

Boris Zbarsky [:bzbarsky]

Assignee

Comment 638

•

10 years ago

This is ridiculous. I'm going to see if I can figure out what's up here.

Andrew Overholt [:overholt]

Comment 639

•

10 years ago

I'm going to clear qawanted since bz is looking into it.

Keywords: qawanted

Comment hidden (Legacy TBPL/Treeherder Robot)

Boris Zbarsky [:bzbarsky]

Assignee

Comment 643

•

10 years ago

Continuing the ridiculousness, I spent a while doing logging and retriggering and stuff to see why the onload event is not firing on the child, but it clearly _is_ firing on the child based on the logs. So then I actually _read_ the test. I am a moron. Patch coming up.

Assignee: nobody → bzbarsky

Flags: needinfo?(jst)

Flags: needinfo?(john)

Flags: needinfo?(bzbarsky)

Whiteboard: [test disabled on OSX and Windows debug][leave open] → [test disabled on OSX and Windows debug]

Boris Zbarsky [:bzbarsky]

Assignee

Comment 644

•

10 years ago

Attached patch Fix the race in test_bug382113.html so we don't set our child-onload-fired boolean to false _after_ the child onload has already fired (deleted) — Details — Splinter Review

Attachment #8574971 - Flags: review?(bugs)

Nathan Froyd [:froydnj]

Comment 645

•

10 years ago

Comment on attachment 8574971 [details] [diff] [review] Fix the race in test_bug382113.html so we don't set our child-onload-fired boolean to false _after_ the child onload has already fired Review of attachment 8574971 [details] [diff] [review]: ----------------------------------------------------------------- bz asked me over IRC whether I could review this.

Attachment #8574971 - Flags: review?(bugs) → review+

Carsten Book [:Tomcat]

Comment 646

•

10 years ago

https://hg.mozilla.org/mozilla-central/rev/88634c2aafcb

Status: NEW → RESOLVED

Closed: 11 years ago → 10 years ago

status-firefox39: --- → fixed

Resolution: --- → FIXED

Target Milestone: --- → mozilla39

Serge Gautherie (:sgautherie)

Reporter

Comment 647

•

10 years ago

Fwiw, note that treeherder comments 613-617 and 619-620 seem unrelated to this bug.

URL: http://mxr.mozilla.org/mozilla-centra... → http://mxr.mozilla.org/mozilla-centra...

Whiteboard: [test disabled on OSX and Windows debug]

Ryan VanderMeulen [:RyanVM]

Updated

•

10 years ago

status-b2g-v1.4: --- → unaffected

status-b2g-v2.0: --- → affected

status-b2g-v2.1: --- → affected

status-b2g-v2.2: --- → affected

status-b2g-master: --- → fixed

status-firefox37: --- → affected

status-firefox38: --- → affected

status-firefox-esr31: --- → affected

Ryan VanderMeulen [:RyanVM]

Comment 648

•

10 years ago

https://hg.mozilla.org/releases/mozilla-beta/rev/8fbda87f6b4d

status-firefox37: affected → fixed

Flags: in-testsuite+

Ryan VanderMeulen [:RyanVM]

Comment 649

•

10 years ago

https://hg.mozilla.org/releases/mozilla-aurora/rev/caac2d955f54

status-firefox38: affected → fixed

Ryan VanderMeulen [:RyanVM]

Comment 650

•

10 years ago

https://hg.mozilla.org/releases/mozilla-b2g37_v2_2/rev/8fbda87f6b4d https://hg.mozilla.org/releases/mozilla-b2g34_v2_1/rev/c5dd23fe6356 https://hg.mozilla.org/releases/mozilla-b2g32_v2_0/rev/78df3174fb10

status-b2g-v2.0: affected → fixed

status-b2g-v2.1: affected → fixed

status-b2g-v2.2: affected → fixed

Vincent Liu[:vliu]

Comment 651

•

10 years ago

https://hg.mozilla.org/releases/mozilla-b2g34_v2_1s/rev/c5dd23fe6356

status-b2g-v2.1S: --- → fixed

Ryan VanderMeulen [:RyanVM]

Comment 652

•

10 years ago

https://hg.mozilla.org/releases/mozilla-esr31/rev/acaa4a0e2f73

status-firefox-esr31: affected → fixed