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

RESOLVED FIXED in Firefox 12

Status

()

defect
RESOLVED FIXED
7 years ago
a month ago

People

(Reporter: philor, Assigned: justin.lebar+bug)

Tracking

({intermittent-failure})

Trunk
mozilla12
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(firefox12 fixed)

Details

(Whiteboard: [qa-])

Attachments

(1 attachment, 1 obsolete attachment)

(Reporter)

Description

7 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

7 years ago
Posted patch fix (obsolete) — Splinter Review
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)
Comment on attachment 590424 [details] [diff] [review]
fix

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

Comment 6

7 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 → ---
Well, one problem is

  addEventListener('load', runTest());

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

Comment 8

7 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

7 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

7 years ago
A little knowledge is a dangerous thing, and that's about how much I have, a little.
Assignee: philringnalda → nobody
Okay, yeah, these tests are totally bogus.  :(
Posted patch Patch v1Splinter Review
Attachment #590431 - Flags: review?(philringnalda)
(Assignee)

Updated

7 years ago
Attachment #590424 - Attachment is obsolete: true
Philor, feel free to push this.  If you don't get around to it tonight, I'll push in the morning.
(Reporter)

Comment 13

7 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 (Legacy TBPL/Treeherder Robot)
At least three of the tests didn't have this failure.  The Windows tests are still pending (??), but I'm going to push anyway.
Comment hidden (Legacy TBPL/Treeherder Robot)
Comment hidden (Legacy TBPL/Treeherder Robot)
Comment hidden (Legacy TBPL/Treeherder Robot)
Comment hidden (Legacy TBPL/Treeherder Robot)
Comment hidden (Legacy TBPL/Treeherder Robot)
https://hg.mozilla.org/mozilla-central/rev/8ff77543b294
Status: NEW → RESOLVED
Last Resolved: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla12
Comment hidden (Legacy TBPL/Treeherder Robot)
Comment hidden (Legacy TBPL/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 (Legacy TBPL/Treeherder Robot)
Whiteboard: [orange] → [orange][qa-]
Whiteboard: [orange][qa-] → [qa-]
Component: DOM → DOM: Core & HTML
Product: Core → Core
You need to log in before you can comment on or make changes to this bug.