wine-commits September 2012

wine-commits@winehq.org

2 participants
828 discussions

Dmitry Timoshkov : gdiplus: Add more tests for GIF image properties.
by Alexandre Julliard 21 Sep '12

21 Sep '12

Module: wine Branch: master Commit: f39c140a88a73b321770fbef954d6af42cbe267d URL: http://source.winehq.org/git/wine.git/?a=commit;h=f39c140a88a73b321770fbef9… Author: Dmitry Timoshkov <dmitry(a)baikal.ru> Date: Fri Sep 21 19:57:08 2012 +0900 gdiplus: Add more tests for GIF image properties. --- dlls/gdiplus/tests/image.c | 81 ++++++++++++++++++++++++++++++++++++++++---- 1 files changed, 74 insertions(+), 7 deletions(-) diff --git a/dlls/gdiplus/tests/image.c b/dlls/gdiplus/tests/image.c index a26c4f5..59a5064 100644 --- a/dlls/gdiplus/tests/image.c +++ b/dlls/gdiplus/tests/image.c @@ -3961,8 +3961,10 @@ static void test_gif_properties(void) GpImage *image; GUID guid; UINT dim_count, frame_count, prop_count, prop_size, i; + UINT total_size, total_count; PROPID *prop_id; PropertyItem *prop_item; + const char *item_data; image = load_image(animatedgif, sizeof(animatedgif)); if (!image) /* XP fails to load this GIF image */ @@ -4003,24 +4005,27 @@ todo_wine status = GdipGetPropertyIdList(image, prop_count, prop_id); expect(Ok, status); + prop_size = 0; for (i = 0; i < prop_count; i++) { - status = GdipGetPropertyItemSize(image, prop_id[i], &prop_size); + UINT size; + status = GdipGetPropertyItemSize(image, prop_id[i], &size); expect(Ok, status); if (status != Ok) break; - ok(prop_size > sizeof(*prop_item), "%u: too small item length %u\n", i, prop_size); + ok(size > sizeof(*prop_item), "%u: too small item length %u\n", i, size); - prop_item = HeapAlloc(GetProcessHeap(), HEAP_ZERO_MEMORY, prop_size); - status = GdipGetPropertyItem(image, prop_id[i], prop_size, prop_item); + prop_size += size; + + prop_item = HeapAlloc(GetProcessHeap(), HEAP_ZERO_MEMORY, size); + status = GdipGetPropertyItem(image, prop_id[i], size, prop_item); expect(Ok, status); ok(prop_item->value == prop_item + 1, "expected item->value %p, got %p\n", prop_item + 1, prop_item->value); ok(td[i].type == prop_item->type, "%u: expected type %u, got %u\n", i, td[i].type, prop_item->type); ok(td[i].id == prop_item->id, "%u: expected id %#x, got %#x\n", i, td[i].id, prop_item->id); - prop_size -= sizeof(*prop_item); - ok(prop_item->length == prop_size, "%u: expected length %u, got %u\n", i, prop_size, prop_item->length); + size -= sizeof(*prop_item); + ok(prop_item->length == size, "%u: expected length %u, got %u\n", i, size, prop_item->length); ok(td[i].length == prop_item->length, "%u: expected length %u, got %u\n", i, td[i].length, prop_item->length); - ok(td[i].length == prop_size, "%u: expected length %u, got %u\n", i, td[i].length, prop_size); if (td[i].length == prop_item->length) { int match = memcmp(td[i].value, prop_item->value, td[i].length) == 0; @@ -4040,6 +4045,68 @@ todo_wine HeapFree(GetProcessHeap(), 0, prop_id); + status = GdipGetPropertySize(NULL, &total_size, &total_count); + expect(InvalidParameter, status); + status = GdipGetPropertySize(image, &total_size, NULL); + expect(InvalidParameter, status); + status = GdipGetPropertySize(image, NULL, &total_count); + expect(InvalidParameter, status); + status = GdipGetPropertySize(image, NULL, NULL); + expect(InvalidParameter, status); + total_size = 0xdeadbeef; + total_count = 0xdeadbeef; + status = GdipGetPropertySize(image, &total_size, &total_count); + expect(Ok, status); + ok(prop_count == total_count, + "expected total property count %u, got %u\n", prop_count, total_count); + ok(prop_size == total_size, + "expected total property size %u, got %u\n", prop_size, total_size); + + prop_item = HeapAlloc(GetProcessHeap(), 0, prop_size); + + status = GdipGetAllPropertyItems(image, 0, prop_count, prop_item); + expect(InvalidParameter, status); + status = GdipGetAllPropertyItems(image, prop_size, 1, prop_item); + expect(InvalidParameter, status); + status = GdipGetAllPropertyItems(image, prop_size, prop_count, NULL); + expect(InvalidParameter, status); + status = GdipGetAllPropertyItems(image, prop_size, prop_count, NULL); + expect(InvalidParameter, status); + status = GdipGetAllPropertyItems(image, 0, 0, NULL); + expect(InvalidParameter, status); + status = GdipGetAllPropertyItems(image, prop_size + 1, prop_count, prop_item); + expect(InvalidParameter, status); + status = GdipGetAllPropertyItems(image, prop_size, prop_count, prop_item); + expect(Ok, status); + + item_data = (const char *)(prop_item + prop_count); + for (i = 0; i < prop_count; i++) + { + ok(prop_item[i].value == item_data, "%u: expected value %p, got %p\n", + i, item_data, prop_item[i].value); + ok(td[i].type == prop_item[i].type, + "%u: expected type %u, got %u\n", i, td[i].type, prop_item[i].type); + ok(td[i].id == prop_item[i].id, "%u: expected id %#x, got %#x\n", i, td[i].id, prop_item[i].id); + ok(td[i].length == prop_item[i].length, "%u: expected length %u, got %u\n", i, td[i].length, prop_item[i].length); + if (td[i].length == prop_item[i].length) + { + int match = memcmp(td[i].value, prop_item[i].value, td[i].length) == 0; + ok(match, "%u: data mismatch\n", i); + if (!match) + { + UINT j; + BYTE *data = prop_item[i].value; + printf("id %#x:", prop_item[i].id); + for (j = 0; j < prop_item[i].length; j++) + printf(" %02x", data[j]); + printf("\n"); + } + } + item_data += prop_item[i].length; + } + + HeapFree(GetProcessHeap(), 0, prop_item); + GdipDisposeImage(image); }

1 0

Dmitry Timoshkov : gdiplus: Test GIF properties using a specially created GIF image with a bunch of extensions .
by Alexandre Julliard 21 Sep '12

21 Sep '12

