Move most importer code to chrome/utility/importer

chrome/browser/importer/importer_uma.cc

Index: chrome/browser/importer/importer_uma.cc
diff --git a/chrome/browser/importer/importer_creator.cc b/chrome/browser/importer/importer_uma.cc
similarity index 66%
rename from chrome/browser/importer/importer_creator.cc
rename to chrome/browser/importer/importer_uma.cc
index 7127ed2c59055054397f5a5864898ab792dc5f9d..714c472b0a4e786315e430595ec219a400c40d78 100644
--- a/chrome/browser/importer/importer_creator.cc
+++ b/chrome/browser/importer/importer_uma.cc
@@ -2,22 +2,8 @@
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
-#include "chrome/browser/importer/importer_creator.h"
-
-#include "base/logging.h"
-#include "base/metrics/histogram.h"
-#include "chrome/browser/importer/bookmarks_file_importer.h"
-#include "chrome/browser/importer/firefox3_importer.h"
-
-#if defined(OS_WIN)
-#include "chrome/browser/importer/ie_importer.h"
-#endif
-
-#if defined(OS_MACOSX)
-#include <CoreFoundation/CoreFoundation.h>
-#include "base/mac/foundation_util.h"
-#include "chrome/browser/importer/safari_importer.h"
-#endif
+#include "build/build_config.h"
+#include "chrome/browser/importer/importer_uma.h"
 
 namespace importer {
 
@@ -42,31 +28,8 @@ enum ImporterTypeMetrics {
   IMPORTER_METRICS_SIZE
 };
 
-
 }  // namespace
 
-Importer* CreateImporterByType(ImporterType type) {
-  switch (type) {
-#if defined(OS_WIN)
-    case TYPE_IE:
-      return new IEImporter();
-#endif
-    case TYPE_BOOKMARKS_FILE:
-      return new BookmarksFileImporter();
-    case TYPE_FIREFOX3:
-      return new Firefox3Importer();
-#if defined(OS_MACOSX)
-    case TYPE_SAFARI:
-      return new SafariImporter(base::mac::GetUserLibraryPath());
-#endif
-    default:
-      NOTREACHED();
-      return NULL;
-  }
-  NOTREACHED();
-  return NULL;
-}
-
 void LogImporterUseToMetrics(const std::string& metric_postfix,
                              ImporterType type) {
   ImporterTypeMetrics metrics_type = IMPORTER_METRICS_UNKNOWN;
@@ -104,4 +67,5 @@ void LogImporterUseToMetrics(const std::string& metric_postfix,
   histogram->Add(metrics_type);
 }
 
+

[gab, 2013/07/10 13:40:44]

nit: Looks like you have more than a single empty line at the end of this file.

[scottmg, 2013/07/15 16:06:01]

Done.

 }  // namespace importer