Closed Bug 1095457 Opened 10 years ago Closed 10 years ago

Intermittent browser_share.js | uncaught exception - TypeError: iframe.messageManager is undefined at chrome://browser/content/browser.js:5037

Categories

(Firefox Graveyard :: SocialAPI, defect)

All
macOS
defect
Not set
normal

Tracking

(firefox34 unaffected, firefox35 unaffected, firefox36 fixed, firefox-esr31 unaffected)

RESOLVED FIXED
Firefox 36
Tracking Status
firefox34 --- unaffected
firefox35 --- unaffected
firefox36 --- fixed
firefox-esr31 --- unaffected

People

(Reporter: cbook, Assigned: mixedpuppy)

References

(Depends on 1 open bug, )

Details

(Keywords: intermittent-failure)

Attachments

(1 file, 1 obsolete file)

Rev4 MacOSX Snow Leopard 10.6 mozilla-inbound opt test mochitest-e10s-browser-chrome-1 https://treeherder.mozilla.org/ui/logviewer.html#?job_id=3685914&repo=mozilla-inbound 04:41:57 INFO - 405 INFO TEST-UNEXPECTED-FAIL | chrome://mochitests/content/browser/browser/base/content/test/social/browser_share.js | uncaught exception - TypeError: iframe.messageManager is undefined at chrome://browser/content/browser.js:5037
Component: General → SocialAPI
Flags: needinfo?(mixedpuppy)
Just noticed that locally, looking into it already.
Flags: needinfo?(mixedpuppy)
Blocks: 915547
DOMContentLoaded worked locally under tests, but isn't working all the time, load appears to work all the time.
Assignee: nobody → mixedpuppy
Attachment #8519151 - Flags: review?(mhammond)
grabbed wrong patch
Attachment #8519151 - Attachment is obsolete: true
Attachment #8519151 - Flags: review?(mhammond)
Attachment #8519153 - Flags: review?(mhammond)
Blocks: 1095723
Comment on attachment 8519153 [details] [diff] [review] fix loading frame script in share panel Review of attachment 8519153 [details] [diff] [review]: ----------------------------------------------------------------- rs=me
Attachment #8519153 - Flags: review?(mhammond) → review+
Argh - this sucks IMO. It seems possible stuff has happened before load that our message manager might want to know about. Shane, Jaws, do you think this is worth a platform bug to make the availability of the message manager guaranteed to be "early enough"?
(In reply to Mark Hammond [:markh] from comment #12) > Argh - this sucks IMO. It seems possible stuff has happened before load > that our message manager might want to know about. Shane, Jaws, do you > think this is worth a platform bug to make the availability of the message > manager guaranteed to be "early enough"? In this particular use case, it's not a problem, but I agree that it should be usable earlier, a platform bug would be good.
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 36
Depends on: 1101074
Product: Firefox → Firefox Graveyard
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: