Re: A problem about CHtmlView::Navigate2

From:

=?Utf-8?B?VG9tIExlZQ==?= <Tom.Lee@community.nospam>

Newsgroups:

microsoft.public.vc.mfc

Date:

Wed, 8 Aug 2007 02:18:17 -0700

Message-ID:

<C635864A-2407-410C-B5DE-CE9A8336DB08@microsoft.com>

The NavigateComplete2 event coudl be fired many times. So we could not rely
on it.
The webbrowser object seems have bug, when the XML file is big and user
calls Navigate2 very quickly ??? it means user calls Navigate2 many times when
IE core is downloading a file. It will hang sometimes.
--
Tom Lee

"Giovanni Dicanio" wrote:

"Tom Lee" <Tom.Lee@community.nospam> ha scritto nel messaggio
news:BC70C71D-74E5-4133-850A-E2E1FC0EB120@microsoft.com...

I found if I call the CHtmlView::Navigate2 very quickly, it will hang the
process.

[...]

//Get the XML files full path and put them into a vector.
TCHAR szPath[MAX_PATH*4];
ZeroMemory(szPath, MAX_PATH*4);

This is unrelated to your original question, but I think you have a bug
here.
The above code is correct in ANSI builds, when TCHAR = char = 1 byte.
But in Unicode builds, you are zero-ing 50% of szPath memory, not 100%.
You should write:

::ZeroMemory( szPath, MAX_PATH * 4 * sizeof(TCHAR) );

or

::ZeroMemory( szPath, sizeof( szPath ) );

this->Navigate2(*m_curIter, NULL, NULL);
CString cs;
cs.Format(_T("i = %d, Navigate2 %s\n"), i, *m_curIter);
OutputDebugString(cs);
m_curIter++;
}

//Use this function to tigger the nvaigate2 again.
void CTestHVView::OnRunAgain()
{
SetTimer(1, 10, NULL);
}

Any comments are very appreciated!

The timer set to 0.01 seconds is very fast... maybe you should wait for a
NavigateComplete2 event to be fired before calling the Navigate2 method
again...

Giovanni