Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(292)

Unified Diff: third_party/WebKit/Source/core/dom/custom/CustomElementTest.cpp

Issue 1916013004: Implement PotentialCustomElementName (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: tkent review Created 4 years, 8 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: third_party/WebKit/Source/core/dom/custom/CustomElementTest.cpp
diff --git a/third_party/WebKit/Source/core/dom/custom/CustomElementTest.cpp b/third_party/WebKit/Source/core/dom/custom/CustomElementTest.cpp
new file mode 100644
index 0000000000000000000000000000000000000000..720b5c1247bc1028c15d8edc8e80eea3a12befdc
--- /dev/null
+++ b/third_party/WebKit/Source/core/dom/custom/CustomElementTest.cpp
@@ -0,0 +1,110 @@
+// Copyright 2016 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 "core/dom/custom/CustomElement.h"
+
+#include "testing/gtest/include/gtest/gtest.h"
+
+namespace blink {
+
+static void testIsPotentialCustomElementName(const AtomicString& str, bool expected)
+{
+ if (expected) {
+ EXPECT_TRUE(CustomElement::isPotentialCustomElementName(str))
+ << str << " should be PotentialCustomElementName.";
+ } else {
+ EXPECT_FALSE(CustomElement::isPotentialCustomElementName(str))
+ << str << " should NOT be PotentialCustomElementName.";
+ }
+}
+
+static void testIsPotentialCustomElementNameChar(UChar32 c, bool expected)
+{
+ LChar str8[] = "a-X";
+ UChar str16[] = { 'a', '-', 'X', '\0', '\0' };
+ AtomicString str;
+ if (c <= 0xFF) {
+ str8[2] = c;
+ str = str8;
+ } else {
+ size_t i = 2;
+ U16_APPEND_UNSAFE(str16, i, c);
+ str16[i] = 0;
+ str = str16;
+ }
+ testIsPotentialCustomElementName(str, expected);
+}
+
+TEST(CustomElementTest, TestIsPotentialCustomElementName)
+{
+ struct {
+ bool expected;
+ AtomicString str;
+ } tests[] = {
+ { false, "" },
+ { false, "a" },
+ { false, "A" },
+
+ { false, "A-" },
+ { false, "0-" },
+
+ { true, "a-" },
+ { true, "a-a" },
+ { true, "aa-" },
+ { true, "aa-a" },
+ { true, reinterpret_cast<const UChar*>(u"aa-\x6F22\x5B57") }, // Two CJK Unified Ideographs
+ { true, reinterpret_cast<const UChar*>(u"aa-\xD840\xDC0B") }, // Surrogate pair U+2000B
+
+ { false, "a-A" },
+ { false, "a-Z" },
+ };
+ for (auto test : tests)
+ testIsPotentialCustomElementName(test.str, test.expected);
+}
+
+TEST(CustomElementTest, TestIsPotentialCustomElementNameChar)
+{
+ struct {
+ UChar32 from, to;
+ } ranges[] = {
+ { '-', '.' }, // "-" | "." need to merge to test -1/+1.
+ { '0', '9' },
+ { '_', '_' },
+ { 'a', 'z' },
+ { 0xB7, 0xB7 },
+ { 0xC0, 0xD6 },
+ { 0xD8, 0xF6 },
+ { 0xF8, 0x37D }, // [#xF8-#x2FF] | [#x300-#x37D] need to merge to test -1/+1.
+ { 0x37F, 0x1FFF },
+ { 0x200C, 0x200D },
+ { 0x203F, 0x2040 },
+ { 0x2070, 0x218F },
+ { 0x2C00, 0x2FEF },
+ { 0x3001, 0xD7FF },
+ { 0xF900, 0xFDCF },
+ { 0xFDF0, 0xFFFD },
+ { 0x10000, 0xEFFFF },
+ };
+ for (auto range : ranges) {
+ testIsPotentialCustomElementNameChar(range.from - 1, false);
+ for (UChar32 c = range.from; c <= range.to; ++c)
+ testIsPotentialCustomElementNameChar(c, true);
+ testIsPotentialCustomElementNameChar(range.to + 1, false);
+ }
+}
+
+TEST(CustomElementTest, TestIsPotentialCustomElementNameCharFalse)
+{
+ struct {
+ UChar32 from, to;
+ } ranges[] = {
+ { 'A', 'Z' },
+ };
+ for (auto range : ranges) {
+ for (UChar32 c = range.from; c <= range.to; ++c)
+ testIsPotentialCustomElementNameChar(c, false);
+ }
+}
+
+} // namespace blink
« no previous file with comments | « third_party/WebKit/Source/core/dom/custom/CustomElement.cpp ('k') | third_party/WebKit/Source/platform/text/Character.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698