Bug 256180 mobile part - Increase the max size for the runtime stack of the Gecko main thread on Android. r=snorp.
☠☠ backed out by e956eb613130 ☠ ☠
authorHenri Sivonen <hsivonen@hsivonen.fi>
Wed, 04 Oct 2017 09:46:05 +0300
changeset 512924 b696df615c8b69589893e90aecefb9558f1b7328
parent 512923 2d69841d2eb77fd7f296a09b979c96a9794ff6f6
child 512925 bc2e0a89d88efca3d0a12f0538e34372dbb6d21d
push id1953
push userffxbld-merge
push dateMon, 11 Mar 2019 12:10:20 +0000
treeherdermozilla-release@9c35dcbaa899 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewerssnorp
bugs256180
milestone66.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 256180 mobile part - Increase the max size for the runtime stack of the Gecko main thread on Android. r=snorp. MozReview-Commit-ID: F85WvVTOP1S
mobile/android/geckoview/src/main/java/org/mozilla/gecko/GeckoThread.java
--- a/mobile/android/geckoview/src/main/java/org/mozilla/gecko/GeckoThread.java
+++ b/mobile/android/geckoview/src/main/java/org/mozilla/gecko/GeckoThread.java
@@ -153,17 +153,19 @@ public class GeckoThread extends Thread 
         public int prefsFd;
         public int prefMapFd;
         public int ipcFd;
         public int crashFd;
         public int crashAnnotationFd;
     }
 
     GeckoThread() {
-        setName("Gecko");
+        // Request more (virtual) stack space to avoid overflows in the CSS frame
+        // constructor. 8 MB matches desktop.
+        super(null, null, "Gecko", 8 * 1024 * 1024);
     }
 
     @WrapForJNI
     private static boolean isChildProcess() {
         final InitInfo info = INSTANCE.mInitInfo;
         return info != null && info.extras.getInt(EXTRA_IPC_FD, -1) != -1;
     }