diff options
Diffstat (limited to 'chrome/browser/importer')
| -rw-r--r-- | chrome/browser/importer/firefox_importer_utils.cc | 13 |
1 files changed, 4 insertions, 9 deletions
diff --git a/chrome/browser/importer/firefox_importer_utils.cc b/chrome/browser/importer/firefox_importer_utils.cc index 023fcf9..c546727 100644 --- a/chrome/browser/importer/firefox_importer_utils.cc +++ b/chrome/browser/importer/firefox_importer_utils.cc @@ -195,14 +195,12 @@ void ParseSearchEnginesFromXMLFiles(const std::vector<std::wstring>& xml_files, std::map<std::wstring, TemplateURL*> search_engine_for_url; std::string content; - bool need_to_increment_iter; // The first XML file represents the default search engine in Firefox 3, so we // need to keep it on top of the list. TemplateURL* default_turl = NULL; - for (std::vector<std::wstring>::const_iterator iter = xml_files.begin(); - iter != xml_files.end();) { - need_to_increment_iter = true; - file_util::ReadFileToString(*iter, &content); + for (std::vector<std::wstring>::const_iterator file_iter = xml_files.begin(); + file_iter != xml_files.end(); ++file_iter) { + file_util::ReadFileToString(*file_iter, &content); TemplateURL* template_url = new TemplateURL(); FirefoxURLParameterFilter param_filter; if (TemplateURLParser::Parse( @@ -218,8 +216,7 @@ void ParseSearchEnginesFromXMLFiles(const std::vector<std::wstring>& xml_files, // returns a vector with first Firefox default search engines and then // the user's ones. We want to give priority to the user ones. delete iter->second; - iter = search_engine_for_url.erase(iter); - need_to_increment_iter = false; + search_engine_for_url.erase(iter); } // Give this a keyword to facilitate tab-to-search, if possible. template_url->set_keyword(TemplateURLModel::GenerateKeyword(GURL(url), @@ -232,8 +229,6 @@ void ParseSearchEnginesFromXMLFiles(const std::vector<std::wstring>& xml_files, delete template_url; } content.clear(); - if (need_to_increment_iter) - ++iter; } // Put the results in the |search_engines| vector. |