Extract BookmarksFileImporter from Firefox2Importer.

chrome/browser/importer/bookmarks_file_importer.h

+// 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.
+
+#ifndef CHROME_BROWSER_IMPORTER_BOOKMARKS_FILE_IMPORTER_H_
+#define CHROME_BROWSER_IMPORTER_BOOKMARKS_FILE_IMPORTER_H_
+
+#include "chrome/browser/importer/importer.h"
+
+class ImporterBridge;
+
+namespace importer {
+struct SourceProfile;
+}
+
+// Importer for bookmarks files.
+class BookmarksFileImporter : public Importer {
+ public:
+  BookmarksFileImporter();
+
+  virtual void StartImport(const importer::SourceProfile& source_profile,
+                           uint16 items,
+                           ImporterBridge* bridge) OVERRIDE;
+
+ private:
+  virtual ~BookmarksFileImporter();

[gab, 2013/05/10 20:37:57]

Why is this private? I don't even think you need to declare a destructor here.

[Avi (use Gerrit), 2013/05/10 21:34:53]

This is an Importer which is ref counted, therefore it needs a private
destructor.

[gab, 2013/05/10 21:47:44]

Ah I see, thanks for clarifying.

+
+  DISALLOW_COPY_AND_ASSIGN(BookmarksFileImporter);
+};
+
+#endif  // CHROME_BROWSER_IMPORTER_BOOKMARKS_FILE_IMPORTER_H_