From 10dec1a5aa4445193d96470c980440481bbc3335 Mon Sep 17 00:00:00 2001 From: Neil Rashbrook Date: Thu, 9 Jun 2011 21:15:10 +0100 Subject: [PATCH] Bug 660535 Remove useless use of PromiseFlatCString r=cjones --- ipc/glue/GeckoChildProcessHost.cpp | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/ipc/glue/GeckoChildProcessHost.cpp b/ipc/glue/GeckoChildProcessHost.cpp index 7352021f0fe..a64edcba8bc 100644 --- a/ipc/glue/GeckoChildProcessHost.cpp +++ b/ipc/glue/GeckoChildProcessHost.cpp @@ -383,7 +383,7 @@ GeckoChildProcessHost::PerformAsyncLaunch(std::vector aExtraOpts, b // or mChildCounter touched by any other thread, so this is safe. static char* restoreOrigLogName = 0; if (!restoreOrigLogName) - restoreOrigLogName = strdup(PromiseFlatCString(setChildLogName).get()); + restoreOrigLogName = strdup(setChildLogName.get()); // Append child-specific postfix to name setChildLogName.AppendLiteral(".child-"); @@ -391,7 +391,7 @@ GeckoChildProcessHost::PerformAsyncLaunch(std::vector aExtraOpts, b // Passing temporary to PR_SetEnv is ok here because env gets copied // by exec, etc., to permanent storage in child when process launched. - PR_SetEnv(PromiseFlatCString(setChildLogName).get()); + PR_SetEnv(setChildLogName.get()); bool retval = PerformAsyncLaunchInternal(aExtraOpts, arch); // Revert to original value