之前的文章 "http://blog.csdn.net/norsd/archive/2008/09/13/2921389.aspx" 实现了一个WebBrowser.
但是万物总不是完美的, 最近有网友问, GetHTMLDocumentX 系列函数的实现,是否需要Release.
当时我回答不需要. 我是这样考虑的:
WebBrowser 类只提供一个(系列) IHtmlDocumentX* 指针, 在析构的时候 Release. 这样的实现听起来不错,实际运行也似乎没有问题.
非常巧合,最近终于发现问题.
考虑这样的情况:
1.打开页面 nothere.html
2.如果这个页面无法打开,WebBrowser会不断重试
3.这时候如果立刻跳转到其他页面,这时候WebBrowser会产生一个新的 IDocument 对象
这时候我们的 IHtmlDocumentX* 指针指向的还是之前的 nothere.html
只有我们 get_Document 才能得到当前页面的 document.
所以代码应该改成这样:
- IHTMLDocument2*
- WebBrowser::GetHTMLDocument2()
- {
- if( _pHtmlDoc2 != NULL )
- _pHtmlDoc2->Release();
- _pHtmlDoc2 = NULL ;
- IWebBrowser2* pWB2 = NULL;
- IDispatch* pDp = NULL;
- NULLTEST_AE(pWB2 = GetWebBrowser2() );
- HRTEST_E(pWB2->get_Document(&pDp));
- NULLTEST_E( pDp ); // ie7情况下如果document还不存在,则 pWB2->get_Document会返回错误, 但是ie6则不会有有任何HR错误, 他只是使pDp=NULL.
- HRTEST_E(pDp->QueryInterface(IID_IHTMLDocument2,(void**)&_pHtmlDoc2));
- return _pHtmlDoc2;
- RETURN:
- return NULL;
- }
p.s: IHTMLDcoumentX , 表示 IHTMLDocument , IHTMLDocument2 , and so on.