diff options
Diffstat (limited to 'chrome/browser/safe_browsing/safe_browsing_test.cc')
-rw-r--r-- | chrome/browser/safe_browsing/safe_browsing_test.cc | 34 |
1 files changed, 18 insertions, 16 deletions
diff --git a/chrome/browser/safe_browsing/safe_browsing_test.cc b/chrome/browser/safe_browsing/safe_browsing_test.cc index 53c65fe..3c8a0aa2 100644 --- a/chrome/browser/safe_browsing/safe_browsing_test.cc +++ b/chrome/browser/safe_browsing/safe_browsing_test.cc @@ -20,9 +20,9 @@ #include "base/environment.h" #include "base/path_service.h" #include "base/process_util.h" +#include "base/stringprintf.h" #include "base/string_number_conversions.h" #include "base/string_split.h" -#include "base/string_util.h" #include "base/synchronization/lock.h" #include "base/test/test_timeouts.h" #include "base/threading/platform_thread.h" @@ -141,7 +141,7 @@ class SafeBrowsingTestServer { CommandLine cmd_line(python_runtime); FilePath datafile = testserver_path.Append(datafile_); cmd_line.AppendArgPath(testserver); - cmd_line.AppendSwitchASCII("port", StringPrintf("%d", kPort_)); + cmd_line.AppendSwitchASCII("port", base::StringPrintf("%d", kPort_)); cmd_line.AppendSwitchPath("datafile", datafile); if (!base::LaunchApp(cmd_line, false, true, &server_handle_)) { @@ -303,15 +303,15 @@ class SafeBrowsingServiceTest : public InProcessBrowserTest { // server. Although in real production, they are served from different // servers. std::string url_prefix = - StringPrintf("http://%s:%d/safebrowsing", - SafeBrowsingTestServer::Host(), - SafeBrowsingTestServer::Port()); + base::StringPrintf("http://%s:%d/safebrowsing", + SafeBrowsingTestServer::Host(), + SafeBrowsingTestServer::Port()); command_line->AppendSwitchASCII(switches::kSbInfoURLPrefix, url_prefix); command_line->AppendSwitchASCII(switches::kSbMacKeyURLPrefix, url_prefix); } void SetTestStep(int step) { - std::string test_step = StringPrintf("test_step=%d", step); + std::string test_step = base::StringPrintf("test_step=%d", step); safe_browsing_service_->protocol_manager_->set_additional_query(test_step); } @@ -448,8 +448,8 @@ class SafeBrowsingServiceTestHelper void WaitTillServerReady(const char* host, int port) { response_status_ = net::URLRequestStatus::FAILED; - GURL url(StringPrintf("http://%s:%d%s?test_step=0", - host, port, kDBResetPath)); + GURL url(base::StringPrintf("http://%s:%d%s?test_step=0", + host, port, kDBResetPath)); // TODO(lzheng): We should have a way to reliably tell when a server is // ready so we could get rid of the Sleep and retry loop. while (true) { @@ -466,19 +466,21 @@ class SafeBrowsingServiceTestHelper net::URLRequestStatus::Status FetchDBToVerify(const char* host, int port, int test_step) { // TODO(lzheng): Remove chunk_type=add once it is not needed by the server. - GURL url(StringPrintf("http://%s:%d%s?" - "client=chromium&appver=1.0&pver=2.2&test_step=%d&" - "chunk_type=add", - host, port, kDBVerifyPath, test_step)); + GURL url(base::StringPrintf( + "http://%s:%d%s?" + "client=chromium&appver=1.0&pver=2.2&test_step=%d&" + "chunk_type=add", + host, port, kDBVerifyPath, test_step)); return FetchUrl(url); } // Calls test server to fetch URLs for verification. net::URLRequestStatus::Status FetchUrlsToVerify(const char* host, int port, int test_step) { - GURL url(StringPrintf("http://%s:%d%s?" - "client=chromium&appver=1.0&pver=2.2&test_step=%d", - host, port, kUrlVerifyPath, test_step)); + GURL url(base::StringPrintf( + "http://%s:%d%s?" + "client=chromium&appver=1.0&pver=2.2&test_step=%d", + host, port, kUrlVerifyPath, test_step)); return FetchUrl(url); } @@ -565,7 +567,7 @@ IN_PROC_BROWSER_TEST_F(SafeBrowsingServiceTest, SafeBrowsingSystemTest) { // This repeats till there is no update data. for (int step = 1;; step++) { // Every step should be a fresh start. - SCOPED_TRACE(StringPrintf("step=%d", step)); + SCOPED_TRACE(base::StringPrintf("step=%d", step)); EXPECT_TRUE(is_database_ready()); EXPECT_FALSE(is_update_scheduled()); |