[PATCH 0/1] MR7582: comctl32: Implement EVENT_OBJECT_VALUECHANGE for datetime control.

[Esme Povirk (＠madewokherd)]

Windows implemented this event only for comctl32 v6. Making this behavior version-specific in Wine would be difficult, and I don't expect it to matter.

-- 
https://gitlab.winehq.org/wine/wine/-/merge_requests/7582

[PATCH 1/1] comctl32: Implement EVENT_OBJECT_VALUECHANGE for datetime control.

[Esme Povirk]

From: Esme Povirk <esme(a)codeweavers.com>

---
 dlls/comctl32/datetime.c       |  2 ++
 dlls/comctl32/tests/datetime.c | 66 +++++++++++++++++++++++++++++++---
 2 files changed, 64 insertions(+), 4 deletions(-)

diff --git a/dlls/comctl32/datetime.c b/dlls/comctl32/datetime.c
index 393406e778b..2772cfec00c 100644
--- a/dlls/comctl32/datetime.c
+++ b/dlls/comctl32/datetime.c
@@ -336,6 +336,8 @@ DATETIME_SetFormatW (DATETIME_INFO *infoPtr, LPCWSTR format)
     DATETIME_UseFormat (infoPtr, format);
     InvalidateRect (infoPtr->hwndSelf, NULL, TRUE);
 
+    NotifyWinEvent(EVENT_OBJECT_VALUECHANGE, infoPtr->hwndSelf, OBJID_CLIENT, CHILDID_SELF);
+
     return TRUE;
 }
 
diff --git a/dlls/comctl32/tests/datetime.c b/dlls/comctl32/tests/datetime.c
index e5dbd6bec45..e2a2967e831 100644
--- a/dlls/comctl32/tests/datetime.c
+++ b/dlls/comctl32/tests/datetime.c
@@ -35,9 +35,60 @@ static BOOL (WINAPI *pInitCommonControlsEx)(const INITCOMMONCONTROLSEX*);
 
 static struct msg_sequence *sequences[NUM_MSG_SEQUENCES];
 
+static void CALLBACK msg_winevent_proc(HWINEVENTHOOK hevent,
+                                       DWORD event,
+                                       HWND hwnd,
+                                       LONG object_id,
+                                       LONG child_id,
+                                       DWORD thread_id,
+                                       DWORD event_time)
+{
+    struct message msg = {0};
+    char class_name[256];
+
+    /* ignore window and other system events */
+    if (object_id != OBJID_CLIENT) return;
+
+    /* ignore events not from a datetime control */
+    if (!GetClassNameA(hwnd, class_name, ARRAY_SIZE(class_name)) ||
+        strcmp(class_name, DATETIMEPICK_CLASSA) != 0)
+        return;
+
+    msg.message = event;
+    msg.flags = winevent_hook|wparam|lparam;
+    msg.wParam = object_id;
+    msg.lParam = child_id;
+    add_message(sequences, DATETIME_SEQ_INDEX, &msg);
+}
+
+static void init_winevent_hook(void) {
+    hwineventhook = SetWinEventHook(EVENT_MIN, EVENT_MAX, GetModuleHandleA(0), msg_winevent_proc,
+                                    0, GetCurrentThreadId(), WINEVENT_INCONTEXT);
+    if (!hwineventhook)
+        win_skip( "no win event hook support\n" );
+}
+
+static void uninit_winevent_hook(void) {
+    if (!hwineventhook)
+        return;
+
+    UnhookWinEvent(hwineventhook);
+    hwineventhook = 0;
+}
+
+static const struct message test_dtm_set_format_v6_seq[] = {
+    { DTM_SETFORMATA, sent|wparam|lparam, 0, 0 },
+    { EVENT_OBJECT_VALUECHANGE, winevent_hook|wparam|lparam, OBJID_CLIENT, CHILDID_SELF },
+    { DTM_SETFORMATA, sent|wparam, 0 },
+    { EVENT_OBJECT_VALUECHANGE, winevent_hook|wparam|lparam, OBJID_CLIENT, CHILDID_SELF },
+    { 0 }
+};
+
 static const struct message test_dtm_set_format_seq[] = {
     { DTM_SETFORMATA, sent|wparam|lparam, 0, 0 },
+    { EVENT_OBJECT_VALUECHANGE, winevent_hook|wparam|lparam|optional, OBJID_CLIENT, CHILDID_SELF }, /* Sent on Wine only */
     { DTM_SETFORMATA, sent|wparam, 0 },
+    { EVENT_OBJECT_VALUECHANGE, winevent_hook|wparam|lparam|optional, OBJID_CLIENT, CHILDID_SELF }, /* Sent on Wine only */
     { 0 }
 };
 
@@ -181,7 +232,7 @@ static HWND create_datetime_control(DWORD style)
     return hWndDateTime;
 }
 
-static void test_dtm_set_format(void)
+static void test_dtm_set_format(BOOL comctl_v6)
 {
     HWND hWnd;
     CHAR txt[256];
@@ -199,7 +250,10 @@ static void test_dtm_set_format(void)
 		    (LPARAM)"'Today is: 'hh':'m':'s dddd MMM dd', 'yyyy");
     expect(1, r);
 
-    ok_sequence(sequences, DATETIME_SEQ_INDEX, test_dtm_set_format_seq, "test_dtm_set_format", FALSE);
+    if (comctl_v6)
+        ok_sequence(sequences, DATETIME_SEQ_INDEX, test_dtm_set_format_v6_seq, "test_dtm_set_format_v6", FALSE);
+    else
+        ok_sequence(sequences, DATETIME_SEQ_INDEX, test_dtm_set_format_seq, "test_dtm_set_format", FALSE);
 
     r = SendMessageA(hWnd, DTM_SETFORMATA, 0, (LPARAM)"'hh' hh");
     expect(1, r);
@@ -859,7 +913,9 @@ START_TEST(datetime)
 
     init_msg_sequences(sequences, NUM_MSG_SEQUENCES);
 
-    test_dtm_set_format();
+    init_winevent_hook();
+
+    test_dtm_set_format(FALSE);
     test_dtm_set_and_get_mccolor();
     test_dtm_set_and_get_mcfont();
     test_dtm_get_monthcal();
@@ -873,7 +929,7 @@ START_TEST(datetime)
     if (!load_v6_module(&cookie, &ctxt))
         return;
 
-    test_dtm_set_format();
+    test_dtm_set_format(TRUE);
     test_dtm_set_and_get_mccolor();
     test_dtm_set_and_get_mcfont();
     test_dtm_get_monthcal();
@@ -881,5 +937,7 @@ START_TEST(datetime)
     test_wm_set_get_text();
     test_dts_shownone();
 
+    uninit_winevent_hook();
+
     unload_v6_module(cookie, ctxt);
 }
-- 
GitLab

https://gitlab.winehq.org/wine/wine/-/merge_requests/7582

[PATCH 0/1] MR7582: comctl32: Implement EVENT_OBJECT_VALUECHANGE for datetime control. - approved

[Nikolay Sivov (＠nsivov)]

This merge request was approved by Nikolay Sivov.
-- 
https://gitlab.winehq.org/wine/wine/-/merge_requests/7582