Non-SFI NaCl: Stop running sandbox tests in components_unittests

sandbox/linux/tests/main.cc

 // Copyright (c) 2012 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/at_exit.h"
 #include "base/logging.h"
 #include "base/test/test_suite.h"
 #include "sandbox/linux/tests/test_utils.h"
 #include "sandbox/linux/tests/unit_tests.h"
 #include "testing/gtest/include/gtest/gtest.h"
 
 namespace sandbox {

[Mark Seaborn, 2014/05/08 15:29:39]

Can this "namespace sandbox" be dropped now?  It only wraps an anon namespace.

[jln (DO NOT USE THIS), 2014/05/08 16:30:02]

Yeah, but it's the right idiom in sandbox/, no?. Everything should always be in
namespace sandbox.
It just so happens that main.cc is only declaring one exported function and it's
outside of the namespace for obvious reasons.

But typically sandbox/ should not have to add sandbox::.

[Mark Seaborn, 2014/05/08 16:42:14]

Oh, I see, yes, it does make a semantic difference.  Ignore my comment, then.

-
-extern const bool kAllowForkWithThreads = false;
-
 namespace {
 
 // Check for leaks in our tests.
 void RunPostTestsChecks() {
   if (TestUtils::CurrentProcessHasChildren()) {
     LOG(ERROR) << "One of the tests created a child that was not waited for. "
                << "Please, clean-up after your tests!";
   }
 }
 
@@ skipping 13 matching lines @@
   ::testing::FLAGS_gtest_death_test_style = "threadsafe";
 #if defined(OS_ANDROID)
   int tests_result = RUN_ALL_TESTS();
 #else
   int tests_result = base::RunUnitTestsUsingBaseTestSuite(argc, argv);
 #endif
 
   sandbox::RunPostTestsChecks();
   return tests_result;
 }