Module: wine Branch: master Commit: d8d4bffb39e4cb3ba0490b32770ae854262c6403 URL: http://source.winehq.org/git/wine.git/?a=commit;h=d8d4bffb39e4cb3ba0490b327… Author: Dmitry Timoshkov <dmitry(a)baikal.ru> Date: Fri Sep 21 19:56:21 2012 +0900 gdiplus: Test GIF properties using a specially created GIF image with a bunch of extensions. --- dlls/gdiplus/tests/image.c | 48 +++++++++++++++++++++++++++++++++++-------- 1 files changed, 39 insertions(+), 9 deletions(-) diff --git a/dlls/gdiplus/tests/image.c b/dlls/gdiplus/tests/image.c index e373a8c..a26c4f5 100644 --- a/dlls/gdiplus/tests/image.c +++ b/dlls/gdiplus/tests/image.c @@ -3920,18 +3920,42 @@ static void test_DrawImage_scale(void) expect(Ok, status); } +static const BYTE animatedgif[] = { +'G','I','F','8','9','a',0x01,0x00,0x01,0x00,0xA1,0x02,0x00, +0x6F,0x6F,0x6F,0x00,0x00,0x00,0x00,0x00,0x00,0x00,0x00,0x00, +/*0x21,0xFF,0x0B,'A','N','I','M','E','X','T','S','1','.','0',*/ +0x21,0xFF,0x0B,'N','E','T','S','C','A','P','E','2','.','0', +0x03,0x01,0x05,0x00,0x00, +0x21,0xFE,0x0C,'H','e','l','l','o',' ','W','o','r','l','d','!',0x00, +0x21,0x01,0x0D,'a','n','i','m','a','t','i','o','n','.','g','i','f',0x00, +0x21,0xF9,0x04,0xff,0x0A,0x00,0x08,0x00, +0x2C,0x00,0x00,0x00,0x00,0x01,0x00,0x01,0x00,0x81, +0xDE,0xDE,0xDE,0x00,0x00,0x00, +0x00,0x00,0x00,0x00,0x00,0x00,0x02,0x02,0x4C,0x01,0x00, +0x21,0xFE,0x08,'i','m','a','g','e',' ','#','1',0x00, +0x21,0x01,0x0C,'p','l','a','i','n','t','e','x','t',' ','#','1',0x00, +0x21,0xF9,0x04,0x00,0x14,0x00,0x01,0x00, +0x2C,0x00,0x00,0x00,0x00,0x01,0x00,0x01,0x00,0x81, +0x4D,0x4D,0x4D,0x00,0x00,0x00, +0x00,0x00,0x00,0x00,0x00,0x00,0x02,0x02,0x44,0x01,0x00, +0x21,0xFE,0x08,'i','m','a','g','e',' ','#','2',0x00, +0x21,0x01,0x0C,'p','l','a','i','n','t','e','x','t',' ','#','2',0x00,0x3B +}; + static void test_gif_properties(void) { static const struct test_data { ULONG type, id, length; - const BYTE value[12]; + const BYTE value[13]; } td[] = { - { PropertyTagTypeLong, PropertyTagFrameDelay, 8, { 10,0,0,0,10,0,0,0 } }, - { PropertyTagTypeShort, PropertyTagLoopCount, 2, { 1 } }, - { PropertyTagTypeByte, PropertyTagGlobalPalette, 12, { 0,0,0,0xff,0xff,0xff,0,0,0,0,0 } }, - { PropertyTagTypeByte, PropertyTagIndexBackground, 1, { 0 } } + { PropertyTagTypeLong, PropertyTagFrameDelay, 8, { 10,0,0,0,20,0,0,0 } }, + { PropertyTagTypeASCII, PropertyTagExifUserComment, 13, { 'H','e','l','l','o',' ','W','o','r','l','d','!',0 } }, + { PropertyTagTypeShort, PropertyTagLoopCount, 2, { 5,0 } }, + { PropertyTagTypeByte, PropertyTagGlobalPalette, 12, { 0x6f,0x6f,0x6f,0,0,0,0,0,0,0,0,0 } }, + { PropertyTagTypeByte, PropertyTagIndexBackground, 1, { 2 } }, + { PropertyTagTypeByte, PropertyTagIndexTransparent, 1, { 8 } } }; GpStatus status; GpImage *image; @@ -3940,9 +3964,12 @@ static void test_gif_properties(void) PROPID *prop_id; PropertyItem *prop_item; - image = load_image(gifanimation, sizeof(gifanimation)); - ok(image != 0, "Failed to load GIF image data\n"); - if (!image) return; + image = load_image(animatedgif, sizeof(animatedgif)); + if (!image) /* XP fails to load this GIF image */ + { + trace("Failed to load GIF image data\n"); + return; + } status = GdipImageGetFrameDimensionsCount(image, &dim_count); expect(Ok, status); @@ -3956,6 +3983,9 @@ static void test_gif_properties(void) expect(Ok, status); expect(2, frame_count); + status = GdipImageSelectActiveFrame(image, &guid, 1); + expect(Ok, status); + status = GdipGetPropertyCount(image, &prop_count); expect(Ok, status); todo_wine @@ -3994,7 +4024,7 @@ todo_wine if (td[i].length == prop_item->length) { int match = memcmp(td[i].value, prop_item->value, td[i].length) == 0; - ok(match || broken(td[i].length <= 4 && !match), "%u: data mismatch\n", i); + ok(match, "%u: data mismatch\n", i); if (!match) { UINT j;

1 0

Jacek Caban : atliface.idl: Added IRegistrarBase declaration.
by Alexandre Julliard 21 Sep '12

21 Sep '12

Module: wine Branch: master Commit: 0c143845891ebf1b622026836b94dfd383aa1870 URL: http://source.winehq.org/git/wine.git/?a=commit;h=0c143845891ebf1b622026836… Author: Jacek Caban <jacek(a)codeweavers.com> Date: Fri Sep 21 13:45:55 2012 +0200 atliface.idl: Added IRegistrarBase declaration. --- dlls/atl/registrar.c | 4 +++- include/atliface.idl | 14 +++++++++++--- 2 files changed, 14 insertions(+), 4 deletions(-) diff --git a/dlls/atl/registrar.c b/dlls/atl/registrar.c index 4fcf28d..7b1fe30 100644 --- a/dlls/atl/registrar.c +++ b/dlls/atl/registrar.c @@ -543,7 +543,9 @@ static HRESULT WINAPI Registrar_QueryInterface(IRegistrar *iface, REFIID riid, v { TRACE("(%p)->(%s %p\n", iface, debugstr_guid(riid), ppvObject); - if(IsEqualGUID(&IID_IUnknown, riid) || IsEqualGUID(&IID_IRegistrar, riid)) { + if(IsEqualGUID(&IID_IUnknown, riid) + || IsEqualGUID(&IID_IRegistrar, riid) + || IsEqualGUID(&IID_IRegistrarBase, riid)) { IRegistrar_AddRef(iface); *ppvObject = iface; return S_OK; diff --git a/include/atliface.idl b/include/atliface.idl index cbdae28..78d8518 100644 --- a/include/atliface.idl +++ b/include/atliface.idl @@ -24,17 +24,25 @@ cpp_quote("#endif") [ object, - oleautomation, - uuid(44EC053B-400F-11D0-9DCD-00A0C90391D3) + uuid(e21f8a85-b05d-4243-8183-c7cb405588f7), + pointer_default(unique) ] -interface IRegistrar : IUnknown +interface IRegistrarBase : IUnknown { HRESULT AddReplacement( [in] LPCOLESTR Key, [in] LPCOLESTR item); HRESULT ClearReplacements(); +} +[ + object, + uuid(44EC053B-400F-11D0-9DCD-00A0C90391D3), + pointer_default(unique) +] +interface IRegistrar : IRegistrarBase +{ HRESULT ResourceRegisterSz( [in] LPCOLESTR resFileName, [in] LPCOLESTR szID,

