| Index: chrome/browser/ui/webui/options/import_data_handler.cc
|
| diff --git a/chrome/browser/ui/webui/options/import_data_handler.cc b/chrome/browser/ui/webui/options/import_data_handler.cc
|
| index 5b2d449ce5ab7de409e5d62b209fc1f5ab357409..eefe4fe2a858fa843e7cd388cc86e08a6057f97d 100644
|
| --- a/chrome/browser/ui/webui/options/import_data_handler.cc
|
| +++ b/chrome/browser/ui/webui/options/import_data_handler.cc
|
| @@ -109,12 +109,16 @@ void ImportDataHandler::ImportData(const ListValue* args) {
|
| state);
|
| import_did_succeed_ = false;
|
|
|
| - // TODO(csilv): Out-of-process import has only been qualified on MacOS X,
|
| - // so we will only use it on that platform since it is required. Remove this
|
| - // conditional logic once oop import is qualified for Linux/Windows.
|
| - // http://crbug.com/22142
|
| -#if defined(OS_MACOSX)
|
| - importer_host_ = new ExternalProcessImporterHost;
|
| + // TODO(gab): Make Linux use OOP import as well (http://crbug.com/56816) and
|
| + // get rid of these ugly ifdefs.
|
| +#if defined(OS_MACOSX) || defined(OS_WIN)
|
| + // The Google Toolbar importer doesn't work for the out-of-process import.
|
| + // This is the only entry point for this importer (it is never used on first
|
| + // run). See discussion on http://crbug.com/219419 for details.
|
| + if (source_profile.importer_type == importer::TYPE_GOOGLE_TOOLBAR5)
|
| + importer_host_ = new ImporterHost;
|
| + else
|
| + importer_host_ = new ExternalProcessImporterHost;
|
| #else
|
| importer_host_ = new ImporterHost;
|
| #endif
|
|
|