From 12eaba020061eb462628b8a0e4e2d643503a345d Mon Sep 17 00:00:00 2001
From: "ssu%netscape.com"
 <ssu%netscape.com@18797224-902f-48f8-a5cc-f745e15eee43>
Date: Fri, 14 Sep 2001 20:52:54 +0000
Subject: [PATCH] fixing bug 99340's part regarding turbo settings in installer
 not retained when running installer in silent/auto modes. r=dveditz,syd
 sr=mscott, a=asa affects windows platforms only.

git-svn-id: svn://10.0.0.236/trunk@102936 18797224-902f-48f8-a5cc-f745e15eee43
---
 .../xpinstall/wizard/windows/setup/dialogs.c  | 81 ++++++++++---------
 1 file changed, 44 insertions(+), 37 deletions(-)

diff --git a/mozilla/xpinstall/wizard/windows/setup/dialogs.c b/mozilla/xpinstall/wizard/windows/setup/dialogs.c
index 931bbf07766..12e90c7b60c 100644
--- a/mozilla/xpinstall/wizard/windows/setup/dialogs.c
+++ b/mozilla/xpinstall/wizard/windows/setup/dialogs.c
@@ -2158,8 +2158,6 @@ LRESULT CALLBACK DlgProcQuickLaunch(HWND hDlg, UINT msg, WPARAM wParam, LONG lPa
   RECT  rDlg;
   LPSTR szMessage = NULL;
   char  szBuf[MAX_BUF];
-  char  szKey[MAX_BUF];
-  char  szData[MAX_BUF];
 
   switch(msg)
   {
@@ -2216,20 +2214,12 @@ LRESULT CALLBACK DlgProcQuickLaunch(HWND hDlg, UINT msg, WPARAM wParam, LONG lPa
       switch(LOWORD(wParam))
       {
         case IDWIZNEXT:
-          wsprintf(szKey, SETUP_STATE_REG_KEY, sgProduct.szCompanyName, sgProduct.szProductName,
-            sgProduct.szUserAgent);
-           
           if(diQuickLaunch.bTurboModeEnabled)
           {
-            if(IsDlgButtonChecked(hDlg, IDC_CHECK_TURBO_MODE) == BST_CHECKED) {
-              strcpy( szData, "turbo=yes" );
+            if(IsDlgButtonChecked(hDlg, IDC_CHECK_TURBO_MODE) == BST_CHECKED)
               diQuickLaunch.bTurboMode = TRUE;
-            }
-            else {
-              strcpy( szData, "turbo=no" );
+            else
               diQuickLaunch.bTurboMode = FALSE;
-            }
-            AppendWinReg(HKEY_CURRENT_USER, szKey, "browserargs", REG_SZ, szData, 0, strlen( szData ) + 1, FALSE, FALSE );
           }
  
           DestroyWindow(hDlg);
@@ -2264,8 +2254,6 @@ LRESULT CALLBACK DlgProcStartInstall(HWND hDlg, UINT msg, WPARAM wParam, LONG lP
 {
   RECT  rDlg;
   LPSTR szMessage = NULL;
-  char  szKey[MAX_BUF];
-  char  szData[MAX_BUF];
 
   switch(msg)
   {
@@ -2316,22 +2304,6 @@ LRESULT CALLBACK DlgProcStartInstall(HWND hDlg, UINT msg, WPARAM wParam, LONG lP
       switch(LOWORD(wParam))
       {
         case IDWIZNEXT:
-          wsprintf(szKey, SETUP_STATE_REG_KEY, sgProduct.szCompanyName, sgProduct.szProductName,
-            sgProduct.szUserAgent);
-           
-          if(diQuickLaunch.bTurboModeEnabled)
-          {
-            if(diQuickLaunch.bTurboMode == TRUE) {
-              strcpy( szData, "turbo=yes" );
-              diQuickLaunch.bTurboMode = TRUE;
-            }
-            else {
-              strcpy( szData, "turbo=no" );
-              diQuickLaunch.bTurboMode = FALSE;
-            }
-            AppendWinReg(HKEY_CURRENT_USER, szKey, "browserargs", REG_SZ, szData, 0, strlen( szData ) + 1, FALSE, FALSE );
-          }
- 
           DestroyWindow(hDlg);
           DlgSequenceNext();
           break;
@@ -2587,6 +2559,44 @@ BOOL CheckWizardStateCustom(DWORD dwDefault)
   return(TRUE);
 }
 
+/*
+ * Check to see if turbo is enabled.  If so, do the following:
+ *   * Log the turbo status that use had chosen.
+ *   * Set the appropriate Windows registry keys/values.
+ */
+void SetTurboArgs(void)
+{
+  char szData[MAX_BUF];
+  char szKey[MAX_BUF];
+
+  if(diQuickLaunch.bTurboModeEnabled)
+  {
+    /* log if the user selected the turbo mode or not */
+    LogISTurboMode(diQuickLaunch.bTurboMode);
+    LogMSTurboMode(diQuickLaunch.bTurboMode);
+
+    if(diQuickLaunch.bTurboMode)
+      strcpy( szData, "turbo=yes" );
+    else
+      strcpy( szData, "turbo=no" );
+
+    wsprintf(szKey,
+             SETUP_STATE_REG_KEY,
+             sgProduct.szCompanyName,
+             sgProduct.szProductName,
+             sgProduct.szUserAgent);
+    AppendWinReg(HKEY_CURRENT_USER,
+                 szKey,
+                 "browserargs",
+                 REG_SZ,
+                 szData,
+                 0,
+                 strlen( szData ) + 1,
+                 FALSE,
+                 FALSE );
+  }
+}
+
 void DlgSequenceNext()
 {
   HRESULT hrValue;
@@ -2802,15 +2812,12 @@ void DlgSequenceNext()
         /* PRE_DOWNLOAD process file manipulation functions */
         ProcessFileOps(T_PRE_DOWNLOAD, NULL);
 
-        /* log if the user selected the turbo mode or not */
-        if(diQuickLaunch.bTurboModeEnabled)
-        {
-          LogISTurboMode(diQuickLaunch.bTurboMode);
-          LogMSTurboMode(diQuickLaunch.bTurboMode);
-        }
-
         if(RetrieveArchives() == WIZ_OK)
         {
+          /* Check to see if Turbo is required.  If so, set the
+           * appropriate Windows registry keys */
+          SetTurboArgs();
+
           /* POST_DOWNLOAD process file manipulation functions */
           ProcessFileOps(T_POST_DOWNLOAD, NULL);
           /* PRE_XPCOM process file manipulation functions */