Test to see if changes to XRE exit have caused the talos regressions. Should be push to try, but try is dead. See bug 701232.
☠☠ backed out by 1cf9e4b19819 ☠ ☠
authorDoug Turner <dougt@dougt.org>
Wed, 09 Nov 2011 18:25:33 -0800
changeset 83430 821f8e1cd0ed8b524a4f39aa80d15358bc83df8f
parent 83429 eaf778e88070a0292cf3c35139686eac4fe307ca
child 83431 1cf9e4b19819eb654976f467861b27965830d6fa
push id519
push userakeybl@mozilla.com
push dateWed, 01 Feb 2012 00:38:35 +0000
treeherdermozilla-beta@788ea1ef610b [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
bugs701232
milestone10.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
Test to see if changes to XRE exit have caused the talos regressions. Should be push to try, but try is dead. See bug 701232.
embedding/android/GeckoAppShell.java
--- a/embedding/android/GeckoAppShell.java
+++ b/embedding/android/GeckoAppShell.java
@@ -621,24 +621,28 @@ public class GeckoAppShell
         GeckoApp.mAppContext.setLaunchState(GeckoApp.LaunchState.GeckoExiting);
         Log.i("GeckoAppJava", "XRE exited");
         if (gRestartScheduled) {
             GeckoApp.mAppContext.doRestart();
         } else {
             Log.i("GeckoAppJava", "we're done, good bye");
             GeckoApp.mAppContext.finish();
         }
+        Log.w("GeckoAppShell", "Killing via System.exit()");
+        System.exit(0);
+        /*
         getHandler().postDelayed(new Runnable() {
             // This is just a watch dog to make sure we die
             // we should never hit this
             public void run() {
                 Log.w("GeckoAppShell", "Killing via System.exit()");
                 System.exit(0);
             }
         }, 5000);
+        */
     }
     static void scheduleRestart() {
         Log.i("GeckoAppJava", "scheduling restart");
         gRestartScheduled = true;
     }
 
     // "Installs" an application by creating a shortcut
     static void createShortcut(String aTitle, String aURI, String aIconData, String aType) {