<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.1//EN"

"http://www.w3.org/TR/xhtml11/DTD/xhtml11.dtd">

<html xmlns="http://www.w3.org/1999/xhtml">

<head><meta http-equiv="content-type" content="text/html; charset=utf-8" />

<title>[203679] trunk/Source/WebCore</title>

</head>

<body>

<style type="text/css"><!--

#msg dl.meta { border: 1px #006 solid; background: #369; padding: 6px; color: #fff; }

#msg dl.meta dt { float: left; width: 6em; font-weight: bold; }

#msg dt:after { content:':';}

#msg dl, #msg dt, #msg ul, #msg li, #header, #footer, #logmsg { font-family: verdana,arial,helvetica,sans-serif; font-size: 10pt;  }

#msg dl a { font-weight: bold}

#msg dl a:link    { color:#fc3; }

#msg dl a:active  { color:#ff0; }

#msg dl a:visited { color:#cc6; }

h3 { font-family: verdana,arial,helvetica,sans-serif; font-size: 10pt; font-weight: bold; }

#msg pre { overflow: auto; background: #ffc; border: 1px #fa0 solid; padding: 6px; }

#logmsg { background: #ffc; border: 1px #fa0 solid; padding: 1em 1em 0 1em; }

#logmsg p, #logmsg pre, #logmsg blockquote { margin: 0 0 1em 0; }

#logmsg p, #logmsg li, #logmsg dt, #logmsg dd { line-height: 14pt; }

#logmsg h1, #logmsg h2, #logmsg h3, #logmsg h4, #logmsg h5, #logmsg h6 { margin: .5em 0; }

#logmsg h1:first-child, #logmsg h2:first-child, #logmsg h3:first-child, #logmsg h4:first-child, #logmsg h5:first-child, #logmsg h6:first-child { margin-top: 0; }

#logmsg ul, #logmsg ol { padding: 0; list-style-position: inside; margin: 0 0 0 1em; }

#logmsg ul { text-indent: -1em; padding-left: 1em; }#logmsg ol { text-indent: -1.5em; padding-left: 1.5em; }

#logmsg > ul, #logmsg > ol { margin: 0 0 1em 0; }

#logmsg pre { background: #eee; padding: 1em; }

#logmsg blockquote { border: 1px solid #fa0; border-left-width: 10px; padding: 1em 1em 0 1em; background: white;}

#logmsg dl { margin: 0; }

#logmsg dt { font-weight: bold; }

#logmsg dd { margin: 0; padding: 0 0 0.5em 0; }

#logmsg dd:before { content:'\00bb';}

#logmsg table { border-spacing: 0px; border-collapse: collapse; border-top: 4px solid #fa0; border-bottom: 1px solid #fa0; background: #fff; }

#logmsg table th { text-align: left; font-weight: normal; padding: 0.2em 0.5em; border-top: 1px dotted #fa0; }

#logmsg table td { text-align: right; border-top: 1px dotted #fa0; padding: 0.2em 0.5em; }

#logmsg table thead th { text-align: center; border-bottom: 1px solid #fa0; }

#logmsg table th.Corner { text-align: left; }

#logmsg hr { border: none 0; border-top: 2px dashed #fa0; height: 1px; }

#header, #footer { color: #fff; background: #636; border: 1px #300 solid; padding: 6px; }

#patch { width: 100%; }

#patch h4 {font-family: verdana,arial,helvetica,sans-serif;font-size:10pt;padding:8px;background:#369;color:#fff;margin:0;}

#patch .propset h4, #patch .binary h4 {margin:0;}

#patch pre {padding:0;line-height:1.2em;margin:0;}

#patch .diff {width:100%;background:#eee;padding: 0 0 10px 0;overflow:auto;}

#patch .propset .diff, #patch .binary .diff  {padding:10px 0;}

#patch span {display:block;padding:0 10px;}

#patch .modfile, #patch .addfile, #patch .delfile, #patch .propset, #patch .binary, #patch .copfile {border:1px solid #ccc;margin:10px 0;}

#patch ins {background:#dfd;text-decoration:none;display:block;padding:0 10px;}

#patch del {background:#fdd;text-decoration:none;display:block;padding:0 10px;}

#patch .lines, .info {color:#888;background:#fff;}

--></style>

<div id="msg">

<dl class="meta">

<dt>Revision</dt> <dd><a href="http://trac.webkit.org/projects/webkit/changeset/203679">203679</a></dd>

<dt>Author</dt> <dd>fred.wang@free.fr</dd>

<dt>Date</dt> <dd>2016-07-25 00:18:47 -0700 (Mon, 25 Jul 2016)</dd>

</dl>

<h3>Log Message</h3>

<pre>Move parsing of display, displaystyle and mathvariant attributes into MathML element classes

https://bugs.webkit.org/show_bug.cgi?id=159623

Patch by Frederic Wang &lt;fwang@igalia.com&gt; on 2016-07-24

Reviewed by Brent Fulgham.

No new tests, already covered by existing tests.

* mathml/MathMLElement.cpp:

(WebCore::MathMLElement::parseMathVariantAttribute): Move helper function to parse the

mathvariant attribute.

(WebCore::MathMLElement::getSpecifiedDisplayStyle): Helper function to set the displaystyle

value from the attribute specified on the MathML element.

(WebCore::MathMLElement::getSpecifiedMathVariant): Helper function to set the mathvariant

value from the attribute specified on the MathML element.

* mathml/MathMLElement.h: Move the enum for mathvariant values and declare new members.

(WebCore::MathMLElement::acceptsDisplayStyleAttribute): Indicate whether the element accepts

displaystyle attribute (false for most of them).

(WebCore::MathMLElement::acceptsMathVariantAttribute): Indicate whether the element accepts

mathvariant attribute (false for most of them).

* mathml/MathMLInlineContainerElement.cpp:

(WebCore::MathMLInlineContainerElement::acceptsDisplayStyleAttribute): Add mstyle and mtable

to the list of elements accepting the displaystyle attribute.

(WebCore::MathMLInlineContainerElement::acceptsMathVariantAttribute): Add mstyle to the list

of elements accepting the mathvariant attribute.

(WebCore::MathMLInlineContainerElement::parseAttribute): Mark displaystyle and mathvariant

dirty if necessary. Also use the new accepts*Attribute function.

* mathml/MathMLInlineContainerElement.h: Declare overridden accepts*Attribute members.

* mathml/MathMLMathElement.cpp:

(WebCore::MathMLMathElement::getSpecifiedDisplayStyle): Override acceptsDisplayStyleAttribute

so that the display attribute is also used to set the default value if the displaystyle

attribute is absent.

(WebCore::MathMLMathElement::parseAttribute): Mark displaystyle and mathvariant dirty if

necessary. We directly MathMLElement::parseAttribute to avoid duplicate work.

* mathml/MathMLMathElement.h: Add the math tag to the list of elements accepting the

displaystyle and mathvariant attributes. Declare overridden getSpecifiedDisplayStyle.

* mathml/MathMLTextElement.cpp:

(WebCore::MathMLTextElement::parseAttribute): Mark mathvariant as dirty.

* mathml/MathMLTextElement.h: Add token elements to the list of elements accepting the

mathvariant attribute.

* rendering/mathml/MathMLStyle.cpp:

(WebCore::MathMLStyle::updateStyleIfNeeded): Use the new MathMLElement::MathVariant enum.

(WebCore::MathMLStyle::resolveMathMLStyle):  We no longer parse the display value to

initialize the default value on the math tag, because this is handled in

getSpecifiedDisplayStyle. In general, we also just call getSpecifiedDisplayStyle and

getSpecifiedMathVariant on the MathML elements instead of parsing the displaystyle and

