Remove "<HR>" tags to import Firefox bookmarks correctly.

chrome/utility/importer/bookmark_html_reader_unittest.cc

Index: chrome/utility/importer/bookmark_html_reader_unittest.cc
diff --git a/chrome/utility/importer/bookmark_html_reader_unittest.cc b/chrome/utility/importer/bookmark_html_reader_unittest.cc
index 72f42cf52bae9db26b5c65e6e7f9bc0119c8cb8f..ecdc8e9522d83642de11afdbde4ae10af1f69b72 100644
--- a/chrome/utility/importer/bookmark_html_reader_unittest.cc
+++ b/chrome/utility/importer/bookmark_html_reader_unittest.cc
@@ -196,6 +196,20 @@ TEST(BookmarkHTMLReaderTest, Firefox2BookmarkFileImport) {
   ExpectThirdFirefox2Bookmark(bookmarks[2]);
 }
 
+TEST(BookmarkHTMLReaderTest, BookmarkFileWithHrTagImport) {
+  base::FilePath path;
+  ASSERT_TRUE(PathService::Get(chrome::DIR_TEST_DATA, &path));
+  path = path.AppendASCII("bookmark_html_reader");
+  path = path.AppendASCII("firefox3.html");
+
+  std::vector<ImportedBookmarkEntry> bookmarks;
+  ImportBookmarksFile(base::Callback<bool(void)>(),
+                      base::Callback<bool(const GURL&)>(),
+                      path, &bookmarks, NULL);
+
+  ASSERT_EQ(7U, bookmarks.size());

[gab, 2013/08/09 13:27:30]

Can you also verify the bookmarks' contents like the other tests do.

[zhchbin, 2013/08/09 14:20:00]

I mean to keep this test case simple and only testing the size seems already
enough for this case. However, if we need it I will add.

[gab, 2013/08/09 14:29:03]

Well, from knowing the implementation, we know that this only makes sure it
works with the <HR> tags, but this test doesn't need to know that and should
check that the full bookmarks were imported as expected.

Feel free to export a smaller test file than 7 bookmarks to make for the
smallest test possible that has full coverage.

+}
+
 TEST(BookmarkHTMLReaderTest, EpiphanyBookmarkFileImport) {
   base::FilePath path;
   ASSERT_TRUE(PathService::Get(chrome::DIR_TEST_DATA, &path));