之前的文章 "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:
- returnNULL;
- }
p.s: IHTMLDcoumentX , 表示 IHTMLDocument , IHTMLDocument2 , and so on.