149134 - AsyncOpen and Suspend/Resume failures for "mailto:" protocol handling

Status: RESOLVED FIXED

(MailNews Core :: Networking: SMTP, defect)

Description

[David Epstein]

ran in testEmbed, compiled from /embedding/qa/testEmbed. "TestYourMethod2" from
Tools menu.
1. Use "mailto:[e-mail]" for the uri test.
2. rv = NS_NewURI(getter_AddRefs(theURI), uri). rv succeeded.
3. rv = NS_NewChannel(getter_AddRefs(theChannel), theURI, nsnull, theLoadGroup).
rv  succeeded.
4. set up stream listener.
5. rv = theChannel->AsyncOpen(listener, nsnull);
Results: AsyncOpen() rv failed. Get Assert "Suspend: 'Not Reached', file
/mozilla/mailnews/compose/src/nsSmtpService.cpp line 488. After QI'ing on the
channel to get request object, nsIRequest->Suspend() and Resume() rv failed.

call stack, exception break point reached:

nsDebug::Assertion(const char * 0x037b4df8, const char * 0x1012b688, const char
* 0x037b4db8, int 488) line 278 + 13 bytes
nsDebug::NotReached(const char * 0x037b4df8, const char * 0x037b4db8, int 488)
line 451 + 22 bytes
nsMailtoChannel::Suspend(nsMailtoChannel * const 0x030f3240) line 488 + 21 bytes
CNsIRequest::SuspendReqTest(nsIRequest * 0x030f3240) line 286 + 12 bytes
CTests::OnToolsTestYourMethod2() line 688 + 14 bytes
_AfxDispatchCmdMsg(CCmdTarget * 0x00f382d0 {CTests hWnd=0x00000000}, unsigned
int 32802, int 0, void (void)* 0x00402234 CTests::OnToolsTestYourMethod2(void),
void * 0x00000000, unsigned int 12, AFX_CMDHANDLERINFO * 0x00000000) line 88
CCmdTarget::OnCmdMsg(unsigned int 32802, int 0, void * 0x00000000,
AFX_CMDHANDLERINFO * 0x00000000) line 302 + 39 bytes
CTests::OnCmdMsg(unsigned int 32802, int 0, void * 0x00000000,
AFX_CMDHANDLERINFO * 0x00000000) line 724
CBrowserView::OnCmdMsg(unsigned int 32802, int 0, void * 0x00000000,
AFX_CMDHANDLERINFO * 0x00000000) line 1062 + 42 bytes
CBrowserFrame::OnCmdMsg(unsigned int 32802, int 0, void * 0x00000000,
AFX_CMDHANDLERINFO * 0x00000000) line 280 + 37 bytes
CWnd::OnCommand(unsigned int 32802, long 0) line 2088
CFrameWnd::OnCommand(unsigned int 32802, long 0) line 321
CWnd::OnWndMsg(unsigned int 273, unsigned int 32802, long 0, long * 0x0012fd98)
line 1597 + 28 bytes
CWnd::WindowProc(unsigned int 273, unsigned int 32802, long 0) line 1585 + 30 bytes
AfxCallWndProc(CWnd * 0x01144340 {CBrowserFrame hWnd=0x00160718}, HWND__ *
0x00160718, unsigned int 273, unsigned int 32802, long 0) line 215 + 26 bytes
AfxWndProc(HWND__ * 0x00160718, unsigned int 273, unsigned int 32802, long 0)
line 368
AfxWndProcBase(HWND__ * 0x00160718, unsigned int 273, unsigned int 32802, long
0) line 220 + 21 bytes

Comment 1

[David Epstein]

Attachment: patch for Tests.cpp for TestEmbed, place in /embedding/qa/testembed

Comment 2

[lchiang]

Is this a bug in the product or with testembed?  

Comment 3

[David Epstein]

Unless there are additional implementations necessary for mailto protocol
handling (i.e. listeners), it looks like it's in the product. The purpose of
these tests is to see if nsIRequest methods work in our embedding environment,
not to find problems in Necko. We get nsIRequest object by QI'ing off
nsIChannel. I've tried different protocol requests, some like http & https are
handled ok, others popup asserts or have problems. With the mailto request,
we're able to get the channel. The problem is that we're getting a failure when
calling AsyncOpen(). After that, suspend/resume doesn't work. 

Comment 4

[Christian :Biesinger (don't email me, ping me on IRC)]

bug 200681 killed nsMailtoChannel. mailto: now uses an InputStreamChannel, which
should work fine w/ Suspend/Resume, as well as asyncOpen. marking fixed.