[ros-diffs] [jimtabor] 51973: [ScreenSaver] - Fixed screen saver load and save plus the registry problems.

Author: jimtabor
Date: Sat May 28 11:39:36 2011
New Revision: 51973

URL: http://svn.reactos.org/svn/reactos?rev=51973&view=rev
Log:
[ScreenSaver]
- Fixed screen saver load and save plus the registry problems.

Modified:
    trunk/reactos/base/system/winlogon/screensaver.c
    trunk/reactos/dll/cpl/desk/screensaver.c
    trunk/reactos/subsystems/win32/win32k/ntuser/input.c
    trunk/reactos/subsystems/win32/win32k/ntuser/sysparams.c

Modified: trunk/reactos/base/system/winlogon/screensaver.c
URL:
http://svn.reactos.org/svn/reactos/trunk/reactos/base/system/winlogon/scree…
==============================================================================
--- trunk/reactos/base/system/winlogon/screensaver.c [iso-8859-1] (original)
+++ trunk/reactos/base/system/winlogon/screensaver.c [iso-8859-1] Sat May 28 11:39:36
2011
@@ -240,7 +240,7 @@
 StartScreenSaver(
 	IN PWLSESSION Session)
 {
-	HKEY hKey = NULL;
+	HKEY hKey = NULL, hCurrentUser = NULL;
 	WCHAR szApplicationName[MAX_PATH];
 	WCHAR szCommandLine[MAX_PATH + 3];
 	DWORD bufferSize = sizeof(szApplicationName) - sizeof(WCHAR);
@@ -258,8 +258,17 @@
 		goto cleanup;
 	}
 
+	rc = RegOpenCurrentUser(
+		KEY_READ,
+		&hCurrentUser);
+	if (rc != ERROR_SUCCESS)
+	{
+		ERR("WL: RegOpenCurrentUser Error!\n");
+		goto cleanup;
+	}
+
 	rc = RegOpenKeyExW(
-		HKEY_CURRENT_USER,
+		hCurrentUser,
 		L"Control Panel\\Desktop",
 		0,
 		KEY_QUERY_VALUE,
@@ -343,6 +352,8 @@
 	RevertToSelf();
 	if (hKey)
 		RegCloseKey(hKey);
+	if (hCurrentUser)
+		RegCloseKey(hCurrentUser);
 	if (!ret)
 	{
 		PostMessageW(Session->SASWindow, WLX_WM_SAS, WLX_SAS_TYPE_SCRNSVR_ACTIVITY, 0);

Modified: trunk/reactos/dll/cpl/desk/screensaver.c
URL:
http://svn.reactos.org/svn/reactos/trunk/reactos/dll/cpl/desk/screensaver.c…
==============================================================================
--- trunk/reactos/dll/cpl/desk/screensaver.c [iso-8859-1] (original)
+++ trunk/reactos/dll/cpl/desk/screensaver.c [iso-8859-1] Sat May 28 11:39:36 2011
@@ -460,6 +460,7 @@
 SetScreenSaver(HWND hwndDlg, PDATA pData)
 {
     HKEY regKey;
+    BOOL DeleteMode = FALSE;
 
     if (RegOpenKeyEx(HKEY_CURRENT_USER,
                      _T("Control Panel\\Desktop"),
@@ -486,6 +487,7 @@
         {
             /* Windows deletes the value if no screensaver is set */
             RegDeleteValue(regKey, _T("SCRNSAVE.EXE"));
+            DeleteMode = TRUE;
         }
 
         /* set the screensaver time delay */
@@ -505,6 +507,10 @@
                       REG_SZ,
                       (PBYTE)szTime,
                       _tcslen(szTime) * sizeof(TCHAR));
+
+        if (DeleteMode) Time = 0;
+
+        SystemParametersInfoW(SPI_SETSCREENSAVETIMEOUT, Time, 0, SPIF_SENDCHANGE);
 
         /* set the secure value */
         Ret = SendDlgItemMessage(hwndDlg,

Modified: trunk/reactos/subsystems/win32/win32k/ntuser/input.c
URL:
http://svn.reactos.org/svn/reactos/trunk/reactos/subsystems/win32/win32k/nt…
==============================================================================
--- trunk/reactos/subsystems/win32/win32k/ntuser/input.c [iso-8859-1] (original)
+++ trunk/reactos/subsystems/win32/win32k/ntuser/input.c [iso-8859-1] Sat May 28 11:39:36
2011
@@ -75,14 +75,15 @@
 VOID FASTCALL DoTheScreenSaver(VOID)
 {
    LARGE_INTEGER TickCount;
-   DWORD Test;
+   DWORD Test, TO;
 
    if (gspv.iScrSaverTimeout > 0) // Zero means Off.
    {
       KeQueryTickCount(&TickCount);
       Test = MsqCalculateMessageTime(&TickCount);
       Test = Test - LastInputTick;
-      if (Test > gspv.iScrSaverTimeout)
+      TO = 1000 * gspv.iScrSaverTimeout;
+      if (Test > TO)
       {
          DPRINT("Screensaver Message Start! Tick %d Timeout %d \n", Test,
gspv.iScrSaverTimeout);
 

Modified: trunk/reactos/subsystems/win32/win32k/ntuser/sysparams.c
URL:
http://svn.reactos.org/svn/reactos/trunk/reactos/subsystems/win32/win32k/nt…
==============================================================================
--- trunk/reactos/subsystems/win32/win32k/ntuser/sysparams.c [iso-8859-1] (original)
+++ trunk/reactos/subsystems/win32/win32k/ntuser/sysparams.c [iso-8859-1] Sat May 28
11:39:36 2011
@@ -59,6 +59,7 @@
 
 static const WCHAR* KEY_DESKTOP = L"Control Panel\\Desktop";
 static const WCHAR* VAL_SCRTO = L"ScreenSaveTimeOut";
+static const WCHAR* VAL_SCRNSV = L"SCRNSAVE.EXE";
 static const WCHAR* VAL_SCRACT = L"ScreenSaveActive";
 static const WCHAR* VAL_GRID = L"GridGranularity";
 static const WCHAR* VAL_DRAG = L"DragFullWindows";
@@ -134,6 +135,20 @@
         return dValue;
     }
     return Result;
+}
+
+static
+DWORD
+SpiLoadTimeOut(VOID)
+{   // Must have the string!
+    WCHAR szApplicationName[MAX_PATH];
+    RtlZeroMemory(&szApplicationName, sizeof(szApplicationName));
+    if (!RegReadUserSetting(KEY_DESKTOP, VAL_SCRNSV, REG_SZ, &szApplicationName,
sizeof(szApplicationName)))
+    {
+        return 0;
+    }
+    if (wcslen(szApplicationName) == 0) return 0;
+    return SpiLoadInt(KEY_DESKTOP, VAL_SCRTO, 0);
 }
 
 static
@@ -280,7 +295,7 @@
     gspv.bDropShadow = 1;
     gspv.dwMenuShowDelay = 100;
 
-    gspv.iScrSaverTimeout = 600000; // TO in 10 minutes.
+    gspv.iScrSaverTimeout = SpiLoadTimeOut();
     gspv.bScrSaverActive = FALSE;
     gspv.bScrSaverRunning = FALSE;
 #if(WINVER >= 0x0600)