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

Unified Diff: chrome/browser/net/url_fixer_upper_unittest.cc

Issue 13196006: Move path functions from file_util to FilePath object. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Move AsAbsolute back out of FilePath Created 7 years, 9 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/net/url_fixer_upper_unittest.cc
diff --git a/chrome/browser/net/url_fixer_upper_unittest.cc b/chrome/browser/net/url_fixer_upper_unittest.cc
index cdfd77a55fa9c22f1d6d3f346030f463a00bd6ff..9f03d7a97a1eab1c7efcb3f8d35cb66d1857a029 100644
--- a/chrome/browser/net/url_fixer_upper_unittest.cc
+++ b/chrome/browser/net/url_fixer_upper_unittest.cc
@@ -468,7 +468,8 @@ TEST(URLFixerUpperTest, FixupRelativeFile) {
FILE_PATH_LITERAL("url_fixer_upper_existing_file.txt"));
ASSERT_TRUE(PathService::Get(chrome::DIR_APP, &dir));
ASSERT_TRUE(MakeTempFile(dir, file_part, &full_path));
- ASSERT_TRUE(file_util::AbsolutePath(&full_path));
+ full_path = base::MakeAbsoluteFilePath(full_path);
+ ASSERT_FALSE(full_path.empty());
// make sure we pass through good URLs
for (size_t i = 0; i < arraysize(fixup_cases); ++i) {
@@ -506,7 +507,8 @@ TEST(URLFixerUpperTest, FixupRelativeFile) {
base::FilePath new_dir = dir.Append(sub_dir);
file_util::CreateDirectory(new_dir);
ASSERT_TRUE(MakeTempFile(new_dir, sub_file, &full_path));
- ASSERT_TRUE(file_util::AbsolutePath(&full_path));
+ full_path = base::MakeAbsoluteFilePath(full_path);
+ ASSERT_FALSE(full_path.empty());
// test file in the subdir
base::FilePath relative_file = sub_dir.Append(sub_file);

Powered by Google App Engine
This is Rietveld 408576698