Mac: remove some now-dead code and add a comment.

chrome/browser/importer/safari_importer.mm

 // 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 <Cocoa/Cocoa.h>
 
 #include "chrome/browser/importer/safari_importer.h"
 
 #include <map>
 #include <vector>
@@ skipping 35 matching lines @@
 SafariImporter::~SafariImporter() {
 }
 
 // static
 bool SafariImporter::CanImport(const FilePath& library_dir,
                                uint16 *services_supported) {
   DCHECK(services_supported);
   *services_supported = NONE;
 
   // Import features are toggled by the following:
-  // bookmarks import: existance of ~/Library/Safari/Bookmarks.plist file.
-  // history import: existance of ~/Library/Safari/History.plist file.
-  // homepage import: existance of appropriate key in defaults.
+  // bookmarks import: existence of ~/Library/Safari/Bookmarks.plist file.
+  // history import: existence of ~/Library/Safari/History.plist file.
   FilePath safari_dir = library_dir.Append("Safari");
   FilePath bookmarks_path = safari_dir.Append("Bookmarks.plist");
   FilePath history_path = safari_dir.Append("History.plist");
 
   using file_util::PathExists;
   if (PathExists(bookmarks_path))
     *services_supported |= FAVORITES;
   if (PathExists(history_path))
     *services_supported |= HISTORY;
 
-  const scoped_nsobject<NSString> homepage_ns(
-      reinterpret_cast<const NSString*>(
-          CFPreferencesCopyAppValue(CFSTR("HomePage"),
-                                    CFSTR("com.apple.Safari"))));
-  if (homepage_ns.get())
-    *services_supported |= HOME_PAGE;
-
   return *services_supported != NONE;
 }
 
 void SafariImporter::StartImport(ProfileInfo profile_info,
                                  uint16 services_supported,
                                  ImporterBridge* bridge) {
   bridge_ = bridge;
 
   // The order here is important!
   bridge_->NotifyStarted();
   // In keeping with import on other platforms (and for other browsers), we
-  // don't import the home page (since it may lead to a useless homepage).
+  // don't import the home page (since it may lead to a useless homepage); see
+  // crbug.com/25603.
   if ((services_supported & HISTORY) && !cancelled()) {
     bridge_->NotifyItemStarted(HISTORY);
     ImportHistory();
     bridge_->NotifyItemEnded(HISTORY);
   }
   if ((services_supported & FAVORITES) && !cancelled()) {
     bridge_->NotifyItemStarted(FAVORITES);
     ImportBookmarks();
     bridge_->NotifyItemEnded(FAVORITES);
   }
@@ skipping 288 matching lines @@
     if (!last_visit_str)
       continue;
 
     // Convert Safari's last visit time to Unix Epoch time.
     double seconds_since_unix_epoch = HistoryTimeToEpochTime(last_visit_str);
     row.set_last_visit(base::Time::FromDoubleT(seconds_since_unix_epoch));
 
     history_items->push_back(row);
   }
 }
-
-void SafariImporter::ImportHomepage() {
-  const scoped_nsobject<NSString> homepage_ns(
-      reinterpret_cast<const NSString*>(
-          CFPreferencesCopyAppValue(CFSTR("HomePage"),
-                                    CFSTR("com.apple.Safari"))));
-  if (!homepage_ns.get())
-    return;
-
-  string16 hompeage_str = base::SysNSStringToUTF16(homepage_ns.get());
-  GURL homepage(hompeage_str);
-  if (homepage.is_valid()) {
-    bridge_->AddHomePage(homepage);
-  }
-}