1 0

Christian Costa : dmusic: Rename params in SynthPortImpl_IDirectMusicPort_QueryInterface.
by Alexandre Julliard 21 Sep '12

21 Sep '12

Module: wine Branch: master Commit: 699fa23bd293346d3f576388eed401e41cdd5e35 URL: http://source.winehq.org/git/wine.git/?a=commit;h=699fa23bd293346d3f576388e… Author: Christian Costa <titan.costa(a)gmail.com> Date: Fri Sep 21 11:47:06 2012 +0200 dmusic: Rename params in SynthPortImpl_IDirectMusicPort_QueryInterface. --- dlls/dmusic/port.c | 50 ++++++++++++++++++++++++++------------------------ 1 files changed, 26 insertions(+), 24 deletions(-) diff --git a/dlls/dmusic/port.c b/dlls/dmusic/port.c index cfe4265..b6489a7 100644 --- a/dlls/dmusic/port.c +++ b/dlls/dmusic/port.c @@ -39,31 +39,33 @@ static inline SynthPortImpl *impl_from_SynthPortImpl_IDirectMusicThru(IDirectMus } /* SynthPortImpl IDirectMusicPort IUnknown part follows: */ -static HRESULT WINAPI SynthPortImpl_IDirectMusicPort_QueryInterface(LPDIRECTMUSICPORT iface, REFIID riid, LPVOID *ppobj) +static HRESULT WINAPI SynthPortImpl_IDirectMusicPort_QueryInterface(LPDIRECTMUSICPORT iface, REFIID riid, LPVOID *ret_iface) { - SynthPortImpl *This = impl_from_SynthPortImpl_IDirectMusicPort(iface); + SynthPortImpl *This = impl_from_SynthPortImpl_IDirectMusicPort(iface); - TRACE("(%p, %s, %p)\n", This, debugstr_dmguid(riid), ppobj); - - if (IsEqualIID (riid, &IID_IUnknown) || - IsEqualGUID(riid, &IID_IDirectMusicPort) || - IsEqualGUID(riid, &IID_IDirectMusicPort8)) { - *ppobj = &This->IDirectMusicPort_iface; - IDirectMusicPort_AddRef((LPDIRECTMUSICPORT)*ppobj); - return S_OK; - } else if (IsEqualGUID(riid, &IID_IDirectMusicPortDownload) || - IsEqualGUID(riid, &IID_IDirectMusicPortDownload8)) { - *ppobj = &This->IDirectMusicPortDownload_iface; - IDirectMusicPortDownload_AddRef((LPDIRECTMUSICPORTDOWNLOAD)*ppobj); - return S_OK; - } else if (IsEqualGUID(riid, &IID_IDirectMusicThru) || - IsEqualGUID(riid, &IID_IDirectMusicThru8)) { - *ppobj = &This->IDirectMusicThru_iface; - IDirectMusicThru_AddRef((LPDIRECTMUSICTHRU)*ppobj); - return S_OK; - } - WARN("(%p, %s, %p): not found\n", This, debugstr_dmguid(riid), ppobj); - return E_NOINTERFACE; + TRACE("(%p/%p)->(%s, %p)\n", iface, This, debugstr_dmguid(riid), ret_iface); + + if (IsEqualIID (riid, &IID_IUnknown) || + IsEqualGUID(riid, &IID_IDirectMusicPort) || + IsEqualGUID(riid, &IID_IDirectMusicPort8)) { + *ret_iface = &This->IDirectMusicPort_iface; + IDirectMusicPort_AddRef((LPDIRECTMUSICPORT)*ret_iface); + return S_OK; + } else if (IsEqualGUID(riid, &IID_IDirectMusicPortDownload) || + IsEqualGUID(riid, &IID_IDirectMusicPortDownload8)) { + *ret_iface = &This->IDirectMusicPortDownload_iface; + IDirectMusicPortDownload_AddRef((LPDIRECTMUSICPORTDOWNLOAD)*ret_iface); + return S_OK; + } else if (IsEqualGUID(riid, &IID_IDirectMusicThru) || + IsEqualGUID(riid, &IID_IDirectMusicThru8)) { + *ret_iface = &This->IDirectMusicThru_iface; + IDirectMusicThru_AddRef((LPDIRECTMUSICTHRU)*ret_iface); + return S_OK; + } + + WARN("(%p, %s, %p): not found\n", This, debugstr_dmguid(riid), ret_iface); + + return E_NOINTERFACE; } static ULONG WINAPI SynthPortImpl_IDirectMusicPort_AddRef(LPDIRECTMUSICPORT iface)

1 0

Christian Costa : dmusic: Rename params in IDirectMusic8Impl_GetDefaultPort .
by Alexandre Julliard 21 Sep '12

21 Sep '12

Module: wine Branch: master Commit: 27b2868899288613acd6378f93e8143afee2b239 URL: http://source.winehq.org/git/wine.git/?a=commit;h=27b2868899288613acd6378f9… Author: Christian Costa <titan.costa(a)gmail.com> Date: Fri Sep 21 11:46:57 2012 +0200 dmusic: Rename params in IDirectMusic8Impl_GetDefaultPort. --- dlls/dmusic/dmusic.c | 45 +++++++++++++++++++++++---------------------- 1 files changed, 23 insertions(+), 22 deletions(-) diff --git a/dlls/dmusic/dmusic.c b/dlls/dmusic/dmusic.c index b4b35ec..2977cd9 100644 --- a/dlls/dmusic/dmusic.c +++ b/dlls/dmusic/dmusic.c @@ -248,29 +248,30 @@ static HRESULT WINAPI IDirectMusic8Impl_Activate(LPDIRECTMUSIC8 iface, BOOL enab return S_OK; } -static HRESULT WINAPI IDirectMusic8Impl_GetDefaultPort(LPDIRECTMUSIC8 iface, LPGUID pguidPort) +static HRESULT WINAPI IDirectMusic8Impl_GetDefaultPort(LPDIRECTMUSIC8 iface, LPGUID guid_port) { - IDirectMusic8Impl *This = impl_from_IDirectMusic8(iface); - HKEY hkGUID; - DWORD returnTypeGUID, sizeOfReturnBuffer = 50; - char returnBuffer[51]; - GUID defaultPortGUID; - WCHAR buff[51]; - - TRACE("(%p, %p)\n", This, pguidPort); - if ((RegOpenKeyExA(HKEY_LOCAL_MACHINE, "Software\\Microsoft\\DirectMusic\\Defaults" , 0, KEY_READ, &hkGUID) != ERROR_SUCCESS) || - (RegQueryValueExA(hkGUID, "DefaultOutputPort", NULL, &returnTypeGUID, (LPBYTE)returnBuffer, &sizeOfReturnBuffer) != ERROR_SUCCESS)) - { - WARN(": registry entry missing\n" ); - *pguidPort = CLSID_DirectMusicSynth; - return S_OK; - } - /* FIXME: Check return types to ensure we're interpreting data right */ - MultiByteToWideChar(CP_ACP, 0, returnBuffer, -1, buff, sizeof(buff) / sizeof(WCHAR)); - CLSIDFromString(buff, &defaultPortGUID); - *pguidPort = defaultPortGUID; - - return S_OK; + IDirectMusic8Impl *This = impl_from_IDirectMusic8(iface); + HKEY hkGUID; + DWORD returnTypeGUID, sizeOfReturnBuffer = 50; + char returnBuffer[51]; + GUID defaultPortGUID; + WCHAR buff[51]; + + TRACE("(%p)->(%p)\n", This, guid_port); + + if ((RegOpenKeyExA(HKEY_LOCAL_MACHINE, "Software\\Microsoft\\DirectMusic\\Defaults" , 0, KEY_READ, &hkGUID) != ERROR_SUCCESS) || + (RegQueryValueExA(hkGUID, "DefaultOutputPort", NULL, &returnTypeGUID, (LPBYTE)returnBuffer, &sizeOfReturnBuffer) != ERROR_SUCCESS)) + { + WARN(": registry entry missing\n" ); + *guid_port = CLSID_DirectMusicSynth; + return S_OK; + } + /* FIXME: Check return types to ensure we're interpreting data right */ + MultiByteToWideChar(CP_ACP, 0, returnBuffer, -1, buff, sizeof(buff) / sizeof(WCHAR)); + CLSIDFromString(buff, &defaultPortGUID); + *guid_port = defaultPortGUID; + + return S_OK; } static HRESULT WINAPI IDirectMusic8Impl_SetDirectSound(LPDIRECTMUSIC8 iface, LPDIRECTSOUND dsound, HWND wnd)

1 0

Jacek Caban : itss: Canonicalize relative paths before resolving object.
by Alexandre Julliard 21 Sep '12

21 Sep '12

Module: wine Branch: master Commit: 1a56a73c861f4e137119f1de86c41d0098fa2e40 URL: http://source.winehq.org/git/wine.git/?a=commit;h=1a56a73c861f4e137119f1de8… Author: Jacek Caban <jacek(a)codeweavers.com> Date: Fri Sep 21 12:28:18 2012 +0200 itss: Canonicalize relative paths before resolving object. --- dlls/itss/protocol.c | 51 ++++++++++++++++++++++++++++++++++++++++++++ dlls/itss/tests/protocol.c | 3 ++ 2 files changed, 54 insertions(+), 0 deletions(-) diff --git a/dlls/itss/protocol.c b/dlls/itss/protocol.c index fa2ad8b..6b188c6 100644 --- a/dlls/itss/protocol.c +++ b/dlls/itss/protocol.c @@ -134,6 +134,55 @@ static LPCWSTR skip_schema(LPCWSTR url) return NULL; } +/* Adopted from urlmon */ +static void remove_dot_segments(WCHAR *path) { + const WCHAR *in = path; + WCHAR *out = path; + + while(1) { + /* Move the first path segment in the input buffer to the end of + * the output buffer, and any subsequent characters up to, including + * the next "/" character (if any) or the end of the input buffer. + */ + while(*in != '/') { + if(!(*out++ = *in++)) + return; + } + + *out++ = *in++; + + while(*in) { + if(*in != '.') + break; + + /* Handle ending "/." */ + if(!in[1]) { + ++in; + break; + } + + /* Handle "/./" */ + if(in[1] == '/') { + in += 2; + continue; + } + + /* If we don't have "/../" or ending "/.." */ + if(in[1] != '.' || (in[2] && in[2] != '/')) + break; + + in += *in ? 3 : 2; + + /* Find the slash preceding out pointer and move out pointer to it */ + if(out > path+1 && *--out == '/') + --out; + while(out > path && *(--out) != '/'); + if(*out == '/') + ++out; + } + } +} + static HRESULT report_result(IInternetProtocolSink *sink, HRESULT hres) { IInternetProtocolSink_ReportResult(sink, hres, 0, NULL); @@ -215,6 +264,8 @@ static HRESULT WINAPI ITSProtocol_Start(IInternetProtocol *iface, LPCWSTR szUrl, *p = '/'; } + remove_dot_segments(object_name); + TRACE("Resolving %s\n", debugstr_w(object_name)); memset(&chm_object, 0, sizeof(chm_object)); diff --git a/dlls/itss/tests/protocol.c b/dlls/itss/tests/protocol.c index b718502..7467d37 100644 --- a/dlls/itss/tests/protocol.c +++ b/dlls/itss/tests/protocol.c @@ -87,6 +87,8 @@ static const WCHAR blank_url7[] = {'m','k',':','@','M','S','I','T','S','t','o',' 't','e','s','t','.','c','h','m',':',':','\\','b','l','a','n','k','.','h','t','m','l',0}; static const WCHAR blank_url8[] = {'m','k',':','@','M','S','I','T','S','t','o','r','e',':', 't','e','s','t','.','c','h','m',':',':','/','b','l','a','n','k','.','h','t','m','l','/',0}; +static const WCHAR blank_url9[] = {'i','t','s',':', + 't','e','s','t','.','c','h','m',':',':','/','d','i','r','/','.','.','/','b','l','a','n','k','.','h','t','m','l',0}; static enum { ITS_PROTOCOL, @@ -612,6 +614,7 @@ static void test_its_protocol(void) test_protocol_url(factory, blank_url5, TRUE); test_protocol_url(factory, blank_url6, TRUE); test_protocol_url(factory, blank_url8, TRUE); + test_protocol_url(factory, blank_url9, TRUE); bindf = BINDF_FROMURLMON | BINDF_NEEDFILE; test_protocol_url(factory, blank_url1, TRUE); }

1 0

Jacek Caban : mshtml: Use IUri as base URL in navigate_url.
by Alexandre Julliard 21 Sep '12

21 Sep '12

