2010-07-25 Adam Barth <abarth@webkit.org>
authorabarth@webkit.org <abarth@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Sun, 25 Jul 2010 21:51:04 +0000 (21:51 +0000)
committerabarth@webkit.org <abarth@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Sun, 25 Jul 2010 21:51:04 +0000 (21:51 +0000)
        Reviewed by Eric Seidel.

        Implement adjustMathMLAttributes
        https://bugs.webkit.org/show_bug.cgi?id=42952

        Yay test progression.

        * html5lib/runner-expected-html5.txt:
2010-07-25  Adam Barth  <abarth@webkit.org>

        Reviewed by Eric Seidel.

        Implement adjustMathMLAttributes
        https://bugs.webkit.org/show_bug.cgi?id=42952

        This is kind of mechanical overkill for adjusting one attribute name,
        but it's nice to use the same machinery as SVG.

        * html/HTMLTreeBuilder.cpp:
        * mathml/mathattrs.in:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@64024 268f45cc-cd09-0410-ab3c-d52691b4dbfc

LayoutTests/ChangeLog
LayoutTests/html5lib/runner-expected-html5.txt
WebCore/ChangeLog
WebCore/html/HTMLTreeBuilder.cpp
WebCore/mathml/mathattrs.in

index 5f36031..6fd77b1 100644 (file)
@@ -1,3 +1,14 @@
+2010-07-25  Adam Barth  <abarth@webkit.org>
+
+        Reviewed by Eric Seidel.
+
+        Implement adjustMathMLAttributes
+        https://bugs.webkit.org/show_bug.cgi?id=42952
+
+        Yay test progression.
+
+        * html5lib/runner-expected-html5.txt:
+
 2010-07-25  Alexey Proskuryakov  <ap@apple.com>
 
         Reviewed by Sam Weinig.
index 6a401de..892a563 100644 (file)
@@ -161,7 +161,6 @@ resources/tests17.dat: PASS
 resources/tests18.dat: PASS
 
 resources/tests19.dat:
-1
 22
 24
 63
@@ -172,25 +171,6 @@ resources/tests19.dat:
 101
 102
 
-Test 1 of 102 in resources/tests19.dat failed. Input:
-<!doctype html><math><mn DefinitionUrl="foo">
-Got:
-| <!DOCTYPE html>
-| <html>
-|   <head>
-|   <body>
-|     <math math>
-|       <math mn>
-|         definitionurl="foo"
-Expected:
-| <!DOCTYPE html>
-| <html>
-|   <head>
-|   <body>
-|     <math math>
-|       <math mn>
-|         definitionURL="foo"
-
 Test 22 of 102 in resources/tests19.dat failed. Input:
 <!doctype html><h1><div><h3><span></h1>foo
 Got:
index 0cca215..532d8d0 100644 (file)
@@ -2,6 +2,19 @@
 
         Reviewed by Eric Seidel.
 
+        Implement adjustMathMLAttributes
+        https://bugs.webkit.org/show_bug.cgi?id=42952
+
+        This is kind of mechanical overkill for adjusting one attribute name,
+        but it's nice to use the same machinery as SVG.
+
+        * html/HTMLTreeBuilder.cpp:
+        * mathml/mathattrs.in:
+
+2010-07-25  Adam Barth  <abarth@webkit.org>
+
+        Reviewed by Eric Seidel.
+
         Functions supporting the legacy tree builder are confusing
         https://bugs.webkit.org/show_bug.cgi?id=42951
 
index f706af8..0a5c859 100644 (file)
@@ -680,14 +680,15 @@ void adjustSVGTagNameCase(AtomicHTMLToken& token)
     token.setName(casedName.localName());
 }
 
-void adjustSVGAttributes(AtomicHTMLToken& token)
+template<QualifiedName** getAttrs(size_t* length)>
+void adjustAttributes(AtomicHTMLToken& token)
 {
     static PrefixedNameToQualifiedNameMap* caseMap = 0;
     if (!caseMap) {
         caseMap = new PrefixedNameToQualifiedNameMap;
         size_t length = 0;
-        QualifiedName** svgAttrs = SVGNames::getSVGAttrs(&length);
-        mapLoweredLocalNameToName(caseMap, svgAttrs, length);
+        QualifiedName** attrs = getAttrs(&length);
+        mapLoweredLocalNameToName(caseMap, attrs, length);
     }
 
     NamedNodeMap* attributes = token.attributes();
@@ -702,9 +703,14 @@ void adjustSVGAttributes(AtomicHTMLToken& token)
     }
 }
 
-void adjustMathMLAttributes(AtomicHTMLToken&)
+void adjustSVGAttributes(AtomicHTMLToken& token)
 {
-    notImplemented();
+    adjustAttributes<SVGNames::getSVGAttrs>(token);
+}
+
+void adjustMathMLAttributes(AtomicHTMLToken& token)
+{
+    adjustAttributes<MathMLNames::getMathMLAttrs>(token);
 }
 
 void addNamesWithPrefix(PrefixedNameToQualifiedNameMap* map, const AtomicString& prefix, QualifiedName** names, size_t length)
index 55a25ed..5431453 100644 (file)
@@ -4,6 +4,7 @@ guardFactoryWith="ENABLE(MATHML)"
 attrsNullNamespace
 
 close
+definitionURL
 denomalign
 linethickness
 mathbackground