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

Unified Diff: extensions/browser/guest_view/web_view/web_view_apitest.cc

Issue 2832153003: Move calling of shared browser test methods like SetUpOnMainThread/TearDownOnMainThread/RunTestOn... (Closed)
Patch Set: sync Created 3 years, 8 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
« no previous file with comments | « extensions/browser/guest_view/web_view/web_view_apitest.h ('k') | extensions/shell/test/shell_test.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: extensions/browser/guest_view/web_view/web_view_apitest.cc
diff --git a/extensions/browser/guest_view/web_view/web_view_apitest.cc b/extensions/browser/guest_view/web_view/web_view_apitest.cc
index bce35f7e7d0f55323ff563a7bf42ea9f886ebb8b..6206dd126d5f9b38f92ff6ede01423ffca341007 100644
--- a/extensions/browser/guest_view/web_view/web_view_apitest.cc
+++ b/extensions/browser/guest_view/web_view/web_view_apitest.cc
@@ -29,6 +29,7 @@
#include "extensions/common/extension.h"
#include "extensions/common/extension_paths.h"
#include "extensions/common/switches.h"
+#include "extensions/shell/browser/desktop_controller.h"
#include "extensions/shell/browser/shell_content_browser_client.h"
#include "extensions/shell/browser/shell_extension_system.h"
#include "extensions/shell/test/shell_test.h"
@@ -175,11 +176,6 @@ void WebViewAPITest::RunTest(const std::string& test_name,
ASSERT_TRUE(done_listener.WaitUntilSatisfied());
}
-void WebViewAPITest::RunTestOnMainThreadLoop() {
- AppShellTest::RunTestOnMainThreadLoop();
- GetGuestViewManager()->WaitForAllGuestsDeleted();
-}
-
void WebViewAPITest::SetUpCommandLine(base::CommandLine* command_line) {
AppShellTest::SetUpCommandLine(command_line);
command_line->AppendSwitchASCII(::switches::kJavaScriptFlags, "--expose-gc");
@@ -236,6 +232,8 @@ void WebViewAPITest::StopTestServer() {
}
void WebViewAPITest::TearDownOnMainThread() {
+ DesktopController::instance()->CloseAppWindows();
+ GetGuestViewManager()->WaitForAllGuestsDeleted();
TestGetConfigFunction::set_test_config_state(nullptr);
AppShellTest::TearDownOnMainThread();
« no previous file with comments | « extensions/browser/guest_view/web_view/web_view_apitest.h ('k') | extensions/shell/test/shell_test.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698