Revert of Stop silently accepting unsupported flags in test binaries (patchset #5 id:150001 of https://codereview.webrtc.org/2968003003/ )

Reason for revert:
Causes failures on perf bots
https://luci-milo.appspot.com/buildbot/client.webrtc.perf/Mac%2010.11/3567

Original issue's description:
> Stop silently accepting unsupported flags in test binaries
>
> Instead explicitly ignore only the flags we know should be ignored.
>
> BUG=webrtc:7568
>
> Review-Url: https://codereview.webrtc.org/2968003003
> Cr-Commit-Position: refs/heads/master@{#19412}
> Committed: a2782f6f5d

TBR=kjellander@webrtc.org,henrika@webrtc.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=webrtc:7568

Review-Url: https://codereview.webrtc.org/3002963002
Cr-Commit-Position: refs/heads/master@{#19413}

tools_webrtc/gtest-parallel-wrapper.py

@@ -81,10 +81,6 @@ def get_args_and_env():
   parser.add_argument('--isolated-script-test-chartjson-output', type=str,
                       default=None)
 
-  # No-sandbox is a Chromium-specific flag, ignore it.
-  # TODO(oprypin): Remove (bugs.webrtc.org/8115)
-  parser.add_argument('--no-sandbox', action='store_true', default=False)
-
   # We have to do this, since --isolated-script-test-output is passed as an
   # argument to the executable by the swarming scripts, and we want to pass it
   # to gtest-parallel instead.

tools_webrtc/valgrind/webrtc_tests.py

@@ -24,7 +24,6 @@ Chrome's suppression files and our own, so we can easily maintain WebRTC
 specific suppressions in our own files.
 """
 
-import argparse
 import logging
 import optparse
 import os
@@ -125,15 +124,6 @@ def main(_):
                     help="run the tests with --test-launcher-shard-index")
   options, args = parser.parse_args()
 
-  ignore_parser = argparse.ArgumentParser()
-  # Ignore Chromium-specific flags
-  # TODO(oprypin): Remove (bugs.webrtc.org/8115)
-  ignore_parser.add_argument('--isolated-script-test-output',
-                             type=str, default=None)
-  ignore_parser.add_argument('--isolated-script-test-chartjson-output',
-                             type=str, default=None)
-  _, args = ignore_parser.parse_known_args(args)
-
   if options.verbose:
     logging_utils.config_root(logging.DEBUG)
   else:

webrtc/modules/audio_coding/codecs/isac/fix/test/kenny.cc

@@ -384,8 +384,10 @@ TEST(IsacFixTest, Kenny) {
     sscanf(argv[CodingMode+1], "%s", bottleneck_file);
     f_bn = fopen(bottleneck_file, "rb");
     if (f_bn  == NULL) {
-      printf("No value provided for BottleNeck\n");
+      printf("No value provided for BottleNeck and cannot read file %s\n",
-      FAIL() << "Cannot read file " << bottleneck_file;
+             bottleneck_file);
+      exit(0);  // TODO(oprypin): don't silence this error
+      // FAIL() << "Cannot read file " << bottleneck_file;
     } else {
       int aux_var;
       printf("reading bottleneck rates from file %s\n\n",bottleneck_file);

webrtc/test/test_main.cc

@@ -19,11 +19,6 @@
 
 #if defined(WEBRTC_IOS)
 #include "webrtc/test/ios/test_support.h"
-
-DEFINE_string(NSTreatUnknownArgumentsAsOpen, "",
-    "Intentionally ignored flag intended for iOS simulator.");
-DEFINE_string(ApplePersistenceIgnoreState, "",
-    "Intentionally ignored flag intended for iOS simulator.");
 #endif
 
 DEFINE_bool(logs, false, "print logs to stderr");
@@ -41,6 +36,9 @@ int main(int argc, char* argv[]) {
   if (rtc::LogMessage::GetLogToDebug() > rtc::LS_INFO)
     rtc::LogMessage::LogToDebug(rtc::LS_INFO);
 
+  // AllowCommandLineParsing allows us to ignore flags passed on to us by
+  // Chromium build bots without having to explicitly disable them.
+  google::AllowCommandLineReparsing();
   google::ParseCommandLineFlags(&argc, &argv, false);
 
   webrtc::test::SetExecutablePath(argv[0]);

webrtc/voice_engine/test/auto_test/voe_standard_test.cc

@@ -101,6 +101,9 @@ int main(int argc, char** argv) {
   // This function and RunInAutomatedMode is defined in automated_mode.cc
   // to avoid macro clashes with googletest (for instance ASSERT_TRUE).
   webrtc::voetest::InitializeGoogleTest(&argc, argv);
+  // AllowCommandLineParsing allows us to ignore flags passed on to us by
+  // Chromium build bots without having to explicitly disable them.
+  google::AllowCommandLineReparsing();
   google::ParseCommandLineFlags(&argc, &argv, true);
 
   if (FLAGS_automated) {