Amebis/wxWidgets
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Ensure wxWebViewIE::SetPage clears the existing content before writing the new output. Also add a basic unit test to verify correctness in the future.

Browse Source 
Fixes #13770

git-svn-id: https://svn.wxwidgets.org/svn/wx/wxWidgets/trunk@70041 c3d73ce0-8a6f-49c7-b76d-6d57e0e08775
This commit is contained in:
Steve Lamerton
2011-12-18 11:51:52 +00:00
parent 913ce2990e
commit 7f98bdd6a5
2 changed files with 49 additions and 18 deletions
Download Patch File Download Diff File Expand all files Collapse all files

32
src/msw/webview_ie.cpp
View File

@@ -102,20 +102,36 @@ void wxWebViewIE::LoadURL(const wxString& url)
void wxWebViewIE::SetPage(const wxString& html, const wxString& baseUrl) void wxWebViewIE::SetPage(const wxString& html, const wxString& baseUrl)
{ {
BSTR bstr = SysAllocString(html.wc_str()); BSTR bstr = SysAllocString(OLESTR(""));
// Creates a new one-dimensional array
SAFEARRAY *psaStrings = SafeArrayCreateVector(VT_VARIANT, 0, 1); SAFEARRAY *psaStrings = SafeArrayCreateVector(VT_VARIANT, 0, 1);
if (psaStrings != NULL) if (psaStrings != NULL)
{ {
VARIANT *param; VARIANT *param;
HRESULT hr = SafeArrayAccessData(psaStrings, (LPVOID*)&param); HRESULT hr = SafeArrayAccessData(psaStrings, (LPVOID*)&param);
param->vt = VT_BSTR; param->vt = VT_BSTR;
param->bstrVal = bstr; param->bstrVal = bstr;
hr = SafeArrayUnaccessData(psaStrings);
IHTMLDocument2* document = GetDocument();
document->write(psaStrings);
document->close();
document->Release();
SafeArrayDestroy(psaStrings);
bstr = SysAllocString(html.wc_str());
// Creates a new one-dimensional array
psaStrings = SafeArrayCreateVector(VT_VARIANT, 0, 1);
if (psaStrings != NULL)
{
hr = SafeArrayAccessData(psaStrings, (LPVOID*)&param);
param->vt = VT_BSTR;
param->bstrVal = bstr;
hr = SafeArrayUnaccessData(psaStrings); hr = SafeArrayUnaccessData(psaStrings);
IHTMLDocument2* document = GetDocument(); document = GetDocument();
document->write(psaStrings); document->write(psaStrings);
document->Release(); document->Release();
@@ -138,7 +154,11 @@ void wxWebViewIE::SetPage(const wxString& html, const wxString& baseUrl)
{ {
wxLogError("wxWebViewIE::SetPage() : psaStrings is NULL"); wxLogError("wxWebViewIE::SetPage() : psaStrings is NULL");
} }
}
else
{
wxLogError("wxWebViewIE::SetPage() : psaStrings is NULL during clear");
}
} }
wxString wxWebViewIE::GetPageSource() const wxString wxWebViewIE::GetPageSource() const

11
tests/controls/webtest.cpp
View File

@@ -44,6 +44,7 @@ private:
CPPUNIT_TEST( Selection ); CPPUNIT_TEST( Selection );
CPPUNIT_TEST( Zoom ); CPPUNIT_TEST( Zoom );
CPPUNIT_TEST( RunScript ); CPPUNIT_TEST( RunScript );
CPPUNIT_TEST( SetPage );
CPPUNIT_TEST_SUITE_END(); CPPUNIT_TEST_SUITE_END();
void Title(); void Title();
@@ -56,6 +57,7 @@ private:
void Selection(); void Selection();
void Zoom(); void Zoom();
void RunScript(); void RunScript();
void SetPage();
void LoadUrl(int times = 1); void LoadUrl(int times = 1);
wxWebView* m_browser; wxWebView* m_browser;
@@ -244,4 +246,13 @@ void WebTestCase::RunScript()
CPPUNIT_ASSERT_EQUAL("Hello World!", m_browser->GetPageText()); CPPUNIT_ASSERT_EQUAL("Hello World!", m_browser->GetPageText());
} }
void WebTestCase::SetPage()
{
m_browser->SetPage("<html><body>text</body></html>", "");
CPPUNIT_ASSERT_EQUAL("text", m_browser->GetPageText());
m_browser->SetPage("<html><body>other text</body></html>", "");
CPPUNIT_ASSERT_EQUAL("other text", m_browser->GetPageText());
}
#endif //wxUSE_WEBVIEW && (wxUSE_WEBVIEW_WEBKIT || wxUSE_WEBVIEW_IE) #endif //wxUSE_WEBVIEW && (wxUSE_WEBVIEW_WEBKIT || wxUSE_WEBVIEW_IE)