Implement CustomElement::isValidName()

third_party/WebKit/Source/core/dom/custom/CustomElement.h

Index: third_party/WebKit/Source/core/dom/custom/CustomElement.h
diff --git a/third_party/WebKit/Source/core/dom/custom/CustomElement.h b/third_party/WebKit/Source/core/dom/custom/CustomElement.h
index 8a2e93c502b95ce0be48810f98726add880865c6..8156666d622cf9606dd2b46dfb52b39e3e7bc219 100644
--- a/third_party/WebKit/Source/core/dom/custom/CustomElement.h
+++ b/third_party/WebKit/Source/core/dom/custom/CustomElement.h
@@ -14,6 +14,7 @@ namespace blink {
 class CORE_EXPORT CustomElement {
     STATIC_ONLY(CustomElement);
 public:
+    static bool isValidName(const AtomicString& name);

[dominicc (has gone to gerrit), 2016/04/27 05:34:23]

Nit: Make the symbol in the title and description match this, ie isValidName not
IsValidName.

[kojii, 2016/04/27 06:55:43]

Done, thanks.

     static bool isPotentialCustomElementName(const AtomicString& name);

[dominicc (has gone to gerrit), 2016/04/27 05:34:23]

You want to at *least* make this private, or even just roll it into isValidName.
Having two hanging around is confusing; when would you call
isPotentialCustomElementName but not want isValidName?

[kojii, 2016/04/27 06:55:43]

Merged them. Confirmed that there are no other references to
PotentialCustomElementName in the spec.

 };