Closed Bug 1019670 Opened 10 years ago Closed 10 years ago

Error in AboutHome.sendAboutHomeData: TypeError: target.messageManager is undefined" {file: "resource://app/modules/AboutHome.jsm" line: 248}

Categories

(Firefox :: General, defect)

29 Branch
defect
Not set
normal

Tracking

()

RESOLVED FIXED
Firefox 33

People

(Reporter: Yoric, Assigned: Yoric)

References

Details

Attachments

(1 file)

Attached patch Prototype fixSplinter Review
This warning seems to be ignored atm.
Attachment #8433386 - Flags: review?(gavin.sharp)
Comment on attachment 8433386 [details] [diff] [review]
Prototype fix

If target is non-null, why does it have a null messageManager?
Attachment #8433386 - Flags: review?(gavin.sharp) → review?(felipc)
So I believe the reason is that the element has been removed from the DOM tree. We still have a reference to it but after being removed it loses its frameLoader and the messageManager.

This is probably the same root cause for all the 3 bugs you've requested review. I think we need to first determine what is causing us to deal with items that are already removed from the DOM, before r+'ing these patches.

My first guess is tab closing...
I'll try and run the investigation. Any suggestion how to get started?
Flags: needinfo?(felipc)
I guess finding the minimal testcase on how this happens.. You can them add some breakpoint; to the code and use the --jsdebugger option to start the test run with the debugger attached to try to find out where the trouble is
Flags: needinfo?(felipc)
Attachment #8433386 - Flags: review?(felipc) → review+
https://hg.mozilla.org/integration/fx-team/rev/74fa63584506
Assignee: nobody → dteller
Keywords: checkin-needed
Whiteboard: [fixed-in-fx-team]
https://hg.mozilla.org/mozilla-central/rev/74fa63584506
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Whiteboard: [fixed-in-fx-team]
Target Milestone: --- → Firefox 33
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: