From 1e1c1310f5cbef72ef5ea7ade6041814f872dc22 Mon Sep 17 00:00:00 2001 From: Nikolay Sivov Date: Sat, 28 Nov 2009 12:04:58 +0300 Subject: [PATCH] comctl32/listview: Get rid of duplicated helper in tests. --- dlls/comctl32/tests/listview.c | 162 ++++++++++++++------------------- 1 file changed, 70 insertions(+), 92 deletions(-) diff --git a/dlls/comctl32/tests/listview.c b/dlls/comctl32/tests/listview.c index 8e2d9d59887..8017521c957 100644 --- a/dlls/comctl32/tests/listview.c +++ b/dlls/comctl32/tests/listview.c @@ -475,7 +475,7 @@ static HWND create_listview_control(DWORD style) GetClientRect(hwndparent, &rect); hwnd = CreateWindowExA(0, WC_LISTVIEW, "foo", - WS_CHILD | WS_BORDER | WS_VISIBLE | LVS_REPORT | style, + WS_CHILD | WS_BORDER | WS_VISIBLE | style, 0, 0, rect.right, rect.bottom, hwndparent, NULL, GetModuleHandleA(NULL), NULL); ok(hwnd != NULL, "gle=%d\n", GetLastError()); @@ -499,7 +499,7 @@ static HWND create_listview_controlW(DWORD style, HWND parent) GetClientRect(parent, &rect); hwnd = CreateWindowExW(0, WC_LISTVIEWW, nameW, - WS_CHILD | WS_BORDER | WS_VISIBLE | LVS_REPORT | style, + WS_CHILD | WS_BORDER | WS_VISIBLE | style, 0, 0, rect.right, rect.bottom, parent, NULL, GetModuleHandleW(NULL), NULL); ok(hwnd != NULL, "gle=%d\n", GetLastError()); @@ -513,28 +513,6 @@ static HWND create_listview_controlW(DWORD style, HWND parent) return hwnd; } -static HWND create_custom_listview_control(DWORD style) -{ - WNDPROC oldproc; - HWND hwnd; - RECT rect; - - GetClientRect(hwndparent, &rect); - hwnd = CreateWindowExA(0, WC_LISTVIEW, "foo", - WS_CHILD | WS_BORDER | WS_VISIBLE | style, - 0, 0, rect.right, rect.bottom, - hwndparent, NULL, GetModuleHandleA(NULL), NULL); - ok(hwnd != NULL, "gle=%d\n", GetLastError()); - - if (!hwnd) return NULL; - - oldproc = (WNDPROC)SetWindowLongPtrA(hwnd, GWLP_WNDPROC, - (LONG_PTR)listview_subclass_proc); - SetWindowLongPtrA(hwnd, GWLP_USERDATA, (LONG_PTR)oldproc); - - return hwnd; -} - static LRESULT WINAPI header_subclass_proc(HWND hwnd, UINT message, WPARAM wParam, LPARAM lParam) { WNDPROC oldproc = (WNDPROC)GetWindowLongPtrA(hwnd, GWLP_USERDATA); @@ -1213,7 +1191,7 @@ static void test_columns(void) DestroyWindow(hwnd); /* LVM_GETCOLUMNORDERARRAY */ - hwnd = create_listview_control(0); + hwnd = create_listview_control(LVS_REPORT); hwndheader = subclass_header(hwnd); memset(&column, 0, sizeof(column)); @@ -1410,7 +1388,7 @@ static void test_create(void) DestroyWindow(hList); /* not report style accepts LVS_EX_HEADERDRAGDROP too */ - hList = create_custom_listview_control(0); + hList = create_listview_control(LVS_ICON); SendMessage(hList, LVM_SETEXTENDEDLISTVIEWSTYLE, 0, LVS_EX_HEADERDRAGDROP); r = SendMessage(hList, LVM_GETEXTENDEDLISTVIEWSTYLE, 0, 0); ok(r & LVS_EX_HEADERDRAGDROP, "Expected LVS_EX_HEADERDRAGDROP to be set\n"); @@ -1436,7 +1414,7 @@ static void test_create(void) /* WM_MEASUREITEM should be sent when created with LVS_OWNERDRAWFIXED */ flush_sequences(sequences, NUM_MSG_SEQUENCES); - hList = create_listview_control(LVS_OWNERDRAWFIXED); + hList = create_listview_control(LVS_OWNERDRAWFIXED | LVS_REPORT); ok_sequence(sequences, PARENT_SEQ_INDEX, create_ownerdrawfixed_parent_seq, "created with LVS_OWNERDRAWFIXED|LVS_REPORT - parent seq", FALSE); DestroyWindow(hList); @@ -1449,7 +1427,7 @@ static void test_redraw(void) BOOL res; DWORD r; - hwnd = create_listview_control(0); + hwnd = create_listview_control(LVS_REPORT); hwndheader = subclass_header(hwnd); flush_sequences(sequences, NUM_MSG_SEQUENCES); @@ -1546,7 +1524,7 @@ static void test_customdraw(void) HWND hwnd; WNDPROC oldwndproc; - hwnd = create_listview_control(0); + hwnd = create_listview_control(LVS_REPORT); insert_column(hwnd, 0); insert_column(hwnd, 1); @@ -1572,7 +1550,7 @@ static void test_icon_spacing(void) WORD w, h; DWORD r; - hwnd = create_custom_listview_control(LVS_ICON); + hwnd = create_listview_control(LVS_ICON); ok(hwnd != NULL, "failed to create a listview window\n"); r = SendMessage(hwnd, WM_NOTIFYFORMAT, (WPARAM)hwndparent, (LPARAM)NF_REQUERY); @@ -1625,7 +1603,7 @@ static void test_color(void) COLORREF color; COLORREF colors[4] = {RGB(0,0,0), RGB(100,50,200), CLR_NONE, RGB(255,255,255)}; - hwnd = create_listview_control(0); + hwnd = create_listview_control(LVS_REPORT); ok(hwnd != NULL, "failed to create a listview window\n"); flush_sequences(sequences, NUM_MSG_SEQUENCES); @@ -1676,7 +1654,7 @@ static void test_item_count(void) static CHAR item1text[] = "item1"; static CHAR item2text[] = "item2"; - hwnd = create_listview_control(0); + hwnd = create_listview_control(LVS_REPORT); ok(hwnd != NULL, "failed to create a listview window\n"); /* resize in dpiaware manner to fit all 3 items added */ @@ -1779,7 +1757,7 @@ static void test_item_position(void) static CHAR item1text[] = "item1"; static CHAR item2text[] = "item2"; - hwnd = create_custom_listview_control(LVS_ICON); + hwnd = create_listview_control(LVS_ICON); ok(hwnd != NULL, "failed to create a listview window\n"); flush_sequences(sequences, NUM_MSG_SEQUENCES); @@ -1844,7 +1822,7 @@ static void test_getorigin(void) position.x = position.y = 0; - hwnd = create_custom_listview_control(LVS_ICON); + hwnd = create_listview_control(LVS_ICON); ok(hwnd != NULL, "failed to create a listview window\n"); flush_sequences(sequences, NUM_MSG_SEQUENCES); trace("test get origin results\n"); @@ -1853,7 +1831,7 @@ static void test_getorigin(void) flush_sequences(sequences, NUM_MSG_SEQUENCES); DestroyWindow(hwnd); - hwnd = create_custom_listview_control(LVS_SMALLICON); + hwnd = create_listview_control(LVS_SMALLICON); ok(hwnd != NULL, "failed to create a listview window\n"); flush_sequences(sequences, NUM_MSG_SEQUENCES); trace("test get origin results\n"); @@ -1862,7 +1840,7 @@ static void test_getorigin(void) flush_sequences(sequences, NUM_MSG_SEQUENCES); DestroyWindow(hwnd); - hwnd = create_custom_listview_control(LVS_LIST); + hwnd = create_listview_control(LVS_LIST); ok(hwnd != NULL, "failed to create a listview window\n"); flush_sequences(sequences, NUM_MSG_SEQUENCES); trace("test get origin results\n"); @@ -1871,7 +1849,7 @@ static void test_getorigin(void) flush_sequences(sequences, NUM_MSG_SEQUENCES); DestroyWindow(hwnd); - hwnd = create_custom_listview_control(LVS_REPORT); + hwnd = create_listview_control(LVS_REPORT); ok(hwnd != NULL, "failed to create a listview window\n"); flush_sequences(sequences, NUM_MSG_SEQUENCES); trace("test get origin results\n"); @@ -1910,7 +1888,7 @@ static void test_multiselect(void) }; - hwnd = create_listview_control(0); + hwnd = create_listview_control(LVS_REPORT); for (i=0;iview mapping */ - hwnd = create_listview_control(0); + hwnd = create_listview_control(LVS_REPORT); ok(hwnd != NULL, "failed to create a listview window\n"); ret = SendMessage(hwnd, LVM_GETVIEW, 0, 0); @@ -3779,7 +3757,7 @@ static void test_canceleditlabel(void) static CHAR test[] = "test"; static const CHAR test1[] = "test1"; - hwnd = create_listview_control(LVS_EDITLABELS); + hwnd = create_listview_control(LVS_EDITLABELS | LVS_REPORT); ok(hwnd != NULL, "failed to create a listview window\n"); insert_item(hwnd, 0); @@ -3828,14 +3806,14 @@ static void test_mapidindex(void) DWORD ret; /* LVM_MAPINDEXTOID unsupported with LVS_OWNERDATA */ - hwnd = create_listview_control(LVS_OWNERDATA); + hwnd = create_listview_control(LVS_OWNERDATA | LVS_REPORT); ok(hwnd != NULL, "failed to create a listview window\n"); insert_item(hwnd, 0); ret = SendMessage(hwnd, LVM_MAPINDEXTOID, 0, 0); expect(-1, ret); DestroyWindow(hwnd); - hwnd = create_listview_control(0); + hwnd = create_listview_control(LVS_REPORT); ok(hwnd != NULL, "failed to create a listview window\n"); /* LVM_MAPINDEXTOID with invalid index */ @@ -3889,7 +3867,7 @@ static void test_getitemspacing(void) cy = GetSystemMetrics(SM_CYICONSPACING) - GetSystemMetrics(SM_CYICON); /* LVS_ICON */ - hwnd = create_custom_listview_control(0); + hwnd = create_listview_control(LVS_ICON); ret = SendMessage(hwnd, LVM_GETITEMSPACING, FALSE, 0); todo_wine { expect(cx, LOWORD(ret)); @@ -3919,7 +3897,7 @@ todo_wine { } DestroyWindow(hwnd); /* LVS_SMALLICON */ - hwnd = create_custom_listview_control(LVS_SMALLICON); + hwnd = create_listview_control(LVS_SMALLICON); ret = SendMessage(hwnd, LVM_GETITEMSPACING, FALSE, 0); todo_wine { expect(cx, LOWORD(ret)); @@ -3927,7 +3905,7 @@ todo_wine { } DestroyWindow(hwnd); /* LVS_REPORT */ - hwnd = create_custom_listview_control(LVS_REPORT); + hwnd = create_listview_control(LVS_REPORT); ret = SendMessage(hwnd, LVM_GETITEMSPACING, FALSE, 0); todo_wine { expect(cx, LOWORD(ret)); @@ -3935,7 +3913,7 @@ todo_wine { } DestroyWindow(hwnd); /* LVS_LIST */ - hwnd = create_custom_listview_control(LVS_LIST); + hwnd = create_listview_control(LVS_LIST); ret = SendMessage(hwnd, LVM_GETITEMSPACING, FALSE, 0); todo_wine { expect(cx, LOWORD(ret)); @@ -3953,7 +3931,7 @@ static void test_getcolumnwidth(void) LVITEMA itema; /* default column width */ - hwnd = create_custom_listview_control(0); + hwnd = create_listview_control(LVS_ICON); ret = SendMessage(hwnd, LVM_GETCOLUMNWIDTH, 0, 0); expect(0, ret); style = GetWindowLong(hwnd, GWL_STYLE); @@ -3970,7 +3948,7 @@ static void test_getcolumnwidth(void) DestroyWindow(hwnd); /* default column width with item added */ - hwnd = create_custom_listview_control(LVS_LIST); + hwnd = create_listview_control(LVS_LIST); memset(&itema, 0, sizeof(itema)); SendMessage(hwnd, LVM_INSERTITEMA, 0, (LPARAM)&itema); ret = SendMessage(hwnd, LVM_GETCOLUMNWIDTH, 0, 0); @@ -3983,7 +3961,7 @@ static void test_scrollnotify(void) HWND hwnd; DWORD ret; - hwnd = create_listview_control(0); + hwnd = create_listview_control(LVS_REPORT); insert_column(hwnd, 0); insert_column(hwnd, 1); @@ -4023,7 +4001,7 @@ static void test_LVS_EX_TRANSPARENTBKGND(void) DWORD ret; HDC hdc; - hwnd = create_listview_control(0); + hwnd = create_listview_control(LVS_REPORT); ret = SendMessage(hwnd, LVM_SETBKCOLOR, 0, RGB(0, 0, 0)); expect(TRUE, ret); @@ -4074,7 +4052,7 @@ static void test_approximate_viewrect(void) cx = GetSystemMetrics(SM_CXICONSPACING) - GetSystemMetrics(SM_CXICON); cy = GetSystemMetrics(SM_CYICONSPACING) - GetSystemMetrics(SM_CYICON); - hwnd = create_custom_listview_control(LVS_ICON); + hwnd = create_listview_control(LVS_ICON); himl = ImageList_Create(40, 40, 0, 4, 4); ok(himl != NULL, "failed to create imagelist\n"); hbmp = CreateBitmap(40, 40, 1, 1, NULL); @@ -4142,7 +4120,7 @@ static void test_finditem(void) HWND hwnd; DWORD r; - hwnd = create_listview_control(0); + hwnd = create_listview_control(LVS_REPORT); insert_item(hwnd, 0); memset(&fi, 0, sizeof(fi)); @@ -4203,7 +4181,7 @@ static void test_LVS_EX_HEADERINALLVIEWS(void) HWND hwnd, header; DWORD style; - hwnd = create_custom_listview_control(LVS_ICON); + hwnd = create_listview_control(LVS_ICON); SendMessage(hwnd, LVM_SETEXTENDEDLISTVIEWSTYLE, LVS_EX_HEADERINALLVIEWS, LVS_EX_HEADERINALLVIEWS); @@ -4236,21 +4214,21 @@ static void test_LVS_EX_HEADERINALLVIEWS(void) DestroyWindow(hwnd); /* check other styles */ - hwnd = create_custom_listview_control(LVS_LIST); + hwnd = create_listview_control(LVS_LIST); SendMessage(hwnd, LVM_SETEXTENDEDLISTVIEWSTYLE, LVS_EX_HEADERINALLVIEWS, LVS_EX_HEADERINALLVIEWS); header = (HWND)SendMessage(hwnd, LVM_GETHEADER, 0, 0); ok(IsWindow(header), "Expected header to be created\n"); DestroyWindow(hwnd); - hwnd = create_custom_listview_control(LVS_SMALLICON); + hwnd = create_listview_control(LVS_SMALLICON); SendMessage(hwnd, LVM_SETEXTENDEDLISTVIEWSTYLE, LVS_EX_HEADERINALLVIEWS, LVS_EX_HEADERINALLVIEWS); header = (HWND)SendMessage(hwnd, LVM_GETHEADER, 0, 0); ok(IsWindow(header), "Expected header to be created\n"); DestroyWindow(hwnd); - hwnd = create_custom_listview_control(LVS_REPORT); + hwnd = create_listview_control(LVS_REPORT); SendMessage(hwnd, LVM_SETEXTENDEDLISTVIEWSTYLE, LVS_EX_HEADERINALLVIEWS, LVS_EX_HEADERINALLVIEWS); header = (HWND)SendMessage(hwnd, LVM_GETHEADER, 0, 0); @@ -4263,7 +4241,7 @@ static void test_hover(void) HWND hwnd; DWORD r; - hwnd = create_custom_listview_control(LVS_ICON); + hwnd = create_listview_control(LVS_ICON); /* test WM_MOUSEHOVER forwarding */ flush_sequences(sequences, NUM_MSG_SEQUENCES);