Remove unused ExceptionCode& argument from Element::setAttribute(QualifiedName)
authoradamk@chromium.org <adamk@chromium.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 20 Dec 2011 03:42:22 +0000 (03:42 +0000)
committeradamk@chromium.org <adamk@chromium.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 20 Dec 2011 03:42:22 +0000 (03:42 +0000)
https://bugs.webkit.org/show_bug.cgi?id=74740

Reviewed by Ryosuke Niwa.

Updated lots of callers to remove third argument. The list of changes
below only lists things other than updating callers.

* dom/Document.cpp:
(WebCore::Document::importNode):
* dom/Element.cpp:
(WebCore::Element::setAttribute):
(WebCore::Element::setAttributeNS):
(WebCore::Element::setIntegralAttribute):
(WebCore::Element::setUnsignedIntegralAttribute):
* dom/Element.h: Removed third arg from method, removed old two-arg
overload which now has an identical signature.
* editing/mac/EditorMac.mm: Updated caller and used
ASSERT_NO_EXCEPTION for nearby code.
(WebCore::styleForSelectionStart):
* html/HTMLElement.cpp:
(WebCore::HTMLElement::setContentEditable):
* inspector/InspectorDOMAgent.cpp:
(WebCore::InspectorDOMAgent::setAttributesAsText):
* svg/SVGAnimationElement.cpp:
(WebCore::SVGAnimationElement::setTargetAttributeAnimatedValue):
* svg/SVGElement.idl: Removed 'setter raises(DOMException)'
* svg/SVGGlyphRefElement.idl: ditto.
* svg/SVGStyleElement.cpp:
(WebCore::SVGStyleElement::setType):
(WebCore::SVGStyleElement::setMedia):
(WebCore::SVGStyleElement::setTitle):
* xml/parser/XMLTreeBuilder.cpp:
(WebCore::XMLTreeBuilder::processAttributes):

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

12 files changed:
Source/WebCore/ChangeLog
Source/WebCore/dom/Document.cpp
Source/WebCore/dom/Element.cpp
Source/WebCore/dom/Element.h
Source/WebCore/editing/mac/EditorMac.mm
Source/WebCore/html/HTMLElement.cpp
Source/WebCore/inspector/InspectorDOMAgent.cpp
Source/WebCore/svg/SVGAnimationElement.cpp
Source/WebCore/svg/SVGElement.idl
Source/WebCore/svg/SVGGlyphRefElement.idl
Source/WebCore/svg/SVGStyleElement.cpp
Source/WebCore/xml/parser/XMLTreeBuilder.cpp

index 99e550c..3589af7 100644 (file)
@@ -1,3 +1,40 @@
+2011-12-19  Adam Klein  <adamk@chromium.org>
+
+        Remove unused ExceptionCode& argument from Element::setAttribute(QualifiedName)
+        https://bugs.webkit.org/show_bug.cgi?id=74740
+
+        Reviewed by Ryosuke Niwa.
+
+        Updated lots of callers to remove third argument. The list of changes
+        below only lists things other than updating callers.
+
+        * dom/Document.cpp:
+        (WebCore::Document::importNode):
+        * dom/Element.cpp:
+        (WebCore::Element::setAttribute):
+        (WebCore::Element::setAttributeNS):
+        (WebCore::Element::setIntegralAttribute):
+        (WebCore::Element::setUnsignedIntegralAttribute):
+        * dom/Element.h: Removed third arg from method, removed old two-arg
+        overload which now has an identical signature.
+        * editing/mac/EditorMac.mm: Updated caller and used
+        ASSERT_NO_EXCEPTION for nearby code.
+        (WebCore::styleForSelectionStart):
+        * html/HTMLElement.cpp:
+        (WebCore::HTMLElement::setContentEditable):
+        * inspector/InspectorDOMAgent.cpp:
+        (WebCore::InspectorDOMAgent::setAttributesAsText):
+        * svg/SVGAnimationElement.cpp:
+        (WebCore::SVGAnimationElement::setTargetAttributeAnimatedValue):
+        * svg/SVGElement.idl: Removed 'setter raises(DOMException)'
+        * svg/SVGGlyphRefElement.idl: ditto.
+        * svg/SVGStyleElement.cpp:
+        (WebCore::SVGStyleElement::setType):
+        (WebCore::SVGStyleElement::setMedia):
+        (WebCore::SVGStyleElement::setTitle):
+        * xml/parser/XMLTreeBuilder.cpp:
+        (WebCore::XMLTreeBuilder::processAttributes):
+
 2011-12-19  Kentaro Hara  <haraken@chromium.org>
 
         Unreviewed. Rebaselined run-bindings-tests results.
