December 2010 - Wine-commits

Alexandre Julliard : quartz: Convert coclasses registration to the IRegistrar mechanism.
by Alexandre Julliard 17 Dec '10

17 Dec '10

Module: wine Branch: master Commit: ae2a992d703d93826de105f2484f388d0a37f49b URL: http://source.winehq.org/git/wine.git/?a=commit;h=ae2a992d703d93826de105f24… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Fri Dec 17 13:18:21 2010 +0100 quartz: Convert coclasses registration to the IRegistrar mechanism. --- dlls/quartz/Makefile.in | 3 +- dlls/quartz/quartz_strmif.idl | 128 ++++++++++++++++++++++++++++++++++++++++- dlls/quartz/regsvr.c | 111 ----------------------------------- 3 files changed, 128 insertions(+), 114 deletions(-) diff --git a/dlls/quartz/Makefile.in b/dlls/quartz/Makefile.in index 0798cdb..9a4e511 100644 --- a/dlls/quartz/Makefile.in +++ b/dlls/quartz/Makefile.in @@ -1,7 +1,7 @@ MODULE = quartz.dll IMPORTLIB = quartz IMPORTS = strmiids strmbase uuid dsound msacm32 msvfw32 ole32 oleaut32 shlwapi rpcrt4 user32 gdi32 advapi32 -EXTRADEFS = -DENTRY_PREFIX=QUARTZ_ -DPROXY_DELEGATION -DREGISTER_PROXY_DLL +EXTRADEFS = -DENTRY_PREFIX=QUARTZ_ -DPROXY_DELEGATION -DWINE_REGISTER_DLL C_SRCS = \ acmwrapper.c \ @@ -30,6 +30,7 @@ C_SRCS = \ RC_SRCS = version.rc IDL_P_SRCS = quartz_strmif.idl +IDL_R_SRCS = quartz_strmif.idl EXTRA_OBJS = dlldata.o diff --git a/dlls/quartz/quartz_strmif.idl b/dlls/quartz/quartz_strmif.idl index aad6232..ede51f6 100644 --- a/dlls/quartz/quartz_strmif.idl +++ b/dlls/quartz/quartz_strmif.idl @@ -16,7 +16,131 @@ * Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301, USA */ -/* just a wrapper for strmif.idl */ - cpp_quote("#include <wingdi.h>") #include "strmif.idl" + +[ + helpstring("Filter Graph"), + threading(both), + uuid(e436ebb3-524f-11ce-9f53-0020af0ba770) +] +coclass FilterGraph { interface IFilterGraph2; } + +[ + helpstring("Filter Graph no thread"), + threading(both), + uuid(e436ebb8-524f-11ce-9f53-0020af0ba770) +] +coclass FilterGraphNoThread { interface IFilterGraph2; } + +[ + helpstring("Filter Mapper"), + threading(both), + uuid(e436ebb2-524f-11ce-9f53-0020af0ba770) +] +coclass FilterMapper { interface IFilterMapper; } + +[ + helpstring("Filter Mapper2"), + threading(both), + uuid(cda42200-bd88-11d0-bd4e-00a0c911ce86) +] +coclass FilterMapper2 { interface IFilterMapper3; } + +[ + helpstring("System Clock"), + threading(both), + uuid(e436ebb1-524f-11ce-9f53-0020af0ba770) +] +coclass SystemClock { interface IReferenceClock; } + +[ + helpstring("Memory Allocator"), + threading(both), + uuid(1e651cc0-b199-11d0-8212-00c04fc32c45) +] +coclass MemoryAllocator { interface IMemAllocator; } + +[ + helpstring("Seeking"), + threading(both), + uuid(060af76c-68dd-11d0-8fc1-00c04fd9189d) +] +coclass SeekingPassThru { interface ISeekingPassThru; } + +[ + helpstring("File Source Filter"), + threading(both), + uuid(e436ebb5-524f-11ce-9f53-0020af0ba770) +] +coclass AsyncReader { interface IBaseFilter; } + +[ + helpstring("AVI Splitter"), + threading(both), + uuid(1b544c20-fd0b-11ce-8c63-00aa0044b51e) +] +coclass AviSplitter { interface IBaseFilter; } + +[ + helpstring("MPEG-I Stream Splitter"), + threading(both), + uuid(336475d0-942a-11ce-a870-00aa002feab5) +] +coclass MPEG1Splitter { interface IBaseFilter; } + +[ + helpstring("AVI Decompressor"), + threading(both), + uuid(cf49d4e0-1115-11ce-b03a-0020af0ba770) +] +coclass AVIDec { interface IBaseFilter; } + +[ + helpstring("DirectSound Audio Renderer"), + threading(both), + uuid(79376820-07d0-11cf-a24d-0020afd79767) +] +coclass DSoundRender { interface IBaseFilter; } + +[ + helpstring("Wave Audio Renderer"), + threading(both), + uuid(e30629d1-27e5-11ce-875d-00608cb78066) +] +coclass AudioRender { interface IBaseFilter; } + +[ + helpstring("Null Renderer"), + threading(both), + uuid(c1f400a4-3f08-11d3-9f0b-006008039e37) +] +coclass NullRenderer { interface IBaseFilter; } + +[ + helpstring("Video Renderer"), + threading(both), + uuid(70e102b0-5556-11ce-97c0-00aa0055595a) +] +coclass VideoRenderer { interface IBaseFilter; } + +[ + helpstring("Default Video Renderer"), + threading(both), + uuid(6bc1cffa-8fc1-4261-ac22-cfb4cc38db50) +] +coclass VideoRendererDefault { interface IBaseFilter; } + +[ + helpstring("ACM wrapper"), + threading(both), + uuid(6a08cf80-0e18-11cf-a24d-0020afd79767) +] +coclass ACMWrapper { interface IBaseFilter; } + +[ + helpstring("Wave Parser"), + threading(both), + uuid(d51bd5a1-7548-11cf-a520-0080c77ef58a) +] +coclass WAVEParser { interface IBaseFilter; } diff --git a/dlls/quartz/regsvr.c b/dlls/quartz/regsvr.c index cdb33d1..d41e2f5 100644 --- a/dlls/quartz/regsvr.c +++ b/dlls/quartz/regsvr.c @@ -778,121 +778,10 @@ error_close_progid_key: return res; } -static GUID const CLSID_PSFactoryBuffer = { - 0x92a3a302, 0xda7c, 0x4a1f, {0xba,0x7e,0x18,0x02,0xbb,0x5d,0x2d,0x02} }; - /*********************************************************************** * coclass list */ static struct regsvr_coclass const coclass_list[] = { - { &CLSID_FilterGraph, - "Filter Graph", - NULL, - "quartz.dll", - "Both" - }, - { &CLSID_FilterGraphNoThread, - "Filter Graph", - NULL, - "quartz.dll", - "Both" - }, - { &CLSID_FilterMapper, - "Filter Mapper", - NULL, - "quartz.dll", - "Both" - }, - { &CLSID_FilterMapper2, - "Filter Mapper2", - NULL, - "quartz.dll", - "Both" - }, - { &CLSID_SystemClock, - "System Clock", - NULL, - "quartz.dll", - "Both" - }, - { &CLSID_MemoryAllocator, - "Memory Allocator", - NULL, - "quartz.dll", - "Both" - }, - { &CLSID_SeekingPassThru, - "Seeking", - NULL, - "quartz.dll", - "Both" - }, - { &CLSID_AsyncReader, - "File Source Filter", - NULL, - "quartz.dll", - "Both" - }, - { &CLSID_AviSplitter, - "AVI Splitter", - NULL, - "quartz.dll", - "Both" - }, - { &CLSID_MPEG1Splitter, - "MPEG-I Stream Splitter", - NULL, - "quartz.dll", - "Both" - }, - { &CLSID_AVIDec, - "AVI Decompressor", - NULL, - "quartz.dll", - "Both" - }, - { &CLSID_DSoundRender, - "DirectSound Audio Renderer", - NULL, - "quartz.dll", - "Both" - }, - { &CLSID_AudioRender, - "Wave Audio Renderer", - NULL, - "quartz.dll", - "Both" - }, - { &CLSID_NullRenderer, - "Null Renderer", - NULL, - "quartz.dll", - "Both" - }, - { &CLSID_VideoRenderer, - "Video Renderer", - NULL, - "quartz.dll", - "Both" - }, - { &CLSID_VideoRendererDefault, - "Default Video Renderer", - NULL, - "quartz.dll", - "Both" - }, - { &CLSID_ACMWrapper, - "ACM wrapper", - NULL, - "quartz.dll", - "Both" - }, - { &CLSID_WAVEParser, - "Wave Parser", - NULL, - "quartz.dll", - "Both" - }, { NULL } /* list terminator */ };

