之前的文章 "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.