Bug 1416113 - Unnecessary to set LOCALE_THOUSANDS_SEP, LOCALE_DECIMAL_POINT and LOCALE_GROUPING. r?snorp
LOCALE_THOUSANDS_SEP, LOCALE_DECIMAL_POINT and LOCALE_GROUPING are only used by
old code without ICU. Since we turn on ICU with ECMA-402 support, it is
unnecessary to set LOCALE_THOUSANDS_SEP, LOCALE_DECIMAL_POINT and
LOCALE_GROUPING in GeckoLoader.setupLocaleEnvironment.
MozReview-Commit-ID: LlGQdLlM7I2
--- a/mobile/android/geckoview/src/main/java/org/mozilla/gecko/mozglue/GeckoLoader.java
+++ b/mobile/android/geckoview/src/main/java/org/mozilla/gecko/mozglue/GeckoLoader.java
@@ -3,19 +3,16 @@
* License, v. 2.0. If a copy of the MPL was not distributed with this
* file, You can obtain one at http://mozilla.org/MPL/2.0/. */
package org.mozilla.gecko.mozglue;
import java.io.File;
import java.io.FileOutputStream;
import java.io.InputStream;
-import java.text.DecimalFormat;
-import java.text.DecimalFormatSymbols;
-import java.text.NumberFormat;
import java.util.Locale;
import java.util.zip.ZipEntry;
import java.util.zip.ZipFile;
import android.content.Context;
import android.content.Intent;
import android.os.Build;
import android.os.Environment;
@@ -461,25 +458,16 @@ public final class GeckoLoader {
public synchronized static void loadGeckoLibs(final Context context, final String apkName) {
loadLibsSetupLocked(context);
loadGeckoLibsNative(apkName);
}
private static void setupLocaleEnvironment() {
putenv("LANG=" + Locale.getDefault().toString());
- NumberFormat nf = NumberFormat.getInstance();
- if (nf instanceof DecimalFormat) {
- DecimalFormat df = (DecimalFormat)nf;
- DecimalFormatSymbols dfs = df.getDecimalFormatSymbols();
-
- putenv("LOCALE_DECIMAL_POINT=" + dfs.getDecimalSeparator());
- putenv("LOCALE_THOUSANDS_SEP=" + dfs.getGroupingSeparator());
- putenv("LOCALE_GROUPING=" + (char)df.getGroupingSize());
- }
}
@SuppressWarnings("serial")
public static class AbortException extends Exception {
public AbortException(String msg) {
super(msg);
}
}