Bug 14202: Support relative paths when installing external extensions through prefs.

First we try the path as absolute path. If the file is not found, we try a relative path: [installation path]\extensions

This will make installing extensions easier from pre-installed prefs, since we can drop them into an extensions directory under the program directory.

BUG=14202
TEST=Specify relative path when installing extensions through prefs. Best to test this with an installer that supports deploying extensions (blocked on 14201)

Review URL: http://codereview.chromium.org/125219

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@18554 0039d316-1c4b-4281-b951-d872f2087c98

1 files changed, 25 insertions, 1 deletions

diff --git a/chrome/browser/extensions/external_pref_extension_provider.cc b/chrome/browser/extensions/external_pref_extension_provider.cc
index 5ed1a5e..5d16ba0 100644
--- a/chrome/browser/extensions/external_pref_extension_provider.cc
+++ b/chrome/browser/extensions/external_pref_extension_provider.cc
@@ -4,7 +4,10 @@
 
 #include "chrome/browser/extensions/external_pref_extension_provider.h"
 
+#include "app/app_paths.h"
 #include "base/file_path.h"
+#include "base/file_util.h"
+#include "base/path_service.h"
 #include "base/string_util.h"
 #include "base/version.h"
 
@@ -56,10 +59,31 @@ void ExternalPrefExtensionProvider::VisitRegisteredExtension(
       continue;
     }
 
+    if (external_crx.find(FilePath::kParentDirectory) != StringPiece::npos) {
+      LOG(WARNING) << "Path traversal not allowed in path: "
+                   << external_crx.c_str();
+      continue;
+    }
+
+    // See if it's an absolute path...
+    FilePath path(external_crx);
+    if (!path.IsAbsolute()) {
+      // Try path as relative path from external extension dir.
+      FilePath base_path;
+      PathService::Get(app::DIR_EXTERNAL_EXTENSIONS, &base_path);
+      path = base_path.Append(external_crx);
+    }
+
+    if (!file_util::PathExists(path)) {
+      LOG(WARNING) << "Cannot find extension: " << external_crx.c_str();
+      continue;  // Path is neither absolute nor relative. Might be
+                 // meta-physical, but we don't support those (yet).
+    }
+
     scoped_ptr<Version> version;
     version.reset(Version::GetVersionFromString(external_version));
     visitor->OnExternalExtensionFound(
-        WideToASCII(extension_id), version.get(), FilePath(external_crx));
+        WideToASCII(extension_id), version.get(), path);
   }
 }