Module: wine Branch: master Commit: e75683b85860b33ba88c663147821ece6b4eafc0 URL: http://source.winehq.org/git/wine.git/?a=commit;h=e75683b85860b33ba88c66314… Author: Jacek Caban <jacek(a)codeweavers.com> Date: Fri Sep 21 12:24:37 2012 +0200 mshtml: Use IUri as base URL in navigate_url. --- dlls/mshtml/binding.h | 2 +- dlls/mshtml/htmlanchor.c | 2 +- dlls/mshtml/htmldoc.c | 2 +- dlls/mshtml/htmlframebase.c | 2 +- dlls/mshtml/htmllocation.c | 4 +- dlls/mshtml/mshtml_private.h | 2 +- dlls/mshtml/navigate.c | 98 ++++++++++++++++++++++++----------------- dlls/mshtml/nsio.c | 2 +- 8 files changed, 65 insertions(+), 49 deletions(-) diff --git a/dlls/mshtml/binding.h b/dlls/mshtml/binding.h index 0053e75..4d19c46 100644 --- a/dlls/mshtml/binding.h +++ b/dlls/mshtml/binding.h @@ -103,7 +103,7 @@ HRESULT create_redirect_nschannel(const WCHAR*,nsChannel*,nsChannel**) DECLSPEC_ nsresult on_start_uri_open(NSContainer*,nsIURI*,cpp_bool*) DECLSPEC_HIDDEN; HRESULT hlink_frame_navigate(HTMLDocument*,LPCWSTR,nsChannel*,DWORD,BOOL*) DECLSPEC_HIDDEN; -HRESULT create_doc_uri(HTMLOuterWindow*,WCHAR*,nsWineURI**) DECLSPEC_HIDDEN; +HRESULT create_doc_uri(HTMLOuterWindow*,const WCHAR*,nsWineURI**) DECLSPEC_HIDDEN; HRESULT load_nsuri(HTMLOuterWindow*,nsWineURI*,nsChannelBSC*,DWORD) DECLSPEC_HIDDEN; HRESULT set_moniker(HTMLDocument*,IMoniker*,IBindCtx*,nsChannelBSC*,BOOL) DECLSPEC_HIDDEN; void prepare_for_binding(HTMLDocument*,IMoniker*,BOOL) DECLSPEC_HIDDEN; diff --git a/dlls/mshtml/htmlanchor.c b/dlls/mshtml/htmlanchor.c index fac2d63..b0ec97e 100644 --- a/dlls/mshtml/htmlanchor.c +++ b/dlls/mshtml/htmlanchor.c @@ -121,7 +121,7 @@ static HRESULT navigate_anchor(HTMLAnchorElement *This) if(*href) { if(!window) window = This->element.node.doc->basedoc.window; - hres = navigate_url(window, href, window->url); + hres = navigate_url(window, href, window->uri); }else { TRACE("empty href\n"); hres = S_OK; diff --git a/dlls/mshtml/htmldoc.c b/dlls/mshtml/htmldoc.c index a6bb56d..ff917d9 100644 --- a/dlls/mshtml/htmldoc.c +++ b/dlls/mshtml/htmldoc.c @@ -609,7 +609,7 @@ static HRESULT WINAPI HTMLDocument_put_URL(IHTMLDocument2 *iface, BSTR v) return E_FAIL; } - return navigate_url(This->window, v, This->window->url); + return navigate_url(This->window, v, This->window->uri); } static HRESULT WINAPI HTMLDocument_get_URL(IHTMLDocument2 *iface, BSTR *p) diff --git a/dlls/mshtml/htmlframebase.c b/dlls/mshtml/htmlframebase.c index bc00761..55a56db 100644 --- a/dlls/mshtml/htmlframebase.c +++ b/dlls/mshtml/htmlframebase.c @@ -135,7 +135,7 @@ static HRESULT WINAPI HTMLFrameBase_put_src(IHTMLFrameBase *iface, BSTR v) return E_FAIL; } - return navigate_url(This->content_window, v, This->element.node.doc->basedoc.window->url); + return navigate_url(This->content_window, v, This->element.node.doc->basedoc.window->uri); } static HRESULT WINAPI HTMLFrameBase_get_src(IHTMLFrameBase *iface, BSTR *p) diff --git a/dlls/mshtml/htmllocation.c b/dlls/mshtml/htmllocation.c index cde1fee..3ce0978 100644 --- a/dlls/mshtml/htmllocation.c +++ b/dlls/mshtml/htmllocation.c @@ -170,7 +170,7 @@ static HRESULT WINAPI HTMLLocation_put_href(IHTMLLocation *iface, BSTR v) return E_FAIL; } - return navigate_url(This->window->base.outer_window, v, This->window->base.outer_window->url); + return navigate_url(This->window->base.outer_window, v, This->window->base.outer_window->uri); } static HRESULT WINAPI HTMLLocation_get_href(IHTMLLocation *iface, BSTR *p) @@ -570,7 +570,7 @@ static HRESULT WINAPI HTMLLocation_replace(IHTMLLocation *iface, BSTR bstr) return E_FAIL; } - return navigate_url(This->window->base.outer_window, bstr, This->window->base.outer_window->url); + return navigate_url(This->window->base.outer_window, bstr, This->window->base.outer_window->uri); } static HRESULT WINAPI HTMLLocation_assign(IHTMLLocation *iface, BSTR bstr) diff --git a/dlls/mshtml/mshtml_private.h b/dlls/mshtml/mshtml_private.h index fa4e073..aec754f 100644 --- a/dlls/mshtml/mshtml_private.h +++ b/dlls/mshtml/mshtml_private.h @@ -775,7 +775,7 @@ void init_node_cc(void); HRESULT nsuri_to_url(LPCWSTR,BOOL,BSTR*) DECLSPEC_HIDDEN; BOOL compare_ignoring_frag(IUri*,IUri*) DECLSPEC_HIDDEN; -HRESULT navigate_url(HTMLOuterWindow*,const WCHAR*,const WCHAR*) DECLSPEC_HIDDEN; +HRESULT navigate_url(HTMLOuterWindow*,const WCHAR*,IUri*) DECLSPEC_HIDDEN; HRESULT set_frame_doc(HTMLFrameBase*,nsIDOMDocument*) DECLSPEC_HIDDEN; void call_property_onchanged(ConnectionPoint*,DISPID) DECLSPEC_HIDDEN; diff --git a/dlls/mshtml/navigate.c b/dlls/mshtml/navigate.c index 15c9e85..3b618e2 100644 --- a/dlls/mshtml/navigate.c +++ b/dlls/mshtml/navigate.c @@ -2245,61 +2245,28 @@ HRESULT hlink_frame_navigate(HTMLDocument *doc, LPCWSTR url, nsChannel *nschanne return hres; } -HRESULT navigate_url(HTMLOuterWindow *window, const WCHAR *new_url, const WCHAR *base_url) +HRESULT navigate_uri(HTMLOuterWindow *window, IUri *uri, const WCHAR *display_uri) { - WCHAR url[INTERNET_MAX_URL_LENGTH]; - nsWineURI *uri; + nsWineURI *nsuri; HRESULT hres; - if(!new_url) { - *url = 0; - }else if(base_url) { - DWORD len = 0; - - hres = CoInternetCombineUrl(base_url, new_url, URL_ESCAPE_SPACES_ONLY|URL_DONT_ESCAPE_EXTRA_INFO, - url, sizeof(url)/sizeof(WCHAR), &len, 0); - if(FAILED(hres)) - return hres; - }else { - strcpyW(url, new_url); - } - - if(window->doc_obj && window->doc_obj->hostui) { - OLECHAR *translated_url = NULL; - - hres = IDocHostUIHandler_TranslateUrl(window->doc_obj->hostui, 0, url, - &translated_url); - if(hres == S_OK) { - TRACE("%08x %s -> %s\n", hres, debugstr_w(url), debugstr_w(translated_url)); - strcpyW(url, translated_url); - CoTaskMemFree(translated_url); - } - } - if(window->doc_obj && window->doc_obj->is_webbrowser && window == window->doc_obj->basedoc.window) { BOOL cancel = FALSE; - IUri *uri; - hres = IDocObjectService_FireBeforeNavigate2(window->doc_obj->doc_object_service, NULL, url, 0x40, + hres = IDocObjectService_FireBeforeNavigate2(window->doc_obj->doc_object_service, NULL, display_uri, 0x40, NULL, NULL, 0, NULL, TRUE, &cancel); if(SUCCEEDED(hres) && cancel) { TRACE("Navigation canceled\n"); return S_OK; } - hres = CreateUri(url, 0, 0, &uri); - if(FAILED(hres)) - return hres; - - hres = super_navigate(window, uri, NULL, NULL, 0); - IUri_Release(uri); - return hres; + return super_navigate(window, uri, NULL, NULL, 0); } if(window->doc_obj && window == window->doc_obj->basedoc.window) { BOOL cancel; - hres = hlink_frame_navigate(&window->base.inner_window->doc->basedoc, url, NULL, 0, &cancel); + hres = hlink_frame_navigate(&window->base.inner_window->doc->basedoc, display_uri, NULL, 0, &cancel); if(FAILED(hres)) return hres; @@ -2309,11 +2276,60 @@ HRESULT navigate_url(HTMLOuterWindow *window, const WCHAR *new_url, const WCHAR } } - hres = create_doc_uri(window, url, &uri); + hres = create_doc_uri(window, display_uri, &nsuri); + if(FAILED(hres)) + return hres; + + hres = load_nsuri(window, nsuri, NULL, LOAD_FLAGS_NONE); + nsISupports_Release((nsISupports*)nsuri); + return hres; +} + +HRESULT navigate_url(HTMLOuterWindow *window, const WCHAR *new_url, IUri *base_uri) +{ + BSTR display_uri; + IUri *uri; + HRESULT hres; + + if(new_url && base_uri) + hres = CoInternetCombineUrlEx(base_uri, new_url, URL_ESCAPE_SPACES_ONLY|URL_DONT_ESCAPE_EXTRA_INFO, + &uri, 0); + else + hres = CreateUri(new_url, 0, 0, &uri); if(FAILED(hres)) return hres; - hres = load_nsuri(window, uri, NULL, LOAD_FLAGS_NONE); - nsISupports_Release((nsISupports*)uri); + hres = IUri_GetDisplayUri(uri, &display_uri); + if(FAILED(hres)) { + IUri_Release(uri); + return hres; + } + + if(window->doc_obj && window->doc_obj->hostui) { + OLECHAR *translated_url = NULL; + + hres = IDocHostUIHandler_TranslateUrl(window->doc_obj->hostui, 0, display_uri, + &translated_url); + if(hres == S_OK) { + TRACE("%08x %s -> %s\n", hres, debugstr_w(display_uri), debugstr_w(translated_url)); + SysFreeString(display_uri); + IUri_Release(uri); + hres = CreateUri(translated_url, 0, 0, &uri); + CoTaskMemFree(translated_url); + if(FAILED(hres)) + return hres; + + hres = IUri_GetDisplayUri(uri, &display_uri); + if(FAILED(hres)) { + IUri_Release(uri); + return hres; + } + } + } + + hres = navigate_uri(window, uri, display_uri); + + IUri_Release(uri); + SysFreeString(display_uri); return hres; } diff --git a/dlls/mshtml/nsio.c b/dlls/mshtml/nsio.c index 4b247ee..cf71ec5 100644 --- a/dlls/mshtml/nsio.c +++ b/dlls/mshtml/nsio.c @@ -2867,7 +2867,7 @@ static nsresult create_nsuri(IUri *iuri, HTMLOuterWindow *window, NSContainer *c return NS_OK; } -HRESULT create_doc_uri(HTMLOuterWindow *window, WCHAR *url, nsWineURI **ret) +HRESULT create_doc_uri(HTMLOuterWindow *window, const WCHAR *url, nsWineURI **ret) { nsWineURI *uri; IUri *iuri;

