 Chromium Code Reviews
 Chromium Code Reviews Issue 160341:
  First cut at Firefox import on OSX + stubs for Safari import.  (Closed)
    
  
    Issue 160341:
  First cut at Firefox import on OSX + stubs for Safari import.  (Closed) 
  | 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 |