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

Unified Diff: chrome/browser/download/download_browsertest.cc

Issue 13145003: Rewrite std::string("") to std::string(), Linux edition. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Ugh Created 7 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/download/download_browsertest.cc
diff --git a/chrome/browser/download/download_browsertest.cc b/chrome/browser/download/download_browsertest.cc
index 7c63cb7c6bf0e61dfced6e4a28fce9b0fcf14ef6..5ec91a2797b812caa3926ab5eb9123464368538d 100644
--- a/chrome/browser/download/download_browsertest.cc
+++ b/chrome/browser/download/download_browsertest.cc
@@ -1655,7 +1655,12 @@ IN_PROC_BROWSER_TEST_F(DownloadTest, NewWindow) {
IN_PROC_BROWSER_TEST_F(DownloadTest, DownloadHistoryCheck) {
GURL download_url(URLRequestSlowDownloadJob::kKnownSizeUrl);
- base::FilePath file(net::GenerateFileName(download_url, "", "", "", "", ""));
+ base::FilePath file(net::GenerateFileName(download_url,
+ std::string(),
+ std::string(),
+ std::string(),
+ std::string(),
+ std::string()));
// We use the server so that we can get a redirect and test url_chain
// persistence.
@@ -2627,7 +2632,7 @@ IN_PROC_BROWSER_TEST_F(DownloadTest, LoadURLExternallyReferrerPolicy) {
// Check that the file contains the expected referrer.
base::FilePath file(download_items[0]->GetFullPath());
- std::string expected_contents = test_server()->GetURL("").spec();
+ std::string expected_contents = test_server()->GetURL(std::string()).spec();
ASSERT_TRUE(VerifyFile(file, expected_contents, expected_contents.length()));
}
« no previous file with comments | « chrome/browser/devtools/devtools_sanity_browsertest.cc ('k') | chrome/browser/download/download_file_picker.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698