1 0

Jacek Caban : include: Added uiautomation.h.
by Alexandre Julliard 21 Sep '12

21 Sep '12

Module: wine Branch: master Commit: 40e7f7522e82223ad4faf9f6b59dd83f173ba766 URL: http://source.winehq.org/git/wine.git/?a=commit;h=40e7f7522e82223ad4faf9f6b… Author: Jacek Caban <jacek(a)codeweavers.com> Date: Fri Sep 21 11:16:21 2012 +0200 include: Added uiautomation.h. --- include/Makefile.in | 1 + include/uiautomation.h | 21 +++++++++++++++++++++ 2 files changed, 22 insertions(+), 0 deletions(-) diff --git a/include/Makefile.in b/include/Makefile.in index a4f00a3..8536a07 100644 --- a/include/Makefile.in +++ b/include/Makefile.in @@ -519,6 +519,7 @@ SRCDIR_INCLUDES = \ traffic.h \ twain.h \ udpmib.h \ + uiautomation.h \ userenv.h \ usp10.h \ uuids.h \ diff --git a/include/uiautomation.h b/include/uiautomation.h new file mode 100644 index 0000000..8bd2b55 --- /dev/null +++ b/include/uiautomation.h @@ -0,0 +1,21 @@ +/* + * Copyright 2012 Jacek Caban for CodeWeavers + * + * 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 <uiautomationcore.h> +/* #include <uiautomationclient.h> */ +/* #include <uiautomationcoreapi.h> */

1 0

Jacek Caban : include: Added uiautomationcore.idl.
by Alexandre Julliard 21 Sep '12

21 Sep '12

