Index: chrome/common/importer/firefox_importer_utils.cc |
diff --git a/chrome/common/importer/firefox_importer_utils.cc b/chrome/common/importer/firefox_importer_utils.cc |
index 46e120c304e4cfeace63f1663c7e0a201cfc8dec..3b74d6f585cf27a9ac4dbafdff22caf36d98f9c0 100644 |
--- a/chrome/common/importer/firefox_importer_utils.cc |
+++ b/chrome/common/importer/firefox_importer_utils.cc |
@@ -138,11 +138,9 @@ bool GetFirefoxVersionAndPathFromProfile(const base::FilePath& profile_path, |
std::string content; |
base::ReadFileToString(compatibility_file, &content); |
base::ReplaceSubstringsAfterOffset(&content, 0, "\r\n", "\n"); |
- std::vector<std::string> lines; |
- base::SplitString(content, '\n', &lines); |
- for (size_t i = 0; i < lines.size(); ++i) { |
- const std::string& line = lines[i]; |
+ for (const std::string& line : base::SplitString( |
+ content, "\n", base::TRIM_WHITESPACE, base::SPLIT_WANT_ALL)) { |
if (line.empty() || line[0] == '#' || line[0] == ';') |
continue; |
size_t equal = line.find('='); |
@@ -242,11 +240,10 @@ bool IsDefaultHomepage(const GURL& homepage, const base::FilePath& app_path) { |
return homepage.spec() == GURL(default_homepages).spec(); |
// Crack the string into separate homepage urls. |
- std::vector<std::string> urls; |
- base::SplitString(default_homepages, '|', &urls); |
- |
- for (size_t i = 0; i < urls.size(); ++i) { |
- if (homepage.spec() == GURL(urls[i]).spec()) |
+ for (const std::string& url : base::SplitString( |
+ default_homepages, "|", |
+ base::TRIM_WHITESPACE, base::SPLIT_WANT_ALL)) { |
+ if (homepage.spec() == GURL(url).spec()) |
return true; |
} |
@@ -303,19 +300,18 @@ base::string16 GetFirefoxImporterName(const base::FilePath& app_path) { |
if (base::PathExists(app_ini_file)) { |
std::string content; |
base::ReadFileToString(app_ini_file, &content); |
- std::vector<std::string> lines; |
- base::SplitString(content, '\n', &lines); |
+ |
const std::string name_attr("Name="); |
bool in_app_section = false; |
- for (size_t i = 0; i < lines.size(); ++i) { |
- base::TrimWhitespace(lines[i], base::TRIM_ALL, &lines[i]); |
- if (lines[i] == "[App]") { |
+ for (const base::StringPiece& line : base::SplitStringPiece( |
+ content, "\n", base::TRIM_WHITESPACE, base::SPLIT_WANT_ALL)) { |
+ if (line == "[App]") { |
in_app_section = true; |
} else if (in_app_section) { |
- if (lines[i].find(name_attr) == 0) { |
- branding_name = lines[i].substr(name_attr.size()); |
+ if (line.find(name_attr) == 0) { |
+ line.substr(name_attr.size()).CopyToString(&branding_name); |
break; |
- } else if (lines[i].length() > 0 && lines[i][0] == '[') { |
+ } else if (line.length() > 0 && line[0] == '[') { |
// No longer in the [App] section. |
break; |
} |