Reland of move --no-sandbox flag from command line for telemetry

base/trace_event/memory_dump_manager.cc

 // Copyright 2015 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "base/trace_event/memory_dump_manager.h"
 
 #include <algorithm>
 #include <utility>
 
 #include "base/atomic_sequence_num.h"
@@ skipping 401 matching lines @@
   //   hence the above |post_task_failed| == true.
   // - The MDP does NOT have a task runner affinity. A locked access is required
   //   to R/W |disabled| (for the UnregisterAndDeleteDumpProviderSoon() case).
   bool should_dump;
   const char* disabled_reason = nullptr;
   {
     AutoLock lock(lock_);
     if (!mdpinfo->disabled) {
       if (mdpinfo->consecutive_failures >= kMaxConsecutiveFailuresCount) {
         mdpinfo->disabled = true;
-        disabled_reason =
-            "Dump failure, possibly related with sandboxing (crbug.com/461788)."
-            " Try --no-sandbox.";
+        disabled_reason = "Dump failed multiple times consecutively.";
       } else if (post_task_failed && mdpinfo->task_runner) {
         // Don't disable unbound dump providers. The utility thread is normally
         // shutdown when disabling the trace and getting here in this case is
         // expected.
         mdpinfo->disabled = true;
         disabled_reason = "The thread it was meant to dump onto is gone.";
       }
     }
     should_dump = !mdpinfo->disabled && !post_task_failed;
   }  // AutoLock lock(lock_);
@@ skipping 237 matching lines @@
   auto iter = process_dumps.find(pid);
   if (iter == process_dumps.end()) {
     scoped_ptr<ProcessMemoryDump> new_pmd(new ProcessMemoryDump(session_state));
     iter = process_dumps.insert(std::make_pair(pid, std::move(new_pmd))).first;
   }
   return iter->second.get();
 }
 
 }  // namespace trace_event
 }  // namespace base