1 0

Alexandre Julliard : avifil32: Move extension registrations to the rgs file .
by Alexandre Julliard 17 Dec '10

17 Dec '10

Module: wine Branch: master Commit: 7372ffddb73ed9ef41031e7878fe3323d6a206d1 URL: http://source.winehq.org/git/wine.git/?a=commit;h=7372ffddb73ed9ef41031e787… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Fri Dec 17 13:50:37 2010 +0100 avifil32: Move extension registrations to the rgs file. --- dlls/avifil32/avifile.rgs | 19 +++++++++++++++++++ tools/wine.inf.in | 7 ------- 2 files changed, 19 insertions(+), 7 deletions(-) diff --git a/dlls/avifil32/avifile.rgs b/dlls/avifil32/avifile.rgs index e9e3fb2..3e6ba98 100644 --- a/dlls/avifil32/avifile.rgs +++ b/dlls/avifil32/avifile.rgs @@ -8,4 +8,23 @@ HKCR '{0002000d-0000-0000-C000-000000000046}' { InprocServer = s 'avifile.dll' } '{0002000f-0000-0000-C000-000000000046}' { InprocServer = s 'avifile.dll' } } + NoRemove AVIFile + { + Compressors + { + auds = s '{0002000F-0000-0000-C000-000000000046}' + vids = s '{00020001-0000-0000-C000-000000000046}' + } + Extensions + { + AU = s '{00020003-0000-0000-C000-000000000046}' + AVI = s '{00020000-0000-0000-C000-000000000046}' + WAV = s '{00020003-0000-0000-C000-000000000046}' + } + RIFFHandlers + { + AVI = s '{00020000-0000-0000-C000-000000000046}' + WAVE = s '{00020003-0000-0000-C000-000000000046}' + } + } } diff --git a/tools/wine.inf.in b/tools/wine.inf.in index aa3b354..8dbe0b7 100644 --- a/tools/wine.inf.in +++ b/tools/wine.inf.in @@ -225,13 +225,6 @@ HKCR,https\shell\open\ddeexec\Application,,2,"IExplore" HKCR,https\shell\open\ddeexec\Topic,,2,"WWW_OpenURL" HKCR,mailto\shell\open\command,,2,"%11%\winebrowser %1" ; FIXME: the following should be done by the dll registration -HKCR,AVIFile\Compressors\auds,,,"{0002000F-0000-0000-C000-000000000046}" -HKCR,AVIFile\Compressors\vids,,,"{00020001-0000-0000-C000-000000000046}" -HKCR,AVIFile\Extensions\AU,,,"{00020003-0000-0000-C000-000000000046}" -HKCR,AVIFile\Extensions\AVI,,,"{00020000-0000-0000-C000-000000000046}" -HKCR,AVIFile\Extensions\WAV,,,"{00020003-0000-0000-C000-000000000046}" -HKCR,AVIFile\RIFFHandlers\AVI,,,"{00020000-0000-0000-C000-000000000046}" -HKCR,AVIFile\RIFFHandlers\WAVE,,,"{00020003-0000-0000-C000-000000000046}" HKCR,TypeLib\{00020430-0000-0000-C000-000000000046}\1.0\0\win16,,,"stdole.tlb" HKCR,TypeLib\{00020430-0000-0000-C000-000000000046}\1.0\FLAGS,,,"1"

1 0

Alexandre Julliard : widl: Register the PSFactoryBuffer class as factory if it' s defined in the idl file.
by Alexandre Julliard 17 Dec '10

17 Dec '10

