Intermittent test_browserFrame1.html, test_browserFrame2.html, test_browserFrame3.html "iframeLoad is not defined"

RESOLVED FIXED in Firefox 12

Status

()

Core
DOM
RESOLVED FIXED
5 years ago
5 years ago

People

(Reporter: philor, Assigned: Justin Lebar (not reading bugmail))

Tracking

({intermittent-failure})

Trunk
mozilla12
intermittent-failure
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(firefox12 fixed)

Details

(Whiteboard: [qa-])

Attachments

(1 attachment, 1 obsolete attachment)

5.39 KB, patch
philor
: review+
Details | Diff | Splinter Review
(Reporter)

Description

5 years ago
Okay, I only have a test_browserFrame3.html in https://tbpl.mozilla.org/php/getParsedLog.php?id=8713381&tree=Mozilla-Inbound so far, but it's just a matter of time for the others. Nothing says that firing onload has to wait for all script in the document to be parsed, so <iframe onload="foo"></iframe><script>function foo(){}</script> is just asking for "foo is not defined."

Of course, the mochitest boilerplate encourages that, but that's for the same reason that the Mozilla Store only sells clownshoes, starting with size 22.
(Reporter)

Comment 1

5 years ago
Created attachment 590424 [details] [diff] [review]
fix

Moving the <script> inside the <head> where you would have put it as a matter of course if you weren't starting with boilerplate works too, but that requires way more cut and paste than this does.
Attachment #590424 - Flags: review?(justin.lebar+bug)
(Assignee)

Comment 2

5 years ago
Comment on attachment 590424 [details] [diff] [review]
fix

Silly me.  Thanks, Philor.
Attachment #590424 - Flags: review?(justin.lebar+bug) → review+
Comment hidden (Treeherder Robot)
(Reporter)

Comment 4

5 years ago
https://hg.mozilla.org/integration/mozilla-inbound/rev/ca4e2ab3717c
Target Milestone: --- → mozilla12
Comment hidden (Treeherder Robot)
(Reporter)

Comment 6

5 years ago
Okay, that was pathetic, where I starred my own brokenness from not paying attention to what I was fixing and what I would be breaking as though it was just the original problem. Backed out in https://hg.mozilla.org/integration/mozilla-inbound/rev/62b602a49394, guess I ought to actually look at what I'm doing.
Target Milestone: mozilla12 → ---
(Assignee)

Comment 7

5 years ago
Well, one problem is

  addEventListener('load', runTest());

I bet if we got rid of those parentheses, that would help a lot.
(Reporter)

Comment 8

5 years ago
Ignoring 4, which I shouldn't have touched, 1-3 fail to consider data:text/html,<body><iframe onload="alert('iframe loaded')"></iframe></body>.

The whole thing is a race, will the iframe fire its first onload from when it loads empty before iframeLoad is defined, or will it fire it after it's defined but before the window load fires and runTest happens? :)
(Reporter)

Updated

5 years ago
Summary: Intermittent test_browserFrame1.html, test_browserFrame2.html, test_browserFrame3.html, test_browserFrame4.html "iframeLoad is not defined" → Intermittent test_browserFrame1.html, test_browserFrame2.html, test_browserFrame3.html "iframeLoad is not defined"
(Reporter)

Comment 9

5 years ago
A little knowledge is a dangerous thing, and that's about how much I have, a little.
Assignee: philringnalda → nobody
(Assignee)

Comment 10

5 years ago
Okay, yeah, these tests are totally bogus.  :(
(Assignee)

Comment 11

5 years ago
Created attachment 590431 [details] [diff] [review]
Patch v1
Attachment #590431 - Flags: review?(philringnalda)
(Assignee)

Updated

5 years ago
Attachment #590424 - Attachment is obsolete: true
(Assignee)

Comment 12

5 years ago
Philor, feel free to push this.  If you don't get around to it tonight, I'll push in the morning.
(Reporter)

Comment 13

5 years ago
Comment on attachment 590431 [details] [diff] [review]
Patch v1

Pretty sure that's what we want to do, but based on my track record so far tonight, I asked the try server to back me up in https://tbpl.mozilla.org/?tree=Try&rev=53a86844e908
Attachment #590431 - Flags: review?(philringnalda) → review+
Comment hidden (Treeherder Robot)
(Assignee)

Comment 15

5 years ago
At least three of the tests didn't have this failure.  The Windows tests are still pending (??), but I'm going to push anyway.
(Assignee)

Comment 16

5 years ago
https://hg.mozilla.org/integration/mozilla-inbound/rev/8ff77543b294
Assignee: nobody → justin.lebar+bug
status-firefox12: --- → fixed
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
https://hg.mozilla.org/mozilla-central/rev/8ff77543b294
Status: NEW → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla12
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
(In reply to TinderboxPushlog Robot from comment #24)
> sgautherie
> https://tbpl.mozilla.org/php/getParsedLog.php?id=8727601&tree=Try
> Rev3 WINNT 5.1 try opt test mochitests-3/5 on 2012-01-21 15:16:20

This one happened before comment 22 fix.
Comment hidden (Treeherder Robot)
Whiteboard: [orange] → [orange][qa-]
Keywords: intermittent-failure
Whiteboard: [orange][qa-] → [qa-]
You need to log in before you can comment on or make changes to this bug.