diff options
author | tfarina <tfarina@chromium.org> | 2015-11-30 14:00:26 -0800 |
---|---|---|
committer | Commit bot <commit-bot@chromium.org> | 2015-11-30 22:01:28 +0000 |
commit | 8dfd114ecca6c1b2c2f28885e6d7173ddad7c1fe (patch) | |
tree | 297643af797bfac43e6e64303a2afdbf31a23672 | |
parent | 46470b4460e191299ba652f0a9979c7099ea7098 (diff) | |
download | chromium_src-8dfd114ecca6c1b2c2f28885e6d7173ddad7c1fe.zip chromium_src-8dfd114ecca6c1b2c2f28885e6d7173ddad7c1fe.tar.gz chromium_src-8dfd114ecca6c1b2c2f28885e6d7173ddad7c1fe.tar.bz2 |
net: remove InitializeAndWaitUntilReady() from EmbeddedTestServer
Right now it is just calling Start(), so as the TODO say, this patch
removes it and update the callsites to call Start() instead.
This patch was generated on Linux by the following command line:
$ g grep -l InitializeAndWaitUntilReady content/ chrome/ | xargs perl -pi -e 's/InitializeAndWaitUntilReady/Start/g'
BUG=None
R=eroman@chromium.org,svaldez@chromium.org
TBR=jochen@chromium.org
Review URL: https://codereview.chromium.org/1482783003
Cr-Commit-Position: refs/heads/master@{#362232}
14 files changed, 13 insertions, 32 deletions
diff --git a/chrome/browser/dom_distiller/distillable_page_utils_browsertest.cc b/chrome/browser/dom_distiller/distillable_page_utils_browsertest.cc index 11c97d9..9575617e 100644 --- a/chrome/browser/dom_distiller/distillable_page_utils_browsertest.cc +++ b/chrome/browser/dom_distiller/distillable_page_utils_browsertest.cc @@ -57,7 +57,7 @@ class DistillablePageUtilsBrowserTestOption : public InProcessBrowserTest { void SetUpOnMainThread() override { InProcessBrowserTest::SetUpOnMainThread(); - ASSERT_TRUE(embedded_test_server()->InitializeAndWaitUntilReady()); + ASSERT_TRUE(embedded_test_server()->Start()); web_contents_ = browser()->tab_strip_model()->GetActiveWebContents(); setDelegate(web_contents_, holder_.GetDelegate()); diff --git a/chrome/browser/extensions/api/gcd_private/privet_v3_context_getter_unittest.cc b/chrome/browser/extensions/api/gcd_private/privet_v3_context_getter_unittest.cc index 6c19ff1..b461d42 100644 --- a/chrome/browser/extensions/api/gcd_private/privet_v3_context_getter_unittest.cc +++ b/chrome/browser/extensions/api/gcd_private/privet_v3_context_getter_unittest.cc @@ -40,7 +40,7 @@ class PrivetV3ContextGetterTest : public testing::Test, void CreateServer(EmbeddedTestServer::Type type) { server_.reset(new EmbeddedTestServer(type)); - ASSERT_TRUE(server_->InitializeAndWaitUntilReady()); + ASSERT_TRUE(server_->Start()); base::FilePath test_data_dir; ASSERT_TRUE(PathService::Get(base::DIR_SOURCE_ROOT, &test_data_dir)); diff --git a/chrome/browser/extensions/extension_bindings_apitest.cc b/chrome/browser/extensions/extension_bindings_apitest.cc index d99e574..07f6ac3 100644 --- a/chrome/browser/extensions/extension_bindings_apitest.cc +++ b/chrome/browser/extensions/extension_bindings_apitest.cc @@ -115,7 +115,7 @@ IN_PROC_BROWSER_TEST_F(ExtensionBindingsApiTest, ModuleSystem) { } IN_PROC_BROWSER_TEST_F(ExtensionBindingsApiTest, NoExportOverriding) { - ASSERT_TRUE(embedded_test_server()->InitializeAndWaitUntilReady()); + ASSERT_TRUE(embedded_test_server()->Start()); // We need to create runtime bindings in the web page. An extension that's // externally connectable will do that for us. @@ -139,7 +139,7 @@ IN_PROC_BROWSER_TEST_F(ExtensionBindingsApiTest, NoExportOverriding) { } IN_PROC_BROWSER_TEST_F(ExtensionBindingsApiTest, NoGinDefineOverriding) { - ASSERT_TRUE(embedded_test_server()->InitializeAndWaitUntilReady()); + ASSERT_TRUE(embedded_test_server()->Start()); // We need to create runtime bindings in the web page. An extension that's // externally connectable will do that for us. @@ -165,7 +165,7 @@ IN_PROC_BROWSER_TEST_F(ExtensionBindingsApiTest, NoGinDefineOverriding) { } IN_PROC_BROWSER_TEST_F(ExtensionBindingsApiTest, HandlerFunctionTypeChecking) { - ASSERT_TRUE(embedded_test_server()->InitializeAndWaitUntilReady()); + ASSERT_TRUE(embedded_test_server()->Start()); ui_test_utils::NavigateToURL( browser(), embedded_test_server()->GetURL( diff --git a/chrome/browser/local_discovery/privet_http_unittest.cc b/chrome/browser/local_discovery/privet_http_unittest.cc index 5c40cf4..a8cb53c 100644 --- a/chrome/browser/local_discovery/privet_http_unittest.cc +++ b/chrome/browser/local_discovery/privet_http_unittest.cc @@ -1036,7 +1036,7 @@ class PrivetHttpWithServerTest : public ::testing::Test, BrowserThread::GetMessageLoopProxyForThread(BrowserThread::IO)); server_.reset(new EmbeddedTestServer(EmbeddedTestServer::TYPE_HTTP)); - ASSERT_TRUE(server_->InitializeAndWaitUntilReady()); + ASSERT_TRUE(server_->Start()); base::FilePath test_data_dir; ASSERT_TRUE(PathService::Get(base::DIR_SOURCE_ROOT, &test_data_dir)); diff --git a/chrome/browser/notifications/notification_browsertest.cc b/chrome/browser/notifications/notification_browsertest.cc index eccbfad..8642d8d 100644 --- a/chrome/browser/notifications/notification_browsertest.cc +++ b/chrome/browser/notifications/notification_browsertest.cc @@ -435,7 +435,7 @@ IN_PROC_BROWSER_TEST_F(NotificationsTest, TestCreateSimpleNotification) { IN_PROC_BROWSER_TEST_F(NotificationsTest, NotificationBlockerTest) { ToggledNotificationBlocker blocker; - ASSERT_TRUE(embedded_test_server()->InitializeAndWaitUntilReady()); + ASSERT_TRUE(embedded_test_server()->Start()); // Creates a simple notification. AllowAllOrigins(); diff --git a/chrome/browser/search_engines/template_url_scraper_browsertest.cc b/chrome/browser/search_engines/template_url_scraper_browsertest.cc index b5beb9a..d8ee05a 100644 --- a/chrome/browser/search_engines/template_url_scraper_browsertest.cc +++ b/chrome/browser/search_engines/template_url_scraper_browsertest.cc @@ -67,7 +67,7 @@ scoped_ptr<net::test_server::HttpResponse> SendResponse( IN_PROC_BROWSER_TEST_F(TemplateURLScraperTest, ScrapeWithOnSubmit) { host_resolver()->AddRule("*.foo.com", "localhost"); embedded_test_server()->RegisterRequestHandler(base::Bind(&SendResponse)); - ASSERT_TRUE(embedded_test_server()->InitializeAndWaitUntilReady()); + ASSERT_TRUE(embedded_test_server()->Start()); TemplateURLService* template_urls = TemplateURLServiceFactory::GetInstance()->GetForProfile( diff --git a/content/browser/devtools/protocol/devtools_protocol_browsertest.cc b/content/browser/devtools/protocol/devtools_protocol_browsertest.cc index f8cae1a..e76eb66 100644 --- a/content/browser/devtools/protocol/devtools_protocol_browsertest.cc +++ b/content/browser/devtools/protocol/devtools_protocol_browsertest.cc @@ -416,7 +416,7 @@ IN_PROC_BROWSER_TEST_F(DevToolsProtocolTest, CrossSitePauseInBeforeUnload) { IN_PROC_BROWSER_TEST_F(DevToolsProtocolTest, InspectDuringFrameSwap) { host_resolver()->AddRule("*", "127.0.0.1"); - ASSERT_TRUE(embedded_test_server()->InitializeAndWaitUntilReady()); + ASSERT_TRUE(embedded_test_server()->Start()); content::SetupCrossSiteRedirector(embedded_test_server()); GURL test_url1 = diff --git a/content/browser/loader/resource_dispatcher_host_browsertest.cc b/content/browser/loader/resource_dispatcher_host_browsertest.cc index 954eab0..c0a4499 100644 --- a/content/browser/loader/resource_dispatcher_host_browsertest.cc +++ b/content/browser/loader/resource_dispatcher_host_browsertest.cc @@ -642,7 +642,7 @@ class LoFiResourceDispatcherHostBrowserTest : public ContentBrowserTest { void SetUpOnMainThread() override { ContentBrowserTest::SetUpOnMainThread(); - ASSERT_TRUE(embedded_test_server()->InitializeAndWaitUntilReady()); + ASSERT_TRUE(embedded_test_server()->Start()); delegate_.reset(new LoFiModeResourceDispatcherHostDelegate( embedded_test_server()->GetURL("/page_with_iframe.html"), diff --git a/content/browser/wake_lock/wake_lock_browsertest.cc b/content/browser/wake_lock/wake_lock_browsertest.cc index 6e7d2d9..3ac4589 100644 --- a/content/browser/wake_lock/wake_lock_browsertest.cc +++ b/content/browser/wake_lock/wake_lock_browsertest.cc @@ -34,7 +34,7 @@ class WakeLockTest : public ContentBrowserTest { void SetUpOnMainThread() override { host_resolver()->AddRule("*", "127.0.0.1"); - EXPECT_TRUE(embedded_test_server()->InitializeAndWaitUntilReady()); + EXPECT_TRUE(embedded_test_server()->Start()); // To prevent occlusion events from changing page visibility. GetWebContents()->IncrementCapturerCount(gfx::Size()); } diff --git a/content/browser/web_contents/web_contents_impl_browsertest.cc b/content/browser/web_contents/web_contents_impl_browsertest.cc index 20cdcca..9b229746 100644 --- a/content/browser/web_contents/web_contents_impl_browsertest.cc +++ b/content/browser/web_contents/web_contents_impl_browsertest.cc @@ -752,7 +752,7 @@ class MockPageScaleObserver : public WebContentsObserver { // When the page scale factor is set in the renderer it should send // a notification to the browser so that WebContentsObservers are notified. IN_PROC_BROWSER_TEST_F(WebContentsImplBrowserTest, ChangePageScale) { - ASSERT_TRUE(embedded_test_server()->InitializeAndWaitUntilReady()); + ASSERT_TRUE(embedded_test_server()->Start()); NavigateToURL(shell(), embedded_test_server()->GetURL("/title1.html")); MockPageScaleObserver observer(shell()); diff --git a/content/test/webrtc_content_browsertest_base.cc b/content/test/webrtc_content_browsertest_base.cc index 271a72f..62b9467 100644 --- a/content/test/webrtc_content_browsertest_base.cc +++ b/content/test/webrtc_content_browsertest_base.cc @@ -75,7 +75,7 @@ std::string WebRtcContentBrowserTest::ExecuteJavascriptAndReturnResult( void WebRtcContentBrowserTest::MakeTypicalCall(const std::string& javascript, const std::string& html_file) { - ASSERT_TRUE(embedded_test_server()->InitializeAndWaitUntilReady()); + ASSERT_TRUE(embedded_test_server()->Start()); GURL url(embedded_test_server()->GetURL(html_file)); NavigateToURL(shell(), url); diff --git a/net/test/android/javatests/src/org/chromium/net/test/EmbeddedTestServer.java b/net/test/android/javatests/src/org/chromium/net/test/EmbeddedTestServer.java index 841a3e5..9f974ce 100644 --- a/net/test/android/javatests/src/org/chromium/net/test/EmbeddedTestServer.java +++ b/net/test/android/javatests/src/org/chromium/net/test/EmbeddedTestServer.java @@ -54,17 +54,6 @@ public class EmbeddedTestServer { nativeServeFilesFromDirectory(mNativeEmbeddedTestServer, directoryPath); } - // TODO(svaldez): Remove once all consumers have switched to start(). - /** Wrapper for start() - * - * start() should be used instead of this. - * - * @return Whether the server was successfully initialized. - */ - public boolean initializeAndWaitUntilReady() { - return start(); - } - /** Starts the server. * * Note that this should be called after handlers are set up, including any relevant calls diff --git a/net/test/embedded_test_server/embedded_test_server.cc b/net/test/embedded_test_server/embedded_test_server.cc index 6131cee..bc401c9 100644 --- a/net/test/embedded_test_server/embedded_test_server.cc +++ b/net/test/embedded_test_server/embedded_test_server.cc @@ -78,10 +78,6 @@ bool EmbeddedTestServer::Start() { return true; } -bool EmbeddedTestServer::InitializeAndWaitUntilReady() { - return Start(); -} - bool EmbeddedTestServer::InitializeAndListen() { DCHECK(!Started()); diff --git a/net/test/embedded_test_server/embedded_test_server.h b/net/test/embedded_test_server/embedded_test_server.h index 1fe1383..044f017a 100644 --- a/net/test/embedded_test_server/embedded_test_server.h +++ b/net/test/embedded_test_server/embedded_test_server.h @@ -131,10 +131,6 @@ class EmbeddedTestServer { // Returns whether a listening socket has been successfully created. bool Start(); - // Deprecated method that calls Start(). - // TODO(svaldez): Remove and replace with Start(). - bool InitializeAndWaitUntilReady() WARN_UNUSED_RESULT; - // Starts listening for incoming connections but will not yet accept them. // Returns whether a listening socket has been succesfully created. bool InitializeAndListen() WARN_UNUSED_RESULT; |