comdlg32: Correctly handle filters with invalid extensions in Save As dialogs.
[wine] / dlls / comdlg32 / tests / filedlg.c
1 /*
2  * Unit test suite for comdlg32 API functions: file dialogs
3  *
4  * Copyright 2007 Google (Lei Zhang)
5  *
6  * This library is free software; you can redistribute it and/or
7  * modify it under the terms of the GNU Lesser General Public
8  * License as published by the Free Software Foundation; either
9  * version 2.1 of the License, or (at your option) any later version.
10  *
11  * This library is distributed in the hope that it will be useful,
12  * but WITHOUT ANY WARRANTY; without even the implied warranty of
13  * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the GNU
14  * Lesser General Public License for more details.
15  *
16  * You should have received a copy of the GNU Lesser General Public
17  * License along with this library; if not, write to the Free Software
18  * Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301, USA
19  *
20  */
21
22 #include <windows.h>
23 #include <wine/test.h>
24
25 #include "shlguid.h"
26 #define COBJMACROS
27 #include "shobjidl.h"
28
29 /* ##### */
30
31 static int resizesupported = TRUE;
32
33 static void toolbarcheck( HWND hDlg)
34 {
35     /* test toolbar properties */
36     /* bug #10532 */
37     int maxtextrows;
38     HWND ctrl;
39     DWORD ret;
40     char classname[20];
41
42     for( ctrl = GetWindow( hDlg, GW_CHILD);
43             ctrl ; ctrl = GetWindow( ctrl, GW_HWNDNEXT)) {
44         GetClassName( ctrl, classname, 10);
45         classname[7] = '\0';
46         if( !strcmp( classname, "Toolbar")) break;
47     }
48     ok( ctrl != NULL, "could not get the toolbar control\n");
49     ret = SendMessage( ctrl, TB_ADDSTRING, 0, (LPARAM)"winetestwinetest\0\0");
50     ok( ret == 0, "addstring returned %d (expected 0)\n", ret);
51     maxtextrows = SendMessage( ctrl, TB_GETTEXTROWS, 0, 0);
52     ok( maxtextrows == 0 || broken(maxtextrows == 1),  /* Win2k and below */
53         "Get(Max)TextRows returned %d (expected 0)\n", maxtextrows);
54 }
55
56
57 static UINT_PTR CALLBACK OFNHookProc( HWND hDlg, UINT msg, WPARAM wParam, LPARAM lParam)
58 {
59     LPNMHDR nmh;
60
61     if( msg == WM_NOTIFY)
62     {
63         nmh = (LPNMHDR) lParam;
64         if( nmh->code == CDN_INITDONE)
65         {
66             PostMessage( GetParent(hDlg), WM_COMMAND, IDCANCEL, FALSE);
67         } else if (nmh->code == CDN_FOLDERCHANGE )
68         {
69             char buf[1024];
70             int ret;
71
72             memset(buf, 0x66, sizeof(buf));
73             ret = SendMessage( GetParent(hDlg), CDM_GETFOLDERIDLIST, 5, (LPARAM)buf);
74             ok(ret > 0, "CMD_GETFOLDERIDLIST not implemented\n");
75             if (ret > 5)
76                 ok(buf[0] == 0x66 && buf[1] == 0x66, "CMD_GETFOLDERIDLIST: The buffer was touched on failure\n");
77             toolbarcheck( GetParent(hDlg));
78         }
79     }
80
81     return 0;
82 }
83
84 /* bug 6829 */
85 static void test_DialogCancel(void)
86 {
87     OPENFILENAMEA ofn;
88     BOOL result;
89     char szFileName[MAX_PATH] = "";
90     char szInitialDir[MAX_PATH];
91
92     GetWindowsDirectory(szInitialDir, MAX_PATH);
93
94     ZeroMemory(&ofn, sizeof(ofn));
95
96     ofn.lStructSize = sizeof(ofn);
97     ofn.hwndOwner = NULL;
98     ofn.lpstrFilter = "Text Files (*.txt)\0*.txt\0All Files (*.*)\0*.*\0";
99     ofn.lpstrFile = szFileName;
100     ofn.nMaxFile = MAX_PATH;
101     ofn.Flags = OFN_EXPLORER | OFN_FILEMUSTEXIST | OFN_HIDEREADONLY | OFN_ENABLEHOOK;
102     ofn.lpstrDefExt = "txt";
103     ofn.lpfnHook = OFNHookProc;
104     ofn.lpstrInitialDir = szInitialDir;
105
106     PrintDlgA(NULL);
107     ok(CDERR_INITIALIZATION == CommDlgExtendedError(),
108        "expected CDERR_INITIALIZATION, got %d\n", CommDlgExtendedError());
109
110     result = GetOpenFileNameA(&ofn);
111     ok(0 == result, "expected 0, got %d\n", result);
112     ok(0 == CommDlgExtendedError(), "expected 0, got %d\n",
113        CommDlgExtendedError());
114
115     PrintDlgA(NULL);
116     ok(CDERR_INITIALIZATION == CommDlgExtendedError(),
117        "expected CDERR_INITIALIZATION, got %d\n", CommDlgExtendedError());
118
119     result = GetSaveFileNameA(&ofn);
120     ok(0 == result, "expected 0, got %d\n", result);
121     ok(0 == CommDlgExtendedError(), "expected 0, got %d\n",
122        CommDlgExtendedError());
123
124     PrintDlgA(NULL);
125     ok(CDERR_INITIALIZATION == CommDlgExtendedError(),
126        "expected CDERR_INITIALIZATION, got %d\n", CommDlgExtendedError());
127
128     /* Before passing the ofn to Unicode functions, remove the ANSI strings */
129     ofn.lpstrFilter = NULL;
130     ofn.lpstrInitialDir = NULL;
131     ofn.lpstrDefExt = NULL;
132
133     PrintDlgA(NULL);
134     ok(CDERR_INITIALIZATION == CommDlgExtendedError(),
135        "expected CDERR_INITIALIZATION, got %d\n", CommDlgExtendedError());
136
137     SetLastError(0xdeadbeef);
138     result = GetOpenFileNameW((LPOPENFILENAMEW) &ofn);
139     if (GetLastError() == ERROR_CALL_NOT_IMPLEMENTED)
140         win_skip("GetOpenFileNameW is not implemented\n");
141     else
142     {
143         ok(0 == result, "expected 0, got %d\n", result);
144         ok(0 == CommDlgExtendedError(), "expected 0, got %d\n", CommDlgExtendedError());
145     }
146
147     SetLastError(0xdeadbeef);
148     result = GetSaveFileNameW((LPOPENFILENAMEW) &ofn);
149     if (GetLastError() == ERROR_CALL_NOT_IMPLEMENTED)
150         win_skip("GetSaveFileNameW is not implemented\n");
151     else
152     {
153         ok(0 == result, "expected 0, got %d\n", result);
154         ok(0 == CommDlgExtendedError(), "expected 0, got %d\n", CommDlgExtendedError());
155     }
156 }
157
158 static UINT_PTR CALLBACK create_view_window2_hook(HWND dlg, UINT msg, WPARAM wParam, LPARAM lParam)
159 {
160     if (msg == WM_NOTIFY)
161     {
162         if (((LPNMHDR)lParam)->code == CDN_FOLDERCHANGE)
163         {
164             IShellBrowser *shell_browser = (IShellBrowser *)SendMessage(GetParent(dlg), WM_USER + 7 /* WM_GETISHELLBROWSER */, 0, 0);
165             IShellView *shell_view = NULL;
166             IShellView2 *shell_view2 = NULL;
167             SV2CVW2_PARAMS view_params;
168             FOLDERSETTINGS folder_settings;
169             HRESULT hr;
170             RECT rect = {0, 0, 0, 0};
171
172             hr = IShellBrowser_QueryActiveShellView(shell_browser, &shell_view);
173             ok(SUCCEEDED(hr), "QueryActiveShellView returned %#x\n", hr);
174             if (FAILED(hr)) goto cleanup;
175
176             hr = IShellView_QueryInterface(shell_view, &IID_IShellView2, (void **)&shell_view2);
177             if (hr == E_NOINTERFACE)
178             {
179                 win_skip("IShellView2 not supported\n");
180                 goto cleanup;
181             }
182             ok(SUCCEEDED(hr), "QueryInterface returned %#x\n", hr);
183             if (FAILED(hr)) goto cleanup;
184
185             hr = IShellView2_DestroyViewWindow(shell_view2);
186             ok(SUCCEEDED(hr), "DestroyViewWindow returned %#x\n", hr);
187
188             folder_settings.ViewMode = FVM_LIST;
189             folder_settings.fFlags = 0;
190
191             view_params.cbSize = sizeof(view_params);
192             view_params.psvPrev = NULL;
193             view_params.pfs = &folder_settings;
194             view_params.psbOwner = shell_browser;
195             view_params.prcView = &rect;
196             view_params.pvid = NULL;
197             view_params.hwndView = NULL;
198
199             hr = IShellView2_CreateViewWindow2(shell_view2, &view_params);
200             if (hr == E_FAIL)
201             {
202                 win_skip("CreateViewWindow2 is broken on Vista/W2K8\n");
203                 goto cleanup;
204             }
205             ok(SUCCEEDED(hr), "CreateViewWindow2 returned %#x\n", hr);
206             if (FAILED(hr)) goto cleanup;
207
208             hr = IShellView2_GetCurrentInfo(shell_view2, &folder_settings);
209             ok(SUCCEEDED(hr), "GetCurrentInfo returned %#x\n", hr);
210             ok(folder_settings.ViewMode == FVM_LIST,
211                "view mode is %d, expected FVM_LIST\n",
212                folder_settings.ViewMode);
213
214             hr = IShellView2_DestroyViewWindow(shell_view2);
215             ok(SUCCEEDED(hr), "DestroyViewWindow returned %#x\n", hr);
216
217             /* XP and W2K3 need this. On W2K the call to DestroyWindow() fails and has
218              * no side effects. NT4 doesn't get here. (FIXME: Vista doesn't get here yet).
219              */
220             DestroyWindow(view_params.hwndView);
221
222             view_params.pvid = &VID_Details;
223             hr = IShellView2_CreateViewWindow2(shell_view2, &view_params);
224             ok(SUCCEEDED(hr), "CreateViewWindow2 returned %#x\n", hr);
225             if (FAILED(hr)) goto cleanup;
226
227             hr = IShellView2_GetCurrentInfo(shell_view2, &folder_settings);
228             ok(SUCCEEDED(hr), "GetCurrentInfo returned %#x\n", hr);
229             ok(folder_settings.ViewMode == FVM_DETAILS || broken(folder_settings.ViewMode == FVM_LIST), /* nt4 */
230                "view mode is %d, expected FVM_DETAILS\n",
231                folder_settings.ViewMode);
232
233 cleanup:
234             if (shell_view2) IShellView2_Release(shell_view2);
235             if (shell_view) IShellView_Release(shell_view);
236             PostMessage(GetParent(dlg), WM_COMMAND, IDCANCEL, 0);
237         }
238     }
239     return 0;
240 }
241
242 static LONG_PTR WINAPI template_hook(HWND dlg, UINT msg, WPARAM wParam, LPARAM lParam)
243 {
244     if (msg == WM_INITDIALOG)
245     {
246         HWND p,cb;
247         INT sel;
248         p = GetParent(dlg);
249         ok(p!=NULL, "Failed to get parent of template\n");
250         cb = GetDlgItem(p,0x470);
251         ok(cb!=NULL, "Failed to get filter combobox\n");
252         sel = SendMessage(cb, CB_GETCURSEL, 0, 0);
253         ok (sel != -1, "Failed to get selection from filter listbox\n");
254     }
255     if (msg == WM_NOTIFY)
256     {
257         if (((LPNMHDR)lParam)->code == CDN_FOLDERCHANGE)
258             PostMessage(GetParent(dlg), WM_COMMAND, IDCANCEL, 0);
259     }
260     return 0;
261 }
262
263 static void test_create_view_window2(void)
264 {
265     OPENFILENAMEA ofn = {0};
266     char filename[1024] = {0};
267     DWORD ret;
268
269     ofn.lStructSize = sizeof(ofn);
270     ofn.lpstrFile = filename;
271     ofn.nMaxFile = 1024;
272     ofn.lpfnHook = create_view_window2_hook;
273     ofn.Flags = OFN_ENABLEHOOK | OFN_EXPLORER;
274     ret = GetOpenFileNameA(&ofn);
275     ok(!ret, "GetOpenFileNameA returned %#x\n", ret);
276     ret = CommDlgExtendedError();
277     ok(!ret, "CommDlgExtendedError returned %#x\n", ret);
278 }
279
280 static void test_create_view_template(void)
281 {
282     OPENFILENAMEA ofn = {0};
283     char filename[1024] = {0};
284     DWORD ret;
285
286     ofn.lStructSize = sizeof(ofn);
287     ofn.lpstrFile = filename;
288     ofn.nMaxFile = 1024;
289     ofn.lpfnHook = (LPOFNHOOKPROC)template_hook;
290     ofn.Flags = OFN_ENABLEHOOK | OFN_EXPLORER| OFN_ENABLETEMPLATE;
291     ofn.hInstance = GetModuleHandleA(NULL);
292     ofn.lpTemplateName = "template1";
293     ofn.lpstrFilter="text\0*.txt\0All\0*\0\0";
294     ret = GetOpenFileNameA(&ofn);
295     ok(!ret, "GetOpenFileNameA returned %#x\n", ret);
296     ret = CommDlgExtendedError();
297     ok(!ret, "CommDlgExtendedError returned %#x\n", ret);
298 }
299
300 /* test cases for resizing of the file dialog */
301 static const struct {
302     DWORD flags;
303     int resize_folderchange;/* change in CDN_FOLDERCHANGE handler */
304     int resize_timer1;      /* change in first WM_TIMER handler */
305     int resize_check;       /* expected change (in second  WM_TIMER handler) */
306     BOOL todo;              /* mark that test todo_wine */
307     BOOL testcontrols;      /* test resizing and moving of the controls */
308 } resize_testcases[] = {
309     { 0                , 10, 10, 20,FALSE,FALSE},   /* 0 */
310     { 0                ,-10,-10,-20,FALSE,FALSE},
311     { OFN_ENABLESIZING ,  0,  0,  0,FALSE,FALSE},
312     { OFN_ENABLESIZING ,  0,-10,  0,FALSE,FALSE},
313     { OFN_ENABLESIZING ,  0, 10, 10,FALSE, TRUE},
314     { OFN_ENABLESIZING ,-10,  0, 10,FALSE,FALSE},   /* 5 */
315     { OFN_ENABLESIZING , 10,  0, 10,FALSE,FALSE},
316     { OFN_ENABLESIZING ,  0, 10, 20,FALSE,FALSE},
317     /* mark the end */
318     { 0xffffffff }
319 };
320
321 static LONG_PTR WINAPI resize_template_hook(HWND dlg, UINT msg, WPARAM wParam, LPARAM lParam)
322 {
323     static RECT initrc, rc;
324     static int index, count;
325     static int gotSWP_bottom, gotShowWindow;
326     HWND parent = GetParent( dlg);
327     int resize;
328 #define MAXNRCTRLS 30
329     static RECT ctrlrcs[MAXNRCTRLS];
330     static int ctrlids[MAXNRCTRLS];
331     static HWND ctrls[MAXNRCTRLS];
332     static int nrctrls;
333
334     switch( msg)
335     {
336         case WM_INITDIALOG:
337         {
338             DWORD style;
339
340             index = ((OPENFILENAME*)lParam)->lCustData;
341             count = 0;
342             gotSWP_bottom = gotShowWindow = 0;
343             /* test style */
344             style = GetWindowLong( parent, GWL_STYLE);
345             if( resize_testcases[index].flags & OFN_ENABLESIZING)
346                 if( !(style & WS_SIZEBOX)) {
347                     win_skip( "OFN_ENABLESIZING flag not supported.\n");
348                     resizesupported = FALSE;
349                     PostMessage( parent, WM_COMMAND, IDCANCEL, 0);
350                 } else
351                     ok( style & WS_SIZEBOX,
352                             "testid %d: dialog should have a WS_SIZEBOX style.\n", index);
353             else
354                 ok( !(style & WS_SIZEBOX),
355                         "testid %d: dialog should not have a WS_SIZEBOX style.\n", index);
356             break;
357         }
358         case WM_NOTIFY:
359         {
360             if(( (LPNMHDR)lParam)->code == CDN_FOLDERCHANGE){
361                 GetWindowRect( parent, &initrc);
362                 if( (resize  = resize_testcases[index].resize_folderchange)){
363                     MoveWindow( parent, initrc.left,initrc.top, initrc.right - initrc.left + resize,
364                             initrc.bottom - initrc.top + resize, TRUE);
365                 }
366                 SetTimer( dlg, 0, 100, 0);
367             }
368             break;
369         }
370         case WM_TIMER:
371         {
372             if( count == 0){
373                 /* store the control rectangles */
374                 if( resize_testcases[index].testcontrols) {
375                     HWND ctrl;
376                     int i;
377                     for( i = 0, ctrl = GetWindow( parent, GW_CHILD);
378                             i < MAXNRCTRLS && ctrl;
379                             i++, ctrl = GetWindow( ctrl, GW_HWNDNEXT)) {
380                         ctrlids[i] = GetDlgCtrlID( ctrl);
381                         GetWindowRect( ctrl, &ctrlrcs[i]);
382                         MapWindowPoints( NULL, parent, (LPPOINT) &ctrlrcs[i], 2);
383                         ctrls[i] = ctrl;
384                     }
385                     nrctrls = i;
386                 }
387                 if( (resize  = resize_testcases[index].resize_timer1)){
388                     GetWindowRect( parent, &rc);
389                     MoveWindow( parent, rc.left,rc.top, rc.right - rc.left + resize,
390                             rc.bottom - rc.top + resize, TRUE);
391                 }
392             } else if( count == 1){
393                 resize  = resize_testcases[index].resize_check;
394                 GetWindowRect( parent, &rc);
395                 if( resize_testcases[index].todo){
396                     todo_wine {
397                         ok( resize == rc.right - rc.left - initrc.right + initrc.left,
398                             "testid %d size-x change %d expected %d\n", index,
399                             rc.right - rc.left - initrc.right + initrc.left, resize);
400                         ok( resize == rc.bottom - rc.top - initrc.bottom + initrc.top,
401                             "testid %d size-y change %d expected %d\n", index,
402                             rc.bottom - rc.top - initrc.bottom + initrc.top, resize);
403                     }
404                 }else{
405                     ok( resize == rc.right - rc.left - initrc.right + initrc.left,
406                         "testid %d size-x change %d expected %d\n", index,
407                         rc.right - rc.left - initrc.right + initrc.left, resize);
408                     ok( resize == rc.bottom - rc.top - initrc.bottom + initrc.top,
409                         "testid %d size-y change %d expected %d\n", index,
410                         rc.bottom - rc.top - initrc.bottom + initrc.top, resize);
411                 }
412                 if( resize_testcases[index].testcontrols) {
413                     int i;
414                     RECT rc;
415                     for( i = 0; i < nrctrls; i++) {
416                         GetWindowRect( ctrls[i], &rc);
417                         MapWindowPoints( NULL, parent, (LPPOINT) &rc, 2);
418                         switch( ctrlids[i]){
419
420 /* test if RECT R1, moved and sized result in R2 */
421 #define TESTRECTS( R1, R2, Mx, My, Sx, Sy) \
422          ((R1).left + (Mx) ==(R2).left \
423         &&(R1).top + (My) ==(R2).top \
424         &&(R1).right + (Mx) + (Sx) == (R2).right \
425         &&(R1).bottom + (My) + (Sy) ==(R2).bottom)
426
427                             /* sized horizontal and moved vertical */
428                             case cmb1:
429                             case edt1:
430                                 ok( TESTRECTS( ctrlrcs[i], rc, 0, 10, 10, 0),
431                                     "control id %03x should have sized horizontally and moved vertically, before %d,%d-%d,%d after  %d,%d-%d,%d\n",
432                                     ctrlids[i], ctrlrcs[i].left, ctrlrcs[i].top,
433                                     ctrlrcs[i].right, ctrlrcs[i].bottom,
434                                     rc.left, rc.top, rc.right, rc.bottom);
435                                 break;
436                             /* sized horizontal and vertical */
437                             case lst2:
438                                 ok( TESTRECTS( ctrlrcs[i], rc, 0, 0, 10, 10),
439                                     "control id %03x should have sized horizontally and vertically, before %d,%d-%d,%d after  %d,%d-%d,%d\n",
440                                     ctrlids[i], ctrlrcs[i].left, ctrlrcs[i].top,
441                                     ctrlrcs[i].right, ctrlrcs[i].bottom,
442                                     rc.left, rc.top, rc.right, rc.bottom);
443                                 break;
444                             /* moved horizontal and vertical */
445                             case IDCANCEL:
446                             case pshHelp:
447                                 ok( TESTRECTS( ctrlrcs[i], rc, 10, 10, 0, 0),
448                                     "control id %03x should have moved horizontally and vertically, before %d,%d-%d,%d after  %d,%d-%d,%d\n",
449                                     ctrlids[i], ctrlrcs[i].left, ctrlrcs[i].top,
450                                     ctrlrcs[i].right, ctrlrcs[i].bottom,
451                                     rc.left, rc.top, rc.right, rc.bottom);
452                                 break;
453                             /* moved vertically */
454                             case chx1:
455                             case stc2:
456                             case stc3:
457                                 ok( TESTRECTS( ctrlrcs[i], rc, 0, 10, 0, 0),
458                                     "control id %03x should have moved vertically, before %d,%d-%d,%d after  %d,%d-%d,%d\n",
459                                     ctrlids[i], ctrlrcs[i].left, ctrlrcs[i].top,
460                                     ctrlrcs[i].right, ctrlrcs[i].bottom,
461                                     rc.left, rc.top, rc.right, rc.bottom);
462                                 break;
463                             /* resized horizontal */
464                             case cmb2: /* aka IDC_LOOKIN */
465                                 ok( TESTRECTS( ctrlrcs[i], rc, 0, 0, 10, 0)||
466                                         TESTRECTS( ctrlrcs[i], rc, 0, 0, 0, 0), /* Vista and higher */
467                                     "control id %03x should have resized horizontally, before %d,%d-%d,%d after  %d,%d-%d,%d\n",
468                                     ctrlids[i], ctrlrcs[i].left, ctrlrcs[i].top,
469                                     ctrlrcs[i].right, ctrlrcs[i].bottom,
470                                     rc.left, rc.top, rc.right, rc.bottom);
471                                 break;
472                             /* non moving non sizing controls */
473                             case stc4:
474                                 ok( TESTRECTS( rc, ctrlrcs[i], 0, 0, 0, 0),
475                                     "control id %03x was moved/resized, before %d,%d-%d,%d after  %d,%d-%d,%d\n",
476                                     ctrlids[i], ctrlrcs[i].left, ctrlrcs[i].top,
477                                     ctrlrcs[i].right, ctrlrcs[i].bottom,
478                                     rc.left, rc.top, rc.right, rc.bottom);
479                                 break;
480                             /* todo_wine: non moving non sizing controls */
481                             case lst1:
482 todo_wine
483                                 ok( TESTRECTS( rc, ctrlrcs[i], 0, 0, 0, 0),
484                                     "control id %03x was moved/resized, before %d,%d-%d,%d after  %d,%d-%d,%d\n",
485                                     ctrlids[i], ctrlrcs[i].left, ctrlrcs[i].top,
486                                     ctrlrcs[i].right, ctrlrcs[i].bottom,
487                                     rc.left, rc.top, rc.right, rc.bottom);
488                                 break;
489                             /* don't test: id is not unique */
490                             case IDOK:
491                             case stc1:
492                             case 0:
493                             case  -1:
494                                 break;
495                             default:
496                                 trace("untested control id %03x before %d,%d-%d,%d after  %d,%d-%d,%d\n",
497                                     ctrlids[i], ctrlrcs[i].left, ctrlrcs[i].top,
498                                     ctrlrcs[i].right, ctrlrcs[i].bottom,
499                                     rc.left, rc.top, rc.right, rc.bottom);
500 #undef TESTRECTS
501 #undef MAXNRCTRLS
502                         }
503                     }
504                 }
505                 KillTimer( dlg, 0);
506                 PostMessage( parent, WM_COMMAND, IDCANCEL, 0);
507             }
508             count++;
509         }
510         break;
511         case WM_WINDOWPOSCHANGING:
512         {
513             WINDOWPOS *pwp = (WINDOWPOS *)lParam;
514             if(  !index && pwp->hwndInsertAfter == HWND_BOTTOM){
515                 gotSWP_bottom = 1;
516                 ok( gotShowWindow == 0, "The WM_WINDOWPOSCHANGING message came after a WM_SHOWWINDOW message\n");
517             }
518         }
519         break;
520         case WM_SHOWWINDOW:
521         {
522             if(  !index){
523                 gotShowWindow = 1;
524                 ok( gotSWP_bottom == 1, "No WM_WINDOWPOSCHANGING message came before a WM_SHOWWINDOW message\n");
525             }
526         }
527         break;
528     }
529     return 0;
530 }
531
532 static void test_resize(void)
533 {
534     OPENFILENAME ofn = { sizeof(OPENFILENAME)};
535     char filename[1024] = {0};
536     DWORD ret;
537     int i;
538
539     ofn.lpstrFile = filename;
540     ofn.nMaxFile = 1024;
541     ofn.lpfnHook = (LPOFNHOOKPROC) resize_template_hook;
542     ofn.hInstance = GetModuleHandle(NULL);
543     ofn.lpTemplateName = "template_sz";
544     for( i = 0; resize_testcases[i].flags != 0xffffffff; i++) {
545         ofn.lCustData = i;
546         ofn.Flags = resize_testcases[i].flags |
547             OFN_ENABLEHOOK | OFN_EXPLORER| OFN_ENABLETEMPLATE | OFN_SHOWHELP ;
548         ret = GetOpenFileName(&ofn);
549         ok(!ret, "GetOpenFileName returned %#x\n", ret);
550         ret = CommDlgExtendedError();
551         ok(!ret, "CommDlgExtendedError returned %#x\n", ret);
552     }
553 }
554
555 /* test cases for control message IDOK */
556 /* Show case for bug #19079 */
557 typedef struct {
558     int  retval;        /* return code of the message handler */
559     BOOL setmsgresult;  /* set the result in the DWLP_MSGRESULT */
560     BOOL usemsgokstr;   /* use the FILEOKSTRING message instead of WM_NOTIFY:CDN_FILEOK */
561     BOOL do_subclass;   /* subclass the dialog hook procedure */
562     BOOL expclose;      /* is the dialog expected to close ? */
563     BOOL actclose;      /* has the dialog actually closed ? */
564 } ok_wndproc_testcase;
565
566 static ok_wndproc_testcase ok_testcases[] = {
567     { 0,        FALSE,  FALSE,  FALSE,  TRUE},
568     { 0,         TRUE,  FALSE,  FALSE,  TRUE},
569     { 0,        FALSE,  FALSE,   TRUE,  TRUE},
570     { 0,         TRUE,  FALSE,   TRUE,  TRUE},
571     { 1,        FALSE,  FALSE,  FALSE,  TRUE},
572     { 1,         TRUE,  FALSE,  FALSE, FALSE},
573     { 1,        FALSE,  FALSE,   TRUE, FALSE},
574     { 1,         TRUE,  FALSE,   TRUE, FALSE},
575     /* FILEOKSTRING tests */
576     { 1,         TRUE,   TRUE,  FALSE, FALSE},
577     { 1,        FALSE,   TRUE,   TRUE, FALSE},
578     /* mark the end */
579     { -1 }
580 };
581
582 /* test_ok_wndproc can be used as hook procedure or a subclass
583  * window proc for the file dialog */
584 static LONG_PTR WINAPI test_ok_wndproc(HWND dlg, UINT msg, WPARAM wParam, LPARAM lParam)
585 {
586     HWND parent = GetParent( dlg);
587     static ok_wndproc_testcase *testcase = NULL;
588     static UINT msgFILEOKSTRING;
589     if (msg == WM_INITDIALOG)
590     {
591         testcase = (ok_wndproc_testcase*)((OPENFILENAME*)lParam)->lCustData;
592         testcase->actclose = TRUE;
593         msgFILEOKSTRING = RegisterWindowMessageA( FILEOKSTRING);
594     }
595     if( msg == WM_NOTIFY) {
596         if(((LPNMHDR)lParam)->code == CDN_FOLDERCHANGE) {
597             SetTimer( dlg, 0, 100, 0);
598             PostMessage( parent, WM_COMMAND, IDOK, 0);
599             return FALSE;
600         } else if(((LPNMHDR)lParam)->code == CDN_FILEOK) {
601             if( testcase->usemsgokstr)
602                 return FALSE;
603             if( testcase->setmsgresult)
604                 SetWindowLongPtrA( dlg, DWLP_MSGRESULT, testcase->retval);
605             return testcase->retval;
606         }
607     }
608     if( msg == msgFILEOKSTRING) {
609         if( !testcase->usemsgokstr)
610             return FALSE;
611         if( testcase->setmsgresult)
612             SetWindowLongPtrA( dlg, DWLP_MSGRESULT, testcase->retval);
613         return testcase->retval;
614     }
615     if( msg == WM_TIMER) {
616         /* the dialog did not close automatically */
617         testcase->actclose = FALSE;
618         KillTimer( dlg, 0);
619         PostMessage( parent, WM_COMMAND, IDCANCEL, 0);
620         return FALSE;
621     }
622     if( testcase && testcase->do_subclass)
623         return DefWindowProc( dlg, msg, wParam, lParam);
624     return FALSE;
625 }
626
627 static LONG_PTR WINAPI ok_template_hook(HWND dlg, UINT msg, WPARAM wParam, LPARAM lParam)
628 {
629     if (msg == WM_SETFONT)
630         SetWindowLongPtrA( dlg, GWLP_WNDPROC, (LONG_PTR) test_ok_wndproc);
631     return FALSE;
632 }
633
634 static void test_ok(void)
635 {
636     OPENFILENAME ofn = { sizeof(OPENFILENAME)};
637     char filename[1024] = {0};
638     char tmpfilename[ MAX_PATH];
639     char curdir[MAX_PATH];
640     int i;
641     DWORD ret;
642     BOOL cdret;
643
644     cdret = GetCurrentDirectoryA(sizeof(curdir), curdir);
645     ok(cdret, "Failed to get current dir err %d\n", GetLastError());
646     if (!GetTempFileNameA(".", "txt", 0, tmpfilename)) {
647         skip("Failed to create a temporary file name\n");
648         return;
649     }
650     ofn.lpstrFile = filename;
651     ofn.nMaxFile = 1024;
652     ofn.hInstance = GetModuleHandle(NULL);
653     ofn.lpTemplateName = "template1";
654     ofn.Flags =  OFN_ENABLEHOOK | OFN_EXPLORER| OFN_ENABLETEMPLATE ;
655     for( i = 0; ok_testcases[i].retval != -1; i++) {
656         strcpy( filename, tmpfilename);
657         ofn.lCustData = (LPARAM)(ok_testcases + i);
658         ofn.lpfnHook = ok_testcases[i].do_subclass
659             ? (LPOFNHOOKPROC) ok_template_hook
660             : (LPOFNHOOKPROC) test_ok_wndproc;
661         ret = GetOpenFileNameA(&ofn);
662         ok( ok_testcases[i].expclose == ok_testcases[i].actclose,
663                 "testid %d: Open File dialog should %shave closed.\n", i,
664                 ok_testcases[i].expclose ? "" : "NOT ");
665         ok(ret == ok_testcases[i].expclose, "testid %d: GetOpenFileName returned %#x\n", i, ret);
666         ret = CommDlgExtendedError();
667         ok(!ret, "CommDlgExtendedError returned %#x\n", ret);
668         cdret = SetCurrentDirectoryA(curdir);
669         ok(cdret, "Failed to restore current dir err %d\n", GetLastError());
670     }
671     ret =  DeleteFileA( tmpfilename);
672     ok( ret, "Failed to delete temporary file %s err %d\n", tmpfilename, GetLastError());
673 }
674
675 /* test arranging with a custom template */
676 typedef struct {
677     int x, y;  /* left, top coordinates */
678     int cx, cy; /* width and height */
679 } posz;
680 static struct {
681    int nrcontrols;    /* 0: no controls, 1: just the stc32 control 2: with button */
682    posz poszDlg;
683    posz poszStc32;
684    posz poszBtn;
685    DWORD ofnflags;
686 } arrange_tests[] = {
687     /* do not change the first two cases: used to get the uncustomized sizes */
688     { 0, {0},{0},{0},0 },
689     { 0, {0},{0},{0}, OFN_SHOWHELP},
690     /* two tests with just a subdialog, no controls */
691     { 0, {0, 0, 316, 76},{0},{0},0 },
692     { 0, {0, 0, 100, 76},{0},{0}, OFN_SHOWHELP},
693     /* now with a control with id stc32 */
694     { 1, {0, 0, 316, 76} ,{0, 0, 204, 76,},{0},0 }, /* bug #17748*/
695     { 1, {0, 0, 316, 76} ,{0, 0, 204, 76,},{0}, OFN_SHOWHELP}, /* bug #17748*/
696     /* tests with size of the stc32 control higher or wider then the standard dialog */
697     { 1, {0, 0, 316, 170} ,{0, 0, 204, 170,},{0},0 },
698     { 1, {0, 0, 316, 165} ,{0, 0, 411, 165,},{0}, OFN_SHOWHELP },
699     /* move the stc32 control around */
700     { 1, {0, 0, 300, 100} ,{73, 17, 50, 50,},{0},0 },
701     /* add control */
702     { 2, {0, 0, 280, 100} ,{0, 0, 50, 50,},{300,20,30,30},0 },
703     /* enable resizing should make the dialog bigger */
704     { 0, {0},{0},{0}, OFN_SHOWHELP|OFN_ENABLESIZING},
705     /* mark the end */
706     { -1 }
707 };
708
709 static LONG_PTR WINAPI template_hook_arrange(HWND dlgChild, UINT msg, WPARAM wParam, LPARAM lParam)
710 {
711     static int index, fixhelp;
712     static posz posz0[2];
713     static RECT clrcParent, clrcChild, rcStc32;
714     static HWND hwndStc32;
715     HWND dlgParent;
716
717     dlgParent = GetParent( dlgChild);
718     if (msg == WM_INITDIALOG) {
719         index = ((OPENFILENAME*)lParam)->lCustData;
720         /* get the positions before rearrangement */
721         GetClientRect( dlgParent, &clrcParent);
722         GetClientRect( dlgChild, &clrcChild);
723         hwndStc32 = GetDlgItem( dlgChild, stc32);
724         if( hwndStc32)  GetWindowRect( hwndStc32, &rcStc32);
725     }
726     if (msg == WM_NOTIFY && ((LPNMHDR)lParam)->code == CDN_FOLDERCHANGE) {
727         RECT wrcParent;
728
729         GetWindowRect( dlgParent, &wrcParent);
730         /* the fist two "tests" just save the dialogs position, with and without
731          * help button */
732         if( index == 0) {
733             posz0[0].x = wrcParent.left;
734             posz0[0].y = wrcParent.top;
735             posz0[0].cx = wrcParent.right - wrcParent.left;
736             posz0[0].cy = wrcParent.bottom - wrcParent.top;
737         } else if( index == 1) {
738             posz0[1].x = wrcParent.left;
739             posz0[1].y = wrcParent.top;
740             posz0[1].cx = wrcParent.right - wrcParent.left;
741             posz0[1].cy = wrcParent.bottom - wrcParent.top;
742             fixhelp = posz0[1].cy - posz0[0].cy;
743         } else {
744             /* the real tests */
745             int withhelp;
746             int expectx, expecty;
747             DWORD style;
748
749             withhelp = (arrange_tests[index].ofnflags & OFN_SHOWHELP) != 0;
750             GetWindowRect( dlgParent, &wrcParent);
751             if( !hwndStc32) {
752                 /* case with no custom subitem with stc32:
753                  * default to all custom controls below the standard */
754                 expecty = posz0[withhelp].cy + clrcChild.bottom;
755                 expectx =  posz0[withhelp].cx;
756             } else {
757                 /* special case: there is a control with id stc32 */
758                 /* expected height */
759                 expecty = posz0[withhelp].cy;
760                 if( rcStc32.bottom - rcStc32.top + (withhelp ? 0 : fixhelp) > clrcParent.bottom) {
761                     expecty +=  clrcChild.bottom -  clrcParent.bottom;
762                     if( !withhelp) expecty += fixhelp;
763                 }
764                 else
765                     expecty +=  clrcChild.bottom - ( rcStc32.bottom - rcStc32.top) ;
766                 /* expected width */
767                 expectx = posz0[withhelp].cx;
768                 if( rcStc32.right - rcStc32.left > clrcParent.right) {
769                     expectx +=  clrcChild.right -  clrcParent.right;
770                 }
771                 else
772                     expectx +=  clrcChild.right - ( rcStc32.right - rcStc32.left) ;
773             }
774             style = GetWindowLong( dlgParent, GWL_STYLE);
775             if( !(style & WS_SIZEBOX)) {
776                 /* without the OFN_ENABLESIZING flag */
777                 ok( wrcParent.bottom - wrcParent.top == expecty,
778                         "Wrong height of dialog %d, expected %d\n",
779                         wrcParent.bottom - wrcParent.top, expecty);
780                 ok( wrcParent.right - wrcParent.left == expectx,
781                         "Wrong width of dialog %d, expected %d\n",
782                         wrcParent.right - wrcParent.left, expectx);
783             } else todo_wine {
784                 /* with the OFN_ENABLESIZING flag */
785                 ok( wrcParent.bottom - wrcParent.top > expecty,
786                         "Wrong height of dialog %d, expected more than %d\n",
787                         wrcParent.bottom - wrcParent.top, expecty);
788                 ok( wrcParent.right - wrcParent.left > expectx,
789                         "Wrong width of dialog %d, expected more than %d\n",
790                         wrcParent.right - wrcParent.left, expectx);
791             }
792
793         }
794         PostMessage( dlgParent, WM_COMMAND, IDCANCEL, 0);
795     }
796     return 0;
797 }
798
799 static void test_arrange(void)
800 {
801     OPENFILENAMEA ofn = {0};
802     char filename[1024] = {0};
803     DWORD ret;
804     HRSRC hRes;
805     HANDLE hDlgTmpl;
806     LPBYTE pv;
807     DLGTEMPLATE *template;
808     DLGITEMTEMPLATE *itemtemplateStc32, *itemtemplateBtn;
809     int i;
810
811     /* load subdialog template into memory */
812     hRes = FindResource( GetModuleHandle(NULL), "template_stc32", (LPSTR)RT_DIALOG);
813     hDlgTmpl = LoadResource( GetModuleHandle(NULL), hRes );
814     /* get pointers to the structures for the dialog and the controls */
815     pv = LockResource( hDlgTmpl );
816     template = (DLGTEMPLATE*)pv;
817     if( template->x != 11111) {
818         win_skip("could not find the dialog template\n");
819         return;
820     }
821     /* skip dialog template, menu, class and title */
822     pv +=  sizeof(DLGTEMPLATE);
823     pv += 3 * sizeof(WORD);
824     /* skip font info */
825     while( *(WORD*)pv)
826         pv += sizeof(WORD);
827     pv += sizeof(WORD);
828     /* align on 32 bit boundaries */
829     pv = (LPBYTE)(((UINT_PTR)pv + 3 ) & ~3);
830     itemtemplateStc32 = (DLGITEMTEMPLATE*)pv;
831     if( itemtemplateStc32->x != 22222) {
832         win_skip("could not find the first item template\n");
833         return;
834     }
835     /* skip itemtemplate, class, title and creation data */
836     pv += sizeof(DLGITEMTEMPLATE);
837     pv +=  4 * sizeof(WORD);
838     /* align on 32 bit boundaries */
839     pv = (LPBYTE)(((UINT_PTR)pv + 3 ) & ~3);
840     itemtemplateBtn = (DLGITEMTEMPLATE*)pv;
841     if( itemtemplateBtn->x != 12345) {
842         win_skip("could not find the second item template\n");
843         return;
844     }
845
846     ofn.lStructSize = sizeof(ofn);
847     ofn.lpstrFile = filename;
848     ofn.nMaxFile = 1024;
849     ofn.lpfnHook = (LPOFNHOOKPROC)template_hook_arrange;
850     ofn.hInstance = hDlgTmpl;
851     ofn.lpstrFilter="text\0*.txt\0All\0*\0\0";
852     for( i = 0; arrange_tests[i].nrcontrols != -1; i++) {
853         ofn.lCustData = i;
854         ofn.Flags = OFN_ENABLEHOOK | OFN_EXPLORER| OFN_ENABLETEMPLATEHANDLE | OFN_HIDEREADONLY |
855             arrange_tests[i].ofnflags;
856         template->cdit = arrange_tests[i].nrcontrols;
857         template->x = arrange_tests[i].poszDlg.x;
858         template->y = arrange_tests[i].poszDlg.y;
859         template->cx = arrange_tests[i].poszDlg.cx;
860         template->cy = arrange_tests[i].poszDlg.cy;
861         itemtemplateStc32->x = arrange_tests[i].poszStc32.x;
862         itemtemplateStc32->y = arrange_tests[i].poszStc32.y;
863         itemtemplateStc32->cx = arrange_tests[i].poszStc32.cx;
864         itemtemplateStc32->cy = arrange_tests[i].poszStc32.cy;
865         itemtemplateBtn->x = arrange_tests[i].poszBtn.x;
866         itemtemplateBtn->y = arrange_tests[i].poszBtn.y;
867         itemtemplateBtn->cx = arrange_tests[i].poszBtn.cx;
868         itemtemplateBtn->cy = arrange_tests[i].poszBtn.cy;
869         ret = GetOpenFileNameA(&ofn);
870         ok(!ret, "GetOpenFileNameA returned %#x\n", ret);
871         ret = CommDlgExtendedError();
872         ok(!ret, "CommDlgExtendedError returned %#x\n", ret);
873     }
874 }
875
876 static CHAR SYSDIR[MAX_PATH];
877
878 static UINT_PTR CALLBACK path_hook_proc( HWND hDlg, UINT msg, WPARAM wParam, LPARAM lParam)
879 {
880     LPNMHDR nmh;
881
882     if( msg == WM_NOTIFY)
883     {
884         nmh = (LPNMHDR) lParam;
885         if( nmh->code == CDN_INITDONE)
886         {
887             PostMessage( GetParent(hDlg), WM_COMMAND, IDCANCEL, FALSE);
888         }
889         else if ( nmh->code == CDN_FOLDERCHANGE)
890         {
891             char buf[1024];
892             int ret;
893
894             memset(buf, 0x66, sizeof(buf));
895             ret = SendMessageA( GetParent(hDlg), CDM_GETFOLDERPATH, sizeof(buf), (LPARAM)buf);
896             ok(!lstrcmpiA(SYSDIR, buf), "Expected '%s', got '%s'\n", SYSDIR, buf);
897             ok(lstrlenA(SYSDIR) + 1 == ret, "Expected %d, got %d\n", lstrlenA(SYSDIR) + 1, ret);
898         }
899     }
900
901     return 0;
902 }
903
904 static void test_getfolderpath(void)
905 {
906     OPENFILENAMEA ofn;
907     BOOL result;
908     char szFileName[MAX_PATH] = "";
909     char szInitialDir[MAX_PATH];
910
911     /* We need to pick a different directory as the other tests because of new
912      * Windows 7 behavior.
913      */
914     GetSystemDirectory(szInitialDir, MAX_PATH);
915     lstrcpyA(SYSDIR, szInitialDir);
916
917     ZeroMemory(&ofn, sizeof(ofn));
918
919     ofn.lStructSize = sizeof(ofn);
920     ofn.hwndOwner = NULL;
921     ofn.lpstrFilter = "Text Files (*.txt)\0*.txt\0All Files (*.*)\0*.*\0";
922     ofn.lpstrFile = szFileName;
923     ofn.nMaxFile = MAX_PATH;
924     ofn.Flags = OFN_EXPLORER | OFN_FILEMUSTEXIST | OFN_HIDEREADONLY | OFN_ENABLEHOOK;
925     ofn.lpstrDefExt = "txt";
926     ofn.lpfnHook = path_hook_proc;
927     ofn.lpstrInitialDir = szInitialDir;
928
929     result = GetOpenFileNameA(&ofn);
930     ok(0 == result, "expected 0, got %d\n", result);
931     ok(0 == CommDlgExtendedError(), "expected 0, got %d\n",
932        CommDlgExtendedError());
933
934     result = GetSaveFileNameA(&ofn);
935     ok(0 == result, "expected 0, got %d\n", result);
936     ok(0 == CommDlgExtendedError(), "expected 0, got %d\n",
937        CommDlgExtendedError());
938 }
939
940 static void test_resizable2(void)
941 {
942     OPENFILENAMEA ofn = {0};
943     char filename[1024] = "pls press Enter if sizable, Esc otherwise";
944     DWORD ret;
945
946     /* interactive because there is no hook function */
947     if( !winetest_interactive) {
948         skip( "some interactive resizable dialog tests (set WINETEST_INTERACTIVE=1)\n");
949         return;
950     }
951     ofn.lStructSize = sizeof(ofn);
952     ofn.lpstrFile = filename;
953     ofn.nMaxFile = 1024;
954     ofn.lpfnHook = NULL;
955     ofn.hInstance = GetModuleHandleA(NULL);
956     ofn.lpTemplateName = "template1";
957     ofn.Flags = OFN_EXPLORER;
958 #define ISSIZABLE 1
959     ret = GetOpenFileNameA(&ofn);
960     ok( ret == ISSIZABLE, "File Dialog should have been sizable\n");
961     ret = CommDlgExtendedError();
962     ok(!ret, "CommDlgExtendedError returned %#x\n", ret);
963     ofn.Flags = OFN_EXPLORER | OFN_ENABLETEMPLATE;
964     ret = GetOpenFileNameA(&ofn);
965     ok( ret != ISSIZABLE, "File Dialog should NOT have been sizable\n");
966     ret = CommDlgExtendedError();
967     ok(!ret, "CommDlgExtendedError returned %#x\n", ret);
968     ofn.Flags = OFN_EXPLORER | OFN_ENABLETEMPLATEHANDLE;
969     ofn.hInstance = LoadResource( GetModuleHandle(NULL), FindResource( GetModuleHandle(NULL), "template1", (LPSTR)RT_DIALOG));
970     ofn.lpTemplateName = NULL;
971     ret = GetOpenFileNameA(&ofn);
972     ok( ret != ISSIZABLE, "File Dialog should NOT have been sizable\n");
973     ret = CommDlgExtendedError();
974     ok(!ret, "CommDlgExtendedError returned %#x\n", ret);
975     ofn.Flags = OFN_EXPLORER | OFN_ENABLEHOOK;
976     ret = GetOpenFileNameA(&ofn);
977     ok( ret != ISSIZABLE, "File Dialog should NOT have been sizable\n");
978     ret = CommDlgExtendedError();
979     ok(!ret, "CommDlgExtendedError returned %#x\n", ret);
980 #undef ISSIZABLE
981 }
982
983 static void test_mru(void)
984 {
985     ok_wndproc_testcase testcase = {0};
986     OPENFILENAME ofn = {sizeof(OPENFILENAME)};
987     const char *test_dir_name = "C:\\mru_test";
988     const char *test_file_name = "test.txt";
989     const char *test_full_path = "C:\\mru_test\\test.txt";
990     char filename_buf[MAX_PATH];
991     DWORD ret;
992
993     ofn.lpstrFile = filename_buf;
994     ofn.nMaxFile = sizeof(filename_buf);
995     ofn.lpTemplateName = "template1";
996     ofn.hInstance = GetModuleHandle(NULL);
997     ofn.Flags =  OFN_ENABLEHOOK | OFN_EXPLORER | OFN_ENABLETEMPLATE | OFN_NOCHANGEDIR;
998     ofn.lCustData = (LPARAM)&testcase;
999     ofn.lpfnHook = (LPOFNHOOKPROC)test_ok_wndproc;
1000
1001     SetLastError(0xdeadbeef);
1002     ret = CreateDirectoryA(test_dir_name, NULL);
1003     ok(ret == TRUE, "CreateDirectoryA should have succeeded: %d\n", GetLastError());
1004
1005     /* "teach" comdlg32 about this directory */
1006     strcpy(filename_buf, test_full_path);
1007     SetLastError(0xdeadbeef);
1008     ret = GetOpenFileNameA(&ofn);
1009     ok(ret, "GetOpenFileNameA should have succeeded: %d\n", GetLastError());
1010     ret = CommDlgExtendedError();
1011     ok(!ret, "CommDlgExtendedError returned %x\n", ret);
1012     ok(testcase.actclose, "Open File dialog should have closed.\n");
1013     ok(!strcmp(ofn.lpstrFile, test_full_path), "Expected to get %s, got %s\n", test_full_path, ofn.lpstrFile);
1014
1015     /* get a filename without a full path. it should return the file in
1016      * test_dir_name, not in the CWD */
1017     strcpy(filename_buf, test_file_name);
1018     SetLastError(0xdeadbeef);
1019     ret = GetOpenFileNameA(&ofn);
1020     ok(ret, "GetOpenFileNameA should have succeeded: %d\n", GetLastError());
1021     ret = CommDlgExtendedError();
1022     ok(!ret, "CommDlgExtendedError returned %x\n", ret);
1023     ok(testcase.actclose, "Open File dialog should have closed.\n");
1024     if(strcmp(ofn.lpstrFile, test_full_path) != 0)
1025         win_skip("Platform doesn't save MRU data\n");
1026
1027     SetLastError(0xdeadbeef);
1028     ret = RemoveDirectoryA(test_dir_name);
1029     ok(ret == TRUE, "RemoveDirectoryA should have succeeded: %d\n", GetLastError());
1030 }
1031
1032 static UINT_PTR WINAPI test_extension_wndproc(HWND dlg, UINT msg, WPARAM wParam, LPARAM lParam)
1033 {
1034     HWND parent = GetParent( dlg);
1035     if( msg == WM_NOTIFY) {
1036         SetTimer( dlg, 0, 1000, 0);
1037         PostMessage( parent, WM_COMMAND, IDOK, 0);
1038     }
1039     if( msg == WM_TIMER) {
1040         /* the dialog did not close automatically */
1041         KillTimer( dlg, 0);
1042         PostMessage( parent, WM_COMMAND, IDCANCEL, 0);
1043     }
1044     return FALSE;
1045 }
1046
1047 #define ARRAY_SIZE(a) (sizeof(a)/sizeof((a)[0]))
1048
1049 static void test_extension_helper(OPENFILENAME* ofn, const char *filter,
1050                                   const char *expected_filename)
1051 {
1052     char *filename_ptr;
1053     DWORD ret;
1054     BOOL boolret;
1055
1056     strcpy(ofn->lpstrFile, "deadbeef");
1057     ofn->lpstrFilter = filter;
1058
1059     boolret = GetSaveFileNameA(ofn);
1060     ok(boolret, "%s: expected TRUE\n", filter);
1061
1062     ret = CommDlgExtendedError();
1063     ok(!ret, "%s: CommDlgExtendedError returned %#x\n", filter, ret);
1064
1065     filename_ptr = ofn->lpstrFile + ofn->nFileOffset;
1066     ok(strcmp(filename_ptr, expected_filename) == 0,
1067         "%s: Filename is %s, expected %s\n", filter, filename_ptr, expected_filename);
1068 }
1069
1070 static void test_extension(void)
1071 {
1072     OPENFILENAME ofn = { sizeof(OPENFILENAME)};
1073     char filename[1024] = {0};
1074     char curdir[MAX_PATH];
1075     unsigned int i;
1076     BOOL boolret;
1077
1078     const char *defext_concrete_filters[] = {
1079         "TestFilter (*.abc)\0*.abc\0",
1080         "TestFilter (*.abc;)\0*.abc;\0",
1081         "TestFilter (*.abc;*.def)\0*.abc;*.def\0",
1082     };
1083
1084     const char *defext_wildcard_filters[] = {
1085         "TestFilter (*.pt*)\0*.pt*\0",
1086         "TestFilter (*.pt*;*.abc)\0*.pt*;*.abc\0",
1087         "TestFilter (*.ab?)\0*.ab?\0",
1088         "TestFilter (*.*)\0*.*\0",
1089         "TestFilter (*sav)\0*sav\0",
1090         NULL    /* is a test, not an endmark! */
1091     };
1092
1093     boolret = GetCurrentDirectoryA(sizeof(curdir), curdir);
1094     ok(boolret, "Failed to get current dir err %d\n", GetLastError());
1095
1096     ofn.lStructSize = sizeof(ofn);
1097     ofn.hwndOwner = NULL;
1098     ofn.lpstrFile = filename;
1099     ofn.nMaxFile = MAX_PATH;
1100     ofn.Flags = OFN_EXPLORER | OFN_ENABLEHOOK;
1101     ofn.lpstrInitialDir = curdir;
1102     ofn.lpfnHook = test_extension_wndproc;
1103     ofn.nFileExtension = 0;
1104
1105     ofn.lpstrDefExt = NULL;
1106
1107     /* Without lpstrDefExt, append no extension */
1108     test_extension_helper(&ofn, "TestFilter (*.abc) lpstrDefExt=NULL\0*.abc\0", "deadbeef");
1109     test_extension_helper(&ofn, "TestFilter (*.ab?) lpstrDefExt=NULL\0*.ab?\0", "deadbeef");
1110
1111     ofn.lpstrDefExt = "";
1112
1113     /* If lpstrDefExt="" and the filter has a concrete extension, append it */
1114     test_extension_helper(&ofn, "TestFilter (*.abc) lpstrDefExt=\"\"\0*.abc\0", "deadbeef.abc");
1115
1116     /* If lpstrDefExt="" and the filter has a wildcard extension, do nothing */
1117     test_extension_helper(&ofn, "TestFilter (*.ab?) lpstrDefExt=\"\"\0*.ab?\0", "deadbeef");
1118
1119     ofn.lpstrDefExt = "xyz";
1120
1121     /* Append concrete extensions from filters */
1122     for (i = 0; i < ARRAY_SIZE(defext_concrete_filters); i++) {
1123         test_extension_helper(&ofn, defext_concrete_filters[i], "deadbeef.abc");
1124     }
1125
1126     /* Append nothing from this filter */
1127     test_extension_helper(&ofn, "TestFilter (*.)\0*.\0", "deadbeef");
1128
1129     /* Ignore wildcard extensions in filters */
1130     for (i = 0; i < ARRAY_SIZE(defext_wildcard_filters); i++) {
1131         test_extension_helper(&ofn, defext_wildcard_filters[i], "deadbeef.xyz");
1132     }
1133 }
1134
1135 #undef ARRAY_SIZE
1136
1137 START_TEST(filedlg)
1138 {
1139     test_DialogCancel();
1140     test_create_view_window2();
1141     test_create_view_template();
1142     test_arrange();
1143     test_resize();
1144     test_ok();
1145     test_getfolderpath();
1146     test_mru();
1147     if( resizesupported) test_resizable2();
1148     test_extension();
1149 }