remove valgrind checks from //components/nacl/zygote

We have not supported valgrind for some time (it was never ported to
the GN build IIUC, and has been obsoleted by ASan/MSan/TSan).

BUG=791518

Change-Id: I9d6f5125de817b9334b295a8a78589004bb08294
Reviewed-on: https://chromium-review.googlesource.com/809132
Commit-Queue: Mostyn Bramley-Moore <mostynb@vewd.com>
Reviewed-by: Mark Seaborn <mseaborn@chromium.org>
Cr-Commit-Position: refs/heads/master@{#522051}

components/nacl/zygote/nacl_fork_delegate_linux.cc

@@ -27,7 +27,6 @@
 #include "base/process/kill.h"
 #include "base/process/launch.h"
 #include "base/strings/string_split.h"
-#include "base/third_party/dynamic_annotations/dynamic_annotations.h"
 #include "build/build_config.h"
 #include "components/nacl/common/nacl_nonsfi_util.h"
 #include "components/nacl/common/nacl_paths.h"
@@ -210,8 +209,6 @@ void NaClForkDelegate::Init(const int sandboxdesc,
              !PathService::Get(nacl::FILE_NACL_HELPER_BOOTSTRAP,
                                &helper_bootstrap_exe)) {
     status_ = kNaClHelperBootstrapMissing;
-  } else if (RunningOnValgrind()) {
-    status_ = kNaClHelperValgrind;
   } else {
     base::CommandLine::StringVector argv_to_launch;
     {

components/nacl/zygote/nacl_fork_delegate_linux.h

@@ -59,7 +59,7 @@ class NaClForkDelegate : public content::ZygoteForkDelegate {
     kNaClHelperUnused = 0,
     kNaClHelperMissing = 1,
     kNaClHelperBootstrapMissing = 2,
-    kNaClHelperValgrind = 3,
+    // kNaClHelperValgrind = 3,  // Running in valgrind no longer supported.
     kNaClHelperLaunchFailed = 4,
     kNaClHelperAckFailed = 5,
     kNaClHelperSuccess = 6,