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

Unified Diff: chrome/browser/extensions/unpacked_installer.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/extensions/unpacked_installer.cc
diff --git a/chrome/browser/extensions/unpacked_installer.cc b/chrome/browser/extensions/unpacked_installer.cc
index aa6b0dc1e1a0211b564418d63c67ad077b9583b4..cc9d9d8e33956d191042d9a93b04dbdb939859b3 100644
--- a/chrome/browser/extensions/unpacked_installer.cc
+++ b/chrome/browser/extensions/unpacked_installer.cc
@@ -129,8 +129,7 @@ void UnpackedInstaller::LoadFromCommandLine(const base::FilePath& path_in,
// between extension loading and loading an URL from the command line.
base::ThreadRestrictions::ScopedAllowIO allow_io;
- extension_path_ = path_in;
- file_util::AbsolutePath(&extension_path_);
+ extension_path_ = base::MakeAbsoluteFilePath(path_in);
if (!IsLoadingUnpackedAllowed()) {
ReportExtensionLoadError(kUnpackedExtensionsBlacklistedError);
@@ -200,7 +199,7 @@ bool UnpackedInstaller::IsLoadingUnpackedAllowed() const {
void UnpackedInstaller::GetAbsolutePath() {
CHECK(BrowserThread::CurrentlyOn(BrowserThread::FILE));
- file_util::AbsolutePath(&extension_path_);
+ extension_path_ = base::MakeAbsoluteFilePath(extension_path_);
BrowserThread::PostTask(BrowserThread::UI, FROM_HERE,
base::Bind(&UnpackedInstaller::CheckExtensionFileAccess, this));

Powered by Google App Engine
This is Rietveld 408576698