Correctly handle accessing a replaced Attr object's attribute value.

Source/core/html/HTMLSelectElement.cpp

Index: Source/core/html/HTMLSelectElement.cpp
diff --git a/Source/core/html/HTMLSelectElement.cpp b/Source/core/html/HTMLSelectElement.cpp
index 812d4a447090b56997276d374bf386d1fd0697dd..669067f6cf6147b25115a7b2508eeed271c5fb64 100644
--- a/Source/core/html/HTMLSelectElement.cpp
+++ b/Source/core/html/HTMLSelectElement.cpp
@@ -323,7 +323,7 @@ void HTMLSelectElement::parseAttribute(const QualifiedName& name, const AtomicSt
         AtomicString attrSize = AtomicString::number(size);
         if (attrSize != value) {
             // FIXME: This is horribly factored.
-            if (Attribute* sizeAttribute = ensureUniqueElementData().getAttributeItem(sizeAttr))
+            if (Attribute* sizeAttribute = ensureUniqueElementData().getAttributeItem(sizeAttr, true))

[Inactive, 2014/06/03 15:11:39]

Gosh, another boolean argument :) Could we make this an enum so that the code
becomes readable?

e.g. getAttributeItem(sizeAttr, IgnoreCase)

[sof, 2014/06/03 15:21:50]

That would break with the convention followed elsewhere for encoding match
case-sensitivity.

                 sizeAttribute->setValue(attrSize);
         }
         size = max(size, 1);