mathvariant attributes here.

(WebCore::MathMLStyle::parseMathVariant): Deleted. This is moved into MathMLElement.

* rendering/mathml/MathMLStyle.h: Use the new MathMLElement::MathVariant enum.

* rendering/mathml/RenderMathMLToken.cpp: Ditto.

(WebCore::mathVariant): Ditto.

(WebCore::RenderMathMLToken::updateMathVariantGlyph): Ditto.</pre>

<h3>Modified Paths</h3>

<ul>

<li><a href="#trunkSourceWebCoreChangeLog">trunk/Source/WebCore/ChangeLog</a></li>

<li><a href="#trunkSourceWebCoremathmlMathMLElementcpp">trunk/Source/WebCore/mathml/MathMLElement.cpp</a></li>

<li><a href="#trunkSourceWebCoremathmlMathMLElementh">trunk/Source/WebCore/mathml/MathMLElement.h</a></li>

<li><a href="#trunkSourceWebCoremathmlMathMLInlineContainerElementcpp">trunk/Source/WebCore/mathml/MathMLInlineContainerElement.cpp</a></li>

<li><a href="#trunkSourceWebCoremathmlMathMLInlineContainerElementh">trunk/Source/WebCore/mathml/MathMLInlineContainerElement.h</a></li>

<li><a href="#trunkSourceWebCoremathmlMathMLMathElementcpp">trunk/Source/WebCore/mathml/MathMLMathElement.cpp</a></li>

<li><a href="#trunkSourceWebCoremathmlMathMLMathElementh">trunk/Source/WebCore/mathml/MathMLMathElement.h</a></li>

<li><a href="#trunkSourceWebCoremathmlMathMLTextElementcpp">trunk/Source/WebCore/mathml/MathMLTextElement.cpp</a></li>

<li><a href="#trunkSourceWebCoremathmlMathMLTextElementh">trunk/Source/WebCore/mathml/MathMLTextElement.h</a></li>

<li><a href="#trunkSourceWebCorerenderingmathmlMathMLStylecpp">trunk/Source/WebCore/rendering/mathml/MathMLStyle.cpp</a></li>

<li><a href="#trunkSourceWebCorerenderingmathmlMathMLStyleh">trunk/Source/WebCore/rendering/mathml/MathMLStyle.h</a></li>

<li><a href="#trunkSourceWebCorerenderingmathmlRenderMathMLTokencpp">trunk/Source/WebCore/rendering/mathml/RenderMathMLToken.cpp</a></li>

</ul>

</div>

<div id="patch">

<h3>Diff</h3>

<a id="trunkSourceWebCoreChangeLog"></a>

<div class="modfile"><h4>Modified: trunk/Source/WebCore/ChangeLog (203678 => 203679)</h4>

<pre class="diff"><span>

<span class="info">--- trunk/Source/WebCore/ChangeLog        2016-07-25 07:10:18 UTC (rev 203678)

+++ trunk/Source/WebCore/ChangeLog        2016-07-25 07:18:47 UTC (rev 203679)

</span><span class="lines">@@ -1,3 +1,57 @@

</span><ins>+2016-07-24  Frederic Wang  &lt;fwang@igalia.com&gt;

+

+        Move parsing of display, displaystyle and mathvariant attributes into MathML element classes

+        https://bugs.webkit.org/show_bug.cgi?id=159623

+

+        Reviewed by Brent Fulgham.

+

+        No new tests, already covered by existing tests.

+

+        * mathml/MathMLElement.cpp:

+        (WebCore::MathMLElement::parseMathVariantAttribute): Move helper function to parse the

+        mathvariant attribute.

+        (WebCore::MathMLElement::getSpecifiedDisplayStyle): Helper function to set the displaystyle

+        value from the attribute specified on the MathML element.

+        (WebCore::MathMLElement::getSpecifiedMathVariant): Helper function to set the mathvariant

+        value from the attribute specified on the MathML element.

+        * mathml/MathMLElement.h: Move the enum for mathvariant values and declare new members.

+        (WebCore::MathMLElement::acceptsDisplayStyleAttribute): Indicate whether the element accepts

+        displaystyle attribute (false for most of them).

+        (WebCore::MathMLElement::acceptsMathVariantAttribute): Indicate whether the element accepts

+        mathvariant attribute (false for most of them).

+        * mathml/MathMLInlineContainerElement.cpp:

+        (WebCore::MathMLInlineContainerElement::acceptsDisplayStyleAttribute): Add mstyle and mtable

+        to the list of elements accepting the displaystyle attribute.

+        (WebCore::MathMLInlineContainerElement::acceptsMathVariantAttribute): Add mstyle to the list

+        of elements accepting the mathvariant attribute.

+        (WebCore::MathMLInlineContainerElement::parseAttribute): Mark displaystyle and mathvariant

+        dirty if necessary. Also use the new accepts*Attribute function.

+        * mathml/MathMLInlineContainerElement.h: Declare overridden accepts*Attribute members.

+        * mathml/MathMLMathElement.cpp:

+        (WebCore::MathMLMathElement::getSpecifiedDisplayStyle): Override acceptsDisplayStyleAttribute

+        so that the display attribute is also used to set the default value if the displaystyle

+        attribute is absent.

+        (WebCore::MathMLMathElement::parseAttribute): Mark displaystyle and mathvariant dirty if

+        necessary. We directly MathMLElement::parseAttribute to avoid duplicate work.

+        * mathml/MathMLMathElement.h: Add the math tag to the list of elements accepting the

+        displaystyle and mathvariant attributes. Declare overridden getSpecifiedDisplayStyle.

+        * mathml/MathMLTextElement.cpp:

+        (WebCore::MathMLTextElement::parseAttribute): Mark mathvariant as dirty.

+        * mathml/MathMLTextElement.h: Add token elements to the list of elements accepting the

+        mathvariant attribute.

+        * rendering/mathml/MathMLStyle.cpp:

+        (WebCore::MathMLStyle::updateStyleIfNeeded): Use the new MathMLElement::MathVariant enum.

+        (WebCore::MathMLStyle::resolveMathMLStyle):  We no longer parse the display value to

+        initialize the default value on the math tag, because this is handled in

+        getSpecifiedDisplayStyle. In general, we also just call getSpecifiedDisplayStyle and

+        getSpecifiedMathVariant on the MathML elements instead of parsing the displaystyle and

+        mathvariant attributes here.

+        (WebCore::MathMLStyle::parseMathVariant): Deleted. This is moved into MathMLElement.

+        * rendering/mathml/MathMLStyle.h: Use the new MathMLElement::MathVariant enum.

+        * rendering/mathml/RenderMathMLToken.cpp: Ditto.

+        (WebCore::mathVariant): Ditto.

+        (WebCore::RenderMathMLToken::updateMathVariantGlyph): Ditto.

+

</ins><span class="cx"> 2016-07-25  Carlos Garcia Campos  &lt;cgarcia@igalia.com&gt;

</span><span class="cx"> 

<span class="cx">         Unreviewed. Remove unneeded header includes from CoordinatedGraphicsLayer.

</span></span></pre></div>

<a id="trunkSourceWebCoremathmlMathMLElementcpp"></a>

<div class="modfile"><h4>Modified: trunk/Source/WebCore/mathml/MathMLElement.cpp (203678 => 203679)</h4>

<pre class="diff"><span>

<span class="info">--- trunk/Source/WebCore/mathml/MathMLElement.cpp        2016-07-25 07:10:18 UTC (rev 203678)

+++ trunk/Source/WebCore/mathml/MathMLElement.cpp        2016-07-25 07:18:47 UTC (rev 203679)

</span><span class="lines">@@ -2,6 +2,7 @@