[PATCH] mshtml: implement ActiveScriptSite_OnScriptError.
Jacek Caban
jacek at codeweavers.com
Wed Sep 29 05:41:24 CDT 2010
On 9/29/10 9:39 AM, Reece Dunn wrote:
> Hi,
>
> This reports any errors sent to the mshtml ActiveScriptSite
> OnScriptError handler to the user and traces it to ERR to aid
> debugging.
static HRESULT WINAPI ActiveScriptSite_OnScriptError(IActiveScriptSite *iface, IActiveScriptError *pscripterror)
{
ScriptHost *This = ACTSCPSITE_THIS(iface);
- FIXME("(%p)->(%p)\n", This, pscripterror);
- return E_NOTIMPL;
+ EXCEPINFO excep;
+ HRESULT hr;
+
+ TRACE("(%p)->(%p)\n", This, pscripterror);
+
+ hr = IActiveScriptError_GetExceptionInfo(pscripterror,&excep);
+
+ ERR("ActiveScript Error: %s source: %s\n",
+ debugstr_w(excep.bstrDescription), debugstr_w(excep.bstrSource));
ERR is not appropriate in this case.
+ MessageBoxW(NULL, excep.bstrDescription, excep.bstrSource, MB_OK|MB_ICONERROR);
This is not how this function should work. If you want to implement it, please start with a test case. It probably should fire onerror event.
Jacek
More information about the wine-devel
mailing list