Commit 27b783e6 authored by Daniel Cheng's avatar Daniel Cheng Committed by Commit Bot

Use STL style naming for some WTF::AtomicString::find() overloads.

Like the WTF::String change, this leaves the the Find() overload that
takes a StringView, start offset, and a case sensitivity option for a
followup CL.

Bug: 709815
Change-Id: I8f98fe0b41eb99a525ca608690c8ae0bdcebf6a6
Reviewed-on: https://chromium-review.googlesource.com/489368Reviewed-by: default avatarKentaro Hara <haraken@chromium.org>
Commit-Queue: Daniel Cheng <dcheng@chromium.org>
Cr-Commit-Position: refs/heads/master@{#467872}
parent 61654002
...@@ -37,7 +37,7 @@ class CORE_EXPORT CustomElement { ...@@ -37,7 +37,7 @@ class CORE_EXPORT CustomElement {
static bool IsValidName(const AtomicString& name) { static bool IsValidName(const AtomicString& name) {
// This quickly rejects all common built-in element names. // This quickly rejects all common built-in element names.
if (name.Find('-', 1) == kNotFound) if (name.find('-', 1) == kNotFound)
return false; return false;
if (!IsASCIILower(name[0])) if (!IsASCIILower(name[0]))
......
...@@ -64,7 +64,7 @@ void V0CustomElement::AddEmbedderCustomElementName(const AtomicString& name) { ...@@ -64,7 +64,7 @@ void V0CustomElement::AddEmbedderCustomElementName(const AtomicString& name) {
} }
static inline bool IsValidNCName(const AtomicString& name) { static inline bool IsValidNCName(const AtomicString& name) {
if (kNotFound != name.Find(':')) if (kNotFound != name.find(':'))
return false; return false;
if (!name.GetString().Is8Bit()) { if (!name.GetString().Is8Bit()) {
...@@ -86,7 +86,7 @@ bool V0CustomElement::IsValidName(const AtomicString& name, ...@@ -86,7 +86,7 @@ bool V0CustomElement::IsValidName(const AtomicString& name,
kNotFound != EmbedderCustomElementNames().Find(name)) kNotFound != EmbedderCustomElementNames().Find(name))
return Document::IsValidName(name); return Document::IsValidName(name);
if ((valid_names & kStandardNames) && kNotFound != name.Find('-')) { if ((valid_names & kStandardNames) && kNotFound != name.find('-')) {
DEFINE_STATIC_LOCAL(Vector<AtomicString>, reserved_names, ()); DEFINE_STATIC_LOCAL(Vector<AtomicString>, reserved_names, ());
if (reserved_names.IsEmpty()) { if (reserved_names.IsEmpty()) {
// FIXME(crbug.com/426605): We should be able to remove this. // FIXME(crbug.com/426605): We should be able to remove this.
......
...@@ -98,14 +98,14 @@ class WTF_EXPORT AtomicString { ...@@ -98,14 +98,14 @@ class WTF_EXPORT AtomicString {
UChar operator[](unsigned i) const { return string_[i]; } UChar operator[](unsigned i) const { return string_[i]; }
// Find characters. // Find characters.
size_t Find(UChar c, unsigned start = 0) const { size_t find(UChar c, unsigned start = 0) const {
return string_.find(c, start); return string_.find(c, start);
} }
size_t Find(LChar c, unsigned start = 0) const { size_t find(LChar c, unsigned start = 0) const {
return string_.find(c, start); return string_.find(c, start);
} }
size_t Find(char c, unsigned start = 0) const { size_t find(char c, unsigned start = 0) const {
return Find(static_cast<LChar>(c), start); return find(static_cast<LChar>(c), start);
} }
size_t Find(CharacterMatchFunctionPtr match_function, size_t Find(CharacterMatchFunctionPtr match_function,
unsigned start = 0) const { unsigned start = 0) const {
...@@ -132,7 +132,7 @@ class WTF_EXPORT AtomicString { ...@@ -132,7 +132,7 @@ class WTF_EXPORT AtomicString {
return string_.FindIgnoringASCIICase(value, start); return string_.FindIgnoringASCIICase(value, start);
} }
bool Contains(char c) const { return Find(c) != kNotFound; } bool Contains(char c) const { return find(c) != kNotFound; }
bool Contains( bool Contains(
const StringView& value, const StringView& value,
TextCaseSensitivity case_sensitivity = kTextCaseSensitive) const { TextCaseSensitivity case_sensitivity = kTextCaseSensitive) const {
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment