Fixing Firefox 21+ password import

chrome/common/importer/firefox_importer_utils.cc

Index: chrome/common/importer/firefox_importer_utils.cc
diff --git a/chrome/common/importer/firefox_importer_utils.cc b/chrome/common/importer/firefox_importer_utils.cc
index 4f6a2019e781551b516cdde924b5aca31f755e1e..cee249aa2d72117babd7ecdd3d1b604dfb66d8c8 100644
--- a/chrome/common/importer/firefox_importer_utils.cc
+++ b/chrome/common/importer/firefox_importer_utils.cc
@@ -115,7 +115,7 @@ bool GetFirefoxVersionAndPathFromProfile(const base::FilePath& profile_path,
       if (key == "LastVersion") {
         base::StringToInt(line.substr(equal + 1), version);
         ret = true;
-      } else if (key == "LastAppDir") {
+      } else if (key == "LastPlatformDir") {

[jeremy, 2013/12/17 11:48:09]

Do you know what the actual semantic meaning of this key is?

My concern is just that we're using this value because it happens to contain the
directory we need and not because of it's real meaning?

My suggestion would be for the caller to look at the value of LastAppDir, search
for Firefox.app manually and build up the path from there rather than switching
to a key which happens to contain the directory we're looking for.

[mpawlowski, 2013/12/17 12:31:25]

What if someone renames Firefox.app? From what I can tell by looking at
Firefox's code, LastPlatformDir is the folder in which the XUL runtime is stored
and LastAppDir is the folder with omni.ja - the package with resources, UI
definitions etc., the actual "app". The question which one should be used to
find what we need for importing is open but IMHO we don't have any control over
this and Mozilla seems to move stuff about. We should use whatever works and
worry again when it breaks, we won't be able to foresee what kind of
rearrangement they'll come up with next, no assumption is guaranteed to be
correct. I could change this to any arbitrary method of arriving at
Firefox.app/Content/MacOS but it won't guarantee they don't move those libs to
Firefox.app/Content/MacOS/Libraries next week.

         // TODO(evanm): If the path in question isn't convertible to
         // UTF-8, what does Firefox do?  If it puts raw bytes in the
         // file, we could go straight from bytes -> filepath;