Module: wine Branch: master Commit: 021cab2882b59a7b9ab2fd224e16016803f51630 URL: http://source.winehq.org/git/wine.git/?a=commit;h=021cab2882b59a7b9ab2fd224… Author: Jacek Caban <jacek(a)codeweavers.com> Date: Fri Sep 21 11:16:06 2012 +0200 include: Added uiautomationcore.idl. --- .gitignore | 1 + include/Makefile.in | 1 + include/uiautomationcore.idl | 91 ++++++++++++++++++++++++++++++++++++++++++ 3 files changed, 93 insertions(+), 0 deletions(-) diff --git a/.gitignore b/.gitignore index 6baea7d..e652488 100644 --- a/.gitignore +++ b/.gitignore @@ -240,6 +240,7 @@ include/structuredquerycondition.h include/textstor.h include/tlogstg.h include/tom.h +include/uiautomationcore.h include/unknwn.h include/urlhist.h include/urlmon.h diff --git a/include/Makefile.in b/include/Makefile.in index 37f2c62..a4f00a3 100644 --- a/include/Makefile.in +++ b/include/Makefile.in @@ -97,6 +97,7 @@ PUBLIC_IDL_H_SRCS = \ textstor.idl \ tlogstg.idl \ tom.idl \ + uiautomationcore.idl \ unknwn.idl \ urlhist.idl \ urlmon.idl \ diff --git a/include/uiautomationcore.idl b/include/uiautomationcore.idl new file mode 100644 index 0000000..3ae6dac --- /dev/null +++ b/include/uiautomationcore.idl @@ -0,0 +1,91 @@ +/* + * Copyright 2012 Jacek Caban for CodeWeavers + * + * 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 + */ + +#define DO_NO_IMPORTS +import "oaidl.idl"; +import "oleacc.idl"; + +enum ProviderOptions { + ProviderOptions_ClientSideProvider = 0x0001, + ProviderOptions_ServerSideProvider = 0x0002, + ProviderOptions_NonClientAreaProvider = 0x0004, + ProviderOptions_OverrideProvider = 0x0008, + ProviderOptions_ProviderOwnsSetFocus = 0x0010, + ProviderOptions_UseComThreading = 0x0020 +}; + +typedef int PROPERTYID; +typedef int PATTERNID; +typedef int EVENTID; +typedef int TEXTATTRIBUTEID; +typedef int CONTROLTYPEID; + +[ + version(1.0), + uuid(930299ce-9965-4dec-b0f4-a54848d4b667), + lcid(0), + hidden +] +library UIA +{ + importlib("stdole2.tlb"); + + [ + object, + uuid(d6dd68d1-86fd-4332-8666-9abedea2d24c), + pointer_default(unique) + ] + interface IRawElementProviderSimple : IUnknown + { + [propget] HRESULT ProviderOptions([out, retval] enum ProviderOptions *pRetVal); + + HRESULT GetPatternProvider( + [in] PATTERNID patternId, + [out, retval] IUnknown **pRetVal); + + HRESULT GetPropertyValue( + [in] PROPERTYID propertyId, + [out, retval] VARIANT *pRetVal); + + [propget] HRESULT HostRawElementProvider([out, retval] IRawElementProviderSimple **pRetVal); + } + + [ + object, + uuid(f8b80ada-2c44-48d0-89be-5ff23c9cd875), + pointer_default(unique), + oleautomation + ] + interface IAccessibleEx : IUnknown + { + HRESULT GetObjectForChild( + [in] long idChild, + [out, retval] IAccessibleEx **pRetVal); + + HRESULT GetIAccessiblePair( + [out] IAccessible **ppAcc, + [out] long *pidChild); + + HRESULT GetRuntimeId( + [out, retval] SAFEARRAY(int) *pRetVal); + + HRESULT ConvertReturnedElement( + [in] IRawElementProviderSimple *pIn, + [out] IAccessibleEx **ppRetValOut); + } +}

1 0

Alexandre Julliard : winex11: Add window data structure locking to the map/ unmap functions.
by Alexandre Julliard 21 Sep '12

21 Sep '12

