Bug 1336712 - toolkit: Drop test checks for OS X <= 10.8 and Windows XP. r=kmag
authorChris Peterson <cpeterson@mozilla.com>
Thu, 08 Nov 2018 23:11:59 +0000
changeset 445382 e006dd6e8fbd62dd57538294ac29e3489cb823ae
parent 445381 4e91b23279a9a0aaa30165b5859e9f6b2aa30bef
child 445383 157be32d85fe86d9a091381ce7e304c834df4ce1
push id35015
push userdluca@mozilla.com
push dateFri, 09 Nov 2018 17:45:20 +0000
treeherdermozilla-central@2f1158e5e0ce [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewerskmag
bugs1336712
milestone65.0a1
first release with
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
last release without
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
Bug 1336712 - toolkit: Drop test checks for OS X <= 10.8 and Windows XP. r=kmag Depends on D6584 Differential Revision: https://phabricator.services.mozilla.com/D6585
toolkit/content/aboutSupport.js
toolkit/content/tests/chrome/test_bug624329.xul
toolkit/locales/en-US/chrome/global/aboutSupport.properties
--- a/toolkit/content/aboutSupport.js
+++ b/toolkit/content/aboutSupport.js
@@ -57,20 +57,16 @@ var snapshotFormatters = {
       case 1:
       case 2:
       case 4:
       case 6:
       case 7:
       case 8:
         statusText = strings.GetStringFromName("multiProcessStatus." + data.autoStartStatus);
         break;
-
-      case 10:
-        statusText = (Services.appinfo.OS == "Darwin" ? "OS X 10.6 - 10.8" : "Windows XP");
-        break;
     }
 
     $("multiprocess-box").textContent = strings.formatStringFromName("multiProcessWindows",
       [data.numRemoteWindows, data.numTotalWindows, statusText], 3);
 
     if (data.remoteAutoStart) {
       $("contentprocesses-box").textContent = data.currentContentProcesses +
                                               "/" +
--- a/toolkit/content/tests/chrome/test_bug624329.xul
+++ b/toolkit/content/tests/chrome/test_bug624329.xul
@@ -49,23 +49,21 @@ function childFocused() {
     // maximizing the window is a simple way to ensure that the menu is near
     // the right edge of the screen.
 
     listenOnce("resize", childResized);
     win.maximize();
 }
 
 function childResized() {
-    const isOSXLion = navigator.userAgent.includes("Mac OS X 10.7");
-    const isOSXMtnLion = navigator.userAgent.includes("Mac OS X 10.8");
     const isOSXMavericks = navigator.userAgent.includes("Mac OS X 10.9");
     const isOSXYosemite = navigator.userAgent.includes("Mac OS X 10.10");
-    if (isOSXLion || isOSXMtnLion || isOSXMavericks || isOSXYosemite) {
+    if (isOSXMavericks || isOSXYosemite) {
         todo_is(win.windowState, win.STATE_MAXIMIZED,
-                "A resize before being maximized breaks this test on 10.7 and 10.8 and 10.9 and 10.10");
+                "A resize before being maximized breaks this test on 10.9 and 10.10");
         finish();
         return;
     }
 
     is(win.windowState, win.STATE_MAXIMIZED,
        "window should be maximized");
 
     isnot(win.innerWidth, 300,
--- a/toolkit/locales/en-US/chrome/global/aboutSupport.properties
+++ b/toolkit/locales/en-US/chrome/global/aboutSupport.properties
@@ -133,18 +133,16 @@ multiProcessWindows = %1$S/%2$S (%3$S)
 multiProcessStatus.0 = Enabled by user
 multiProcessStatus.1 = Enabled by default
 multiProcessStatus.2 = Disabled
 multiProcessStatus.4 = Disabled by accessibility tools
 multiProcessStatus.5 = Disabled by lack of graphics hardware acceleration on Mac OS X
 multiProcessStatus.6 = Disabled by unsupported text input
 multiProcessStatus.7 = Disabled by add-ons
 multiProcessStatus.8 = Disabled forcibly
-# No longer in use (bug 1296353) but we might bring this back.
-multiProcessStatus.9 = Disabled by graphics hardware acceleration on Windows XP
 multiProcessStatus.unknown = Unknown status
 
 asyncPanZoom = Asynchronous Pan/Zoom
 apzNone = none
 wheelEnabled = wheel input enabled
 touchEnabled = touch input enabled
 dragEnabled = scrollbar drag enabled
 keyboardEnabled = keyboard enabled