Prevent loading libffmpegsumo.so to python (autotests) process.

chrome/test/base/chrome_test_suite.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 "chrome/test/base/chrome_test_suite.h"
 
+#if defined(OS_CHROMEOS)
+#include <stdio.h>
+#include <unistd.h>
+#endif
+
 #include "base/command_line.h"
 #include "base/file_util.h"
 #include "base/memory/ref_counted.h"
 #include "base/metrics/stats_table.h"
 #include "base/path_service.h"
 #include "base/process_util.h"
 #include "base/stringprintf.h"
 #include "base/utf_string_conversions.h"
 #include "chrome/browser/browser_process.h"
 #include "chrome/browser/chrome_content_browser_client.h"
@@ skipping 135 matching lines @@
 
 const char ChromeTestSuite::kLaunchAsBrowser[] = "as-browser";
 
 ChromeTestSuite::ChromeTestSuite(int argc, char** argv)
     : content::ContentTestSuiteBase(argc, argv) {
 }
 
 ChromeTestSuite::~ChromeTestSuite() {
 }
 
+static bool IsCrosPythonProcess() {

[jam, 2012/07/24 15:32:19]

nit: put static method in the annonymous namespace beside the other static
method, i.e. line 56. then the "static" should be redundant

[glotov, 2012/07/24 16:23:41]

I'd leave it here: locality is more important for readability IMO.

On 2012/07/24 15:32:19, John Abd-El-Malek wrote:

[jam, 2012/07/24 16:27:55]

it's chrome style to put static functions that aren't members of a class at the
top of the file, instead of interspersing them within the class definitions

[glotov, 2012/07/24 17:05:05]

Done.

+#if defined(OS_CHROMEOS)
+  char buf[80];
+  int num_read = readlink("/proc/self/exe", buf, sizeof(buf) - 1);
+  if (num_read == -1)
+    return false;
+  buf[num_read] = 0;
+  const char kPythonPrefix[] = "/python";
+  return !strncmp(strrchr(buf, '/'), kPythonPrefix, sizeof(kPythonPrefix) - 1);
+#endif  // defined(OS_CHROMEOS)
+  return false;
+}
+
 void ChromeTestSuite::Initialize() {
 #if defined(OS_MACOSX)
   base::mac::ScopedNSAutoreleasePool autorelease_pool;
   chrome_browser_application_mac::RegisterBrowserCrApp();
 #endif
 
   chrome::RegisterPathProvider();
   if (!browser_dir_.empty()) {
     PathService::Override(base::DIR_EXE, browser_dir_);
     PathService::Override(base::DIR_MODULE, browser_dir_);
   }
 
+  // Disable external libraries load if we are under python process in
+  // ChromeOS.  That means we are autotest and, if ASAN is used,
+  // external libraries load crashes.
+  if (!IsCrosPythonProcess())
+    content::ContentTestSuiteBase::DisableLoadingExternalLibraries();
+
   // Initialize after overriding paths as some content paths depend on correct
   // values for DIR_EXE and DIR_MODULE.
   content::ContentTestSuiteBase::Initialize();
 
 #if defined(OS_MACOSX)
   // Look in the framework bundle for resources.
   FilePath path;
   PathService::Get(base::DIR_EXE, &path);
   path = path.Append(chrome::kFrameworkName);
   base::mac::SetOverrideFrameworkBundlePath(path);
@@ skipping 30 matching lines @@
 #if defined(OS_MACOSX)
   base::mac::SetOverrideFrameworkBundle(NULL);
 #endif
 
   base::StatsTable::set_current(NULL);
   stats_table_.reset();
   RemoveSharedMemoryFile(stats_filename_);
 
   base::TestSuite::Shutdown();
 }