Back out changeset ad743e667004 (bug 950401) for Mac OS X 10.6 browser-chrome orange, mostly but not entirely on mochitest-e10s, on a CLOSED TREE.
authorL. David Baron <dbaron@dbaron.org>
Mon, 06 Apr 2015 16:07:13 -0700
changeset 267660 335f1295e99bece2c42c90f0405b1eefb60f5f41
parent 267659 0d4b7cd5b0cd7766118df58f99d6be6d1b2d3544
child 267670 f08e25c64e8ad52e0f6d8f92b3e55bd56f06f652
child 267706 c8e78a6795d2d6e7327f9a181b638ee9b0bf6bab
push id4830
push userjlund@mozilla.com
push dateMon, 29 Jun 2015 20:18:48 +0000
treeherdermozilla-beta@4c2175bb0420 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
bugs950401
milestone40.0a1
backs outad743e6670044f926c72e6e8abd6d817f04d1624
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
Back out changeset ad743e667004 (bug 950401) for Mac OS X 10.6 browser-chrome orange, mostly but not entirely on mochitest-e10s, on a CLOSED TREE.
ipc/chromium/src/base/process_util_bsd.cc
ipc/chromium/src/base/process_util_mac.mm
--- a/ipc/chromium/src/base/process_util_bsd.cc
+++ b/ipc/chromium/src/base/process_util_bsd.cc
@@ -175,19 +175,16 @@ bool LaunchApp(const std::vector<std::st
   FreeEnvVarsArray(vars, varsLen);
 
   posix_spawn_file_actions_destroy(&file_actions);
 
   bool process_handle_valid = pid > 0;
   if (!spawn_succeeded || !process_handle_valid) {
     retval = false;
   } else {
-    gProcessLog.print("==> process %d launched child process %d\n",
-                      GetCurrentProcId(), pid);
-
     if (wait)
       HANDLE_EINTR(waitpid(pid, 0, 0));
 
     if (process_handle)
       *process_handle = pid;
   }
 
   return retval;
--- a/ipc/chromium/src/base/process_util_mac.mm
+++ b/ipc/chromium/src/base/process_util_mac.mm
@@ -13,18 +13,16 @@
 #include <string>
 
 #include "base/eintr_wrapper.h"
 #include "base/logging.h"
 #include "base/rand_util.h"
 #include "base/string_util.h"
 #include "base/time.h"
 
-static mozilla::EnvironmentLog gProcessLog("MOZ_PROCESS_LOG");
-
 namespace base {
 
 void FreeEnvVarsArray(char* array[], int length)
 {
   for (int i = 0; i < length; i++) {
     free(array[i]);
   }
   delete[] array;
@@ -166,19 +164,16 @@ bool LaunchApp(const std::vector<std::st
   posix_spawn_file_actions_destroy(&file_actions);
 
   posix_spawnattr_destroy(&spawnattr);
 
   bool process_handle_valid = pid > 0;
   if (!spawn_succeeded || !process_handle_valid) {
     retval = false;
   } else {
-    gProcessLog.print("==> process %d launched child process %d\n",
-                      GetCurrentProcId(), pid);
-
     if (wait)
       HANDLE_EINTR(waitpid(pid, 0, 0));
 
     if (process_handle)
       *process_handle = pid;
   }
 
   return retval;