⚠ Archived content — this site is no longer maintained.   Current WebKit documentation is at docs.webkit.org.

Changeset 249272 in webkit


Ignore:
Timestamp:
Aug 29, 2019, 7:54:54 AM (7 years ago)
Author:
commit-queue@webkit.org
Message:

Address some MathML FIXMEs
​https://bugs.webkit.org/show_bug.cgi?id=201293

Patch by Rob Buis <​rbuis@igalia.com> on 2019-08-29
Reviewed by Frédéric Wang.

Source/WebCore:

Add element tag checks for data, time and dialog.

  • mathml/MathMLOperatorDictionary.h:
  • mathml/MathMLPresentationElement.cpp:

(WebCore::MathMLPresentationElement::isPhrasingContent):
(WebCore::MathMLPresentationElement::isFlowContent):

LayoutTests:

Also test newly exposed data and time elements.

  • mathml/presentation/foreign-element-in-token-expected.txt:
  • mathml/presentation/foreign-element-in-token.html:
  • platform/ios-wk2/mathml/presentation/foreign-element-in-token-expected.txt:
  • platform/mac-wk2/mathml/presentation/foreign-element-in-token-expected.txt:
Location:
trunk
Files:
8 edited

Legend:

Unmodified
Added
Removed

  • trunk/LayoutTests/ChangeLog

    r249237 r249272  
     12019-08-29  Rob Buis  <rbuis@igalia.com>
     2
     3        Address some MathML FIXMEs
     4        https://bugs.webkit.org/show_bug.cgi?id=201293
     5
     6        Reviewed by Frédéric Wang.
     7
     8        Also test newly exposed data and time elements.
     9
     10        * mathml/presentation/foreign-element-in-token-expected.txt:
     11        * mathml/presentation/foreign-element-in-token.html:
     12        * platform/ios-wk2/mathml/presentation/foreign-element-in-token-expected.txt:
     13        * platform/mac-wk2/mathml/presentation/foreign-element-in-token-expected.txt:
     14
    1152019-08-28  Ryosuke Niwa  <rniwa@webkit.org>
    216

  • trunk/LayoutTests/mathml/presentation/foreign-element-in-token-expected.txt

    r202572 r249272  
    2828mi
    2929
     30data:
     31mi
     32
    3033del:
    3134mi
    … …  
    103106mi
    104107
     108time:
     109mi
     110
    105111u:
    106112mi
    … …  
    136142mn
    137143
     144data:
     145mn
     146
    138147del:
    139148mn
    … …  
    211220mn
    212221
     222time:
     223mn
     224
    213225u:
    214226mn
    … …  
    244256mtext
    245257
     258data:
     259mtext
     260
    246261del:
    247262mtext
    … …  
    319334mtext
    320335
     336time:
     337mtext
     338
    321339u:
    322340mtext
    … …  
    352370mo
    353371
     372data:
     373mo
     374
    354375del:
    355376mo
    … …  
    427448mo
    428449
     450time:
     451mo
     452
    429453u:
    430454mo

  • trunk/LayoutTests/mathml/presentation/foreign-element-in-token.html

    r202572 r249272  
    1212    <p>Foreign phrasing content should create renderers:</p>
    1313
    14     <!-- The following elements are not covered by this test: wbr, audio, data,
    15          time, canvas, iframe, map, video, textarea, meter, noscript, progress,
     14    <!-- The following elements are not covered by this test: wbr, audio,
     15         canvas, iframe, map, video, textarea, meter, noscript, progress,
    1616         script, select, template, br, area. -->
    1717
    … …  
    2525    <p>code: <math><mi><code>mi</code></mi></math></p>
    2626    <p>datalist: <math><mi><datalist>mi</datalist></mi></math></p>
     27    <p>data: <math><mi><data>mi</data></mi></math></p>
    2728    <p>del: <math><mi><del>mi</del></mi></math></p>
    2829    <p>dfn: <math><mi><dfn>mi</dfn></mi></math></p>
    … …  
    5051    <p>sup: <math><mi><sup>mi</sup></mi></math></p>
    5152    <p>svg: <math><mi><svg><text>mi</text></svg></mi></math></p>
     53    <p>time: <math><mi><time>mi</time></mi></math></p>
    5254    <p>u: <math><mi><u>mi</u></mi></math></p>
    5355    <p>var: <math><mi><var>mi</var></mi></math></p>
    … …  
    6264    <p>code: <math><mn><code>mn</code></mn></math></p>
    6365    <p>datalist: <math><mn><datalist>mn</datalist></mn></math></p>
     66    <p>data: <math><mn><data>mn</data></mn></math></p>
    6467    <p>del: <math><mn><del>mn</del></mn></math></p>
    6568    <p>dfn: <math><mn><dfn>mn</dfn></mn></math></p>
    … …  
    8790    <p>sup: <math><mn><sup>mn</sup></mn></math></p>
    8891    <p>svg: <math><mn><svg><text>mn</text></svg></mn></math></p>
     92    <p>time: <math><mn><time>mn</time></mn></math></p>
    8993    <p>u: <math><mn><u>mn</u></mn></math></p>
    9094    <p>var: <math><mn><var>mn</var></mn></math></p>
    … …  
    99103    <p>code: <math><mtext><code>mtext</code></mtext></math></p>
    100104    <p>datalist: <math><mtext><datalist>mtext</datalist></mtext></math></p>
     105    <p>data: <math><mtext><data>mtext</data></mtext></math></p>
    101106    <p>del: <math><mtext><del>mtext</del></mtext></math></p>
    102107    <p>dfn: <math><mtext><dfn>mtext</dfn></mtext></math></p>
    … …  
    124129    <p>sup: <math><mtext><sup>mtext</sup></mtext></math></p>
    125130    <p>svg: <math><mtext><svg><text>mtext</text></svg></mtext></math></p>
     131    <p>time: <math><mtext><time>mtext</time></mtext></math></p>
    126132    <p>u: <math><mtext><u>mtext</u></mtext></math></p>
    127133    <p>var: <math><mtext><var>mtext</var></mtext></math></p>
    … …  
    136142    <p>code: <math><mo><code>mo</code></mo></math></p>
    137143    <p>datalist: <math><mo><datalist>mo</datalist></mo></math></p>
     144    <p>data: <math><mo><data>mo</data></mo></math></p>
    138145    <p>del: <math><mo><del>mo</del></mo></math></p>
    139146    <p>dfn: <math><mo><dfn>mo</dfn></mo></math></p>
    … …  
    161168    <p>sup: <math><mo><sup>mo</sup></mo></math></p>
    162169    <p>svg: <math><mo><svg><text>mo</text></svg></mo></math></p>
     170    <p>time: <math><mo><time>mo</time></mo></math></p>
    163171    <p>u: <math><mo><u>mo</u></mo></math></p>
    164172    <p>var: <math><mo><var>mo</var></mo></math></p>

  • trunk/LayoutTests/platform/ios-wk2/mathml/presentation/foreign-element-in-token-expected.txt

    r237245 r249272  
    2727datalist:
    2828
     29data:
     30mi
     31
    2932del:
    3033mi
    … …  
    102105mi
    103106
     107time:
     108mi
     109
    104110u:
    105111mi
    … …  
    134140datalist:
    135141
     142data:
     143mn
     144
    136145del:
    137146mn
    … …  
    209218mn
    210219
     220time:
     221mn
     222
    211223u:
    212224mn
    … …  
    241253datalist:
    242254
     255data:
     256mtext
     257
    243258del:
    244259mtext
    … …  
    316331mtext
    317332
     333time:
     334mtext
     335
    318336u:
    319337mtext
    … …  
    348366datalist:
    349367
     368data:
     369mo
     370
    350371del:
    351372mo
    … …  
    423444mo
    424445
     446time:
     447mo
     448
    425449u:
    426450mo

  • trunk/LayoutTests/platform/mac-wk2/mathml/presentation/foreign-element-in-token-expected.txt

    r237245 r249272  
    2727datalist:
    2828
     29data:
     30mi
     31
    2932del:
    3033mi
    … …  
    102105mi
    103106
     107time:
     108mi
     109
    104110u:
    105111mi
    … …  
    134140datalist:
    135141
     142data:
     143mn
     144
    136145del:
    137146mn
    … …  
    209218mn
    210219
     220time:
     221mn
     222
    211223u:
    212224mn
    … …  
    241253datalist:
    242254
     255data:
     256mtext
     257
    243258del:
    244259mtext
    … …  
    316331mtext
    317332
     333time:
     334mtext
     335
    318336u:
    319337mtext
    … …  
    348366datalist:
    349367
     368data:
     369mo
     370
    350371del:
    351372mo
    … …  
    423444mo
    424445
     446time:
     447mo
     448
    425449u:
    426450mo

  • trunk/Source/WebCore/ChangeLog

    r249254 r249272  
     12019-08-29  Rob Buis  <rbuis@igalia.com>
     2
     3        Address some MathML FIXMEs
     4        https://bugs.webkit.org/show_bug.cgi?id=201293
     5
     6        Reviewed by Frédéric Wang.
     7
     8        Add element tag checks for data, time and dialog.
     9
     10        * mathml/MathMLOperatorDictionary.h:
     11        * mathml/MathMLPresentationElement.cpp:
     12        (WebCore::MathMLPresentationElement::isPhrasingContent):
     13        (WebCore::MathMLPresentationElement::isFlowContent):
     14
    1152019-08-29  Zan Dobersek  <zdobersek@igalia.com>
    216

  • trunk/Source/WebCore/mathml/MathMLOperatorDictionary.h

    r242776 r249272  
    3636enum Form { Infix, Prefix, Postfix };
    3737enum Flag {
    38     Accent = 0x1, // FIXME: This must be used to implement accentunder/accent on munderover (https://bugs.webkit.org/show_bug.cgi?id=124826).
     38    Accent = 0x1,
    3939    Fence = 0x2, // This has no visual effect but allows to expose semantic information via the accessibility tree.
    4040    LargeOp = 0x4,

  • trunk/Source/WebCore/mathml/MathMLPresentationElement.cpp

    r246490 r249272  
    8787
    8888    if (is<HTMLElement>(node)) {
    89         // FIXME: add the <data> and <time> tags when they are implemented.
    9089        auto& htmlElement = downcast<HTMLElement>(node);
    9190        return htmlElement.hasTagName(HTMLNames::aTag)
    … …  
    102101            || htmlElement.hasTagName(HTMLNames::codeTag)
    103102            || htmlElement.hasTagName(HTMLNames::datalistTag)
     103            || htmlElement.hasTagName(HTMLNames::dataTag)
    104104            || htmlElement.hasTagName(HTMLNames::delTag)
    105105            || htmlElement.hasTagName(HTMLNames::dfnTag)
    … …  
    134134            || htmlElement.hasTagName(HTMLNames::templateTag)
    135135            || htmlElement.hasTagName(HTMLNames::textareaTag)
     136            || htmlElement.hasTagName(HTMLNames::timeTag)
    136137            || htmlElement.hasTagName(HTMLNames::uTag)
    137138            || htmlElement.hasTagName(HTMLNames::varTag)
    … …  
    155156
    156157    auto& htmlElement = downcast<HTMLElement>(node);
    157     // FIXME add the <dialog> tag when it is implemented.
    158158    return htmlElement.hasTagName(HTMLNames::addressTag)
    159159        || htmlElement.hasTagName(HTMLNames::articleTag)
    … …  
    161161        || htmlElement.hasTagName(HTMLNames::blockquoteTag)
    162162        || htmlElement.hasTagName(HTMLNames::detailsTag)
     163        || htmlElement.hasTagName(HTMLNames::dialogTag)
    163164        || htmlElement.hasTagName(HTMLNames::divTag)
    164165        || htmlElement.hasTagName(HTMLNames::dlTag)
Note: See TracChangeset for help on using the changeset viewer.