Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(8576)

Unified Diff: chrome/browser/zygote_main_linux.cc

Issue 6347045: linux: set LC_NUMERIC to C for all non-browser processes (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 9 years, 11 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« chrome/app/chrome_main.cc ('K') | « chrome/app/chrome_main.cc ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/zygote_main_linux.cc
diff --git a/chrome/browser/zygote_main_linux.cc b/chrome/browser/zygote_main_linux.cc
index 86f1decf77bf849d7892e310d45e90cff58bfc5d..b03e59c3b19ac92adb2aa6eb576222ad4cf0492c 100644
--- a/chrome/browser/zygote_main_linux.cc
+++ b/chrome/browser/zygote_main_linux.cc
@@ -589,14 +589,6 @@ static void PreSandboxInit() {
base::SysInfo::MaxSharedMemorySize();
- // To make wcstombs/mbstowcs work in a renderer, setlocale() has to be
- // called before the sandbox is triggered. It's possible to avoid calling
Evan Martin 2011/01/31 22:51:37 I repeated this test that prompted this extra code
- // setlocale() by pulling out the conversion between FilePath and
- // WebCore String out of the renderer and using string16 in place of
- // FilePath for IPC.
- const char* locale = setlocale(LC_ALL, "");
- LOG_IF(WARNING, locale == NULL) << "setlocale failed.";
-
// ICU DateFormat class (used in base/time_format.cc) needs to get the
// Olson timezone ID by accessing the zoneinfo files on disk. After
// TimeZone::createDefault is called once here, the timezone ID is
« chrome/app/chrome_main.cc ('K') | « chrome/app/chrome_main.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698