Patch to solve the problem in import export bookmarks

chrome/browser/importer/firefox2_importer.cc

 // Copyright (c) 2006-2008 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/browser/importer/firefox2_importer.h"
 
 #include "app/l10n_util.h"
 #include "base/file_path.h"
 #include "base/file_util.h"
 #include "base/path_service.h"
@@ skipping 429 matching lines @@
   if (GetAttribute(attribute_list, kFeedURLAttribute, &value))
     return false;
 
   // Title
   CodepageToWide(line.substr(tag_end, end - tag_end), charset.c_str(),
                  OnStringUtilConversionError::SKIP, title);
   HTMLUnescape(title);
 
   // URL
   if (GetAttribute(attribute_list, kHrefAttribute, &value)) {
-    ReplaceSubstringsAfterOffset(&value, 0, "%22", "\"");
-    *url = GURL(value);
+    std::wstring w_url;
+    CodepageToWide(value, charset.c_str(), OnStringUtilConversionError::SKIP,
+      &w_url);
+    HTMLUnescape(&w_url);
+
+    string16 url16 = WideToUTF16Hack(w_url);
+
+    *url = GURL(url16);
   }
 
   // Favicon
   if (GetAttribute(attribute_list, kIconAttribute, &value))
     *favicon = GURL(value);
 
   // Keyword
   if (GetAttribute(attribute_list, kShortcutURLAttribute, &value)) {
     CodepageToWide(value, charset.c_str(), OnStringUtilConversionError::SKIP,
                    shortcut);
@@ skipping 22 matching lines @@
 bool Firefox2Importer::GetAttribute(const std::string& attribute_list,
                                     const std::string& attribute,
                                     std::string* value) {
   const char kQuote[] = "\"";
 
   size_t begin = attribute_list.find(attribute + "=" + kQuote);
   if (begin == std::string::npos)
     return false;  // Can't find the attribute.
 
   begin = attribute_list.find(kQuote, begin) + 1;
-  size_t end = attribute_list.find(kQuote, begin);
-  if (end == std::string::npos)
+
+  size_t end = begin + 1;
+  while (end < attribute_list.size()) {
+    if (attribute_list[end] == '"' &&
+        attribute_list[end - 1] != '\\') {
+      break;
+    }
+    end++;
+  }
+
+  if (end == attribute_list.size())
     return false;  // The value is not quoted.
 
   *value = attribute_list.substr(begin, end - begin);
   return true;
 }
 
 // static
 void Firefox2Importer::HTMLUnescape(std::wstring *text) {
   string16 text16 = WideToUTF16Hack(*text);
   ReplaceSubstringsAfterOffset(
@@ skipping 45 matching lines @@
 
   // We need to make up a URL for the favicon. We use a version of the page's
   // URL so that we can be sure it will not collide.
   usage.favicon_url = GURL(std::string("made-up-favicon:") + link_url.spec());
 
   // We only have one URL per favicon for Firefox 2 bookmarks.
   usage.urls.insert(link_url);
 
   favicons->push_back(usage);
 }