Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(852)

Unified Diff: content/test/content_test_launcher.cc

Issue 190663012: Run ContentMain in a browser_test's browser process. This removes duplication of code in the browse… (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src/
Patch Set: cleanup Created 6 years, 9 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: content/test/content_test_launcher.cc
===================================================================
--- content/test/content_test_launcher.cc (revision 257128)
+++ content/test/content_test_launcher.cc (working copy)
@@ -6,15 +6,16 @@
#include "base/base_paths.h"
#include "base/command_line.h"
+#include "base/debug/stack_trace.h"
#include "base/logging.h"
#include "base/path_service.h"
+#include "base/process/memory.h"
#include "base/sys_info.h"
#include "base/test/test_suite.h"
+#include "base/test/test_timeouts.h"
#include "content/public/common/content_switches.h"
#include "content/public/test/content_test_suite_base.h"
#include "content/shell/app/shell_main_delegate.h"
-#include "content/shell/browser/shell_content_browser_client.h"
-#include "content/shell/common/shell_content_client.h"
#include "content/shell/common/shell_switches.h"
#include "testing/gtest/include/gtest/gtest.h"
@@ -23,43 +24,8 @@
#include "content/public/test/nested_message_pump_android.h"
#endif
-#if defined(OS_WIN)
-#include "content/public/app/startup_helper_win.h"
-#include "sandbox/win/src/sandbox_types.h"
-#endif // defined(OS_WIN)
-
namespace content {
-class ContentShellTestSuiteInitializer
- : public testing::EmptyTestEventListener {
- public:
- ContentShellTestSuiteInitializer() {
- }
-
- virtual void OnTestStart(const testing::TestInfo& test_info) OVERRIDE {
- content_client_.reset(new ShellContentClient);
- browser_content_client_.reset(new ShellContentBrowserClient());
- SetContentClient(content_client_.get());
- SetBrowserClientForTesting(browser_content_client_.get());
- }
-
- virtual void OnTestEnd(const testing::TestInfo& test_info) OVERRIDE {
-#if !defined(OS_ANDROID)
- // On Android, production code doesn't reset ContentClient during shutdown.
- // We try to do the same thing as production. Refer to crbug.com/181069.
- browser_content_client_.reset();
- content_client_.reset();
- SetContentClient(NULL);
-#endif
- }
-
- private:
- scoped_ptr<ShellContentClient> content_client_;
- scoped_ptr<ShellContentBrowserClient> browser_content_client_;
-
- DISALLOW_COPY_AND_ASSIGN(ContentShellTestSuiteInitializer);
-};
-
#if defined(OS_ANDROID)
scoped_ptr<base::MessagePump> CreateMessagePumpForUI() {
return scoped_ptr<base::MessagePump>(new NestedMessagePumpAndroid());
@@ -86,19 +52,8 @@
#endif
ContentTestSuiteBase::Initialize();
-
- testing::TestEventListeners& listeners =
- testing::UnitTest::GetInstance()->listeners();
- listeners.Append(new ContentShellTestSuiteInitializer);
}
- virtual void Shutdown() OVERRIDE {
- base::TestSuite::Shutdown();
- }
- virtual ContentClient* CreateClientForInitialization() OVERRIDE {
- return new ShellContentClient();
- }
-
DISALLOW_COPY_AND_ASSIGN(ContentBrowserTestSuite);
};

Powered by Google App Engine
This is Rietveld 408576698