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)
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•10 years ago
|
Comment 1•10 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•10 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•10 years ago
|
||
I'll try and run the investigation. Any suggestion how to get started?
Flags: needinfo?(felipc)
Comment 4•10 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•10 years ago
|
Attachment #8433386 -
Flags: review?(felipc) → review+
Assignee | ||
Updated•10 years ago
|
Keywords: checkin-needed
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.
Description
•