Module: wine Branch: master Commit: 875c1b8739982187dfa923fcd3336d47aefa80c0 URL: http://source.winehq.org/git/wine.git/?a=commit;h=875c1b8739982187dfa923fcd… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Fri Dec 17 14:58:47 2010 +0100 widl: Register the PSFactoryBuffer class as factory if it's defined in the idl file. --- tools/widl/register.c | 32 ++++++++++++++++++++++++++------ 1 files changed, 26 insertions(+), 6 deletions(-) diff --git a/tools/widl/register.c b/tools/widl/register.c index f709214..9913f8c 100644 --- a/tools/widl/register.c +++ b/tools/widl/register.c @@ -61,9 +61,26 @@ static const char *get_coclass_threading( const type_t *class ) return models[get_attrv( class->attrs, ATTR_THREADING )]; } -static int write_interface( const type_t *iface ) +static const type_t *find_ps_factory( const statement_list_t *stmts ) +{ + const statement_t *stmt; + + if (stmts) LIST_FOR_EACH_ENTRY( stmt, stmts, const statement_t, entry ) + { + if (stmt->type == STMT_TYPE) + { + const type_t *type = stmt->u.type; + if (type_get_type(type) == TYPE_COCLASS && !strcmp( type->name, "PSFactoryBuffer" )) + return type; + } + } + return NULL; +} + +static int write_interface( const type_t *iface, const type_t *ps_factory ) { const UUID *uuid = get_attrp( iface->attrs, ATTR_UUID ); + const UUID *ps_uuid = ps_factory ? get_attrp( ps_factory->attrs, ATTR_UUID ) : NULL; if (!uuid) return 0; if (!is_object( iface )) return 0; @@ -77,13 +94,14 @@ static int write_interface( const type_t *iface ) put_str( indent, "{\n" ); indent++; put_str( indent, "NumMethods = s %u\n", count_methods( iface )); - put_str( indent, "ProxyStubClsid32 = s '%%CLSID_PSFactoryBuffer%%'\n" ); + put_str( indent, "ProxyStubClsid32 = s '%s'\n", + ps_uuid ? format_uuid( ps_uuid ) : "%CLSID_PSFactoryBuffer%" ); indent--; put_str( indent, "}\n" ); return 1; } -static int write_interfaces( const statement_list_t *stmts ) +static int write_interfaces( const statement_list_t *stmts, const type_t *ps_factory ) { const statement_t *stmt; int count = 0; @@ -91,7 +109,7 @@ static int write_interfaces( const statement_list_t *stmts ) if (stmts) LIST_FOR_EACH_ENTRY( stmt, stmts, const statement_t, entry ) { if (stmt->type == STMT_TYPE && type_get_type( stmt->u.type ) == TYPE_INTERFACE) - count += write_interface( stmt->u.type ); + count += write_interface( stmt->u.type, ps_factory ); } return count; } @@ -204,6 +222,7 @@ static inline void put_string( const char *str ) void write_regscript( const statement_list_t *stmts ) { int count; + const type_t *ps_factory; if (!do_regscript) return; if (do_everything && !need_proxy_file( stmts )) return; @@ -215,12 +234,13 @@ void write_regscript( const statement_list_t *stmts ) put_str( indent, "NoRemove Interface\n" ); put_str( indent++, "{\n" ); - count = write_interfaces( stmts ); + ps_factory = find_ps_factory( stmts ); + count = write_interfaces( stmts, ps_factory ); put_str( --indent, "}\n" ); put_str( indent, "NoRemove CLSID\n" ); put_str( indent++, "{\n" ); - if (count) + if (count && !ps_factory) { put_str( indent, "ForceRemove '%%CLSID_PSFactoryBuffer%%' = s 'PSFactoryBuffer'\n" ); put_str( indent++, "{\n" );

1 0

Alexandre Julliard : msi: Convert the dll registration to the IRegistrar mechanism.
by Alexandre Julliard 17 Dec '10

17 Dec '10

Module: wine Branch: master Commit: 17e66e06d613026387753f350aa2eaedbcb02ac1 URL: http://source.winehq.org/git/wine.git/?a=commit;h=17e66e06d613026387753f350… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Fri Dec 17 14:59:49 2010 +0100 msi: Convert the dll registration to the IRegistrar mechanism. --- dlls/msi/Makefile.in | 2 +- dlls/msi/custom.c | 2 +- dlls/msi/msi.rc | 3 + dlls/msi/msi_main.c | 32 ++- dlls/msi/msipriv.h | 14 - dlls/msi/msiserver.idl | 101 ++++++++ dlls/msi/msiserver.rgs | 8 + dlls/msi/regsvr.c | 663 ------------------------------------------------ tools/wine.inf.in | 1 + 9 files changed, 140 insertions(+), 686 deletions(-) Diff: http://source.winehq.org/git/wine.git/?a=commitdiff;h=17e66e06d613026387753…

1 0

Jacek Caban : mshtml: Added support for forwarding object element' s DOM calls to ActiveX control.
by Alexandre Julliard 17 Dec '10

17 Dec '10

Module: wine Branch: master Commit: 42d9e0c813e6f8f76a14ecd328906b7e38cd258d URL: http://source.winehq.org/git/wine.git/?a=commit;h=42d9e0c813e6f8f76a14ecd32… Author: Jacek Caban <jacek(a)codeweavers.com> Date: Fri Dec 17 03:38:33 2010 +0100 mshtml: Added support for forwarding object element's DOM calls to ActiveX control. --- dlls/mshtml/htmlobject.c | 24 +++++++++++++- dlls/mshtml/pluginhost.c | 76 +++++++++++++++++++++++++++++++++++++++++++ dlls/mshtml/pluginhost.h | 6 +++ dlls/mshtml/tests/activex.c | 60 ++++++++++++++++++++++++++++++++- 4 files changed, 163 insertions(+), 3 deletions(-) diff --git a/dlls/mshtml/htmlobject.c b/dlls/mshtml/htmlobject.c index fe97eb9..8f8bef2 100644 --- a/dlls/mshtml/htmlobject.c +++ b/dlls/mshtml/htmlobject.c @@ -439,6 +439,26 @@ static HRESULT HTMLObjectElement_get_readystate(HTMLDOMNode *iface, BSTR *p) return E_NOTIMPL; } +static HRESULT HTMLObjectElement_get_dispid(HTMLDOMNode *iface, BSTR name, + DWORD grfdex, DISPID *pid) +{ + HTMLObjectElement *This = HTMLOBJECT_NODE_THIS(iface); + + TRACE("(%p)->(%s %x %p)\n", This, debugstr_w(name), grfdex, pid); + + return get_plugin_dispid(&This->plugin_container, name, pid); +} + +static HRESULT HTMLObjectElement_invoke(HTMLDOMNode *iface, DISPID id, LCID lcid, + WORD flags, DISPPARAMS *params, VARIANT *res, EXCEPINFO *ei, IServiceProvider *caller) +{ + HTMLObjectElement *This = HTMLOBJECT_NODE_THIS(iface); + + TRACE("(%p)->(%d)\n", This, id); + + return invoke_plugin_prop(&This->plugin_container, id, lcid, flags, params, res, ei); +} + #undef HTMLOBJECT_NODE_THIS static const NodeImplVtbl HTMLObjectElementImplVtbl = { @@ -450,7 +470,9 @@ static const NodeImplVtbl HTMLObjectElementImplVtbl = { NULL, NULL, NULL, - HTMLObjectElement_get_readystate + HTMLObjectElement_get_readystate, + HTMLObjectElement_get_dispid, + HTMLObjectElement_invoke }; static const tid_t HTMLObjectElement_iface_tids[] = { diff --git a/dlls/mshtml/pluginhost.c b/dlls/mshtml/pluginhost.c index 91b6fda..c6df421 100644 --- a/dlls/mshtml/pluginhost.c +++ b/dlls/mshtml/pluginhost.c @@ -318,6 +318,82 @@ HRESULT get_plugin_disp(HTMLPluginContainer *plugin_container, IDispatch **ret) return S_OK; } +HRESULT get_plugin_dispid(HTMLPluginContainer *plugin_container, WCHAR *name, DISPID *ret) +{ + IDispatch *disp; + DISPID id; + DWORD i; + HRESULT hres; + + if(!plugin_container->plugin_host) { + WARN("no plugin host\n"); + return DISP_E_UNKNOWNNAME; + } + + disp = plugin_container->plugin_host->disp; + if(!disp) + return DISP_E_UNKNOWNNAME; + + hres = IDispatch_GetIDsOfNames(disp, &IID_NULL, &name, 1, 0, &id); + if(FAILED(hres)) { + TRACE("no prop %s\n", debugstr_w(name)); + return DISP_E_UNKNOWNNAME; + } + + for(i=0; i < plugin_container->props_len; i++) { + if(id == plugin_container->props[i]) { + *ret = MSHTML_DISPID_CUSTOM_MIN+i; + return S_OK; + } + } + + if(!plugin_container->props) { + plugin_container->props = heap_alloc(8*sizeof(DISPID)); + if(!plugin_container->props) + return E_OUTOFMEMORY; + plugin_container->props_size = 8; + }else if(plugin_container->props_len == plugin_container->props_size) { + DISPID *new_props; + + new_props = heap_realloc(plugin_container->props, plugin_container->props_size*2); + if(!new_props) + return E_OUTOFMEMORY; + + plugin_container->props = new_props; + plugin_container->props_size *= 2; + } + + plugin_container->props[plugin_container->props_len] = id; + *ret = MSHTML_DISPID_CUSTOM_MIN+plugin_container->props_len; + plugin_container->props_len++; + return S_OK; +} + +HRESULT invoke_plugin_prop(HTMLPluginContainer *plugin_container, DISPID id, LCID lcid, WORD flags, DISPPARAMS *params, + VARIANT *res, EXCEPINFO *ei) +{ + PluginHost *host; + + host = plugin_container->plugin_host; + if(!host || !host->disp) { + FIXME("Called with no disp\n"); + return E_UNEXPECTED; + } + + if(!check_script_safety(host)) { + FIXME("Insecure object\n"); + return E_FAIL; + } + + if(id < MSHTML_DISPID_CUSTOM_MIN || id > MSHTML_DISPID_CUSTOM_MIN + plugin_container->props_len) { + ERR("Invalid id\n"); + return E_FAIL; + } + + return IDispatch_Invoke(host->disp, plugin_container->props[id-MSHTML_DISPID_CUSTOM_MIN], &IID_NULL, + lcid, flags, params, res, ei, NULL); +} + static inline PluginHost *impl_from_IOleClientSite(IOleClientSite *iface) { return CONTAINING_RECORD(iface, PluginHost, IOleClientSite_iface); diff --git a/dlls/mshtml/pluginhost.h b/dlls/mshtml/pluginhost.h index bedb8ca..014e3ac 100644 --- a/dlls/mshtml/pluginhost.h +++ b/dlls/mshtml/pluginhost.h @@ -50,6 +50,10 @@ struct HTMLPluginContainer { HTMLElement element; PluginHost *plugin_host; + + DISPID *props; + DWORD props_size; + DWORD props_len; }; extern const IID IID_HTMLPluginContainer; @@ -64,3 +68,5 @@ HRESULT create_ip_window(IOleInPlaceUIWindow**); HRESULT create_ip_frame(IOleInPlaceFrame**); HRESULT get_plugin_disp(HTMLPluginContainer*,IDispatch**); +HRESULT get_plugin_dispid(HTMLPluginContainer*,WCHAR*,DISPID*); +HRESULT invoke_plugin_prop(HTMLPluginContainer*,DISPID,LCID,WORD,DISPPARAMS*,VARIANT*,EXCEPINFO*); diff --git a/dlls/mshtml/tests/activex.c b/dlls/mshtml/tests/activex.c index 809453b..d3cbb9d 100644 --- a/dlls/mshtml/tests/activex.c +++ b/dlls/mshtml/tests/activex.c @@ -70,6 +70,8 @@ DEFINE_EXPECT(Invoke_READYSTATE); DEFINE_EXPECT(Invoke_ENABLED); DEFINE_EXPECT(Invoke_VALID); DEFINE_EXPECT(Invoke_SECURITYCTX); +DEFINE_EXPECT(Invoke_SCRIPTPROP); +DEFINE_EXPECT(GetIDsOfNames_scriptprop); DEFINE_EXPECT(DoVerb); DEFINE_EXPECT(SetExtent); DEFINE_EXPECT(GetExtent); @@ -570,8 +572,15 @@ static HRESULT WINAPI Dispatch_GetTypeInfo(IDispatch *iface, UINT iTInfo, LCID l static HRESULT WINAPI Dispatch_GetIDsOfNames(IDispatch *iface, REFIID riid, LPOLESTR *rgszNames, UINT cNames, LCID lcid, DISPID *rgDispId) { - ok(0, "unexpected call\n"); - return E_FAIL; + CHECK_EXPECT(GetIDsOfNames_scriptprop); + ok(IsEqualGUID(riid, &IID_NULL), "riid = %s\n", debugstr_guid(riid)); + ok(cNames == 1, "cNames = %d\n", cNames); + ok(rgszNames != NULL, "rgszNames == NULL\n"); + ok(!strcmp_wa(rgszNames[0], "scriptprop"), "rgszNames[0] = %s\n", wine_dbgstr_w(rgszNames[0])); + ok(rgDispId != NULL, "rgDispId == NULL\n"); + + *rgDispId = DISPID_SCRIPTPROP; + return S_OK; } static HRESULT WINAPI Dispatch_Invoke(IDispatch *iface, DISPID dispIdMember, REFIID riid, @@ -620,6 +629,17 @@ static HRESULT WINAPI Dispatch_Invoke(IDispatch *iface, DISPID dispIdMember, REF ok(!pExcepInfo, "pExcepInfo != NULL\n"); ok(puArgErr != NULL, "puArgErr == NULL\n"); return DISP_E_MEMBERNOTFOUND; + case DISPID_SCRIPTPROP: + CHECK_EXPECT(Invoke_SCRIPTPROP); + ok(wFlags == DISPATCH_PROPERTYGET, "wFlags = %x\n", wFlags); + ok(!pDispParams->cArgs, "pDispParams->cArgs = %d\n", pDispParams->cArgs); + ok(!pDispParams->rgvarg, "pDispParams->rgvarg != NULL\n"); + ok(pExcepInfo != NULL, "pExcepInfo == NULL\n"); + ok(!puArgErr, "puArgErr != NULL\n"); + + V_VT(pVarResult) = VT_I4; + V_I4(pVarResult) = 4; + return S_OK; default: ok(0, "unexpected call %d\n", dispIdMember); } @@ -1197,10 +1217,42 @@ static const IClassFactoryVtbl ClassFactoryVtbl = { static IClassFactory activex_cf = { &ClassFactoryVtbl }; +static void test_elem_dispex(IDispatchEx *dispex) +{ + DISPPARAMS dp; + EXCEPINFO ei; + VARIANT v; + DISPID id; + BSTR str; + HRESULT hres; + + str = a2bstr("scriptprop"); + SET_EXPECT(GetIDsOfNames_scriptprop); + hres = IDispatchEx_GetDispID(dispex, str, 0, &id); + CHECK_CALLED(GetIDsOfNames_scriptprop); + SysFreeString(str); + ok(hres == S_OK, "GetDispID failed: %08x\n", hres); + todo_wine + ok(id == DISPID_SCRIPTPROP, "id = %d\n", id); + + SET_EXPECT(Invoke_SECURITYCTX); + SET_EXPECT(Invoke_SCRIPTPROP); + memset(&dp, 0, sizeof(dp)); + memset(&ei, 0, sizeof(ei)); + V_VT(&v) = VT_EMPTY; + hres = IDispatchEx_InvokeEx(dispex, id, LOCALE_NEUTRAL, DISPATCH_PROPERTYGET, &dp, &v, &ei, NULL); + ok(hres == S_OK, "InvokeEx failed: %08x\n", hres); + ok(V_VT(&v) == VT_I4, "V_VT(v) = %d\n", V_VT(&v)); + ok(V_I4(&v) == 4, "V_I4(v) = %d\n", V_I4(&v)); + CHECK_CALLED(Invoke_SECURITYCTX); + CHECK_CALLED(Invoke_SCRIPTPROP); +} + static void test_object_elem(IHTMLDocument2 *doc) { IHTMLObjectElement *objelem; IHTMLDocument3 *doc3; + IDispatchEx *dispex; IHTMLElement *elem; IDispatch *disp; BSTR str; @@ -1227,6 +1279,10 @@ static void test_object_elem(IHTMLDocument2 *doc) ok(disp == &Dispatch, "disp != Dispatch\n"); CHECK_CALLED(Invoke_SECURITYCTX); + hres = IHTMLObjectElement_QueryInterface(objelem, &IID_IDispatchEx, (void**)&dispex); + test_elem_dispex(dispex); + IDispatchEx_Release(dispex); + IHTMLObjectElement_Release(objelem); }

1 0

Jacek Caban : mshtml: Improved QueryService debug traces.
by Alexandre Julliard 17 Dec '10

17 Dec '10

Module: wine Branch: master Commit: 5cfe10b3e8d94f84265e40a1872969b5af42733e URL: http://source.winehq.org/git/wine.git/?a=commit;h=5cfe10b3e8d94f84265e40a18… Author: Jacek Caban <jacek(a)codeweavers.com> Date: Fri Dec 17 03:38:44 2010 +0100 mshtml: Improved QueryService debug traces. --- dlls/mshtml/service.c | 5 +++-- 1 files changed, 3 insertions(+), 2 deletions(-) diff --git a/dlls/mshtml/service.c b/dlls/mshtml/service.c index cb8893d..4b9327d 100644 --- a/dlls/mshtml/service.c +++ b/dlls/mshtml/service.c @@ -245,6 +245,8 @@ static HRESULT WINAPI ServiceProvider_QueryService(IServiceProvider *iface, REFG return IOleUndoManager_QueryInterface(This->doc_obj->undomgr, riid, ppv); } + TRACE("(%p)->(%s %s %p)\n", This, debugstr_guid(guidService), debugstr_guid(riid), ppv); + if(This->doc_obj->client) { IServiceProvider *sp; HRESULT hres; @@ -260,8 +262,7 @@ static HRESULT WINAPI ServiceProvider_QueryService(IServiceProvider *iface, REFG } } - FIXME("(%p)->(%s %s %p)\n", This, debugstr_guid(guidService), debugstr_guid(riid), ppv); - + FIXME("unknown service %s\n", debugstr_guid(guidService)); return E_NOINTERFACE; }

1 0

Jacek Caban : mshtml: Improved IOmNavigator::javaEnabled stub.
by Alexandre Julliard 17 Dec '10

17 Dec '10

Module: wine Branch: master Commit: f01d831b9b12d04300d2b31d417501b5d78c902d URL: http://source.winehq.org/git/wine.git/?a=commit;h=f01d831b9b12d04300d2b31d4… Author: Jacek Caban <jacek(a)codeweavers.com> Date: Fri Dec 17 03:38:56 2010 +0100 mshtml: Improved IOmNavigator::javaEnabled stub. --- dlls/mshtml/omnavigator.c | 7 +++++-- 1 files changed, 5 insertions(+), 2 deletions(-) diff --git a/dlls/mshtml/omnavigator.c b/dlls/mshtml/omnavigator.c index 66bfd87..af36537 100644 --- a/dlls/mshtml/omnavigator.c +++ b/dlls/mshtml/omnavigator.c @@ -368,8 +368,11 @@ static HRESULT WINAPI OmNavigator_get_userAgent(IOmNavigator *iface, BSTR *p) static HRESULT WINAPI OmNavigator_javaEnabled(IOmNavigator *iface, VARIANT_BOOL *enabled) { OmNavigator *This = impl_from_IOmNavigator(iface); - FIXME("(%p)->(%p)\n", This, enabled); - return E_NOTIMPL; + + FIXME("(%p)->(%p) semi-stub\n", This, enabled); + + *enabled = VARIANT_FALSE; + return S_OK; } static HRESULT WINAPI OmNavigator_taintEnabled(IOmNavigator *iface, VARIANT_BOOL *enabled)

1 0

Jacek Caban : mshtml: Added IHTMLScriptElement::get_scr implementation.
by Alexandre Julliard 17 Dec '10

17 Dec '10

Module: wine Branch: master Commit: 96efe799c191027f3e58e58f663fc465525568bc URL: http://source.winehq.org/git/wine.git/?a=commit;h=96efe799c191027f3e58e58f6… Author: Jacek Caban <jacek(a)codeweavers.com> Date: Fri Dec 17 03:39:11 2010 +0100 mshtml: Added IHTMLScriptElement::get_scr implementation. --- dlls/mshtml/htmlscript.c | 10 ++++++++-- dlls/mshtml/mshtml_private.h | 1 + dlls/mshtml/nsembed.c | 24 ++++++++++++++++++++++++ dlls/mshtml/tests/dom.c | 5 +++++ 4 files changed, 38 insertions(+), 2 deletions(-) diff --git a/dlls/mshtml/htmlscript.c b/dlls/mshtml/htmlscript.c index 91e1d22..bd066ac 100644 --- a/dlls/mshtml/htmlscript.c +++ b/dlls/mshtml/htmlscript.c @@ -106,8 +106,14 @@ static HRESULT WINAPI HTMLScriptElement_put_src(IHTMLScriptElement *iface, BSTR static HRESULT WINAPI HTMLScriptElement_get_src(IHTMLScriptElement *iface, BSTR *p) { HTMLScriptElement *This = impl_from_IHTMLScriptElement(iface); - FIXME("(%p)->(%p)\n", This, p); - return E_NOTIMPL; + nsAString src_str; + nsresult nsres; + + TRACE("(%p)->(%p)\n", This, p); + + nsAString_Init(&src_str, NULL); + nsres = nsIDOMHTMLScriptElement_GetSrc(This->nsscript, &src_str); + return return_nsstr(nsres, &src_str, p); } static HRESULT WINAPI HTMLScriptElement_put_htmlFor(IHTMLScriptElement *iface, BSTR v) diff --git a/dlls/mshtml/mshtml_private.h b/dlls/mshtml/mshtml_private.h index 7162abb..326439f 100644 --- a/dlls/mshtml/mshtml_private.h +++ b/dlls/mshtml/mshtml_private.h @@ -781,6 +781,7 @@ void nsAString_InitDepend(nsAString*,const PRUnichar*); void nsAString_SetData(nsAString*,const PRUnichar*); PRUint32 nsAString_GetData(const nsAString*,const PRUnichar**); void nsAString_Finish(nsAString*); +HRESULT return_nsstr(nsresult,nsAString*,BSTR*); nsICommandParams *create_nscommand_params(void); HRESULT nsnode_to_nsstring(nsIDOMNode*,nsAString*); diff --git a/dlls/mshtml/nsembed.c b/dlls/mshtml/nsembed.c index 556ae08..d766af8 100644 --- a/dlls/mshtml/nsembed.c +++ b/dlls/mshtml/nsembed.c @@ -733,6 +733,30 @@ void nsAString_Finish(nsAString *str) NS_StringContainerFinish(str); } +HRESULT return_nsstr(nsresult nsres, nsAString *nsstr, BSTR *p) +{ + const PRUnichar *str; + + if(NS_FAILED(nsres)) { + ERR("failed: %08x\n", nsres); + nsAString_Finish(nsstr); + return E_FAIL; + } + + nsAString_GetData(nsstr, &str); + TRACE("ret %s\n", debugstr_w(str)); + if(*str) { + *p = SysAllocString(str); + if(!*p) + return E_OUTOFMEMORY; + }else { + *p = NULL; + } + + nsAString_Finish(nsstr); + return S_OK; +} + nsICommandParams *create_nscommand_params(void) { nsICommandParams *ret = NULL; diff --git a/dlls/mshtml/tests/dom.c b/dlls/mshtml/tests/dom.c index 19b7d14..c29f616 100644 --- a/dlls/mshtml/tests/dom.c +++ b/dlls/mshtml/tests/dom.c @@ -6211,6 +6211,11 @@ static void test_elems(IHTMLDocument2 *doc) hres = IHTMLScriptElement_put_defer(script, VARIANT_FALSE); ok(hres == S_OK, "put_defer failed: %08x\n", hres); + + str = (BSTR)0xdeadbeef; + hres = IHTMLScriptElement_get_src(script, &str); + ok(hres == S_OK, "get_src failed: %08x\n", hres); + ok(!str, "src = %s\n", wine_dbgstr_w(str)); } IHTMLScriptElement_Release(script);

1 0

Jacek Caban : mshtml: Added IHTMLStyleElement:: media property implementation.
by Alexandre Julliard 17 Dec '10

17 Dec '10

Module: wine Branch: master Commit: 09c63b22152cdfbe1ee76d5b4baffee8119f3e65 URL: http://source.winehq.org/git/wine.git/?a=commit;h=09c63b22152cdfbe1ee76d5b4… Author: Jacek Caban <jacek(a)codeweavers.com> Date: Fri Dec 17 03:39:24 2010 +0100 mshtml: Added IHTMLStyleElement::media property implementation. --- dlls/mshtml/htmlstyleelem.c | 39 +++++++++++++++++++++++++++--- dlls/mshtml/nsiface.idl | 15 ++++++++++++ dlls/mshtml/tests/dom.c | 54 ++++++++++++++++++++++++++++++++++++++++++- 3 files changed, 103 insertions(+), 5 deletions(-) diff --git a/dlls/mshtml/htmlstyleelem.c b/dlls/mshtml/htmlstyleelem.c index ee62472..a4cc98e 100644 --- a/dlls/mshtml/htmlstyleelem.c +++ b/dlls/mshtml/htmlstyleelem.c @@ -36,6 +36,8 @@ typedef struct { HTMLElement element; const IHTMLStyleElementVtbl *lpIHTMLStyleElementVtbl; + + nsIDOMHTMLStyleElement *nsstyle; } HTMLStyleElement; #define HTMLSTYLE(x) (&(x)->lpIHTMLStyleElementVtbl) @@ -180,15 +182,33 @@ static HRESULT WINAPI HTMLStyleElement_get_disabled(IHTMLStyleElement *iface, VA static HRESULT WINAPI HTMLStyleElement_put_media(IHTMLStyleElement *iface, BSTR v) { HTMLStyleElement *This = HTMLSTYLE_THIS(iface); - FIXME("(%p)->(%s)\n", This, debugstr_w(v)); - return E_NOTIMPL; + nsAString media_str; + nsresult nsres; + + TRACE("(%p)->(%s)\n", This, debugstr_w(v)); + + nsAString_InitDepend(&media_str, v); + nsres = nsIDOMHTMLStyleElement_SetMedia(This->nsstyle, &media_str); + nsAString_Finish(&media_str); + if(NS_FAILED(nsres)) { + ERR("SetMedia failed: %08x\n", nsres); + return E_FAIL; + } + + return S_OK; } static HRESULT WINAPI HTMLStyleElement_get_media(IHTMLStyleElement *iface, BSTR *p) { HTMLStyleElement *This = HTMLSTYLE_THIS(iface); - FIXME("(%p)->(%p)\n", This, p); - return E_NOTIMPL; + nsAString nsstr; + nsresult nsres; + + TRACE("(%p)->(%p)\n", This, p); + + nsAString_Init(&nsstr, NULL); + nsres = nsIDOMHTMLStyleElement_GetMedia(This->nsstyle, &nsstr); + return return_nsstr(nsres, &nsstr, p); } #undef HTMLSTYLE_THIS @@ -244,6 +264,9 @@ static void HTMLStyleElement_destructor(HTMLDOMNode *iface) { HTMLStyleElement *This = HTMLSTYLE_NODE_THIS(iface); + if(This->nsstyle) + nsIDOMHTMLStyleElement_Release(This->nsstyle); + HTMLElement_destructor(&This->element.node); } @@ -270,6 +293,7 @@ static dispex_static_data_t HTMLStyleElement_dispex = { HRESULT HTMLStyleElement_Create(HTMLDocumentNode *doc, nsIDOMHTMLElement *nselem, HTMLElement **elem) { HTMLStyleElement *ret; + nsresult nsres; ret = heap_alloc_zero(sizeof(*ret)); if(!ret) @@ -278,6 +302,13 @@ HRESULT HTMLStyleElement_Create(HTMLDocumentNode *doc, nsIDOMHTMLElement *nselem ret->lpIHTMLStyleElementVtbl = &HTMLStyleElementVtbl; ret->element.node.vtbl = &HTMLStyleElementImplVtbl; + nsres = nsIDOMHTMLElement_QueryInterface(nselem, &IID_nsIDOMHTMLStyleElement, (void**)&ret->nsstyle); + if(NS_FAILED(nsres)) { + ERR("Could not get nsIDOMHTMLStyleElement iface: %08x\n", nsres); + heap_free(ret); + return E_FAIL; + } + HTMLElement_Init(&ret->element, doc, nselem, &HTMLStyleElement_dispex); *elem = &ret->element; return S_OK; diff --git a/dlls/mshtml/nsiface.idl b/dlls/mshtml/nsiface.idl index fc00b25..4248768 100644 --- a/dlls/mshtml/nsiface.idl +++ b/dlls/mshtml/nsiface.idl @@ -1704,6 +1704,21 @@ interface nsIDOMHTMLParamElement : nsIDOMHTMLElement [ object, + uuid(a6cf908d-15b3-11d2-932e-00805f8add32), + local +] +interface nsIDOMHTMLStyleElement : nsIDOMHTMLElement +{ + nsresult GetDisabled(PRBool *aDisabled); + nsresult SetDisabled(PRBool aDisabled); + nsresult GetMedia(nsAString *aMedia); + nsresult SetMedia(const nsAString *aMedia); + nsresult GetType(nsAString *aType); + nsresult SetType(const nsAString *aType); +} + +[ + object, uuid(94928ab3-8b63-11d3-989d-001083010e9b), local ] diff --git a/dlls/mshtml/tests/dom.c b/dlls/mshtml/tests/dom.c index c29f616..536d528 100644 --- a/dlls/mshtml/tests/dom.c +++ b/dlls/mshtml/tests/dom.c @@ -40,7 +40,7 @@ static const char range_test_str[] = static const char range_test2_str[] = "<html><body>abc<hr />123<br /><hr />def</body></html>"; static const char elem_test_str[] = - "<html><head><title>test</title><style>.body { margin-right: 0px; }</style>" + "<html><head><title>test</title><style id=\"styleid\">.body { margin-right: 0px; }</style>" "<body onload=\"Testing()\">text test" "<a id=\"a\" href=\"http://test\" name=\"x\">link</a>" "<input id=\"in\" class=\"testclass\" tabIndex=\"2\" title=\"test title\" />" @@ -787,6 +787,17 @@ static IHTMLObjectElement *_get_object_iface(unsigned line, IUnknown *unk) return obj; } +#define get_style_iface(u) _get_style_iface(__LINE__,u) +static IHTMLStyleElement *_get_style_iface(unsigned line, IUnknown *unk) +{ + IHTMLStyleElement *obj; + HRESULT hres; + + hres = IUnknown_QueryInterface(unk, &IID_IHTMLStyleElement, (void**)&obj); + ok_(__FILE__,line) (hres == S_OK, "Could not get IHTMLStyleElement: %08x\n", hres); + return obj; +} + #define test_node_name(u,n) _test_node_name(__LINE__,u,n) static void _test_node_name(unsigned line, IUnknown *unk, const char *exname) { @@ -2435,6 +2446,40 @@ static void _test_elem_set_tabindex(unsigned line, IUnknown *unk, short index) _test_elem_tabindex(line, unk, index); } +#define test_style_media(s,m) _test_style_media(__LINE__,s,m) +static void _test_style_media(unsigned line, IUnknown *unk, const char *exmedia) +{ + IHTMLStyleElement *style = _get_style_iface(line, unk); + BSTR media; + HRESULT hres; + + hres = IHTMLStyleElement_get_media(style, &media); + ok_(__FILE__,line)(hres == S_OK, "get_media failed: %08x\n", hres); + if(exmedia) + ok_(__FILE__,line)(!strcmp_wa(media, exmedia), "media = %s, expected %s\n", wine_dbgstr_w(media), exmedia); + else + ok_(__FILE__,line)(!media, "media = %s, expected NULL\n", wine_dbgstr_w(media)); + + IHTMLStyleElement_Release(style); + SysFreeString(media); +} + +#define test_style_put_media(s,m) _test_style_put_media(__LINE__,s,m) +static void _test_style_put_media(unsigned line, IUnknown *unk, const char *media) +{ + IHTMLStyleElement *style = _get_style_iface(line, unk); + BSTR str; + HRESULT hres; + + str = a2bstr(media); + hres = IHTMLStyleElement_put_media(style, str); + ok_(__FILE__,line)(hres == S_OK, "put_media failed: %08x\n", hres); + IHTMLStyleElement_Release(style); + SysFreeString(str); + + _test_style_media(line, unk, media); +} + #define test_elem_filters(u) _test_elem_filters(__LINE__,u) static void _test_elem_filters(unsigned line, IUnknown *unk) { @@ -6279,6 +6324,13 @@ static void test_elems(IHTMLDocument2 *doc) IHTMLElement_Release(elem); } + elem = get_elem_by_id(doc, "styleid", TRUE); + if(elem) { + test_style_media((IUnknown*)elem, NULL); + test_style_put_media((IUnknown*)elem, "screen"); + IHTMLElement_Release(elem); + } + elem = get_doc_elem_by_id(doc, "tbl"); ok(elem != NULL, "elem == NULL\n"); if(elem) {

1 0

Jacek Caban : mshtml: Added IHTMLStyleElement::type property implementation .
by Alexandre Julliard 17 Dec '10

17 Dec '10

Module: wine Branch: master Commit: 98e1433779e5db8e387a66723d7b230c691082f6 URL: http://source.winehq.org/git/wine.git/?a=commit;h=98e1433779e5db8e387a66723… Author: Jacek Caban <jacek(a)codeweavers.com> Date: Fri Dec 17 03:39:36 2010 +0100 mshtml: Added IHTMLStyleElement::type property implementation. --- dlls/mshtml/htmlstyleelem.c | 26 ++++++++++++++++++++++---- dlls/mshtml/tests/dom.c | 36 ++++++++++++++++++++++++++++++++++++ 2 files changed, 58 insertions(+), 4 deletions(-) diff --git a/dlls/mshtml/htmlstyleelem.c b/dlls/mshtml/htmlstyleelem.c index a4cc98e..294a9d5 100644 --- a/dlls/mshtml/htmlstyleelem.c +++ b/dlls/mshtml/htmlstyleelem.c @@ -98,15 +98,33 @@ static HRESULT WINAPI HTMLStyleElement_Invoke(IHTMLStyleElement *iface, DISPID d static HRESULT WINAPI HTMLStyleElement_put_type(IHTMLStyleElement *iface, BSTR v) { HTMLStyleElement *This = HTMLSTYLE_THIS(iface); - FIXME("(%p)->(%s)\n", This, debugstr_w(v)); - return E_NOTIMPL; + nsAString type_str; + nsresult nsres; + + TRACE("(%p)->(%s)\n", This, debugstr_w(v)); + + nsAString_InitDepend(&type_str, v); + nsres = nsIDOMHTMLStyleElement_SetType(This->nsstyle, &type_str); + nsAString_Finish(&type_str); + if(NS_FAILED(nsres)) { + ERR("SetType failed: %08x\n", nsres); + return E_FAIL; + } + + return S_OK; } static HRESULT WINAPI HTMLStyleElement_get_type(IHTMLStyleElement *iface, BSTR *p) { HTMLStyleElement *This = HTMLSTYLE_THIS(iface); - FIXME("(%p)->(%p)\n", This, p); - return E_NOTIMPL; + nsAString nsstr; + nsresult nsres; + + TRACE("(%p)->(%p)\n", This, p); + + nsAString_Init(&nsstr, NULL); + nsres = nsIDOMHTMLStyleElement_GetType(This->nsstyle, &nsstr); + return return_nsstr(nsres, &nsstr, p); } static HRESULT WINAPI HTMLStyleElement_get_readyState(IHTMLStyleElement *iface, BSTR *p) diff --git a/dlls/mshtml/tests/dom.c b/dlls/mshtml/tests/dom.c index 536d528..6601d2d 100644 --- a/dlls/mshtml/tests/dom.c +++ b/dlls/mshtml/tests/dom.c @@ -2480,6 +2480,40 @@ static void _test_style_put_media(unsigned line, IUnknown *unk, const char *medi _test_style_media(line, unk, media); } +#define test_style_type(s,m) _test_style_type(__LINE__,s,m) +static void _test_style_type(unsigned line, IUnknown *unk, const char *extype) +{ + IHTMLStyleElement *style = _get_style_iface(line, unk); + BSTR type; + HRESULT hres; + + hres = IHTMLStyleElement_get_type(style, &type); + ok_(__FILE__,line)(hres == S_OK, "get_type failed: %08x\n", hres); + if(extype) + ok_(__FILE__,line)(!strcmp_wa(type, extype), "type = %s, expected %s\n", wine_dbgstr_w(type), extype); + else + ok_(__FILE__,line)(!type, "type = %s, expected NULL\n", wine_dbgstr_w(type)); + + IHTMLStyleElement_Release(style); + SysFreeString(type); +} + +#define test_style_put_type(s,m) _test_style_put_type(__LINE__,s,m) +static void _test_style_put_type(unsigned line, IUnknown *unk, const char *type) +{ + IHTMLStyleElement *style = _get_style_iface(line, unk); + BSTR str; + HRESULT hres; + + str = a2bstr(type); + hres = IHTMLStyleElement_put_type(style, str); + ok_(__FILE__,line)(hres == S_OK, "put_type failed: %08x\n", hres); + IHTMLStyleElement_Release(style); + SysFreeString(str); + + _test_style_type(line, unk, type); +} + #define test_elem_filters(u) _test_elem_filters(__LINE__,u) static void _test_elem_filters(unsigned line, IUnknown *unk) { @@ -6328,6 +6362,8 @@ static void test_elems(IHTMLDocument2 *doc) if(elem) { test_style_media((IUnknown*)elem, NULL); test_style_put_media((IUnknown*)elem, "screen"); + test_style_type((IUnknown*)elem, NULL); + test_style_put_type((IUnknown*)elem, "text/css"); IHTMLElement_Release(elem); }

1 0