Commit 99dc179c authored by adamk@chromium.org's avatar adamk@chromium.org

Initialize ExceptionCode in Element::removeAttribute

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

Reviewed by Darin Adler.

Silences valgrind warning reported in http://crbug.com/76490.

No new tests since this would only very occasionally be flaky,
and in the codepath in the valgrind report, the ec is ignored anyway.

* dom/Element.cpp:
(WebCore::Element::removeAttribute): Initialize ec to 0.


git-svn-id: http://svn.webkit.org/repository/webkit/trunk@94869 268f45cc-cd09-0410-ab3c-d52691b4dbfc
parent adbe4011
2011-09-09 Adam Klein <adamk@chromium.org>
Initialize ExceptionCode in Element::removeAttribute
https://bugs.webkit.org/show_bug.cgi?id=67820
Reviewed by Darin Adler.
Silences valgrind warning reported in http://crbug.com/76490.
No new tests since this would only very occasionally be flaky,
and in the codepath in the valgrind report, the ec is ignored anyway.
* dom/Element.cpp:
(WebCore::Element::removeAttribute): Initialize ec to 0.
2011-09-09 Laszlo Gombos <laszlo.1.gombos@nokia.com>
[Qt] Remove common.pri
......@@ -1520,6 +1520,7 @@ void Element::removeAttribute(const String& name, ExceptionCode& ec)
String localName = shouldIgnoreAttributeCase(this) ? name.lower() : name;
if (m_attributeMap) {
ec = 0;
m_attributeMap->removeNamedItem(localName, ec);
if (ec == NOT_FOUND_ERR)
ec = 0;
......
Markdown is supported
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment