Bug 1531376: Fix more unused code -Werrors in GeckoChildProcessHost.cpp
authorDavid Major <dmajor@mozilla.com>
Fri, 01 Mar 2019 14:29:39 -0500
changeset 519988 a5009e4ff8296c813335469174a1b7753e8a017d
parent 519987 2c8674d20ba67335e850efd59ce5a7c1ad4811f7
child 519989 c27077a06c6dc7d542034a36434f1041e3dcab31
push id10862
push userffxbld-merge
push dateMon, 11 Mar 2019 13:01:11 +0000
treeherdermozilla-beta@a2e7f5c935da [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
bugs1531376
milestone67.0a1
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
Bug 1531376: Fix more unused code -Werrors in GeckoChildProcessHost.cpp
ipc/glue/GeckoChildProcessHost.cpp
--- a/ipc/glue/GeckoChildProcessHost.cpp
+++ b/ipc/glue/GeckoChildProcessHost.cpp
@@ -675,25 +675,25 @@ AddAppDirToCommandLine(std::vector<std::
         aCmdLine.push_back("-profile");
         aCmdLine.push_back(path.get());
       }
 #endif
     }
   }
 }
 
-#ifdef XP_WIN
+#if defined(XP_WIN) && (defined(MOZ_SANDBOX) || defined(_ARM64_))
 static bool Contains(const std::vector<std::string>& aExtraOpts,
                      const char* aValue) {
   return std::any_of(aExtraOpts.begin(), aExtraOpts.end(),
                      [&](const std::string arg) {
                        return arg.find(aValue) != std::string::npos;
                      });
 }
-#endif  // XP_WIN
+#endif  // defined(XP_WIN) && (defined(MOZ_SANDBOX) || defined(_ARM64_))
 
 bool GeckoChildProcessHost::PerformAsyncLaunch(
     std::vector<std::string> aExtraOpts) {
 #ifdef MOZ_GECKO_PROFILER
   GetProfilerEnvVarsForChildProcess([this](const char* key, const char* value) {
     mLaunchOptions->env_map[ENVIRONMENT_STRING(key)] =
         ENVIRONMENT_STRING(value);
   });