index 07de87d..b1db43d 100644 (file)
@@ -839,9 +839,7 @@ PassRefPtr<Node> Document::importNode(Node* importedNode, bool deep, ExceptionCo
             unsigned length = attrs->length();
             for (unsigned i = 0; i < length; i++) {
                 Attribute* attr = attrs->attributeItem(i);
-                newElement->setAttribute(attr->name(), attr->value().impl(), ec);
-                if (ec)
-                    return 0;
+                newElement->setAttribute(attr->name(), attr->value().impl());
             }
         }
 
index 27f3469..c69252f 100644 (file)
@@ -201,12 +201,6 @@ void Element::removeAttribute(const QualifiedName& name, ExceptionCode& ec)
     }
 }
 
-void Element::setAttribute(const QualifiedName& name, const AtomicString& value)
-{
-    ExceptionCode ec;
-    setAttribute(name, value, ec);
-}
-
 void Element::setBooleanAttribute(const QualifiedName& name, bool b)
 {
     if (b)
@@ -641,7 +635,7 @@ void Element::setAttribute(const AtomicString& name, const AtomicString& value,
     setAttributeInternal(old, old ? old->name() : QualifiedName(nullAtom, localName, nullAtom), value);
 }
 
-void Element::setAttribute(const QualifiedName& name, const AtomicString& value, ExceptionCode&)
+void Element::setAttribute(const QualifiedName& name, const AtomicString& value)
 {
     // Allocate attribute map if necessary.
     Attribute* old = attributes(false)->getAttributeItem(name);
@@ -1502,7 +1496,7 @@ void Element::setAttributeNS(const AtomicString& namespaceURI, const AtomicStrin
     if (scriptingPermission == FragmentScriptingNotAllowed && (isEventHandlerAttribute(qName) || isAttributeToRemove(qName, value)))
         return;
 
-    setAttribute(qName, value, ec);
+    setAttribute(qName, value);
 }
 
 void Element::removeAttribute(const String& name, ExceptionCode& ec)
@@ -1894,8 +1888,7 @@ int Element::getIntegralAttribute(const QualifiedName& attributeName) const
 void Element::setIntegralAttribute(const QualifiedName& attributeName, int value)
 {
     // FIXME: Need an AtomicString version of String::number.
-    ExceptionCode ec;
-    setAttribute(attributeName, String::number(value), ec);
+    setAttribute(attributeName, String::number(value));
 }
 
 unsigned Element::getUnsignedIntegralAttribute(const QualifiedName& attributeName) const
@@ -1906,8 +1899,7 @@ unsigned Element::getUnsignedIntegralAttribute(const QualifiedName& attributeNam
 void Element::setUnsignedIntegralAttribute(const QualifiedName& attributeName, unsigned value)
 {
     // FIXME: Need an AtomicString version of String::number.
-    ExceptionCode ec;
-    setAttribute(attributeName, String::number(value), ec);
+    setAttribute(attributeName, String::number(value));
 }
 
 #if ENABLE(SVG)
index 9b39cd0..ff044a4 100644 (file)
@@ -109,7 +109,7 @@ public:
 
     bool hasAttribute(const QualifiedName&) const;
     const AtomicString& getAttribute(const QualifiedName&) const;
-    void setAttribute(const QualifiedName&, const AtomicString& value, ExceptionCode&);
+    void setAttribute(const QualifiedName&, const AtomicString& value);
     void removeAttribute(const QualifiedName&, ExceptionCode&);
 
     // Typed getters and setters for language bindings.
@@ -209,8 +209,6 @@ public:
     void normalizeAttributes();
     String nodeNamePreservingCase() const;
 
-    // convenience methods which ignore exceptions
-    void setAttribute(const QualifiedName&, const AtomicString& value);
     void setBooleanAttribute(const QualifiedName& name, bool);
 
     NamedNodeMap* attributes(bool readonly = false) const;
index 27b2141..fca360d 100644 (file)
@@ -112,16 +112,12 @@ static RenderStyle* styleForSelectionStart(Frame* frame, Node *&nodeToRemove)
 
     RefPtr<Element> styleElement = frame->document()->createElement(spanTag, false);
 
-    ExceptionCode ec = 0;
     String styleText = typingStyle->style()->cssText() + " display: inline";
-    styleElement->setAttribute(styleAttr, styleText.impl(), ec);
-    ASSERT(!ec);
+    styleElement->setAttribute(styleAttr, styleText.impl());
 
-    styleElement->appendChild(frame->document()->createEditingTextNode(""), ec);
-    ASSERT(!ec);
+    styleElement->appendChild(frame->document()->createEditingTextNode(""), ASSERT_NO_EXCEPTION);
 
-    position.deprecatedNode()->parentNode()->appendChild(styleElement, ec);
-    ASSERT(!ec);
+    position.deprecatedNode()->parentNode()->appendChild(styleElement, ASSERT_NO_EXCEPTION);
 
     nodeToRemove = styleElement.get();
     return styleElement->renderer() ? styleElement->renderer()->style() : 0;
index f3106c3..6573ca6 100644 (file)
@@ -746,9 +746,9 @@ void HTMLElement::setContentEditable(Attribute* attr)
 void HTMLElement::setContentEditable(const String& enabled, ExceptionCode& ec)
 {
     if (equalIgnoringCase(enabled, "true"))
-        setAttribute(contenteditableAttr, "true", ec);
+        setAttribute(contenteditableAttr, "true");
     else if (equalIgnoringCase(enabled, "false"))
-        setAttribute(contenteditableAttr, "false", ec);
+        setAttribute(contenteditableAttr, "false");
     else if (equalIgnoringCase(enabled, "plaintext-only"))
         setAttribute(contenteditableAttr, "plaintext-only");
     else if (equalIgnoringCase(enabled, "inherit"))
index bf08f3c..7d243ef 100644 (file)
@@ -540,11 +540,7 @@ void InspectorDOMAgent::setAttributesAsText(ErrorString* errorString, int elemen
         // Add attribute pair
         const Attribute* attribute = attrMap->attributeItem(i);
         foundOriginalAttribute = foundOriginalAttribute || (name && attribute->name().toString() == *name);
-        element->setAttribute(attribute->name(), attribute->value(), ec);
-        if (ec) {
-            *errorString = "Internal error: could not set attribute value";
-            return;
-        }
+        element->setAttribute(attribute->name(), attribute->value());
     }
 
     if (!foundOriginalAttribute && name) {
index 097de38..97fbad7 100644 (file)
@@ -356,7 +356,7 @@ void SVGAnimationElement::setTargetAttributeAnimatedValue(const String& value)
     } else {
         // FIXME: This should set the 'presentation' value, not the actual 
         // attribute value. Whatever that means in practice.
-        targetElement->setAttribute(attributeName, value, ec);
+        targetElement->setAttribute(attributeName, value);
     }
     
     if (targetElement->isStyled())
@@ -372,7 +372,7 @@ void SVGAnimationElement::setTargetAttributeAnimatedValue(const String& value)
         if (attributeIsCSSProperty)
             shadowTreeElement->style()->setProperty(attributeName.localName(), value, "", ec);
         else
-            shadowTreeElement->setAttribute(attributeName, value, ec);
+            shadowTreeElement->setAttribute(attributeName, value);
         (*it)->correspondingUseElement()->setNeedsStyleRecalc();
     }
 }
@@ -630,4 +630,3 @@ void SVGAnimationElement::endedActiveInterval()
 
 }
 #endif // ENABLE(SVG)
-
index ca4fb4c..1c9ee2d 100644 (file)
@@ -26,7 +26,7 @@ module svg {
         GenerateNativeConverter,
         Conditional=SVG
     ] SVGElement : Element {
-        attribute [Reflect] DOMString id setter raises(DOMException);
+        attribute [Reflect] DOMString id;
         attribute [ConvertNullToNullString] DOMString xmlbase setter raises(DOMException);
         readonly attribute SVGSVGElement ownerSVGElement;
         readonly attribute SVGElement viewportElement;
index 6d7af76..8a1cdb1 100644 (file)
@@ -25,8 +25,7 @@ module svg {
         // FIXME: Use [Reflect] after https://bugs.webkit.org/show_bug.cgi?id=64843 is fixed.
         attribute DOMString glyphRef
             setter raises(DOMException);
-        attribute [Reflect] DOMString format
-            setter raises(DOMException);
+        attribute [Reflect] DOMString format;
         attribute float x
             setter raises(DOMException);
         attribute float y
index b3844bd..cec0378 100644 (file)
@@ -58,9 +58,9 @@ const AtomicString& SVGStyleElement::type() const
     return n.isNull() ? defaultValue : n;
 }
 
-void SVGStyleElement::setType(const AtomicString& type, ExceptionCode& ec)
+void SVGStyleElement::setType(const AtomicString& type, ExceptionCode&)
 {
-    setAttribute(SVGNames::typeAttr, type, ec);
+    setAttribute(SVGNames::typeAttr, type);
 }
 
 const AtomicString& SVGStyleElement::media() const
@@ -70,9 +70,9 @@ const AtomicString& SVGStyleElement::media() const
     return n.isNull() ? defaultValue : n;
 }
 
-void SVGStyleElement::setMedia(const AtomicString& media, ExceptionCode& ec)
+void SVGStyleElement::setMedia(const AtomicString& media, ExceptionCode&)
 {
-    setAttribute(SVGNames::mediaAttr, media, ec);
+    setAttribute(SVGNames::mediaAttr, media);
 }
 
 String SVGStyleElement::title() const
@@ -80,9 +80,9 @@ String SVGStyleElement::title() const
     return fastGetAttribute(SVGNames::titleAttr);
 }
 
-void SVGStyleElement::setTitle(const AtomicString& title, ExceptionCode& ec)
+void SVGStyleElement::setTitle(const AtomicString& title, ExceptionCode&)
 {
-    setAttribute(SVGNames::titleAttr, title, ec);
+    setAttribute(SVGNames::titleAttr, title);
 }
 
 bool SVGStyleElement::isSupportedAttribute(const QualifiedName& attrName)
index e3d793e..6476a1d 100644 (file)
@@ -324,7 +324,7 @@ void XMLTreeBuilder::processAttributes(const AtomicXMLToken& token, NodeStackIte
             newElement->setAttributeNS(XMLNSNames::xmlnsNamespaceURI, xmlnsAtom, attribute->value(), ec);
         else {
             QualifiedName qName(attribute->prefix(), attribute->localName(), stackItem.namespaceForPrefix(attribute->prefix(), nullAtom));
-            newElement->setAttribute(qName, attribute->value(), ec);
+            newElement->setAttribute(qName, attribute->value());
         }
         if (ec) {
             m_parser->stopParsing();