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

chrome/utility/importer/bookmark_html_reader.cc

 // 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/utility/importer/bookmark_html_reader.h"
 
 #include "base/callback.h"
 #include "base/file_util.h"
 #include "base/i18n/icu_string_conversions.h"
 #include "base/strings/string_number_conversions.h"
@@ skipping 94 matching lines @@
   std::vector<base::string16> path;
   size_t toolbar_folder_index = 0;
   std::string charset;
   for (size_t i = 0;
        i < lines.size() &&
            (cancellation_callback.is_null() || !cancellation_callback.Run());
        ++i) {
     std::string line;
     TrimString(lines[i], " ", &line);
 
+    // Remove "<HR>" if |line| starts with it. See http://crbug.com/257474.
+    static const char kHrTag[] = "<HR>";
+    while (StartsWithASCII(line, kHrTag, false)) {

[zhchbin, 2013/08/10 06:06:02]

Note this, I have change it from "if" to "while". Actually speaking, "<HR>" is
the "separator" in the firefox. I have upload a video[0] to show it. My test
case file "firefox23.html" is also exported with these bookmarks.

[0]:
https://docs.google.com/file/d/0B_7w63Sy3AuycWlfTmhvanZLUEE/edit?usp=sharing 

[tfarina, 2013/08/10 14:30:51]

Could also note in the comments that we are removing/skipping the <HR> tag
because it's the separator in Firefox that Chrome does not support?

[zhchbin, 2013/08/10 14:45:39]

Done.

+      line.erase(0, arraysize(kHrTag) - 1);
+      TrimString(line, " ", &line);
+    }
+
     // Get the encoding of the bookmark file.
     if (internal::ParseCharsetFromLine(line, &charset))
       continue;
 
     // Get the folder name.
     if (internal::ParseFolderNameFromLine(line,
                                           charset,
                                           &last_folder,
                                           &last_folder_on_toolbar,
                                           &last_folder_add_date)) {
@@ skipping 298 matching lines @@
       *url = GURL(value);
     }
   }
 
   return true;
 }
 
 }  // namespace internal
 
 }  // namespace bookmark_html_reader