Closed Bug 1019670 Opened 11 years ago Closed 11 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+
Assignee: nobody → dteller
Keywords: checkin-needed
Whiteboard: [fixed-in-fx-team]
Status: NEW → RESOLVED
Closed: 11 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: