March 2010 - Wine-Commits - list.winehq.org

Damjan Jovanovic : usbd.sys: Add stub usbd.sys.
by Alexandre Julliard March 18, 2010

March 18, 2010

Module: wine Branch: master Commit: c177e938ae3eba7e967650b9edc4a1bc0c89cd45 URL: http://source.winehq.org/git/wine.git/?a=commit;h=c177e938ae3eba7e967650b9e… Author: Damjan Jovanovic <damjan.jov(a)gmail.com> Date: Thu Mar 18 09:30:43 2010 +0200 usbd.sys: Add stub usbd.sys. --- configure | 6 ++++++ configure.ac | 1 + dlls/usbd.sys/Makefile.in | 13 +++++++++++++ dlls/usbd.sys/usbd.c | 38 ++++++++++++++++++++++++++++++++++++++ dlls/usbd.sys/usbd.sys.spec | 33 +++++++++++++++++++++++++++++++++ 5 files changed, 91 insertions(+), 0 deletions(-) diff --git a/configure b/configure index 1532d88..bb09801 100755 --- a/configure +++ b/configure @@ -15613,6 +15613,11 @@ wine_fn_config_test dlls/urlmon/tests urlmon_test wine_fn_config_makefile dlls/urlmon/tests "Maketest.rules \$(MAKEDEP)" test "x$enable_tests" != xno && wine_fn_append_file ALL_TEST_DIRS "dlls/urlmon/tests" +wine_fn_append_file ALL_IMPORT_LIBS "dlls/usbd.sys/libusbd.sys.$IMPLIBEXT" +wine_fn_config_dll usbd.sys usbd.sys +wine_fn_config_makefile dlls/usbd.sys "dlls/Makedll.rules \$(MAKEDEP)" +test "x$enable_usbd_sys" != xno && wine_fn_append_file ALL_DLL_DIRS "dlls/usbd.sys" + wine_fn_config_dll user.exe16 wine_fn_config_makefile dlls/user.exe16 "dlls/Makedll.rules \$(MAKEDEP)" test "x$enable_win16" != xno && wine_fn_append_file ALL_DLL_DIRS "dlls/user.exe16" @@ -17342,6 +17347,7 @@ do "dlls/url/Makefile") CONFIG_FILES="$CONFIG_FILES dlls/url/Makefile" ;; "dlls/urlmon/Makefile") CONFIG_FILES="$CONFIG_FILES dlls/urlmon/Makefile" ;; "dlls/urlmon/tests/Makefile") CONFIG_FILES="$CONFIG_FILES dlls/urlmon/tests/Makefile" ;; + "dlls/usbd.sys/Makefile") CONFIG_FILES="$CONFIG_FILES dlls/usbd.sys/Makefile" ;; "dlls/user.exe16/Makefile") CONFIG_FILES="$CONFIG_FILES dlls/user.exe16/Makefile" ;; "dlls/user32/Makefile") CONFIG_FILES="$CONFIG_FILES dlls/user32/Makefile" ;; "dlls/user32/tests/Makefile") CONFIG_FILES="$CONFIG_FILES dlls/user32/tests/Makefile" ;; diff --git a/configure.ac b/configure.ac index 3bb3536..1decf12 100644 --- a/configure.ac +++ b/configure.ac @@ -2537,6 +2537,7 @@ WINE_CONFIG_DLL(updspapi) WINE_CONFIG_DLL(url,,[url]) WINE_CONFIG_DLL(urlmon,,[urlmon]) WINE_CONFIG_TEST(dlls/urlmon/tests) +WINE_CONFIG_DLL(usbd.sys,,[usbd.sys]) WINE_CONFIG_DLL(user.exe16,enable_win16) WINE_CONFIG_DLL(user32,,[user32]) WINE_CONFIG_TEST(dlls/user32/tests) diff --git a/dlls/usbd.sys/Makefile.in b/dlls/usbd.sys/Makefile.in new file mode 100644 index 0000000..9fe2fed --- /dev/null +++ b/dlls/usbd.sys/Makefile.in @@ -0,0 +1,13 @@ +TOPSRCDIR = @top_srcdir@ +TOPOBJDIR = ../.. +SRCDIR = @srcdir@ +VPATH = @srcdir@ +MODULE = usbd.sys +IMPORTLIB = usbd.sys +IMPORTS = kernel32 ntoskrnl.exe +EXTRADLLFLAGS = -Wb,--subsystem,native + +C_SRCS = \ + usbd.c + +(a)MAKE_DLL_RULES@ diff --git a/dlls/usbd.sys/usbd.c b/dlls/usbd.sys/usbd.c new file mode 100644 index 0000000..e734acd --- /dev/null +++ b/dlls/usbd.sys/usbd.c @@ -0,0 +1,38 @@ +/* + * Copyright (C) 2010 Damjan Jovanovic + * + * This library is free software; you can redistribute it and/or + * modify it under the terms of the GNU Lesser General Public + * License as published by the Free Software Foundation; either + * version 2.1 of the License, or (at your option) any later version. + * + * This library is distributed in the hope that it will be useful, + * but WITHOUT ANY WARRANTY; without even the implied warranty of + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU + * Lesser General Public License for more details. + * + * You should have received a copy of the GNU Lesser General Public + * License along with this library; if not, write to the Free Software + * Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301, USA + */ + +#include <stdarg.h> + +#define NONAMELESSUNION +#define NONAMELESSSTRUCT + +#include "ntstatus.h" +#define WIN32_NO_STATUS +#include "windef.h" +#include "winbase.h" +#include "winternl.h" +#include "ddk/wdm.h" +#include "wine/debug.h" + +WINE_DEFAULT_DEBUG_CHANNEL(usbd); + +NTSTATUS WINAPI DriverEntry( DRIVER_OBJECT *driver, UNICODE_STRING *path ) +{ + TRACE( "(%p, %s)\n", driver, debugstr_w(path->Buffer) ); + return STATUS_SUCCESS; +} diff --git a/dlls/usbd.sys/usbd.sys.spec b/dlls/usbd.sys/usbd.sys.spec new file mode 100644 index 0000000..c8f1d68 --- /dev/null +++ b/dlls/usbd.sys/usbd.sys.spec @@ -0,0 +1,33 @@ +@ stub USBD_CreateConfigurationRequestEx +@ stub USBD_ParseConfigurationDescriptorEx +@ stub USBD_ParseDescriptors +@ stub USBD_AllocateDeviceName +@ stub USBD_CalculateUsbBandwidth +@ stub USBD_CompleteRequest +@ stub USBD_CreateConfigurationRequest +@ stub _USBD_CreateConfigurationRequestEx(a)8 +@ stub USBD_CreateDevice +@ stub USBD_Debug_GetHeap +@ stub USBD_Debug_LogEntry +@ stub USBD_Debug_RetHeap +@ stub USBD_Dispatch +@ stub USBD_FreeDeviceMutex +@ stub USBD_FreeDeviceName +@ stub USBD_GetDeviceInformation +@ stub USBD_GetInterfaceLength +@ stub USBD_GetPdoRegistryParameter +@ stub USBD_GetSuspendPowerState +@ stub USBD_GetUSBDIVersion +@ stub USBD_InitializeDevice +@ stub USBD_MakePdoName +@ stub USBD_ParseConfigurationDescriptor +@ stub _USBD_ParseConfigurationDescriptorEx(a)28 +@ stub _USBD_ParseDescriptors(a)16 +@ stub USBD_QueryBusTime +@ stub USBD_RegisterHcDeviceCapabilities +@ stub USBD_RegisterHcFilter +@ stub USBD_RegisterHostController +@ stub USBD_RemoveDevice +@ stub USBD_RestoreDevice +@ stub USBD_SetSuspendPowerState +@ stub USBD_WaitDeviceMutex

1 0

Maarten Lankhorst : winmm: Allow SND_ALIAS|SND_FILENAME in PlaySound.
by Alexandre Julliard March 18, 2010

March 18, 2010

Module: wine Branch: master Commit: b0337ada1abce28d86d0cd063b494b8ad66e9d73 URL: http://source.winehq.org/git/wine.git/?a=commit;h=b0337ada1abce28d86d0cd063… Author: Maarten Lankhorst <m.b.lankhorst(a)gmail.com> Date: Wed Mar 17 23:41:09 2010 +0100 winmm: Allow SND_ALIAS|SND_FILENAME in PlaySound. --- dlls/winmm/playsound.c | 7 ++++--- 1 files changed, 4 insertions(+), 3 deletions(-) diff --git a/dlls/winmm/playsound.c b/dlls/winmm/playsound.c index 54db68f..7b6c946 100644 --- a/dlls/winmm/playsound.c +++ b/dlls/winmm/playsound.c @@ -200,6 +200,7 @@ static BOOL PlaySound_IsString(DWORD fdwSound, const void* psz) case SND_MEMORY: return FALSE; case SND_ALIAS: case SND_FILENAME: + case SND_ALIAS|SND_FILENAME: case 0: return TRUE; default: FIXME("WTF\n"); return FALSE; } @@ -308,7 +309,7 @@ static DWORD WINAPI proc_PlaySound(LPVOID arg) TRACE("Memory sound %p\n", data); hmmio = mmioOpenW(NULL, &mminfo, MMIO_READ); } - else if (wps->fdwSound & SND_ALIAS) + if (!hmmio && wps->fdwSound & SND_ALIAS) { if ((wps->fdwSound & SND_ALIAS_ID) == SND_ALIAS_ID) { @@ -342,11 +343,11 @@ static DWORD WINAPI proc_PlaySound(LPVOID arg) } hmmio = get_mmioFromProfile(wps->fdwSound, wps->pszSound); } - else if (wps->fdwSound & SND_FILENAME) + if (!hmmio && wps->fdwSound & SND_FILENAME) { hmmio = get_mmioFromFile(wps->pszSound); } - else + if (!(wps->fdwSound & (SND_FILENAME|SND_ALIAS|SND_MEMORY))) { if ((hmmio = get_mmioFromProfile(wps->fdwSound | SND_NODEFAULT, wps->pszSound)) == 0) {

1 0

Alexandre Julliard : user32/tests: Add some tests demonstrating that SetCursor uses the thread input data.
by Alexandre Julliard March 18, 2010

March 18, 2010

Module: wine Branch: master Commit: 5a18e2a925144e410517463b1c2bf1238615414e URL: http://source.winehq.org/git/wine.git/?a=commit;h=5a18e2a925144e410517463b1… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Thu Mar 18 13:03:39 2010 +0100 user32/tests: Add some tests demonstrating that SetCursor uses the thread input data. --- dlls/user32/tests/cursoricon.c | 85 ++++++++++++++++++++++++++++++++++++++++ 1 files changed, 85 insertions(+), 0 deletions(-) diff --git a/dlls/user32/tests/cursoricon.c b/dlls/user32/tests/cursoricon.c index ff86cd0..1ebe665 100644 --- a/dlls/user32/tests/cursoricon.c +++ b/dlls/user32/tests/cursoricon.c @@ -1353,6 +1353,90 @@ cleanup: DeleteDC(hdcDst); } +static DWORD parent_id; + +static DWORD CALLBACK set_cursor_thread( void *arg ) +{ + HCURSOR ret; + + PeekMessage( 0, 0, 0, 0, PM_NOREMOVE ); /* create a msg queue */ + if (parent_id) + { + BOOL ret = AttachThreadInput( GetCurrentThreadId(), parent_id, TRUE ); + ok( ret, "AttachThreadInput failed\n" ); + } + if (arg) ret = SetCursor( (HCURSOR)arg ); + else ret = GetCursor(); + return (DWORD_PTR)ret; +} + +static void test_SetCursor(void) +{ + static const BYTE bmp_bits[4096]; + ICONINFO cursorInfo; + HCURSOR cursor, old_cursor; + DWORD error, id, result; + UINT display_bpp; + HDC hdc; + HANDLE thread; + + cursor = GetCursor(); + thread = CreateThread( NULL, 0, set_cursor_thread, 0, 0, &id ); + WaitForSingleObject( thread, 1000 ); + GetExitCodeThread( thread, &result ); + ok( result == (DWORD_PTR)cursor, "wrong thread cursor %x/%p\n", result, cursor ); + + hdc = GetDC(0); + display_bpp = GetDeviceCaps(hdc, BITSPIXEL); + ReleaseDC(0, hdc); + + cursorInfo.fIcon = FALSE; + cursorInfo.xHotspot = 0; + cursorInfo.yHotspot = 0; + cursorInfo.hbmMask = CreateBitmap(32, 32, 1, 1, bmp_bits); + cursorInfo.hbmColor = CreateBitmap(32, 32, 1, display_bpp, bmp_bits); + + cursor = CreateIconIndirect(&cursorInfo); + ok(cursor != NULL, "CreateIconIndirect returned %p\n", cursor); + old_cursor = SetCursor( cursor ); + + thread = CreateThread( NULL, 0, set_cursor_thread, 0, 0, &id ); + WaitForSingleObject( thread, 1000 ); + GetExitCodeThread( thread, &result ); + ok( result == (DWORD_PTR)old_cursor, "wrong thread cursor %x/%p\n", result, old_cursor ); + + SetCursor( 0 ); + ok( GetCursor() == 0, "wrong cursor %p\n", GetCursor() ); + thread = CreateThread( NULL, 0, set_cursor_thread, 0, 0, &id ); + WaitForSingleObject( thread, 1000 ); + GetExitCodeThread( thread, &result ); + ok( result == (DWORD_PTR)old_cursor, "wrong thread cursor %x/%p\n", result, old_cursor ); + + thread = CreateThread( NULL, 0, set_cursor_thread, cursor, 0, &id ); + WaitForSingleObject( thread, 1000 ); + GetExitCodeThread( thread, &result ); + ok( result == (DWORD_PTR)old_cursor, "wrong thread cursor %x/%p\n", result, old_cursor ); + ok( GetCursor() == 0, "wrong cursor %p/0\n", GetCursor() ); + + parent_id = GetCurrentThreadId(); + thread = CreateThread( NULL, 0, set_cursor_thread, cursor, 0, &id ); + WaitForSingleObject( thread, 1000 ); + GetExitCodeThread( thread, &result ); + ok( result == (DWORD_PTR)old_cursor, "wrong thread cursor %x/%p\n", result, old_cursor ); + todo_wine ok( GetCursor() == cursor, "wrong cursor %p/0\n", cursor ); + + SetCursor( old_cursor ); + DestroyCursor( cursor ); + + SetLastError( 0xdeadbeef ); + cursor = SetCursor( (HCURSOR)0xbadbad ); + error = GetLastError(); + ok( cursor == 0, "wrong cursor %p/0\n", cursor ); + todo_wine ok( error == ERROR_INVALID_CURSOR_HANDLE || broken( error == 0xdeadbeef ), /* win9x */ + "wrong error %u\n", error ); +} + + static void test_DestroyCursor(void) { static const BYTE bmp_bits[4096]; @@ -1463,6 +1547,7 @@ START_TEST(cursoricon) test_DrawIcon(); test_DrawIconEx(); test_DrawState(); + test_SetCursor(); test_DestroyCursor(); do_parent(); test_child_process();

1 0

Alexandre Julliard : kernel32: Don' t try to open builtin exe files during CreateProcess, only check the path syntax.
by Alexandre Julliard March 18, 2010

March 18, 2010

Module: wine Branch: master Commit: c7bed19c0b71188799eff67f562e9072de4e0dd0 URL: http://source.winehq.org/git/wine.git/?a=commit;h=c7bed19c0b71188799eff67f5… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Thu Mar 18 13:02:42 2010 +0100 kernel32: Don't try to open builtin exe files during CreateProcess, only check the path syntax. --- dlls/kernel32/process.c | 86 +++++++++++++++++----------------------------- 1 files changed, 32 insertions(+), 54 deletions(-) diff --git a/dlls/kernel32/process.c b/dlls/kernel32/process.c index d9d4520..115cd05 100644 --- a/dlls/kernel32/process.c +++ b/dlls/kernel32/process.c @@ -90,6 +90,7 @@ const WCHAR *DIR_SysWow64 = NULL; #define PDB32_FILE_APIS_OEM 0x0040 /* File APIs are OEM */ #define PDB32_WIN32S_PROC 0x8000 /* Win32s process */ +static const WCHAR exeW[] = {'.','e','x','e',0}; static const WCHAR comW[] = {'.','c','o','m',0}; static const WCHAR batW[] = {'.','b','a','t',0}; static const WCHAR cmdW[] = {'.','c','m','d',0}; @@ -152,6 +153,9 @@ static BOOL get_builtin_path( const WCHAR *libname, const WCHAR *ext, WCHAR *fil void *redir_disabled = 0; unsigned int flags = (sizeof(void*) > sizeof(int) ? BINARY_FLAG_64BIT : 0); + /* builtin names cannot be empty or contain spaces */ + if (!libname[0] || strchrW( libname, ' ' ) || strchrW( libname, '\t' )) return FALSE; + if (is_wow64 && Wow64DisableWow64FsRedirection( &redir_disabled )) Wow64RevertWow64FsRedirection( redir_disabled ); @@ -198,35 +202,6 @@ static BOOL get_builtin_path( const WCHAR *libname, const WCHAR *ext, WCHAR *fil /*********************************************************************** - * open_builtin_exe_file - * - * Open an exe file for a builtin exe. - */ -static void *open_builtin_exe_file( const WCHAR *name, char *error, int error_size, - int test_only, int *file_exists ) -{ - char exename[MAX_PATH]; - WCHAR *p; - UINT i, len; - - *file_exists = 0; - if ((p = strrchrW( name, '/' ))) name = p + 1; - if ((p = strrchrW( name, '\\' ))) name = p + 1; - - /* we don't want to depend on the current codepage here */ - len = strlenW( name ) + 1; - if (len >= sizeof(exename)) return NULL; - for (i = 0; i < len; i++) - { - if (name[i] > 127) return NULL; - exename[i] = (char)name[i]; - if (exename[i] >= 'A' && exename[i] <= 'Z') exename[i] += 'a' - 'A'; - } - return wine_dll_load_main_exe( exename, error, error_size, test_only, file_exists ); -} - - -/*********************************************************************** * open_exe_file * * Open a specific exe file, taking load order into account. @@ -263,27 +238,16 @@ static HANDLE open_exe_file( const WCHAR *name, struct binary_info *binary_info static BOOL find_exe_file( const WCHAR *name, WCHAR *buffer, int buflen, HANDLE *handle, struct binary_info *binary_info ) { - static const WCHAR exeW[] = {'.','e','x','e',0}; - int file_exists; - TRACE("looking for %s\n", debugstr_w(name) ); if (!SearchPathW( NULL, name, exeW, buflen, buffer, NULL )) { - if (get_builtin_path( name, exeW, buffer, buflen, binary_info )) + if (contains_path( name ) && get_builtin_path( name, exeW, buffer, buflen, binary_info )) { - TRACE( "Trying built-in exe %s\n", debugstr_w(buffer) ); - open_builtin_exe_file( buffer, NULL, 0, 1, &file_exists ); - if (file_exists) - { - *handle = 0; - return TRUE; - } - return FALSE; + *handle = 0; + return TRUE; } - /* no builtin found, try native without extension in case it is a Unix app */ - if (!SearchPathW( NULL, name, NULL, buflen, buffer, NULL )) return FALSE; } @@ -1075,7 +1039,6 @@ void CDECL __wine_kernel_init(void) { static const WCHAR kernel32W[] = {'k','e','r','n','e','l','3','2',0}; static const WCHAR dotW[] = {'.',0}; - static const WCHAR exeW[] = {'.','e','x','e',0}; WCHAR *p, main_exe_name[MAX_PATH+1]; PEB *peb = NtCurrentTeb()->Peb; @@ -1181,6 +1144,8 @@ void CDECL __wine_kernel_init(void) debugstr_w(__wine_main_wargv[3]) ); ExitProcess( ERROR_BAD_EXE_FORMAT ); } + MESSAGE( "wine: cannot find %s\n", debugstr_w(main_exe_name) ); + ExitProcess( ERROR_FILE_NOT_FOUND ); } args[0] = (DWORD_PTR)main_exe_name; FormatMessageW( FORMAT_MESSAGE_FROM_SYSTEM | FORMAT_MESSAGE_ARGUMENT_ARRAY, @@ -1898,9 +1863,8 @@ static LPWSTR get_file_name( LPCWSTR appname, LPWSTR cmdline, LPWSTR buffer, { static const WCHAR quotesW[] = {'"','%','s','"',0}; - WCHAR *name, *pos, *ret = NULL; + WCHAR *name, *pos, *first_space, *ret = NULL; const WCHAR *p; - BOOL got_space; /* if we have an app name, everything is easy */ @@ -1928,8 +1892,12 @@ static LPWSTR get_file_name( LPCWSTR appname, LPWSTR cmdline, LPWSTR buffer, memcpy( name, cmdline + 1, len * sizeof(WCHAR) ); name[len] = 0; - if (find_exe_file( name, buffer, buflen, handle, binary_info )) - ret = cmdline; /* no change necessary */ + if (!find_exe_file( name, buffer, buflen, handle, binary_info )) + { + if (!get_builtin_path( name, exeW, buffer, buflen, binary_info )) goto done; + *handle = 0; + } + ret = cmdline; /* no change necessary */ goto done; } @@ -1939,28 +1907,38 @@ static LPWSTR get_file_name( LPCWSTR appname, LPWSTR cmdline, LPWSTR buffer, return NULL; pos = name; p = cmdline; - got_space = FALSE; + first_space = NULL; - while (*p) + for (;;) { - do *pos++ = *p++; while (*p && *p != ' ' && *p != '\t'); + while (*p && *p != ' ' && *p != '\t') *pos++ = *p++; *pos = 0; if (find_exe_file( name, buffer, buflen, handle, binary_info )) { ret = cmdline; break; } - if (*p) got_space = TRUE; + if (!(*pos++ = *p++)) break; + if (!first_space) first_space = pos; } - if (ret && got_space) /* now build a new command-line with quotes */ + if (!ret) + { + if (first_space) *first_space = 0; /* try only the first word as a builtin */ + if (get_builtin_path( name, exeW, buffer, buflen, binary_info )) + { + *handle = 0; + ret = cmdline; + } + else SetLastError( ERROR_FILE_NOT_FOUND ); + } + else if (first_space) /* build a new command-line with quotes */ { if (!(ret = HeapAlloc( GetProcessHeap(), 0, (strlenW(cmdline) + 3) * sizeof(WCHAR) ))) goto done; sprintfW( ret, quotesW, name ); strcatW( ret, p ); } - else if (!ret) SetLastError( ERROR_FILE_NOT_FOUND ); done: HeapFree( GetProcessHeap(), 0, name );

1 0

Jacek Caban : mshtml: Added IHTMLImgElement::onload tests.
by Alexandre Julliard March 18, 2010

March 18, 2010

Module: wine Branch: master Commit: e7615e004a85160f70cabec977733801a2831691 URL: http://source.winehq.org/git/wine.git/?a=commit;h=e7615e004a85160f70cabec97… Author: Jacek Caban <jacek(a)codeweavers.com> Date: Thu Mar 18 01:06:28 2010 +0100 mshtml: Added IHTMLImgElement::onload tests. --- dlls/mshtml/tests/events.c | 60 +++++++++++++++++++++++++++++++++++++++++++- 1 files changed, 59 insertions(+), 1 deletions(-) diff --git a/dlls/mshtml/tests/events.c b/dlls/mshtml/tests/events.c index 55171be..d62df7a 100644 --- a/dlls/mshtml/tests/events.c +++ b/dlls/mshtml/tests/events.c @@ -68,6 +68,7 @@ DEFINE_EXPECT(iframe_onreadystatechange_loading); DEFINE_EXPECT(iframe_onreadystatechange_interactive); DEFINE_EXPECT(iframe_onreadystatechange_complete); DEFINE_EXPECT(iframedoc_onreadystatechange); +DEFINE_EXPECT(img_onload); static HWND container_hwnd = NULL; static IHTMLWindow2 *window; @@ -93,7 +94,10 @@ static const char click_doc_str[] = "</body></html>"; static const char readystate_doc_str[] = - "<<html><body><iframe id=\"iframe\"></iframe></body></html>"; + "<html><body><iframe id=\"iframe\"></iframe></body></html>"; + +static const char img_doc_str[] = + "<html><body><img id=\"imgid\"></img></body></html>"; static const char *debugstr_guid(REFIID riid) { @@ -880,6 +884,17 @@ static HRESULT WINAPI body_onclick(IDispatchEx *iface, DISPID id, LCID lcid, WOR EVENT_HANDLER_FUNC_OBJ(body_onclick); +static HRESULT WINAPI img_onload(IDispatchEx *iface, DISPID id, LCID lcid, WORD wFlags, DISPPARAMS *pdp, + VARIANT *pvarRes, EXCEPINFO *pei, IServiceProvider *pspCaller) +{ + CHECK_EXPECT(img_onload); + test_event_args(&DIID_DispHTMLImg, id, wFlags, pdp, pvarRes, pei, pspCaller); + test_event_src("IMG"); + return S_OK; +} + +EVENT_HANDLER_FUNC_OBJ(img_onload); + static HRESULT WINAPI iframedoc_onreadystatechange(IDispatchEx *iface, DISPID id, LCID lcid, WORD wFlags, DISPPARAMS *pdp, VARIANT *pvarRes, EXCEPINFO *pei, IServiceProvider *pspCaller) { @@ -1359,6 +1374,48 @@ static void test_onreadystatechange(IHTMLDocument2 *doc) IHTMLFrameBase_Release(iframe); } +static void test_imgload(IHTMLDocument2 *doc) +{ + IHTMLImgElement *img; + IHTMLElement *elem; + VARIANT v; + BSTR str; + HRESULT hres; + + elem = get_elem_id(doc, "imgid"); + hres = IHTMLElement_QueryInterface(elem, &IID_IHTMLImgElement, (void**)&img); + IHTMLElement_Release(elem); + ok(hres == S_OK, "Could not get IHTMLImgElement iface: %08x\n", hres); + + V_VT(&v) = VT_EMPTY; + hres = IHTMLImgElement_get_onload(img, &v); + ok(hres == S_OK, "get_onload failed: %08x\n", hres); + ok(V_VT(&v) == VT_NULL, "V_VT(onload) = %d\n", V_VT(&v)); + + V_VT(&v) = VT_DISPATCH; + V_DISPATCH(&v) = (IDispatch*)&img_onload_obj; + hres = IHTMLImgElement_put_onload(img, v); + ok(hres == S_OK, "put_onload failed: %08x\n", hres); + + V_VT(&v) = VT_EMPTY; + hres = IHTMLImgElement_get_onload(img, &v); + ok(hres == S_OK, "get_onload failed: %08x\n", hres); + ok(V_VT(&v) == VT_DISPATCH, "V_VT(onload) = %d\n", V_VT(&v)); + ok(V_DISPATCH(&v) == (IDispatch*)&img_onload_obj, "V_DISPATCH(onload) != onloadkFunc\n"); + VariantClear(&v); + + str = a2bstr("http://www.winehq.org/images/winehq_logo_text.png"); + hres = IHTMLImgElement_put_src(img, str); + ok(hres == S_OK, "put_src failed: %08x\n", hres); + SysFreeString(str); + + SET_EXPECT(img_onload); + pump_msgs(&called_img_onload); + CHECK_CALLED(img_onload); + + IHTMLImgElement_Release(img); +} + static void test_timeout(IHTMLDocument2 *doc) { IHTMLWindow3 *win3; @@ -1998,6 +2055,7 @@ START_TEST(events) run_test(empty_doc_str, test_timeout); run_test(click_doc_str, test_onclick); run_test(readystate_doc_str, test_onreadystatechange); + run_test(img_doc_str, test_imgload); DestroyWindow(container_hwnd); CoUninitialize();

1 0

Jacek Caban : cmd: Added @or_broken@ handling to test templates and use it to fix tests on win2k.
by Alexandre Julliard March 18, 2010

March 18, 2010

Module: wine Branch: master Commit: e486a02157f145ccd5853d73cf5b3514d41be9d1 URL: http://source.winehq.org/git/wine.git/?a=commit;h=e486a02157f145ccd5853d73c… Author: Jacek Caban <jacek(a)codeweavers.com> Date: Thu Mar 18 01:28:19 2010 +0100 cmd: Added @or_broken@ handling to test templates and use it to fix tests on win2k. --- programs/cmd/tests/batch.c | 39 ++++++++++++++++++++++++----- programs/cmd/tests/test_builtins.cmd | 1 + programs/cmd/tests/test_builtins.cmd.exp | 1 + 3 files changed, 34 insertions(+), 7 deletions(-) diff --git a/programs/cmd/tests/batch.c b/programs/cmd/tests/batch.c index ca40ada..e31fdc0 100644 --- a/programs/cmd/tests/batch.c +++ b/programs/cmd/tests/batch.c @@ -101,9 +101,11 @@ static const char *compare_line(const char *out_line, const char *out_end, const const char *exp_end) { const char *out_ptr = out_line, *exp_ptr = exp_line; + const char *err = NULL; static const char pwd_cmd[] = {'@','p','w','d','@'}; static const char todo_space_cmd[] = {'@','t','o','d','o','_','s','p','a','c','e','@'}; + static const char or_broken_cmd[] = {'@','o','r','_','b','r','o','k','e','n','@'}; while(exp_ptr < exp_end) { if(*exp_ptr == '@') { @@ -111,11 +113,13 @@ static const char *compare_line(const char *out_line, const char *out_end, const && !memcmp(exp_ptr, pwd_cmd, sizeof(pwd_cmd))) { exp_ptr += sizeof(pwd_cmd); if(out_end-out_ptr < workdir_len - || (CompareStringA(LOCALE_SYSTEM_DEFAULT, NORM_IGNORECASE, out_ptr, workdir_len, - workdir, workdir_len) != CSTR_EQUAL)) - return out_ptr; - out_ptr += workdir_len; - continue; + || (CompareStringA(LOCALE_SYSTEM_DEFAULT, NORM_IGNORECASE, out_ptr, workdir_len, + workdir, workdir_len) != CSTR_EQUAL)) { + err = out_ptr; + }else { + out_ptr += workdir_len; + continue; + } }else if(exp_ptr+sizeof(todo_space_cmd) <= exp_end && !memcmp(exp_ptr, todo_space_cmd, sizeof(todo_space_cmd))) { exp_ptr += sizeof(todo_space_cmd); @@ -123,10 +127,31 @@ static const char *compare_line(const char *out_line, const char *out_end, const if(out_ptr < out_end && *out_ptr == ' ') out_ptr++; continue; + }else if(exp_ptr+sizeof(or_broken_cmd) <= exp_end + && !memcmp(exp_ptr, or_broken_cmd, sizeof(or_broken_cmd))) { + exp_ptr = exp_end; + continue; } + }else if(out_ptr == out_end || *out_ptr != *exp_ptr) { + err = out_ptr; } - if(out_ptr == out_end || *out_ptr != *exp_ptr++) - return out_ptr; + + if(err) { + if(!broken(1)) + return err; + + while(exp_ptr+sizeof(or_broken_cmd) <= exp_end && memcmp(exp_ptr, or_broken_cmd, sizeof(or_broken_cmd))) + exp_ptr++; + if(!exp_ptr) + return err; + + exp_ptr += sizeof(or_broken_cmd); + out_ptr = out_line; + err = NULL; + continue; + } + + exp_ptr++; out_ptr++; } diff --git a/programs/cmd/tests/test_builtins.cmd b/programs/cmd/tests/test_builtins.cmd index 696ce13..bae19b8 100644 --- a/programs/cmd/tests/test_builtins.cmd +++ b/programs/cmd/tests/test_builtins.cmd @@ -23,6 +23,7 @@ cd dummydir echo %~dp0 cd .. rmdir dummydir +echo CD value %CD% echo %% echo P% echo %P diff --git a/programs/cmd/tests/test_builtins.cmd.exp b/programs/cmd/tests/test_builtins.cmd.exp index 55fc9f2..ad50358 100644 --- a/programs/cmd/tests/test_builtins.cmd.exp +++ b/programs/cmd/tests/test_builtins.cmd.exp @@ -16,6 +16,7 @@ bar ~dp0 should be directory containing batch file @pwd@\ @pwd@\ +CD value @pwd@@or_broken(a)CD value % P P

