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

Unified Diff: LayoutTests/fast/dom/custom/invalid-first-char-combinators.html

Issue 493713002: Invalid chars should not be allowed as first character in custom element name (Closed) Base URL: https://chromium.googlesource.com/chromium/blink.git@master
Patch Set: Addrressing comments Created 6 years, 1 month 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: LayoutTests/fast/dom/custom/invalid-first-char-combinators.html
diff --git a/LayoutTests/fast/dom/custom/invalid-first-char-combinators.html b/LayoutTests/fast/dom/custom/invalid-first-char-combinators.html
new file mode 100644
index 0000000000000000000000000000000000000000..f69422a0dfc238ea453bfbc2aaa525cb4e0911b0
--- /dev/null
+++ b/LayoutTests/fast/dom/custom/invalid-first-char-combinators.html
@@ -0,0 +1,18 @@
+<!DOCTYPE html>
+<script src="../../../resources/testharness.js"></script>
+<script src="../../../resources/testharnessreport.js"></script>
+
+<script>
+test(function () {
+ var invalidCombinators = [ 0x0B83, 0x0F88, 0x0F89, 0x0F8A, 0x0F8B ];
+
+ for (var i = 0; i < invalidCombinators.length; i++) {
+ var invalidCombinator = String.fromCharCode(invalidCombinators[i]) + '-xfoo';
+ assert_throws('SYNTAX_ERR', function () {
+ document.registerElement(invalidCombinator);
+ }, 'registering custom element: *' +
+ invalidCombinator + '* should throw syntax error');
+ }
+
+}, 'registering invalid first letter combinators, not covered in Document::validNames');
dglazkov 2014/11/06 05:52:43 and here too?
deepak.sa 2014/11/06 06:20:02 As the expected files for such test cases has been
+</script>

Powered by Google App Engine
This is Rietveld 408576698