In bug 1059001, openListener has changed. See for example https://reviewboard.mozilla.org/r/1113/diff/2/#4 The Thunderbird actor needs to be adapted.
The fix ports the the approach from bug 1059001 to Thunderbird.
Assignee: nobody → makemyday
Status: NEW → ASSIGNED
Attachment #8539837 - Flags: review?(mkmelin+mozilla)
There's another occurrence of DebuggerServer.openListener in CC - see http://mxr.mozilla.org/comm-central/source//suite/common/src/nsSuiteGlue.js#981 I don't know if this is an issue for Seamonkey - if so, they probably haven't noticed this, because they're swallowing any error for this.
The Seamonkey folks probably wouldn't mind if you fixed it for them :-P Up to you though if you have time.
I don't mind to do so - who would be the one to request the review from?
I'd go with the same reviewer(s) as in bug 973530
Updated patch for Thunderbird with removed whitespaces.
Patch for Seamonkey.
Attachment #8540070 - Flags: review?(neil)
Comment on attachment 8540070 [details] [diff] [review] FixRemoteDebuggerServer-SM-v1.diff Thanks for the patch!
Attachment #8540070 - Flags: review?(neil) → review+
Comment on attachment 8540066 [details] [diff] [review] FixRemoteDebuggerServer-TB-v2.diff Review of attachment 8540066 [details] [diff] [review]: ----------------------------------------------------------------- Looks good, thx! r=mkmelin
Attachment #8540066 - Flags: review?(mkmelin+mozilla) → review+
Status: ASSIGNED → RESOLVED
Closed: 5 years ago
Hardware: x86 → All
Resolution: --- → FIXED
Target Milestone: --- → Thunderbird 37.0
Confirm this now works. I had tried 2014-12-26 build which failed.
Status: RESOLVED → VERIFIED
Summary: Adapt Debugger Server startup code for changes in bug 1059001 → Adapt Debugger Server startup code for changes in bug 1059001. error DebuggerServer.openListener is not a function
You need to log in before you can comment on or make changes to this bug.