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

Unified Diff: chrome/browser/chromeos/app_mode/kiosk_app_manager_browsertest.cc

Issue 2847313002: Update some host_resolver()->AddRules in chrome/browser. (Closed)
Patch Set: 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
Index: chrome/browser/chromeos/app_mode/kiosk_app_manager_browsertest.cc
diff --git a/chrome/browser/chromeos/app_mode/kiosk_app_manager_browsertest.cc b/chrome/browser/chromeos/app_mode/kiosk_app_manager_browsertest.cc
index 8cb8d09e365b1ffaae763589c391ae64097a2419..a6b51cec7c99a680979d4b691e909c9fb79f4ac8 100644
--- a/chrome/browser/chromeos/app_mode/kiosk_app_manager_browsertest.cc
+++ b/chrome/browser/chromeos/app_mode/kiosk_app_manager_browsertest.cc
@@ -252,6 +252,8 @@ class KioskAppManagerTest : public InProcessBrowserTest {
void SetUpOnMainThread() override {
InProcessBrowserTest::SetUpOnMainThread();
+ host_resolver()->AddRule("*", "127.0.0.1");
+
// Start the accept thread as the sandbox host process has already been
// spawned.
embedded_test_server()->StartAcceptingConnections();
@@ -263,12 +265,6 @@ class KioskAppManagerTest : public InProcessBrowserTest {
void TearDownOnMainThread() override { settings_helper_.RestoreProvider(); }
- void SetUpInProcessBrowserTestFixture() override {
- InProcessBrowserTest::SetUpInProcessBrowserTestFixture();
-
- host_resolver()->AddRule("*", "127.0.0.1");
- }
-
std::string GetAppIds() const {
KioskAppManager::Apps apps;
manager()->GetApps(&apps);

Powered by Google App Engine
This is Rietveld 408576698