Move most importer code to chrome/utility/importer

chrome/common/importer/safari_importer_utils.mm

Index: chrome/common/importer/safari_importer_utils.mm
diff --git a/chrome/common/importer/safari_importer_utils.mm b/chrome/common/importer/safari_importer_utils.mm
new file mode 100644
index 0000000000000000000000000000000000000000..1e7a7c817c5827ff2cb60dc69ec89af00c58fbba
--- /dev/null
+++ b/chrome/common/importer/safari_importer_utils.mm
@@ -0,0 +1,29 @@
+// Copyright 2013 The Chromium Authors. All rights reserved.
+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+#include "chrome/common/importer/safari_importer_utils.h"
+
+#include "base/logging.h"
+#include "base/file_util.h"
+#include "chrome/common/importer/importer_data_types.h"
+
+bool SafariImporterCanImport(const base::FilePath& library_dir,

[gab, 2013/07/08 15:18:08]

I would also prefer if this was done in a separate follow-up CL, to keep this CL
to a pure move-only CL.

[scottmg, 2013/07/09 16:20:49]

hrm, ok. moved back into safari_importer.

[gab, 2013/07/09 20:48:31]

err, my bad, I meant as a precursor CL rather than follow-up CL, either works I
guess.

+                             uint16* services_supported) {
+  DCHECK(services_supported);
+  *services_supported = importer::NONE;
+
+  // Import features are toggled by the following:
+  // bookmarks import: existence of ~/Library/Safari/Bookmarks.plist file.
+  // history import: existence of ~/Library/Safari/History.plist file.
+  base::FilePath safari_dir = library_dir.Append("Safari");
+  base::FilePath bookmarks_path = safari_dir.Append("Bookmarks.plist");
+  base::FilePath history_path = safari_dir.Append("History.plist");
+
+  if (file_util::PathExists(bookmarks_path))
+    *services_supported |= importer::FAVORITES;
+  if (file_util::PathExists(history_path))
+    *services_supported |= importer::HISTORY;
+
+  return *services_supported != importer::NONE;
+}