diff options
| author | viettrungluu@chromium.org <viettrungluu@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2009-10-23 22:08:26 +0000 |
|---|---|---|
| committer | viettrungluu@chromium.org <viettrungluu@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2009-10-23 22:08:26 +0000 |
| commit | bca45b6fc47bc0588e96cd64e7a41dc6a1d98d65 (patch) | |
| tree | 0a03533c6de1b668ceda6a46ee479f2f20d50a58 | |
| parent | 105d168e1ed331cb2a84ab11e9d058f10085b636 (diff) | |
| download | chromium_src-bca45b6fc47bc0588e96cd64e7a41dc6a1d98d65.zip chromium_src-bca45b6fc47bc0588e96cd64e7a41dc6a1d98d65.tar.gz chromium_src-bca45b6fc47bc0588e96cd64e7a41dc6a1d98d65.tar.bz2 | |
Mac: don't import Safari's home page setting.
Apparently it often leads to a useless home page. I kept the ImportHomepage() method in case we ever change our minds.
BUG=25603
TEST=Set the home page in Safari to something interesting; run Chromium with --first-run to bring up import dialog; select Safari and continue; make sure home page isn't imported.
Review URL: http://codereview.chromium.org/328010
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@29954 0039d316-1c4b-4281-b951-d872f2087c98
| -rw-r--r-- | chrome/browser/importer/safari_importer.mm | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/chrome/browser/importer/safari_importer.mm b/chrome/browser/importer/safari_importer.mm index 54a6d2a..0dace3b2 100644 --- a/chrome/browser/importer/safari_importer.mm +++ b/chrome/browser/importer/safari_importer.mm @@ -83,8 +83,8 @@ void SafariImporter::StartImport(ProfileInfo profile_info, // The order here is important! bridge_->NotifyStarted(); - if ((services_supported & HOME_PAGE) && !cancelled()) - ImportHomepage(); // Doesn't have a UI item. + // 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). if ((services_supported & HISTORY) && !cancelled()) { bridge_->NotifyItemStarted(HISTORY); ImportHistory(); |