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

Unified Diff: chrome/browser/extensions/extension_service_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: 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/extensions/extension_service_unittest.cc
diff --git a/chrome/browser/extensions/extension_service_unittest.cc b/chrome/browser/extensions/extension_service_unittest.cc
index b1253ebc3b3b74a79ad79ccf9283a23627d150a4..c2a229eb7bf2a01b61e3dce6ccc30eab19f6df6a 100644
--- a/chrome/browser/extensions/extension_service_unittest.cc
+++ b/chrome/browser/extensions/extension_service_unittest.cc
@@ -1168,14 +1168,13 @@ TEST_F(ExtensionServiceTest, LoadAllExtensionsFromDirectorySuccess) {
ExtensionResource resource00(extension->id(),
scripts[0].js_scripts()[0].extension_root(),
scripts[0].js_scripts()[0].relative_path());
- base::FilePath expected_path(extension->path().AppendASCII("script1.js"));
- ASSERT_TRUE(file_util::AbsolutePath(&expected_path));
+ base::FilePath expected_path(
+ extension->path().AppendASCII("script1.js")).AsAbsolute();
EXPECT_TRUE(resource00.ComparePathWithDefault(expected_path));
ExtensionResource resource01(extension->id(),
scripts[0].js_scripts()[1].extension_root(),
scripts[0].js_scripts()[1].relative_path());
- expected_path = extension->path().AppendASCII("script2.js");
- ASSERT_TRUE(file_util::AbsolutePath(&expected_path));
+ expected_path = extension->path().AppendASCII("script2.js").AsAbsolute();
EXPECT_TRUE(resource01.ComparePathWithDefault(expected_path));
EXPECT_TRUE(!extensions::PluginInfo::HasPlugins(extension));
EXPECT_EQ(1u, scripts[1].url_patterns().patterns().size());
@@ -1186,7 +1185,7 @@ TEST_F(ExtensionServiceTest, LoadAllExtensionsFromDirectorySuccess) {
scripts[1].js_scripts()[0].relative_path());
expected_path =
extension->path().AppendASCII("js_files").AppendASCII("script3.js");
- ASSERT_TRUE(file_util::AbsolutePath(&expected_path));
+ expected_path = expected_path.AsAbsolute();
EXPECT_TRUE(resource10.ComparePathWithDefault(expected_path));
expected_patterns.ClearPatterns();

Powered by Google App Engine
This is Rietveld 408576698