Bug 1447154 - Remove unused getOSCPU method from devtools/shared/system;r=jryans draft
authorJulian Descottes <jdescottes@mozilla.com>
Mon, 19 Mar 2018 22:19:09 +0100
changeset 769624 d4187b69cd416c30c7adc588e8116e71bec24d5d
parent 769320 4f1014eb5039bdfdd7a39fb7785d102df1994a6f
child 769625 c3cedd6207e75e240e722d74a0b7fc669c983c75
child 769627 5d92e79f68c666b2e16d4955388957c5409a1c1d
child 769648 22390ff2fe8f963f7c65191537d25262319e9a80
child 769649 f59611637c0fc331da5160ae3aa183a2a910d74b
child 769896 f268dc9ed0d9dc76aa0c27fb495693ec1ca9b89d
child 769900 0ec5e4d320c8b1543d5fe579d8adf51c5fa4286d
push id103192
push userjdescottes@mozilla.com
push dateMon, 19 Mar 2018 21:37:03 +0000
reviewersjryans
bugs1447154
milestone61.0a1
Bug 1447154 - Remove unused getOSCPU method from devtools/shared/system;r=jryans MozReview-Commit-ID: JyzCszMF1UF
devtools/shared/system.js
--- a/devtools/shared/system.js
+++ b/devtools/shared/system.js
@@ -260,49 +260,16 @@ function getScreenDimensions() {
   if (width.value > 2880 || height.value > 1800) {
     return 12;
   }
 
   // Other dimension such as a VM.
   return 11;
 }
 
-/**
- * Function for fetching OS CPU and returning
- * an enum for Telemetry.
- */
-function getOSCPU() {
-  if (oscpu.includes("NT 5.1") || oscpu.includes("NT 5.2")) {
-    return 0;
-  }
-  if (oscpu.includes("NT 6.0")) {
-    return 1;
-  }
-  if (oscpu.includes("NT 6.1")) {
-    return 2;
-  }
-  if (oscpu.includes("NT 6.2")) {
-    return 3;
-  }
-  if (oscpu.includes("NT 6.3")) {
-    return 4;
-  }
-  if (oscpu.includes("OS X")) {
-    return 5;
-  }
-  if (oscpu.includes("Linux")) {
-    return 6;
-  }
-  if (oscpu.includes("NT 10.")) {
-    return 7;
-  }
-  // Other OS.
-  return 12;
-}
-
 function getSetting(name) {
   let deferred = defer();
 
   if ("@mozilla.org/settingsService;1" in Cc) {
     let settingsService;
 
     // settingsService fails in b2g child processes
     // TODO bug 1205797, make this work in child processes.
@@ -321,10 +288,9 @@ function getSetting(name) {
     deferred.reject(new Error("No settings service"));
   }
   return deferred.promise;
 }
 
 exports.getSystemInfo = getSystemInfo;
 exports.getSetting = getSetting;
 exports.getScreenDimensions = getScreenDimensions;
-exports.getOSCPU = getOSCPU;
 exports.constants = AppConstants;