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)
Tracking
()
RESOLVED
FIXED
Firefox 33
People
(Reporter: Yoric, Assigned: Yoric)
References
Details
Attachments
(1 file)
1.04 KB,
patch
|
Felipe
:
review+
|
Details | Diff | Splinter Review |
This warning seems to be ignored atm.
Attachment #8433386 -
Flags: review?(gavin.sharp)
Assignee | ||
Updated•11 years ago
|
Comment 1•11 years ago
|
||
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)
Comment 2•11 years ago
|
||
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...
Assignee | ||
Comment 3•11 years ago
|
||
I'll try and run the investigation. Any suggestion how to get started?
Flags: needinfo?(felipc)
Comment 4•11 years ago
|
||
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)
Updated•11 years ago
|
Attachment #8433386 -
Flags: review?(felipc) → review+
Assignee | ||
Updated•11 years ago
|
Keywords: checkin-needed
Comment 5•11 years ago
|
||
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.
Description
•