Add ?mail.forward_message_mode preference? then forward = Assert

VERIFIED FIXED in M18

Status

P3
normal
VERIFIED FIXED
19 years ago
10 years ago

People

(Reporter: ppandit, Assigned: bugzilla)

Tracking

({polish})

Trunk
x86
Windows NT
polish

Firefox Tracking Flags

(Not tracked)

Details

(Reporter)

Description

19 years ago
Using a debug build from 11/29/1999 on Windows NT 4.0 SP5
Not sure what exactly to make of this.

1) Added the following to my prefs.js file:
user_pref("mail.forward_message_mode", 1); // 0 = attachment, 1 = inline

2) Then brought up /mozilla/dist/win32_D.obj/bin/mozilla
3) Went to messenger and selected a message
4) Pressed forward button and filled in the field to send it to myself
5) Got a crash - here is the trace

NTDLL! 77f7629c()
nsDebug::NotReached(const char * 0x0131cbbc, const char * 0x0131cb80, int 935)
line 301 + 13 bytes
nsGenericHTMLElement::GetAttribute(int 0, nsIAtom * 0x00ccc3d0, nsString &
{...}) line 935 + 21 bytes
nsGenericHTMLContainerElement::BeginConvertToXIF(nsXIFConverter & {...}) line
2831
nsHTMLTableCellElement::BeginConvertToXIF(const nsHTMLTableCellElement * const
0x04739a00, nsXIFConverter & {...}) line 112 + 18 bytes
nsDocument::BeginConvertToXIF(nsXIFConverter & {...}, nsIDOMNode * 0x047399f4)
line 2650
nsDocument::ToXIF(nsDocument * const 0x04738a30, nsXIFConverter & {...},
nsIDOMNode * 0x047399f4) line 2718
nsDocument::ConvertChildrenToXIF(nsXIFConverter & {...}, nsIDOMNode *
0x04739d10) line 2666 + 23 bytes
nsDocument::ToXIF(nsDocument * const 0x04738a30, nsXIFConverter & {...},
nsIDOMNode * 0x04739d10) line 2719
nsDocument::ConvertChildrenToXIF(nsXIFConverter & {...}, nsIDOMNode *
0x047385e0) line 2666 + 23 bytes
nsDocument::ToXIF(nsDocument * const 0x04738a30, nsXIFConverter & {...},
nsIDOMNode * 0x047385e0) line 2719
nsDocument::ConvertChildrenToXIF(nsXIFConverter & {...}, nsIDOMNode *
0x046adfa0) line 2666 + 23 bytes
nsDocument::ToXIF(nsDocument * const 0x04738a30, nsXIFConverter & {...},
nsIDOMNode * 0x046adfa0) line 2719
nsDocument::ConvertChildrenToXIF(nsXIFConverter & {...}, nsIDOMNode *
0x04739f50) line 2666 + 23 bytes
nsDocument::ToXIF(nsDocument * const 0x04738a30, nsXIFConverter & {...},
nsIDOMNode * 0x04739f50) line 2719
nsDocument::CreateXIF(nsDocument * const 0x04738a30, nsString & {...},
nsIDOMSelection * 0x00000000) line 2757 + 29 bytes
nsTextEncoder::EncodeToString(nsTextEncoder * const 0x0466e2f0, nsString &
{...}) line 188
nsHTMLEditor::OutputToString(nsHTMLEditor * const 0x04744940, nsString & {...},
const nsString & {...}, unsigned int 0) line 3882 + 39 bytes
nsEditorShell::GetContentsAs(nsEditorShell * const 0x047216a0, const unsigned
short * 0x0466e3e0, unsigned int 0, unsigned short * * 0x0012d364) line 2567 +
53 bytes
nsMsgComposeAndSend::GetBodyFromEditor(nsMsgComposeAndSend * const 0x04661b50)
line 1177
nsMsgComposeAndSend::Init(nsIMsgIdentity * 0x03175730, nsMsgCompFields *
0x047205b0, nsFileSpec * 0x00000000, int 0, int 0, int 0, nsIMessage *
0x00000000, const char * 0x016a33ec, const char * 0x00be9350, unsigned int 984,
const nsMsgAttachmentData * 0x00000000, const nsMsgAttachedFile * 0x00000000)
line 2303
nsMsgComposeAndSend::CreateAndSendMessage(nsMsgComposeAndSend * const
0x04661b50, nsIEditorShell * 0x047216a0, nsIMsgIdentity * 0x03175730,
nsIMsgCompFields * 0x047205b0, int 0, int 0, int 0, nsIMessage * 0x00000000,
const char * 0x016a33ec, const char * 0x00be9350, unsigned int 984, const
nsMsgAttachmentData * 0x00000000, const nsMsgAttachedFile * 0x00000000, void *
...) line 3010
nsMsgCompose::_SendMsg(int 0, nsIMsgIdentity * 0x03175730, const unsigned short
* 0x00000000) line 453 + 125 bytes
nsMsgCompose::SendMsg(nsMsgCompose * const 0x047206b0, int 0, nsIMsgIdentity *
0x03175730, const unsigned short * 0x00000000) line 531 + 20 bytes
XPTC_InvokeByIndex(nsISupports * 0x047206b0, unsigned int 7, unsigned int 3,
nsXPTCVariant * 0x0012d7a4) line 139
nsXPCWrappedNativeClass::CallWrappedMethod(JSContext * 0x040b3890,
nsXPCWrappedNative * 0x04721e50, const XPCNativeMemberDescriptor * 0x04721f14,
nsXPCWrappedNativeClass::CallMode CALL_METHOD, unsigned int 3, long *
0x02a368d0, long * 0x0012d954) line 894 + 43 bytes
WrappedNative_CallMethod(JSContext * 0x040b3890, JSObject * 0x04112300, unsigned
int 3, long * 0x02a368d0, long * 0x0012d954) line 191 + 34 bytes
js_Invoke(JSContext * 0x040b3890, unsigned int 3, unsigned int 0) line 665 + 26
bytes
js_Interpret(JSContext * 0x040b3890, long * 0x0012e1c4) line 2226 + 15 bytes
js_Invoke(JSContext * 0x040b3890, unsigned int 1, unsigned int 0) line 681 + 13
bytes
js_Interpret(JSContext * 0x040b3890, long * 0x0012e9f0) line 2226 + 15 bytes
js_Invoke(JSContext * 0x040b3890, unsigned int 0, unsigned int 0) line 681 + 13
bytes
js_Interpret(JSContext * 0x040b3890, long * 0x0012f21c) line 2226 + 15 bytes
js_Invoke(JSContext * 0x040b3890, unsigned int 1, unsigned int 2) line 681 + 13
bytes
js_InternalCall(JSContext * 0x040b3890, JSObject * 0x04133430, long 68367432,
unsigned int 1, long * 0x0012f398, long * 0x0012f340) line 758 + 15 bytes
JS_CallFunction(JSContext * 0x040b3890, JSObject * 0x04133430, JSFunction *
0x04805760, unsigned int 1, long * 0x0012f398, long * 0x0012f340) line 2720 + 32
bytes
nsJSContext::CallFunction(nsJSContext * const 0x040b3a20, void * 0x04133430,
void * 0x04805760, unsigned int 1, void * 0x0012f398, int * 0x0012f394) line 468
+ 33 bytes
nsJSEventListener::HandleEvent(nsIDOMEvent * 0x048028d4) line 133 + 51 bytes
nsEventListenerManager::HandleEventSubType(nsListenerStruct * 0x0457c170,
nsIDOMEvent * 0x048028d4, unsigned int 4) line 623 + 19 bytes
nsEventListenerManager::HandleEvent(nsIPresContext * 0x040dff10, nsEvent *
0x0012f7e8, nsIDOMEvent * * 0x0012f7b0, unsigned int 7, nsEventStatus *
0x0012fac0) line 758 + 25 bytes
nsXULElement::HandleDOMEvent(nsXULElement * const 0x0457c2a0, nsIPresContext *
0x040dff10, nsEvent * 0x0012f7e8, nsIDOMEvent * * 0x0012f7b0, unsigned int 1,
nsEventStatus * 0x0012fac0) line 2588
nsEventStateManager::CheckForAndDispatchClick(nsEventStateManager * const
0x0460c0a0, nsIPresContext * 0x040dff10, nsMouseEvent * 0x0012fbb4,
nsEventStatus * 0x0012fac0) line 1363 + 42 bytes
nsEventStateManager::PostHandleEvent(nsEventStateManager * const 0x0460c0a0,
nsIPresContext * 0x040dff10, nsGUIEvent * 0x0012fbb4, nsIFrame * 0x04661500,
nsEventStatus * 0x0012fac0, nsIView * 0x040df860) line 629 + 24 bytes
PresShell::HandleEvent(PresShell * const 0x040df444, nsIView * 0x040df860,
nsGUIEvent * 0x0012fbb4, nsEventStatus * 0x0012fac0) line 2458 + 43 bytes
nsView::HandleEvent(nsView * const 0x040df860, nsGUIEvent * 0x0012fbb4, unsigned
int 28, nsEventStatus * 0x0012fac0, int & 0) line 841
nsViewManager::DispatchEvent(nsViewManager * const 0x040dfa30, nsGUIEvent *
0x0012fbb4, nsEventStatus * 0x0012fac0) line 1725
HandleEvent(nsGUIEvent * 0x0012fbb4) line 69
nsWindow::DispatchEvent(nsWindow * const 0x040df724, nsGUIEvent * 0x0012fbb4,
nsEventStatus & nsEventStatus_eIgnore) line 438 + 10 bytes
nsWindow::DispatchWindowEvent(nsGUIEvent * 0x0012fbb4) line 459
nsWindow::DispatchMouseEvent(unsigned int 301, nsPoint * 0x00000000) line 3482 +
21 bytes
ChildWindow::DispatchMouseEvent(unsigned int 301, nsPoint * 0x00000000) line
3700
nsWindow::ProcessMessage(unsigned int 514, unsigned int 0, long 2949144, long *
0x0012fde0) line 2760 + 24 bytes
nsWindow::WindowProc(HWND__ * 0x001506aa, unsigned int 514, unsigned int 0, long
2949144) line 625 + 27 bytes
USER32! 77e71820()
js_GetLocalVariable(JSContext *, JSObject *, long, lon
(Reporter)

Updated

19 years ago
Assignee: phil → jefft
QA Contact: lchiang → ppandit
(Reporter)

Comment 1

19 years ago
Make ppandit QA Contact
Assign to jefft for initial look

Updated

19 years ago
Severity: normal → major

Updated

19 years ago
Assignee: jefft → ducarroz

Comment 2

19 years ago
This smell like send + ender problem fromt the stack trace. Reassign to ducarroz
for further investigation. Jean-Francois can you take a look on this? Thanks,

Comment 3

19 years ago
Actually, it sends okay if you kept continuing executing the program. F5 all the
way through the assertion sends and receives okay.
(Assignee)

Updated

19 years ago
Severity: major → normal
Status: NEW → ASSIGNED
Summary: Add "mail.forward_message_mode preference" then forward = crash → Add "mail.forward_message_mode preference" then forward = Assert
Target Milestone: M15
(Assignee)

Comment 4

19 years ago
Therefore, is not a crash but just an assert problem that affect only a debug build. I have updated the summary to
reflect the real situation.

Comment 5

19 years ago
seth fixed something with these prefs.
http://bugzilla.mozilla.org/show_bug.cgi?id=22791.

Comment 6

19 years ago
The forward mode is getting set to 1, and mozilla only supports
modes 0 and 2. (as attachment and inline)

Comment 7

19 years ago
Mass moving to M16 to get these off the M15 radar.  Please let me know if this
is really an M15 stopper.
Target Milestone: M15 → M16

Comment 8

19 years ago
Not M16 stopper.  Marking M17.
Target Milestone: M16 → M17

Comment 9

19 years ago
moving to M18 and nominating for beta3
Keywords: nsbeta3
Target Milestone: M17 → M18
(Assignee)

Updated

18 years ago
Keywords: polish

Comment 10

18 years ago
pratik - can you try to see what the current behavior is?
QA Contact: ppandit → pratikd

Comment 11

18 years ago
Pratik, 

When you are testing this out, could you try migrating a profile that has 
forward quoting as its default and see what happens?  Because I don't think we 
need to fix a bug that can only occur when hand editing preferences, but if 
migration can cause this then this is different.
Summary: Add "mail.forward_message_mode preference" then forward = Assert → Add ?mail.forward_message_mode preference? then forward = Assert

Comment 12

18 years ago
Tested using build: 2000-07-29-08 M18 on comm NS6 and Mozilla ver. 

I do not see the crash, either by adding user_pref("mail.forward_message_mode", 
1) directly or by migrating a profile that has forward quoted as default pref.

marking as fixed. 
Status: ASSIGNED → RESOLVED
Last Resolved: 18 years ago
Resolution: --- → FIXED

Updated

18 years ago
Status: RESOLVED → VERIFIED

Comment 13

18 years ago
Build: 2000-07-29-08 M18 on comm NS6 and Mozilla ver. 
Verified: Fixed
Product: MailNews → Core
Product: Core → MailNews Core
You need to log in before you can comment on or make changes to this bug.