Bug 1342347 - Disable 64-bit as default for new installs. r=rstrong a=jcristau
authorMatt Howell <mhowell@mozilla.com>
Wed, 08 Mar 2017 08:52:42 -0800
changeset 395149 efbf7a1aff2739dfcbbfc01eb4a6e51daa9d31ff
parent 395148 6f8621db856faa258ba94c62b5fa283868789136
child 395150 b1c941e9f9cc319c756d3ed10302d1e299dc1bbe
push id1468
push userasasaki@mozilla.com
push dateMon, 05 Jun 2017 19:31:07 +0000
treeherdermozilla-release@0641fc6ee9d1 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersrstrong, jcristau
bugs1342347
milestone54.0a2
Bug 1342347 - Disable 64-bit as default for new installs. r=rstrong a=jcristau MozReview-Commit-ID: Lnnmk1xlyqv
browser/installer/windows/nsis/stub.nsi
--- a/browser/installer/windows/nsis/stub.nsi
+++ b/browser/installer/windows/nsis/stub.nsi
@@ -336,21 +336,22 @@ Function .onInit
 
   ; SSE2 CPU support
   ${If} "$R7" == "0"
     MessageBox MB_OKCANCEL|MB_ICONSTOP "$(WARN_MIN_SUPPORTED_CPU_MSG)" IDCANCEL +2
     ExecShell "open" "${URLSystemRequirements}"
     Quit
   ${EndIf}
 
-  ${If} ${RunningX64}
-    StrCpy $INSTDIR "${DefaultInstDir64bit}"
-  ${Else}
+  ; Bug 1342347: disable 64-bit as default until Flash issues are resolved
+  ;${If} ${RunningX64}
+  ;  StrCpy $INSTDIR "${DefaultInstDir64bit}"
+  ;${Else}
     StrCpy $INSTDIR "${DefaultInstDir32bit}"
-  ${EndIf}
+  ;${EndIf}
 
   ; Require elevation if the user can elevate
   ${ElevateUAC}
 
   ; If we have any existing installation, use its location as the default
   ; path for this install, even if it's not the same architecture.
   SetRegView 32
   SetShellVarContext all ; Set SHCTX to HKLM
@@ -441,21 +442,22 @@ Function .onInit
   StrCpy $CheckboxShortcuts "1"
   StrCpy $CheckboxSendPing "1"
 !ifdef MOZ_MAINTENANCE_SERVICE
   StrCpy $CheckboxInstallMaintSvc "1"
 !else
   StrCpy $CheckboxInstallMaintSvc "0"
 !endif
   StrCpy $WasOptionsButtonClicked "0"
-  ${If} ${RunningX64}
-    StrCpy $DroplistArch "$(VERSION_64BIT)"
-  ${Else}
+  ; Bug 1342347: disable 64-bit as default until Flash issues are resolved
+  ;${If} ${RunningX64}
+    ;StrCpy $DroplistArch "$(VERSION_64BIT)"
+  ;${Else}
     StrCpy $DroplistArch "$(VERSION_32BIT)"
-  ${EndIf}
+  ;${EndIf}
 
   StrCpy $0 ""
 !ifdef FONT_FILE1
   ${If} ${FileExists} "$FONTS\${FONT_FILE1}"
     StrCpy $0 "${FONT_NAME1}"
   ${EndIf}
 !endif
 
@@ -1145,17 +1147,18 @@ Function createOptions
     Pop $DroplistArch
     ${NSD_CB_AddString} $DroplistArch "$(VERSION_32BIT)"
     ${NSD_CB_AddString} $DroplistArch "$(VERSION_64BIT)"
     ${NSD_OnChange} $DroplistArch OnChange_DroplistArch
     ; The uxtheme must be disabled in order to override the system colors.
     System::Call 'uxtheme::SetWindowTheme(i $DroplistArch, w " ", w " ")'
     SetCtlColors $DroplistArch ${COMMON_TEXT_COLOR_NORMAL} ${COMMON_BKGRD_COLOR}
     SendMessage $DroplistArch ${WM_SETFONT} $FontNormal 0
-    ${NSD_CB_SelectString} $DroplistArch "$(VERSION_64BIT)"
+    ; Bug 1342347: disable 64-bit as default until Flash issues are resolved
+    ${NSD_CB_SelectString} $DroplistArch "$(VERSION_32BIT)"
   ${EndIf}
 
   GetDlgItem $0 $HWNDPARENT 1 ; Install button
   ${If} ${FileExists} "$INSTDIR\${FileMainEXE}"
     SendMessage $0 ${WM_SETTEXT} 0 "STR:$(UPGRADE_BUTTON)"
   ${Else}
     SendMessage $0 ${WM_SETTEXT} 0 "STR:$(INSTALL_BUTTON)"
   ${EndIf}