Bug 1923470 - Removing code related to FissionDecisionStatus for Experiments r=nika,fluent-reviewers

This patch removes code related to FissionDecisionStatus because fission
has shipped on Desktop and Android is not planning on using this mechanism.

Differential Revision: https://phabricator.services.mozilla.com/D226497
This commit is contained in:
Olivia Hall 2024-10-23 13:14:14 +00:00
parent b1146d27a1
commit 90259d1d20
6 changed files with 0 additions and 35 deletions

View File

@ -148,8 +148,6 @@ var snapshotFormatters = {
} catch (e) {}
const STATUS_STRINGS = {
experimentControl: "fission-status-experiment-control",
experimentTreatment: "fission-status-experiment-treatment",
disabledByE10sEnv: "fission-status-disabled-by-e10s-env",
enabledByEnv: "fission-status-enabled-by-env",
disabledByEnv: "fission-status-disabled-by-env",
@ -158,7 +156,6 @@ var snapshotFormatters = {
enabledByUserPref: "fission-status-enabled-by-user-pref",
disabledByUserPref: "fission-status-disabled-by-user-pref",
disabledByE10sOther: "fission-status-disabled-by-e10s-other",
enabledByRollout: "fission-status-enabled-by-rollout",
};
let statusTextId = STATUS_STRINGS[data.fissionDecisionStatus];

View File

@ -397,8 +397,6 @@ multi-process-windows = { $remoteWindows }/{ $totalWindows }
# $fissionWindows (integer) - Number of remote windows
# $totalWindows (integer) - Number of total windows
fission-windows = { $fissionWindows }/{ $totalWindows }
fission-status-experiment-control = Disabled by experiment
fission-status-experiment-treatment = Enabled by experiment
fission-status-disabled-by-e10s-env = Disabled by environment
fission-status-enabled-by-env = Enabled by environment
fission-status-disabled-by-env = Disabled by environment
@ -407,7 +405,6 @@ fission-status-disabled-by-default = Disabled by default
fission-status-enabled-by-user-pref = Enabled by user
fission-status-disabled-by-user-pref = Disabled by user
fission-status-disabled-by-e10s-other = E10s disabled
fission-status-enabled-by-rollout = Enabled by phased rollout
async-pan-zoom = Asynchronous Pan/Zoom
apz-none = none

View File

@ -1382,12 +1382,6 @@ nsXULAppInfo::GetFissionDecisionStatusString(nsACString& aResult) {
EnsureFissionAutostartInitialized();
switch (gFissionDecisionStatus) {
case eFissionExperimentControl:
aResult = "experimentControl";
break;
case eFissionExperimentTreatment:
aResult = "experimentTreatment";
break;
case eFissionDisabledByE10sEnv:
aResult = "disabledByE10sEnv";
break;
@ -1412,9 +1406,6 @@ nsXULAppInfo::GetFissionDecisionStatusString(nsACString& aResult) {
case eFissionDisabledByE10sOther:
aResult = "disabledByE10sOther";
break;
case eFissionEnabledByRollout:
aResult = "enabledByRollout";
break;
default:
MOZ_ASSERT_UNREACHABLE("Unexpected enum value");
}

View File

@ -3,13 +3,6 @@ from contextlib import contextmanager
from marionette_harness import MarionetteTestCase
class ExperimentStatus:
UNENROLLED = 0
ENROLLED_CONTROL = 1
ENROLLED_TREATMENT = 2
DISQUALIFIED = 3
class Prefs:
FISSION_AUTOSTART = "fission.autostart"
FISSION_AUTOSTART_SESSION = "fission.autostart.session"
@ -21,8 +14,6 @@ ENV_DISABLE_E10S = "MOZ_FORCE_DISABLE_E10S"
DECISION_STATUS = {
"experimentControl": 1,
"experimentTreatment": 2,
"disabledByE10sEnv": 3,
"enabledByEnv": 4,
"disabledByEnv": 5,

View File

@ -14544,8 +14544,6 @@ enum ContentWin32kLockdownState {
enum FissionDecisionStatus {
eFissionStatusUnknown = 0,
eFissionExperimentControl = 1,
eFissionExperimentTreatment = 2,
eFissionDisabledByE10sEnv = 3,
eFissionEnabledByEnv = 4,
eFissionDisabledByEnv = 5,
@ -14554,7 +14552,6 @@ enum FissionDecisionStatus {
eFissionEnabledByUserPref = 9,
eFissionDisabledByUserPref = 10,
eFissionDisabledByE10sOther = 11,
eFissionEnabledByRollout = 12,
}
}

View File

@ -117,12 +117,6 @@ interface nsIXULRuntime : nsISupports
// mapping in aboutSupport.js
cenum FissionDecisionStatus : 8 {
eFissionStatusUnknown = 0,
// Fission is disabled because the user is in the control group of a
// Normandy experiment.
eFissionExperimentControl = 1,
// Fission is enabled because the user is in the treatment group of a
// Normandy experiment.
eFissionExperimentTreatment = 2,
// Fission is disabled because the `MOZ_FORCE_DISABLE_E10S` environment
// variable is set.
eFissionDisabledByE10sEnv = 3,
@ -146,8 +140,6 @@ interface nsIXULRuntime : nsISupports
eFissionDisabledByUserPref = 10,
// Fission is disabled because e10s is disabled for some other reason.
eFissionDisabledByE10sOther = 11,
// Fission is enabled by a Normandy phased rollout.
eFissionEnabledByRollout = 12,
};
/**