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

Unified Diff: chrome/browser/safe_browsing/safe_browsing_test.cc

Issue 1378123003: Adding SSL ETS Tests (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@ets
Patch Set: Rebase. Created 5 years, 2 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/safe_browsing/safe_browsing_test.cc
diff --git a/chrome/browser/safe_browsing/safe_browsing_test.cc b/chrome/browser/safe_browsing/safe_browsing_test.cc
index e50cd25f9a4599ca1a702b67c331c30f1d188ba8..d05ee8b78aa6de5ad20c1e9caea7026fa77c1003 100644
--- a/chrome/browser/safe_browsing/safe_browsing_test.cc
+++ b/chrome/browser/safe_browsing/safe_browsing_test.cc
@@ -252,7 +252,7 @@ class SafeBrowsingServerTest : public InProcessBrowserTest {
return local_database_manager()->safe_browsing_task_runner_;
}
- const net::SpawnedTestServer& test_server() const {
+ const net::SpawnedTestServer& spawned_test_server() const {
return *test_server_;
}
@@ -538,8 +538,9 @@ IN_PROC_BROWSER_TEST_F(SafeBrowsingServerTest,
}
// Fetches URLs to verify and waits till server responses with data.
- EXPECT_EQ(net::URLRequestStatus::SUCCESS,
- safe_browsing_helper->FetchUrlsToVerify(test_server(), step));
+ EXPECT_EQ(
+ net::URLRequestStatus::SUCCESS,
+ safe_browsing_helper->FetchUrlsToVerify(spawned_test_server(), step));
std::vector<PhishingUrl> phishing_urls;
EXPECT_TRUE(ParsePhishingUrls(safe_browsing_helper->response_data(),
@@ -567,14 +568,16 @@ IN_PROC_BROWSER_TEST_F(SafeBrowsingServerTest,
}
// TODO(lzheng): We should verify the fetched database with local
// database to make sure they match.
- EXPECT_EQ(net::URLRequestStatus::SUCCESS,
- safe_browsing_helper->FetchDBToVerify(test_server(), step));
+ EXPECT_EQ(
+ net::URLRequestStatus::SUCCESS,
+ safe_browsing_helper->FetchDBToVerify(spawned_test_server(), step));
EXPECT_GT(safe_browsing_helper->response_data().size(), 0U);
last_step = step;
}
// Verifies with server if test is done and waits till server responses.
EXPECT_EQ(net::URLRequestStatus::SUCCESS,
- safe_browsing_helper->VerifyTestComplete(test_server(), last_step));
+ safe_browsing_helper->VerifyTestComplete(spawned_test_server(),
+ last_step));
EXPECT_EQ("yes", safe_browsing_helper->response_data());
}

Powered by Google App Engine
This is Rietveld 408576698