Module: wine Branch: master Commit: a6c543d0e6e2c76f7a357f3d72c242df8fdba78c URL: http://source.winehq.org/git/wine.git/?a=commit;h=a6c543d0e6e2c76f7a357f3d7… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Fri Sep 21 16:50:17 2012 +0200 winex11: Add window data structure locking to the map/unmap functions. --- dlls/winex11.drv/event.c | 60 +++++++++++++++++++++++------------ dlls/winex11.drv/window.c | 76 ++++++++++++++++++++++++++------------------- 2 files changed, 84 insertions(+), 52 deletions(-) diff --git a/dlls/winex11.drv/event.c b/dlls/winex11.drv/event.c index 1f49c3c..4e4b816 100644 --- a/dlls/winex11.drv/event.c +++ b/dlls/winex11.drv/event.c @@ -1163,7 +1163,7 @@ static int get_window_wm_state( Display *display, Window window ) */ static void handle_wm_state_notify( HWND hwnd, XPropertyEvent *event, BOOL update_window ) { - struct x11drv_win_data *data = X11DRV_get_win_data( hwnd ); + struct x11drv_win_data *data = get_win_data( hwnd ); DWORD style; if (!data) return; @@ -1185,13 +1185,13 @@ static void handle_wm_state_notify( HWND hwnd, XPropertyEvent *event, BOOL updat data->wm_state = new_state; /* ignore the initial state transition out of withdrawn state */ /* metacity does Withdrawn->NormalState->IconicState when mapping an iconic window */ - if (!old_state) return; + if (!old_state) goto done; } } break; } - if (!update_window || !data->managed || !data->mapped) return; + if (!update_window || !data->managed || !data->mapped) goto done; style = GetWindowLongW( data->hwnd, GWL_STYLE ); @@ -1203,17 +1203,20 @@ static void handle_wm_state_notify( HWND hwnd, XPropertyEvent *event, BOOL updat if ((style & WS_MAXIMIZEBOX) && !(style & WS_DISABLED)) { TRACE( "restoring to max %p/%lx\n", data->hwnd, data->whole_window ); - SendMessageW( data->hwnd, WM_SYSCOMMAND, SC_MAXIMIZE, 0 ); + release_win_data( data ); + SendMessageW( hwnd, WM_SYSCOMMAND, SC_MAXIMIZE, 0 ); + return; } - else TRACE( "not restoring to max win %p/%lx style %08x\n", - data->hwnd, data->whole_window, style ); + TRACE( "not restoring to max win %p/%lx style %08x\n", data->hwnd, data->whole_window, style ); } else if (style & (WS_MINIMIZE | WS_MAXIMIZE)) { TRACE( "restoring win %p/%lx\n", data->hwnd, data->whole_window ); - SendMessageW( data->hwnd, WM_SYSCOMMAND, SC_RESTORE, 0 ); + release_win_data( data ); + SendMessageW( hwnd, WM_SYSCOMMAND, SC_RESTORE, 0 ); + return; } - else TRACE( "not restoring win %p/%lx style %08x\n", data->hwnd, data->whole_window, style ); + TRACE( "not restoring win %p/%lx style %08x\n", data->hwnd, data->whole_window, style ); } else if (!data->iconic && data->wm_state == IconicState) { @@ -1221,10 +1224,14 @@ static void handle_wm_state_notify( HWND hwnd, XPropertyEvent *event, BOOL updat if ((style & WS_MINIMIZEBOX) && !(style & WS_DISABLED)) { TRACE( "minimizing win %p/%lx\n", data->hwnd, data->whole_window ); - SendMessageW( data->hwnd, WM_SYSCOMMAND, SC_MINIMIZE, 0 ); + release_win_data( data ); + SendMessageW( hwnd, WM_SYSCOMMAND, SC_MINIMIZE, 0 ); + return; } - else TRACE( "not minimizing win %p/%lx style %08x\n", data->hwnd, data->whole_window, style ); + TRACE( "not minimizing win %p/%lx style %08x\n", data->hwnd, data->whole_window, style ); } +done: + release_win_data( data ); } @@ -1254,20 +1261,33 @@ static Bool is_wm_state_notify( Display *display, XEvent *event, XPointer arg ) void wait_for_withdrawn_state( HWND hwnd, BOOL set ) { Display *display = thread_display(); - struct x11drv_win_data *data = X11DRV_get_win_data( hwnd ); + struct x11drv_win_data *data; DWORD end = GetTickCount() + 2000; - if (!data || !data->managed) return; - - TRACE( "waiting for window %p/%lx to become %swithdrawn\n", - data->hwnd, data->whole_window, set ? "" : "not " ); + TRACE( "waiting for window %p to become %swithdrawn\n", hwnd, set ? "" : "not " ); - while (data->whole_window && ((data->wm_state == WithdrawnState) == !set)) + for (;;) { XEvent event; + Window window; int count = 0; - while (XCheckIfEvent( display, &event, is_wm_state_notify, (char *)data->whole_window )) + if (!(data = get_win_data( hwnd ))) break; + if (!data->managed || data->embedded || data->display != display) break; + if (!(window = data->whole_window)) break; + if (!data->mapped == !set) + { + TRACE( "window %p/%lx now %smapped\n", hwnd, window, data->mapped ? "" : "un" ); + break; + } + if ((data->wm_state == WithdrawnState) != !set) + { + TRACE( "window %p/%lx state now %d\n", hwnd, window, data->wm_state ); + break; + } + release_win_data( data ); + + while (XCheckIfEvent( display, &event, is_wm_state_notify, (char *)window )) { count++; if (XFilterEvent( &event, None )) continue; /* filtered, ignore it */ @@ -1284,12 +1304,12 @@ void wait_for_withdrawn_state( HWND hwnd, BOOL set ) pfd.events = POLLIN; if (timeout <= 0 || poll( &pfd, 1, timeout ) != 1) { - FIXME( "window %p/%lx wait timed out\n", data->hwnd, data->whole_window ); - break; + FIXME( "window %p/%lx wait timed out\n", hwnd, window ); + return; } } } - TRACE( "window %p/%lx state now %d\n", data->hwnd, data->whole_window, data->wm_state ); + release_win_data( data ); } diff --git a/dlls/winex11.drv/window.c b/dlls/winex11.drv/window.c index 4ccf067..931f60e 100644 --- a/dlls/winex11.drv/window.c +++ b/dlls/winex11.drv/window.c @@ -1068,24 +1068,31 @@ static void set_xembed_flags( struct x11drv_win_data *data, unsigned long flags */ static void map_window( HWND hwnd, DWORD new_style ) { - struct x11drv_win_data *data = X11DRV_get_win_data( hwnd ); - - TRACE( "win %p/%lx\n", data->hwnd, data->whole_window ); - - remove_startup_notification( data->display, data->whole_window ); + struct x11drv_win_data *data; wait_for_withdrawn_state( hwnd, TRUE ); - if (!data->embedded) + if (!(data = get_win_data( hwnd ))) return; + + if (data->whole_window && !data->mapped) { - update_net_wm_states( data ); - sync_window_style( data ); - XMapWindow( data->display, data->whole_window ); - } - else set_xembed_flags( data, XEMBED_MAPPED ); + TRACE( "win %p/%lx\n", data->hwnd, data->whole_window ); - data->mapped = TRUE; - data->iconic = (new_style & WS_MINIMIZE) != 0; + remove_startup_notification( data->display, data->whole_window ); + set_wm_hints( data ); + + if (!data->embedded) + { + update_net_wm_states( data ); + sync_window_style( data ); + XMapWindow( data->display, data->whole_window ); + } + else set_xembed_flags( data, XEMBED_MAPPED ); + + data->mapped = TRUE; + data->iconic = (new_style & WS_MINIMIZE) != 0; + } + release_win_data( data ); } @@ -1094,20 +1101,24 @@ static void map_window( HWND hwnd, DWORD new_style ) */ static void unmap_window( HWND hwnd ) { - struct x11drv_win_data *data = X11DRV_get_win_data( hwnd ); + struct x11drv_win_data *data; - TRACE( "win %p/%lx\n", data->hwnd, data->whole_window ); + wait_for_withdrawn_state( hwnd, FALSE ); + + if (!(data = get_win_data( hwnd ))) return; - if (!data->embedded) + if (data->mapped) { - wait_for_withdrawn_state( hwnd, FALSE ); - if (!data->managed) XUnmapWindow( data->display, data->whole_window ); + TRACE( "win %p/%lx\n", data->hwnd, data->whole_window ); + + if (data->embedded) set_xembed_flags( data, 0 ); + else if (!data->managed) XUnmapWindow( data->display, data->whole_window ); else XWithdrawWindow( data->display, data->whole_window, DefaultScreen(data->display) ); - } - else set_xembed_flags( data, 0 ); - data->mapped = FALSE; - data->net_wm_state = 0; + data->mapped = FALSE; + data->net_wm_state = 0; + } + release_win_data( data ); } @@ -1116,21 +1127,23 @@ static void unmap_window( HWND hwnd ) */ void make_window_embedded( HWND hwnd ) { - struct x11drv_win_data *data = X11DRV_get_win_data( hwnd ); - BOOL was_mapped = data->mapped; + struct x11drv_win_data *data = get_win_data( hwnd ); - /* the window cannot be mapped before being embedded */ - if (data->mapped) unmap_window( hwnd ); + if (!data) return; + /* the window cannot be mapped before being embedded */ + if (data->mapped) + { + if (data->managed) XUnmapWindow( data->display, data->whole_window ); + else XWithdrawWindow( data->display, data->whole_window, DefaultScreen(data->display) ); + data->net_wm_state = 0; + } data->embedded = TRUE; data->managed = TRUE; SetPropA( hwnd, managed_prop, (HANDLE)1 ); sync_window_style( data ); - - if (was_mapped) - map_window( hwnd, 0 ); - else - set_xembed_flags( data, 0 ); + set_xembed_flags( data, data->mapped ? XEMBED_MAPPED : 0 ); + release_win_data( data ); } @@ -2161,7 +2174,6 @@ void CDECL X11DRV_WindowPosChanged( HWND hwnd, HWND insert_after, UINT swp_flags { make_owner_managed( hwnd ); if (!data->icon_pixmap) fetch_icon_data( hwnd, 0, 0 ); - set_wm_hints( data ); map_window( hwnd, new_style ); } else if ((swp_flags & SWP_STATECHANGED) && (!data->iconic != !(new_style & WS_MINIMIZE)))

1 0

← Newer
1
...
24
25
26
27
28
29
30
...
83
Older →

Jump to page:

2025

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

2010

2009

2008

2007

2006

2005

2004

2003

2002

2001

wine-commits September 2012