Index: chrome/common/pref_names_util.cc |
diff --git a/chrome/common/pref_names_util.cc b/chrome/common/pref_names_util.cc |
new file mode 100644 |
index 0000000000000000000000000000000000000000..311d2e2e14c441fe57dda1dc698f93d2056deaf8 |
--- /dev/null |
+++ b/chrome/common/pref_names_util.cc |
@@ -0,0 +1,34 @@ |
+// Copyright (c) 2012 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/common/pref_names_util.h" |
+ |
+#include "base/string_util.h" |
+ |
+namespace { |
Nico
2012/10/30 04:24:29
const has implicit internal linkage, no need for a
falken
2012/10/30 07:01:34
Done.
|
+ |
+const char kWebKitFontPrefPrefix[] = "webkit.webprefs.fonts."; |
Nico
2012/10/30 04:24:29
Add a comment to pref_names.cc where the webkit.we
falken
2012/10/30 07:01:34
Done. I also rearranged the pref orders to make a
|
+ |
+} |
+ |
+namespace pref_names_util { |
+ |
+bool ParseFontNamePrefPath(const std::string& pref_path, |
+ std::string* generic_family, |
+ std::string* script) { |
+ if (!StartsWithASCII(pref_path, kWebKitFontPrefPrefix, true)) |
+ return false; |
+ |
+ size_t start = strlen(kWebKitFontPrefPrefix); |
+ size_t pos = pref_path.find('.', start); |
+ if (pos == std::string::npos || pos + 1 == pref_path.length()) |
+ return false; |
+ if (generic_family) |
+ *generic_family = pref_path.substr(start, pos - start); |
+ if (script) |
+ *script = pref_path.substr(pos + 1); |
+ return true; |
+} |
Nico
2012/10/30 04:24:29
no test for this?
falken
2012/10/30 07:01:34
I added a unit test.
This was refactored out of t
|
+ |
+} // namespace pref_names_util |