added support for validating contenteditable attribute

Mark Pilgrim · Mark Pilgrim · commit dadf71174ae8 · 2007-08-30T03:31:29.000Z
--HG--
extra : convert_revision : svn%3Aacbfec75-9323-0410-a652-858a13e371e0/trunk%40974
diff --git a/src/html5lib/filters/validator.py b/src/html5lib/filters/validator.py
@@ -37,10 +37,10 @@
         _(u"'%(attributeName)s' attribute is not allowed on <input type=%(inputType)s>."),
     "deprecated-attribute":
         _(u"'%(attributeName)s' attribute is deprecated on <%(tagName)s>."),
-    "invalid-class-attribute":
-        _(u"Invalid class attribute value on <%(tagName)s>."),
     "duplicate-value-in-token-list":
         _(u"Duplicate value '%(attributeValue)s' in token list in '%(attributeName)s' attribute on <%(tagName)s>."),
+    "invalid-attribute-value":
+        _(u"Invalid value for '%(attributeName)s' attribute on <%(tagName)s>."),
 })
 
 globalAttributes = frozenset(('class', 'contenteditable', 'contextmenu', 'dir',
@@ -317,8 +317,17 @@ def validateAttributeValueClass(self, token, tagName, attrName, attrValue):
         for t in self.checkTokenList(tagName, attrName, attrValue) or []:
             yield t
             yield {"type": "ParseError",
-                   "data": "invalid-class-attribute",
-                   "datavars": {"tagName": tagName}}
+                   "data": "invalid-attribute-value",
+                   "datavars": {"tagName": tagName,
+                                "attributeName": attrName}}
+
+    def validateAttributeValueContenteditable(self, token, tagName, attrName, attrValue):
+        attrValue = attrValue.lower()
+        if attrValue not in frozenset(('true', 'false', '')):
+            yield {"type": "ParseError",
+                   "data": "invalid-attribute-value",
+                   "datavars": {"tagName": tagName,
+                                "attributeName": attrName}}
         
     def checkTokenList(self, tagName, attrName, attrValue):
         # The "token" in the method name refers to tokens in an attribute value