1 0

Jacek Caban : mshtml: Added IHTMLImgElement::onload property implementation .
by Alexandre Julliard March 18, 2010

March 18, 2010

Module: wine Branch: master Commit: 21f0c64de48e4a744ad12a2b614d1ccaac75de1e URL: http://source.winehq.org/git/wine.git/?a=commit;h=21f0c64de48e4a744ad12a2b6… Author: Jacek Caban <jacek(a)codeweavers.com> Date: Thu Mar 18 01:06:05 2010 +0100 mshtml: Added IHTMLImgElement::onload property implementation. --- dlls/mshtml/htmlimg.c | 13 +++++++++---- 1 files changed, 9 insertions(+), 4 deletions(-) diff --git a/dlls/mshtml/htmlimg.c b/dlls/mshtml/htmlimg.c index 17b6cce..a4448f9 100644 --- a/dlls/mshtml/htmlimg.c +++ b/dlls/mshtml/htmlimg.c @@ -28,6 +28,7 @@ #include "wine/debug.h" #include "mshtml_private.h" +#include "htmlevent.h" WINE_DEFAULT_DEBUG_CHANNEL(mshtml); @@ -388,15 +389,19 @@ static HRESULT WINAPI HTMLImgElement_get_align(IHTMLImgElement *iface, BSTR *p) static HRESULT WINAPI HTMLImgElement_put_onload(IHTMLImgElement *iface, VARIANT v) { HTMLImgElement *This = HTMLIMG_THIS(iface); - FIXME("(%p)->()\n", This); - return E_NOTIMPL; + + TRACE("(%p)->(%s)\n", This, debugstr_variant(&v)); + + return set_node_event(&This->element.node, EVENTID_LOAD, &v); } static HRESULT WINAPI HTMLImgElement_get_onload(IHTMLImgElement *iface, VARIANT *p) { HTMLImgElement *This = HTMLIMG_THIS(iface); - FIXME("(%p)->(%p)\n", This, p); - return E_NOTIMPL; + + TRACE("(%p)->(%p)\n", This, p); + + return get_node_event(&This->element.node, EVENTID_LOAD, p); } static HRESULT WINAPI HTMLImgElement_put_onerror(IHTMLImgElement *iface, VARIANT v)

