Changeset 204030 in webkit

Timestamp:

Aug 2, 2016 10:27:37 AM (8 years ago)

Author:

commit-queue@webkit.org

Message:

[MathML] Use more auto for local variables
​https://bugs.webkit.org/show_bug.cgi?id=160453

Patch by Frederic Wang <​fwang@igalia.com> on 2016-08-02
Reviewed by Darin Adler.

No new tests, behavior is unchanged.

• mathml/MathMLElement.cpp:

(WebCore::MathMLElement::colSpan):
(WebCore::MathMLElement::rowSpan):
(WebCore::MathMLElement::childShouldCreateRenderer):
(WebCore::MathMLElement::defaultEventHandler):

• mathml/MathMLSelectElement.cpp:

(WebCore::MathMLSelectElement::getSelectedActionChild):
(WebCore::MathMLSelectElement::getSelectedSemanticsChild):

• rendering/mathml/RenderMathMLFenced.cpp:

(WebCore::RenderMathMLFenced::updateFromElement):

Location:

trunk/Source/WebCore

Files:

• ChangeLog (modified) (1 diff)
• mathml/MathMLElement.cpp (modified) (4 diffs)
• mathml/MathMLSelectElement.cpp (modified) (2 diffs)
• rendering/mathml/RenderMathMLFenced.cpp (modified) (1 diff)

trunk/Source/WebCore/ChangeLog

@@ +1 @@
+2016-08-02  Frederic Wang  <fwang@igalia.com>
+
+        [MathML] Use more auto for local variables
+        https://bugs.webkit.org/show_bug.cgi?id=160453
+
+        Reviewed by Darin Adler.
+
+        No new tests, behavior is unchanged.
+
+        * mathml/MathMLElement.cpp:
+        (WebCore::MathMLElement::colSpan):
+        (WebCore::MathMLElement::rowSpan):
+        (WebCore::MathMLElement::childShouldCreateRenderer):
+        (WebCore::MathMLElement::defaultEventHandler):
+        * mathml/MathMLSelectElement.cpp:
+        (WebCore::MathMLSelectElement::getSelectedActionChild):
+        (WebCore::MathMLSelectElement::getSelectedSemanticsChild):
+        * rendering/mathml/RenderMathMLFenced.cpp:
+        (WebCore::RenderMathMLFenced::updateFromElement):
+
 2016-08-02  Ryan Haddad  <ryanhaddad@apple.com>
 

trunk/Source/WebCore/mathml/MathMLElement.cpp

@@ -208 +208 @@
     if (!hasTagName(mtdTag))
         return 1u;
-    const AtomicString& colSpanValue = attributeWithoutSynchronization(columnspanAttr);
+    auto& colSpanValue = attributeWithoutSynchronization(columnspanAttr);
     return std::max(1u, limitToOnlyHTMLNonNegative(colSpanValue, 1u));
 }
@@ -216 +216 @@
     if (!hasTagName(mtdTag))
         return 1u;
-    const AtomicString& rowSpanValue = attributeWithoutSynchronization(rowspanAttr);
+    auto& rowSpanValue = attributeWithoutSynchronization(rowspanAttr);
     static const unsigned maxRowspan = 8190; // This constant comes from HTMLTableCellElement.
     return std::max(1u, std::min(limitToOnlyHTMLNonNegative(rowSpanValue, 1u), maxRowspan));
@@ -281 +281 @@
 {
     if (hasTagName(annotation_xmlTag)) {
-        const AtomicString& value = attributeWithoutSynchronization(MathMLNames::encodingAttr);
+        auto& value = attributeWithoutSynchronization(MathMLNames::encodingAttr);
 
         // See annotation-xml.model.mathml, annotation-xml.model.svg and annotation-xml.model.xhtml in the HTML5 RelaxNG schema.
@@ -331 +331 @@
         }
         if (MouseEvent::canTriggerActivationBehavior(*event)) {
-            const AtomicString& href = attributeWithoutSynchronization(hrefAttr);
-            String url = stripLeadingAndTrailingHTMLSpaces(href);
+            auto& href = attributeWithoutSynchronization(hrefAttr);
+            const auto& url = stripLeadingAndTrailingHTMLSpaces(href);
             event->setDefaultHandled();
-            if (Frame* frame = document().frame())
+            if (auto* frame = document().frame())
                 frame->loader().urlSelected(document().completeURL(url), "_self", event, LockHistory::No, LockBackForwardList::No, MaybeSendReferrer, document().shouldOpenExternalURLsPolicyToPropagate());
             return;

trunk/Source/WebCore/mathml/MathMLSelectElement.cpp

@@ -137 +137 @@
 
     // The value of the actiontype attribute is case-sensitive.
-    const AtomicString& actiontype = attributeWithoutSynchronization(MathMLNames::actiontypeAttr);
+    auto& actiontype = attributeWithoutSynchronization(MathMLNames::actiontypeAttr);
     if (actiontype == "statusline")
         // FIXME: implement user interaction for the "statusline" action type (http://wkbug/124922).
@@ -186 +186 @@
                 continue;
             // If the <annotation-xml> element has an encoding attribute describing presentation MathML, SVG or HTML we assume the content can be displayed and we stop here.
-            const AtomicString& value = child->attributeWithoutSynchronization(MathMLNames::encodingAttr);
+            auto& value = child->attributeWithoutSynchronization(MathMLNames::encodingAttr);
             if (isMathMLEncoding(value) || isSVGEncoding(value) || isHTMLEncoding(value))
                 return child;

trunk/Source/WebCore/rendering/mathml/RenderMathMLFenced.cpp

@@ -55 +55 @@
 
     // The open operator defaults to a left parenthesis.
-    AtomicString open = fenced.attributeWithoutSynchronization(MathMLNames::openAttr);
+    auto& open = fenced.attributeWithoutSynchronization(MathMLNames::openAttr);
     m_open = open.isNull() ? gOpeningBraceChar : open;
 
     // The close operator defaults to a right parenthesis.
-    AtomicString close = fenced.attributeWithoutSynchronization(MathMLNames::closeAttr);
+    auto& close = fenced.attributeWithoutSynchronization(MathMLNames::closeAttr);
     m_close = close.isNull() ? gClosingBraceChar : close;
 
-    AtomicString separators = fenced.attributeWithoutSynchronization(MathMLNames::separatorsAttr);
+    auto& separators = fenced.attributeWithoutSynchronization(MathMLNames::separatorsAttr);
     if (!separators.isNull()) {
         StringBuilder characters;