Nikolay Sivov : comctl32/treeview: TVM_EXPAND with TVE_TOGGLE action shouldn' t always notify as it does on user input.
Alexandre Julliard
julliard at winehq.org
Wed May 25 11:28:14 CDT 2011
Module: wine
Branch: master
Commit: bcb98d11ba6702165bf80e0fbc2b3aaef7b43c74
URL: http://source.winehq.org/git/wine.git/?a=commit;h=bcb98d11ba6702165bf80e0fbc2b3aaef7b43c74
Author: Nikolay Sivov <nsivov at codeweavers.com>
Date: Tue May 24 13:25:47 2011 +0400
comctl32/treeview: TVM_EXPAND with TVE_TOGGLE action shouldn't always notify as it does on user input.
---
dlls/comctl32/tests/treeview.c | 33 +++++++++++++++++++++++++--------
dlls/comctl32/treeview.c | 10 +++++-----
2 files changed, 30 insertions(+), 13 deletions(-)
diff --git a/dlls/comctl32/tests/treeview.c b/dlls/comctl32/tests/treeview.c
index 0437d65..5fd26a0 100644
--- a/dlls/comctl32/tests/treeview.c
+++ b/dlls/comctl32/tests/treeview.c
@@ -1302,12 +1302,12 @@ static void test_expandnotify(void)
item.state = TVIS_EXPANDED;
ret = SendMessageA(hTree, TVM_GETITEMA, 0, (LPARAM)&item);
- ok(ret == TRUE, "got %d\n", ret);
+ expect(TRUE, ret);
ok((item.state & TVIS_EXPANDED) == 0, "expected collapsed\n");
/* preselect root node here */
ret = SendMessageA(hTree, TVM_SELECTITEM, TVGN_CARET, (LPARAM)hRoot);
- ok(ret == TRUE, "got %d\n", ret);
+ expect(TRUE, ret);
g_get_from_expand = TRUE;
/* expand */
@@ -1315,7 +1315,7 @@ static void test_expandnotify(void)
g_item_expanding.state = 0xdeadbeef;
g_item_expanded.state = 0xdeadbeef;
ret = SendMessageA(hTree, TVM_EXPAND, TVE_EXPAND, (LPARAM)hRoot);
- ok(ret == TRUE, "got %d\n", ret);
+ expect(TRUE, ret);
ok(g_item_expanding.state == TVIS_SELECTED, "got state on TVN_ITEMEXPANDING 0x%08x\n",
g_item_expanding.state);
ok(g_item_expanded.state == (TVIS_SELECTED|TVIS_EXPANDED), "got state on TVN_ITEMEXPANDED 0x%08x\n",
@@ -1326,16 +1326,16 @@ static void test_expandnotify(void)
/* check that it's expanded */
item.state = TVIS_EXPANDED;
ret = SendMessageA(hTree, TVM_GETITEMA, 0, (LPARAM)&item);
- ok(ret == TRUE, "got %d\n", ret);
+ expect(TRUE, ret);
ok((item.state & TVIS_EXPANDED) == TVIS_EXPANDED, "expected expanded\n");
/* collapse */
flush_sequences(sequences, NUM_MSG_SEQUENCES);
ret = SendMessageA(hTree, TVM_EXPAND, TVE_COLLAPSE, (LPARAM)hRoot);
- ok(ret == TRUE, "got %d\n", ret);
+ expect(TRUE, ret);
item.state = TVIS_EXPANDED;
ret = SendMessageA(hTree, TVM_GETITEMA, 0, (LPARAM)&item);
- ok(ret == TRUE, "got %d\n", ret);
+ expect(TRUE, ret);
ok((item.state & TVIS_EXPANDED) == 0, "expected collapsed\n");
/* all next collapse/expand attempts won't produce any notifications,
the only way is to reset with all children removed */
@@ -1345,7 +1345,7 @@ static void test_expandnotify(void)
flush_sequences(sequences, NUM_MSG_SEQUENCES);
ret = SendMessageA(hTree, TVM_EXPAND, TVE_TOGGLE, (LPARAM)hChild);
expect(FALSE, ret);
- ok_sequence(sequences, PARENT_SEQ_INDEX, empty_seq, "toggle node without children", TRUE);
+ ok_sequence(sequences, PARENT_SEQ_INDEX, empty_seq, "toggle node without children", FALSE);
DestroyWindow(hTree);
@@ -1354,8 +1354,25 @@ static void test_expandnotify(void)
fill_tree(hTree);
g_get_rect_in_expand = TRUE;
ret = TreeView_Select(hTree, hChild, TVGN_CARET);
+ expect(TRUE, ret);
g_get_rect_in_expand = FALSE;
- ok(ret, "got %d\n", ret);
+
+ DestroyWindow(hTree);
+
+ /* TVE_TOGGLE acts as any other TVM_EXPAND */
+ hTree = create_treeview_control(0);
+ fill_tree(hTree);
+
+ flush_sequences(sequences, NUM_MSG_SEQUENCES);
+ ret = SendMessageA(hTree, TVM_EXPAND, TVE_TOGGLE, (LPARAM)hRoot);
+ expect(TRUE, ret);
+ ok_sequence(sequences, PARENT_SEQ_INDEX, parent_expand_seq, "toggle node (expand)", FALSE);
+
+ /* toggle again - no notifications */
+ flush_sequences(sequences, NUM_MSG_SEQUENCES);
+ ret = SendMessageA(hTree, TVM_EXPAND, TVE_TOGGLE, (LPARAM)hRoot);
+ expect(TRUE, ret);
+ ok_sequence(sequences, PARENT_SEQ_INDEX, empty_seq, "toggle node (collapse)", FALSE);
DestroyWindow(hTree);
}
diff --git a/dlls/comctl32/treeview.c b/dlls/comctl32/treeview.c
index 2388d3c..bd65fca 100644
--- a/dlls/comctl32/treeview.c
+++ b/dlls/comctl32/treeview.c
@@ -3489,14 +3489,14 @@ static void TREEVIEW_SingleExpand(TREEVIEW_INFO *infoPtr,
}
static BOOL
-TREEVIEW_Toggle(TREEVIEW_INFO *infoPtr, TREEVIEW_ITEM *item, BOOL bUser)
+TREEVIEW_Toggle(TREEVIEW_INFO *infoPtr, TREEVIEW_ITEM *item, BOOL user)
{
- TRACE("\n");
+ TRACE("item=%p, user=%d\n", item, user);
if (item->state & TVIS_EXPANDED)
- return TREEVIEW_Collapse(infoPtr, item, FALSE, bUser);
+ return TREEVIEW_Collapse(infoPtr, item, FALSE, user);
else
- return TREEVIEW_Expand(infoPtr, item, FALSE, bUser);
+ return TREEVIEW_Expand(infoPtr, item, FALSE, user);
}
static VOID
@@ -3538,7 +3538,7 @@ TREEVIEW_ExpandMsg(TREEVIEW_INFO *infoPtr, UINT flag, HTREEITEM item)
FALSE);
case TVE_TOGGLE:
- return TREEVIEW_Toggle(infoPtr, item, TRUE);
+ return TREEVIEW_Toggle(infoPtr, item, FALSE);
default:
return 0;
More information about the wine-cvs
mailing list