Expose the sandbox related code through the content API. I did a bit of cleanup while I was doing...

chrome/test/base/in_process_browser_test.cc

 // Copyright (c) 2011 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/in_process_browser_test.h"
 
 #include "base/command_line.h"
 #include "base/debug/stack_trace.h"
 #include "base/file_path.h"
 #include "base/file_util.h"
-#include "base/mac/scoped_nsautorelease_pool.h"
 #include "base/path_service.h"
 #include "base/string_number_conversions.h"
 #include "base/test/test_file_util.h"
 #include "chrome/browser/browser_process.h"
 #include "chrome/browser/io_thread.h"
 #include "chrome/browser/profiles/profile.h"
 #include "chrome/browser/profiles/profile_manager.h"
 #include "chrome/browser/tabs/tab_strip_model.h"
 #include "chrome/browser/ui/browser.h"
 #include "chrome/browser/ui/browser_list.h"
@@ skipping 11 matching lines @@
 #include "content/browser/renderer_host/render_process_host.h"
 #include "content/browser/tab_contents/tab_contents.h"
 #include "content/public/browser/notification_service.h"
 #include "content/public/browser/notification_types.h"
 #include "content/renderer/mock_content_renderer_client.h"
 #include "net/base/mock_host_resolver.h"
 #include "net/test/test_server.h"
 
 #if defined(OS_CHROMEOS)
 #include "chrome/browser/chromeos/audio_handler.h"
+#elif defined(OS_MACOSX)
+#include "base/mac/scoped_nsautorelease_pool.h"
 #endif
 
 // Passed as value of kTestType.
 static const char kBrowserTestType[] = "browser";
 
 InProcessBrowserTest::InProcessBrowserTest()
     : browser_(NULL),
       show_window_(false),
       dom_automation_enabled_(false),
       tab_closeable_state_watcher_enabled_(false) {
@@ skipping 195 matching lines @@
   base::debug::StackTrace().PrintBacktrace();
   _exit(128 + signal);
 }
 #endif  // defined(OS_POSIX)
 
 void InProcessBrowserTest::RunTestOnMainThreadLoop() {
 #if defined(OS_POSIX)
   signal(SIGTERM, DumpStackTraceSignalHandler);
 #endif  // defined(OS_POSIX)
 
+#if defined(OS_MACOSX)
   // On Mac, without the following autorelease pool, code which is directly
   // executed (as opposed to executed inside a message loop) would autorelease
   // objects into a higher-level pool. This pool is not recycled in-sync with
   // the message loops' pools and causes problems with code relying on
   // deallocation via an autorelease pool (such as browser window closure and
   // browser shutdown). To avoid this, the following pool is recycled after each
   // time code is directly executed.
   base::mac::ScopedNSAutoreleasePool pool;
+#endif
 
   // Pump startup related events.
   MessageLoopForUI::current()->RunAllPending();
+#if defined(OS_MACOSX)
   pool.Recycle();
+#endif
 
   browser_ = CreateBrowser(ProfileManager::GetDefaultProfile());
+#if defined(OS_MACOSX)
   pool.Recycle();
+#endif
 
   // Pump any pending events that were created as a result of creating a
   // browser.
   MessageLoopForUI::current()->RunAllPending();
 
   SetUpOnMainThread();
+#if defined(OS_MACOSX)

[cpu_(ooo_6.6-7.5), 2011/10/29 01:35:39]

I think the sprinkling of ifdefs on the test are a bit too much, how about
enhancing BrowserTestBase to do this so the derived classes are not cluttered?

[jam, 2011/10/29 01:49:10]

I agree this test suffered the most. I wasn't that concerned though because it's
testing code that people don't really look at since it's part of the
infrastructure. Enhancing BrowserTestBase to add platform specific hooks in all
these places, and implementing stub versions for each platform, doesn't seem to
be worth it IMO

   pool.Recycle();
+#endif
 
   RunTestOnMainThread();
+#if defined(OS_MACOSX)
   pool.Recycle();
+#endif
 
   CleanUpOnMainThread();
+#if defined(OS_MACOSX)
   pool.Recycle();
+#endif
 
   QuitBrowsers();
+#if defined(OS_MACOSX)
   pool.Recycle();
+#endif
 }
 
 void InProcessBrowserTest::QuitBrowsers() {
   if (BrowserList::size() == 0)
     return;
 
   // Invoke CloseAllBrowsersAndMayExit on a running message loop.
   // CloseAllBrowsersAndMayExit exits the message loop after everything has been
   // shut down properly.
   MessageLoopForUI::current()->PostTask(
       FROM_HERE,
       NewRunnableFunction(&BrowserList::AttemptExit));
   ui_test_utils::RunMessageLoop();
 }