forked from mirrors/gecko-dev
		
	Bug 1887716 - Improve MOZ_FORCE_DISABLE_E10S check r=nika
Differential Revision: https://phabricator.services.mozilla.com/D206289
This commit is contained in:
		
							parent
							
								
									80663eebde
								
							
						
					
					
						commit
						19da69d55e
					
				
					 4 changed files with 11 additions and 21 deletions
				
			
		|  | @ -575,7 +575,7 @@ bool BrowserTabsRemoteAutostart() { | |||
|   // then we could remove this automation-only env variable.
 | ||||
|   if (gBrowserTabsRemoteAutostart && xpc::AreNonLocalConnectionsDisabled()) { | ||||
|     const char* forceDisable = PR_GetEnv("MOZ_FORCE_DISABLE_E10S"); | ||||
|     if (forceDisable && *forceDisable == '1') { | ||||
|     if (forceDisable && !strcmp(forceDisable, "1")) { | ||||
|       gBrowserTabsRemoteAutostart = false; | ||||
|       status = kE10sForceDisabled; | ||||
|     } | ||||
|  |  | |||
|  | @ -58,8 +58,7 @@ def set_e10s(enable): | |||
|     if enable: | ||||
|         output.write( | ||||
|             """ | ||||
|         app_version = self.execute_script("return Services.appinfo.version") | ||||
|         self.restart(env={ENV_DISABLE_E10S: app_version}) | ||||
|         self.restart(env={ENV_DISABLE_E10S: "1"}) | ||||
|         self.set_env(ENV_DISABLE_E10S, "null")\n""" | ||||
|         ) | ||||
|     else: | ||||
|  |  | |||
|  | @ -238,8 +238,7 @@ class TestFissionAutostart(MarionetteTestCase): | |||
|             decision="enabledByDefault", | ||||
|         ) | ||||
| 
 | ||||
|         app_version = self.execute_script("return Services.appinfo.version") | ||||
|         self.restart(env={ENV_DISABLE_E10S: app_version}) | ||||
|         self.restart(env={ENV_DISABLE_E10S: "1"}) | ||||
|         self.check_fission_status( | ||||
|             enabled=False, | ||||
|             decision="disabledByE10sEnv", | ||||
|  |  | |||
|  | @ -389,8 +389,7 @@ class TestWin32kAutostart(MarionetteTestCase): | |||
| 
 | ||||
|         self.marionette.set_pref(Prefs.WIN32K, True) | ||||
| 
 | ||||
|         app_version = self.execute_script("return Services.appinfo.version") | ||||
|         self.restart(env={ENV_DISABLE_E10S: app_version}) | ||||
|         self.restart(env={ENV_DISABLE_E10S: "1"}) | ||||
|         self.set_env(ENV_DISABLE_E10S, "null") | ||||
| 
 | ||||
|         self.check_win32k_status( | ||||
|  | @ -623,8 +622,7 @@ class TestWin32kAutostart(MarionetteTestCase): | |||
| 
 | ||||
|         self.marionette.set_pref(Prefs.WIN32K, True) | ||||
| 
 | ||||
|         app_version = self.execute_script("return Services.appinfo.version") | ||||
|         self.restart(env={ENV_DISABLE_E10S: app_version}) | ||||
|         self.restart(env={ENV_DISABLE_E10S: "1"}) | ||||
|         self.set_env(ENV_DISABLE_E10S, "null") | ||||
| 
 | ||||
|         self.check_win32k_status( | ||||
|  | @ -869,8 +867,7 @@ class TestWin32kAutostart(MarionetteTestCase): | |||
|             enrollmentStatusPref=ExperimentStatus.UNENROLLED, | ||||
|         ) | ||||
| 
 | ||||
|         app_version = self.execute_script("return Services.appinfo.version") | ||||
|         self.restart(env={ENV_DISABLE_E10S: app_version}) | ||||
|         self.restart(env={ENV_DISABLE_E10S: "1"}) | ||||
|         self.set_env(ENV_DISABLE_E10S, "null") | ||||
| 
 | ||||
|         self.check_win32k_status( | ||||
|  | @ -1137,8 +1134,7 @@ class TestWin32kAutostart(MarionetteTestCase): | |||
|             enrollmentStatusPref=ExperimentStatus.UNENROLLED, | ||||
|         ) | ||||
| 
 | ||||
|         app_version = self.execute_script("return Services.appinfo.version") | ||||
|         self.restart(env={ENV_DISABLE_E10S: app_version}) | ||||
|         self.restart(env={ENV_DISABLE_E10S: "1"}) | ||||
|         self.set_env(ENV_DISABLE_E10S, "null") | ||||
| 
 | ||||
|         self.check_win32k_status( | ||||
|  | @ -1366,8 +1362,7 @@ class TestWin32kAutostart(MarionetteTestCase): | |||
|         # Re-set enrollment pref, like Normandy would do | ||||
|         self.set_enrollment_status(ExperimentStatus.ENROLLED_CONTROL) | ||||
| 
 | ||||
|         app_version = self.execute_script("return Services.appinfo.version") | ||||
|         self.restart(env={ENV_DISABLE_E10S: app_version}) | ||||
|         self.restart(env={ENV_DISABLE_E10S: "1"}) | ||||
|         self.set_env(ENV_DISABLE_E10S, "null") | ||||
| 
 | ||||
|         self.check_win32k_status( | ||||
|  | @ -1603,8 +1598,7 @@ class TestWin32kAutostart(MarionetteTestCase): | |||
|         # Re-set enrollment pref, like Normandy would do | ||||
|         self.set_enrollment_status(ExperimentStatus.ENROLLED_CONTROL) | ||||
| 
 | ||||
|         app_version = self.execute_script("return Services.appinfo.version") | ||||
|         self.restart(env={ENV_DISABLE_E10S: app_version}) | ||||
|         self.restart(env={ENV_DISABLE_E10S: "1"}) | ||||
|         self.set_env(ENV_DISABLE_E10S, "null") | ||||
| 
 | ||||
|         self.check_win32k_status( | ||||
|  | @ -1853,8 +1847,7 @@ class TestWin32kAutostart(MarionetteTestCase): | |||
|         # Re-set enrollment pref, like Normandy would do | ||||
|         self.set_enrollment_status(ExperimentStatus.ENROLLED_TREATMENT) | ||||
| 
 | ||||
|         app_version = self.execute_script("return Services.appinfo.version") | ||||
|         self.restart(env={ENV_DISABLE_E10S: app_version}) | ||||
|         self.restart(env={ENV_DISABLE_E10S: "1"}) | ||||
|         self.set_env(ENV_DISABLE_E10S, "null") | ||||
| 
 | ||||
|         self.check_win32k_status( | ||||
|  | @ -2103,8 +2096,7 @@ class TestWin32kAutostart(MarionetteTestCase): | |||
|         # Re-set enrollment pref, like Normandy would do | ||||
|         self.set_enrollment_status(ExperimentStatus.ENROLLED_TREATMENT) | ||||
| 
 | ||||
|         app_version = self.execute_script("return Services.appinfo.version") | ||||
|         self.restart(env={ENV_DISABLE_E10S: app_version}) | ||||
|         self.restart(env={ENV_DISABLE_E10S: "1"}) | ||||
|         self.set_env(ENV_DISABLE_E10S, "null") | ||||
| 
 | ||||
|         self.check_win32k_status( | ||||
|  |  | |||
		Loading…
	
		Reference in a new issue
	
	 Gregory Pappas
						Gregory Pappas