Commit f12b5127 authored by jschuh@chromium.org's avatar jschuh@chromium.org

Revert 96565 - Warm up locale and KsecDD in renderer sandbox

I did this in the other sandboxed processes, but missed the renderer because these susbsystems get warmed up implicitly by other initialization. However, changing initialization order (or dependency changes in the underlying libraries) could introduce random crashes. So, for consistency explicit warmup is best, and will be needed for a later patch that closes the LPC ports.


BUG=None
TEST=None
Review URL: http://codereview.chromium.org/7629015

TBR=jschuh@chromium.org
Review URL: http://codereview.chromium.org/7637014

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@96566 0039d316-1c4b-4281-b951-d872f2087c98
parent b40d59ad
......@@ -208,12 +208,6 @@ int RendererMain(const MainFunctionParams& parameters) {
#endif
bool run_loop = true;
if (!no_sandbox) {
// Cause advapi32 to load before the sandbox is turned on.
unsigned int dummy_rand;
rand_s(&dummy_rand);
// Warm up language subsystems before the sandbox is turned on.
::GetUserDefaultLangID();
::GetUserDefaultLCID();
run_loop = platform.EnableSandbox();
} else {
LOG(ERROR) << "Running without renderer sandbox";
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment