Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(163)

Unified Diff: trunk/src/chrome/browser/ui/webui/options/import_data_handler.cc

Issue 15968002: Revert 201837 "OOP import on Windows." (Closed) Base URL: svn://svn.chromium.org/chrome/
Patch Set: Created 7 years, 7 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: trunk/src/chrome/browser/ui/webui/options/import_data_handler.cc
===================================================================
--- trunk/src/chrome/browser/ui/webui/options/import_data_handler.cc (revision 201967)
+++ trunk/src/chrome/browser/ui/webui/options/import_data_handler.cc (working copy)
@@ -109,16 +109,12 @@
state);
import_did_succeed_ = false;
- // 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;
+ // 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;
#else
importer_host_ = new ImporterHost;
#endif
Property changes on: trunk/src/chrome/browser/ui/webui/options/import_data_handler.cc
___________________________________________________________________
Deleted: svn:mergeinfo

Powered by Google App Engine
This is Rietveld 408576698