Chromium Code Reviews| Index: chrome/browser/importer/importer.cc |
| diff --git a/chrome/browser/importer/importer.cc b/chrome/browser/importer/importer.cc |
| index f3115402d2fffc02b765d2c880530cb5e4215bbe..bff54c64470b1d4d827b3fe3fa8ec2c7522e8b35 100644 |
| --- a/chrome/browser/importer/importer.cc |
| +++ b/chrome/browser/importer/importer.cc |
| @@ -650,6 +650,10 @@ Importer* ImporterHost::CreateImporterByType(ProfileType type) { |
| return new Firefox3Importer(); |
| case GOOGLE_TOOLBAR5: |
| return new Toolbar5Importer(); |
| + case SAFARI: |
|
stuartmorgan
2009/07/29 20:22:37
Should this have platform ifdef-ing, since MS_IE h
|
| + // TODO(jeremy): Implement. |
| + NOTIMPLEMENTED(); |
| + return NULL; |
| } |
| NOTREACHED(); |
| return NULL; |
| @@ -694,6 +698,9 @@ void ImporterHost::DetectSourceProfiles() { |
| // TODO(brg) : Current UI requires win_util. |
| DetectGoogleToolbarProfiles(); |
| #else |
| +#if defined(OS_MACOSX) |
| + DetectSafariProfiles(); |
| +#endif |
| DetectFirefoxProfiles(); |
| #endif |
| } |
| @@ -807,3 +814,17 @@ void ImporterHost::DetectGoogleToolbarProfiles() { |
| source_profiles_.push_back(google_toolbar); |
| } |
| } |
| + |
| +#if defined(OS_MACOSX) |
| +void ImporterHost::DetectSafariProfiles() { |
| + // TODO(jeremy):Check that Safari folder is in fact present. |
| + ProfileInfo* safari = new ProfileInfo(); |
| + safari->browser_type = SAFARI; |
| + safari->description = l10n_util::GetString(IDS_IMPORT_FROM_SAFARI); |
| + safari->source_path.clear(); |
| + safari->app_path.clear(); |
| + safari->services_supported = HISTORY | FAVORITES | COOKIES | PASSWORDS | |
| + SEARCH_ENGINES; |
| + source_profiles_.push_back(safari); |
| +} |
| +#endif // OS_MACOSX |