Fix Issue 18530....

chrome/browser/importer/firefox2_importer.cc

 // Copyright (c) 2006-2008 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/browser/importer/firefox2_importer.h"
 
 #include "app/l10n_util.h"
 #include "base/file_path.h"
 #include "base/file_util.h"
 #include "base/message_loop.h"
@@ skipping 57 matching lines @@
     NotifyItemStarted(HISTORY);
     ImportHistory();
     NotifyItemEnded(HISTORY);
   }
   NotifyEnded();
 }
 
 // static
 void Firefox2Importer::LoadDefaultBookmarks(const std::wstring& app_path,
                                             std::set<GURL> *urls) {
-  // TODO(port): Code below is correct only on Windows.
-  // Firefox keeps its default bookmarks in a bookmarks.html file that
-  // lives at: <Firefox install dir>\defaults\profile\bookmarks.html
   FilePath file = FilePath::FromWStringHack(app_path);
-  file.AppendASCII("defaults");
-  file.AppendASCII("profile");
-  file.AppendASCII("bookmarks.html");
+  file = file.AppendASCII("defaults");
+  file = file.AppendASCII("profile");
+  file = file.AppendASCII("bookmarks.html");
 
   urls->clear();
 
   // Read the whole file.
   std::string content;
   file_util::ReadFileToString(file.ToWStringHack(), &content);
   std::vector<std::string> lines;
   SplitString(content, '\n', &lines);
 
   std::string charset;
@@ skipping 475 matching lines @@
 
   // We need to make up a URL for the favicon. We use a version of the page's
   // URL so that we can be sure it will not collide.
   usage.favicon_url = GURL(std::string("made-up-favicon:") + link_url.spec());
 
   // We only have one URL per favicon for Firefox 2 bookmarks.
   usage.urls.insert(link_url);
 
   favicons->push_back(usage);
 }