Move Firefox importer's INI parser to c/browser/common.

chrome/browser/common/ini_parser.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/browser/common/ini_parser.h"
+
+#include "base/strings/string_tokenizer.h"
+
+INIParser::INIParser() {}
+
+INIParser::~INIParser() {
+}
+
+void INIParser::Parse(const std::string& content) {
+  // Reads the whole INI file.
+  base::StringTokenizer tokenizer(content, "\r\n");
+
+  // Parses the file.
+  std::string current_section;

[erikwright (departed), 2013/06/14 19:26:46]

This code probably predates StringPiece support in StringTokenizer.

You could avoid a fair bit of the string copying by using StringPiece instead of
String up until the call to HandlePair.

But that's not required. Only if you enjoy that sort of thing.

[tommycli, 2013/06/17 18:01:56]

That's a good suggestion. I will try to get this committed as a simple move
first, then optimize/modify after.

+  while (tokenizer.GetNext()) {
+    std::string line = tokenizer.token();
+    if (line.empty()) {
+      // Skips the empty line.
+      continue;
+    }
+    if (line[0] == '#' || line[0] == ';') {
+      // This line is a comment.
+      continue;
+    }
+    if (line[0] == '[') {
+      // It is a section header.
+      current_section = line.substr(1);
+      size_t end = current_section.rfind(']');
+      if (end != std::string::npos)
+        current_section.erase(end);
+    } else {
+      std::string key, value;
+      size_t equal = line.find('=');
+      if (equal != std::string::npos) {
+        key = line.substr(0, equal);
+        value = line.substr(equal + 1);
+        HandlePair(current_section, key, value);
+      }
+    }
+  }
+}
+
+DictionaryValueINIParser::DictionaryValueINIParser() {
+}
+
+DictionaryValueINIParser::~DictionaryValueINIParser() {
+}
+
+void DictionaryValueINIParser::HandlePair(const std::string& section,
+                                          const std::string& key,
+                                          const std::string& value) {
+
+  // Checks whether the section and key contain a '.' character.
+  // Those sections and keys break DictionaryValue's path format,
+  // so we discard them.
+  if (section.find('.') == std::string::npos &&
+      key.find('.') == std::string::npos)
+    root_.SetString(section + "." + key, value);
+}