1 0

Jacek Caban : mshtml: Added support for load event on all nodes.
by Alexandre Julliard March 18, 2010

March 18, 2010

Module: wine Branch: master Commit: df0a35e3d32d7f3621d7ffcf00810015c461fe65 URL: http://source.winehq.org/git/wine.git/?a=commit;h=df0a35e3d32d7f3621d7ffcf0… Author: Jacek Caban <jacek(a)codeweavers.com> Date: Thu Mar 18 01:05:38 2010 +0100 mshtml: Added support for load event on all nodes. --- dlls/mshtml/htmldoc.c | 2 +- dlls/mshtml/htmldoc3.c | 2 +- dlls/mshtml/htmlelem2.c | 2 +- dlls/mshtml/htmlevent.c | 64 +++++++++++++++++++++++++++++++++-------- dlls/mshtml/htmlevent.h | 8 ++-- dlls/mshtml/htmlwindow.c | 4 +- dlls/mshtml/mshtml_private.h | 2 +- dlls/mshtml/nsevents.c | 9 ++++- 8 files changed, 68 insertions(+), 25 deletions(-) Diff: http://source.winehq.org/git/wine.git/?a=commitdiff;h=df0a35e3d32d7f3621d7f…

1 0

Nikolay Sivov : shlwapi: Add a forward to DelayLoadFailureHook.
by Alexandre Julliard March 18, 2010

March 18, 2010

Module: wine Branch: master Commit: d16bf5f9ba95394dff19bfe38e67238b91bac866 URL: http://source.winehq.org/git/wine.git/?a=commit;h=d16bf5f9ba95394dff19bfe38… Author: Nikolay Sivov <nsivov(a)codeweavers.com> Date: Thu Mar 18 02:22:50 2010 +0300 shlwapi: Add a forward to DelayLoadFailureHook. --- dlls/shlwapi/shlwapi.spec | 1 + 1 files changed, 1 insertions(+), 0 deletions(-) diff --git a/dlls/shlwapi/shlwapi.spec b/dlls/shlwapi/shlwapi.spec index 6047646..d899232 100644 --- a/dlls/shlwapi/shlwapi.spec +++ b/dlls/shlwapi/shlwapi.spec @@ -559,6 +559,7 @@ @ stdcall ColorAdjustLuma(long long long) @ stdcall ColorHLSToRGB(long long long) @ stdcall ColorRGBToHLS(long ptr ptr ptr) +@ stdcall DelayLoadFailureHook(str str) kernel32.DelayLoadFailureHook @ stdcall -private DllGetVersion(ptr) @ stdcall GetMenuPosFromID(ptr long) @ stdcall HashData (ptr long ptr long)

1 0

Jacek Caban : mshtml: Make sure that readState is READYSTATE_LOADING in read_stream_data.
by Alexandre Julliard March 18, 2010

March 18, 2010

Module: wine Branch: master Commit: 2b94ad52871294847b34304c3ba5b08adcc9b481 URL: http://source.winehq.org/git/wine.git/?a=commit;h=2b94ad52871294847b34304c3… Author: Jacek Caban <jacek(a)codeweavers.com> Date: Thu Mar 18 01:04:15 2010 +0100 mshtml: Make sure that readState is READYSTATE_LOADING in read_stream_data. --- dlls/mshtml/navigate.c | 5 ++++- dlls/mshtml/tests/events.c | 1 - 2 files changed, 4 insertions(+), 2 deletions(-) diff --git a/dlls/mshtml/navigate.c b/dlls/mshtml/navigate.c index a09ea7f..cc2aeb3 100644 --- a/dlls/mshtml/navigate.c +++ b/dlls/mshtml/navigate.c @@ -972,8 +972,11 @@ static HRESULT read_stream_data(nsChannelBSC *This, IStream *stream) on_start_nsrequest(This); - if(This->window) + if(This->window) { update_window_doc(This->window); + if(This->window->readystate != READYSTATE_LOADING) + set_ready_state(This->window, READYSTATE_LOADING); + } } This->bsc.readed += This->nsstream->buf_size; diff --git a/dlls/mshtml/tests/events.c b/dlls/mshtml/tests/events.c index 16f727f..d2d1db7 100644 --- a/dlls/mshtml/tests/events.c +++ b/dlls/mshtml/tests/events.c @@ -1310,7 +1310,6 @@ static void test_onreadystatechange(IHTMLDocument2 *doc) SET_EXPECT(iframe_onreadystatechange_interactive); SET_EXPECT(iframe_onreadystatechange_complete); pump_msgs(&called_iframe_onreadystatechange_complete); - todo_wine CHECK_CALLED(iframe_onreadystatechange_loading); CHECK_CALLED(iframe_onreadystatechange_interactive); CHECK_CALLED(iframe_onreadystatechange_complete);

1 0