Revert 30149 - The existing file_util::AbsolutePath() function was already do...

chrome/common/extensions/extension_resource.cc

 // Copyright (c) 2009 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "chrome/common/extensions/extension_resource.h"
 
 #include "base/file_path.h"
 #include "base/file_util.h"
 #include "base/string_util.h"
 #include "chrome/browser/extensions/extension_l10n_util.h"
@@ skipping 22 matching lines @@
   full_resource_path_ = GetFilePath(extension_root_, relative_path_);
   return full_resource_path_;
 }
 
 // Static version...
 FilePath ExtensionResource::GetFilePath(const FilePath& extension_root,
                                         const FilePath& relative_path) {
   std::vector<FilePath> l10n_relative_paths;
   extension_l10n_util::GetL10nRelativePaths(relative_path,
                                             &l10n_relative_paths);
-  // We need to resolve the parent references in the extension_root
-  // path on its own because IsParent doesn't like parent references.
-  FilePath clean_extension_root(extension_root);
-  if (!file_util::AbsolutePath(&clean_extension_root))
-    return FilePath();
 
   // Stat l10n file(s), and return new path if it exists.
   for (size_t i = 0; i < l10n_relative_paths.size(); ++i) {
-    FilePath full_path = clean_extension_root.Append(l10n_relative_paths[i]);
-    if (file_util::AbsolutePath(&full_path) &&
-        clean_extension_root.IsParent(full_path) &&
+    FilePath full_path;
+    if (extension_root.AppendAndResolveRelative(l10n_relative_paths[i],
+                                                &full_path) &&
+        extension_root.IsParent(full_path) &&
         file_util::PathExists(full_path)) {
       return full_path;
     }
   }
 
   // Fall back to root resource.
-  FilePath full_path = clean_extension_root.Append(relative_path);
-  if (file_util::AbsolutePath(&full_path) &&
-      clean_extension_root.IsParent(full_path)) {
+  FilePath full_path;
+  if (extension_root.AppendAndResolveRelative(relative_path, &full_path) &&
+      extension_root.IsParent(full_path)) {
     return full_path;
   }
 
   return FilePath();
 }
 
 // Unittesting helpers.
 FilePath::StringType ExtensionResource::NormalizeSeperators(
     FilePath::StringType path) const {
 #if defined(FILE_PATH_USES_WIN_SEPARATORS)
@@ skipping 10 matching lines @@
   // Make sure we have a cached value to test against...
   if (full_resource_path_.empty())
     GetFilePath();
   if (NormalizeSeperators(path.value()) ==
     NormalizeSeperators(full_resource_path_.value())) {
     return true;
   } else {
     return false;
   }
 }