[reactos] 01/01: [SYSSETUP][TIMEDATE] Use exact setTimer() timeouts, to stay in sync (#3355)

[Serge Gautherie]

https://git.reactos.org/?p=reactos.git;a=commitdiff;h=4cd92f0d01b342bbbbf46…

commit 4cd92f0d01b342bbbbf4637e6b4ccbb0b5318291
Author:     Serge Gautherie <32623169+SergeGautherie(a)users.noreply.github.com>
AuthorDate: Sun Mar 21 04:01:22 2021 +0100
Commit:     GitHub <noreply(a)github.com>
CommitDate: Sun Mar 21 12:01:22 2021 +0900

    [SYSSETUP][TIMEDATE] Use exact setTimer() timeouts, to stay in sync (#3355)
    
    CORE-13121
---
 base/shell/explorer/trayclock.cpp |  9 ---------
 dll/cpl/timedate/clock.c          | 16 ++++++++++++----
 dll/cpl/timedate/dateandtime.c    | 12 +++++++++---
 dll/cpl/timedate/monthcal.c       |  5 -----
 dll/win32/syssetup/wizard.c       | 27 +++++++++++++++++++++------
 5 files changed, 42 insertions(+), 27 deletions(-)

diff --git a/base/shell/explorer/trayclock.cpp b/base/shell/explorer/trayclock.cpp
index 9736ffbc8b8..cc289ddc0e2 100644
--- a/base/shell/explorer/trayclock.cpp
+++ b/base/shell/explorer/trayclock.cpp
@@ -429,15 +429,6 @@ UINT CTrayClockWnd::CalculateDueTime()
     if (!g_TaskbarSettings.bShowSeconds)
         uiDueTime += (59 - (UINT) LocalTime.wSecond) * 1000;
 
-    if (uiDueTime < USER_TIMER_MINIMUM || uiDueTime > USER_TIMER_MAXIMUM)
-        uiDueTime = 1000;
-    else
-    {
-        /* Add an artificial delay of 0.05 seconds to make sure the timer
-            doesn't fire too early*/
-        uiDueTime += 50;
-    }
-
     return uiDueTime;
 }
 
diff --git a/dll/cpl/timedate/clock.c b/dll/cpl/timedate/clock.c
index 3fb30f6d8b8..d5667a988db 100644
--- a/dll/cpl/timedate/clock.c
+++ b/dll/cpl/timedate/clock.c
@@ -152,10 +152,10 @@ ClockWndProc(HWND hwnd,
             pClockData->hGreyPen = CreatePen(PS_SOLID, 1, RGB(128, 128, 128));
             pClockData->hGreyBrush = CreateSolidBrush(RGB(128, 128, 128));
 
-            SetTimer(hwnd, ID_TIMER, 1000, NULL);
-            pClockData->bTimer = TRUE;
             GetLocalTime(&pClockData->stCurrent);
             pClockData->stPrevious = pClockData->stCurrent;
+
+            pClockData->bTimer = (SetTimer(hwnd, ID_TIMER, 1000 -
pClockData->stCurrent.wMilliseconds, NULL) != 0);
             break;
 
         case WM_SIZE:
@@ -168,6 +168,12 @@ ClockWndProc(HWND hwnd,
             GetLocalTime(&pClockData->stCurrent);
             InvalidateRect(hwnd, NULL, FALSE);
             pClockData->stPrevious = pClockData->stCurrent;
+
+            // Reset timeout.
+            if (pClockData->bTimer)
+            {
+                SetTimer(hwnd, ID_TIMER, 1000 - pClockData->stCurrent.wMilliseconds,
NULL);
+            }
             break;
 
         case WM_PAINT:
@@ -255,8 +261,10 @@ ClockWndProc(HWND hwnd,
         case CLM_STARTCLOCK:
             if (!pClockData->bTimer)
             {
-                SetTimer(hwnd, ID_TIMER, 1000, NULL);
-                pClockData->bTimer = TRUE;
+                SYSTEMTIME LocalTime;
+
+                GetLocalTime(&LocalTime);
+                pClockData->bTimer = (SetTimer(hwnd, ID_TIMER, 1000 -
LocalTime.wMilliseconds, NULL) != 0);
             }
             break;
 
diff --git a/dll/cpl/timedate/dateandtime.c b/dll/cpl/timedate/dateandtime.c
index 026cd305cfc..807bf76faf8 100644
--- a/dll/cpl/timedate/dateandtime.c
+++ b/dll/cpl/timedate/dateandtime.c
@@ -276,17 +276,20 @@ DateTimePageProc(HWND hwndDlg,
             FillMonthsComboBox(GetDlgItem(hwndDlg,
                                           IDC_MONTHCB));
 
-            SetTimer(hwndDlg, ID_TIMER, 1000, NULL);
-
             /* Set range and current year */
             SendMessageW(GetDlgItem(hwndDlg, IDC_YEAR), UDM_SETRANGE, 0, MAKELONG
((short) 9999, (short) 1900));
             SendMessageW(GetDlgItem(hwndDlg, IDC_YEAR), UDM_SETPOS, 0, MAKELONG( (short)
st.wYear, 0));
 
             pOldWndProc = (WNDPROC)SetWindowLongPtrW(GetDlgItem(hwndDlg, IDC_TIMEPICKER),
GWLP_WNDPROC, (LONG_PTR)DTPProc);
+
+            SetTimer(hwndDlg, ID_TIMER, 1000 - st.wMilliseconds, NULL);
             break;
 
         case WM_TIMER:
             SendMessageW(GetDlgItem(hwndDlg, IDC_TIMEPICKER), DTM_SETSYSTEMTIME,
GDT_VALID, (LPARAM) &st);
+
+            // Reset timeout.
+            SetTimer(hwndDlg, ID_TIMER, 1000 - st.wMilliseconds, NULL);
             break;
 
         case WM_COMMAND:
@@ -347,6 +350,8 @@ DateTimePageProc(HWND hwndDlg,
                             SendDlgItemMessageW(hwndDlg, IDC_CLOCKWND, CLM_STOPCLOCK,
                                                 0, 0);
 
+                            // TODO: Set the clock to the input time.
+
                             /* Enable the 'Apply' button */
                             PropSheet_Changed(GetParent(hwndDlg), hwndDlg);
                             break;
@@ -377,11 +382,12 @@ DateTimePageProc(HWND hwndDlg,
 
                         case PSN_APPLY:
                             SetLocalSystemTime(hwndDlg);
-                            SetTimer(hwndDlg, ID_TIMER, 1000, NULL);
 
                             /* Tell the clock to start ticking */
                             SendDlgItemMessageW(hwndDlg, IDC_CLOCKWND, CLM_STARTCLOCK,
                                                 0, 0);
+
+                            SetTimer(hwndDlg, ID_TIMER, 1000 - st.wMilliseconds, NULL);
                             return TRUE;
                     }
                     break;
diff --git a/dll/cpl/timedate/monthcal.c b/dll/cpl/timedate/monthcal.c
index 3caf00fc96f..4f23396e7e4 100644
--- a/dll/cpl/timedate/monthcal.c
+++ b/dll/cpl/timedate/monthcal.c
@@ -238,11 +238,6 @@ MonthCalSetupDayTimer(IN PMONTHCALWND infoPtr)
     uElapse += (59 - (UINT)LocalTime.wMinute) * 60 * 1000;
     uElapse += (23 - (UINT)LocalTime.wHour) * 60 * 60 * 1000;
 
-    if (uElapse < USER_TIMER_MINIMUM || uElapse > USER_TIMER_MAXIMUM)
-        uElapse = 1000;
-    else
-        uElapse += 100; /* Add a delay of 0.1 seconds */
-
     /* Setup the new timer */
     if (SetTimer(infoPtr->hSelf,
                  ID_DAYTIMER,
diff --git a/dll/win32/syssetup/wizard.c b/dll/win32/syssetup/wizard.c
index 75913c67d69..dd24193f5cd 100644
--- a/dll/win32/syssetup/wizard.c
+++ b/dll/win32/syssetup/wizard.c
@@ -1542,11 +1542,8 @@ SetSystemLocalTime(HWND hwnd, PSETUPDATA SetupData)
 
 
 static VOID
-UpdateLocalSystemTime(HWND hwnd)
+UpdateLocalSystemTime(HWND hwnd, SYSTEMTIME LocalTime)
 {
-    SYSTEMTIME LocalTime;
-
-    GetLocalTime(&LocalTime);
     DateTime_SetSystemtime(GetDlgItem(hwnd, IDC_DATEPICKER), GDT_VALID, &LocalTime);
     DateTime_SetSystemtime(GetDlgItem(hwnd, IDC_TIMEPICKER), GDT_VALID, &LocalTime);
 }
@@ -1625,25 +1622,43 @@ DateTimePageDlgProc(HWND hwndDlg,
         }
 
         case WM_TIMER:
-            UpdateLocalSystemTime(hwndDlg);
+        {
+            SYSTEMTIME LocalTime;
+
+            GetLocalTime(&LocalTime);
+            UpdateLocalSystemTime(hwndDlg, LocalTime);
+
+            // Reset timeout.
+            SetTimer(hwndDlg, 1, 1000 - LocalTime.wMilliseconds, NULL);
             break;
+        }
 
         case WM_NOTIFY:
             switch (((LPNMHDR)lParam)->code)
             {
                 case PSN_SETACTIVE:
+                {
+                    SYSTEMTIME LocalTime;
+
+                    GetLocalTime(&LocalTime);
+                    UpdateLocalSystemTime(hwndDlg, LocalTime);
+
                     /* Enable the Back and Next buttons */
                     PropSheet_SetWizButtons(GetParent(hwndDlg), PSWIZB_BACK |
PSWIZB_NEXT);
+
                     if (SetupData->UnattendSetup &&
WriteDateTimeSettings(hwndDlg, SetupData))
                     {
                         SetWindowLongPtr(hwndDlg, DWLP_MSGRESULT,
SetupData->uFirstNetworkWizardPage);
                         return TRUE;
                     }
-                    SetTimer(hwndDlg, 1, 1000, NULL);
+
+                    SetTimer(hwndDlg, 1, 1000 - LocalTime.wMilliseconds, NULL);
                     break;
+                }
 
                 case PSN_KILLACTIVE:
                 case DTN_DATETIMECHANGE:
+                    // NB: Not re-set until changing page (PSN_SETACTIVE).
                     KillTimer(hwndDlg, 1);
                     break;