Changed accessibility properies to meet W3C standard.

Source/modules/accessibility/AXRenderObject.cpp

Index: Source/modules/accessibility/AXRenderObject.cpp
diff --git a/Source/modules/accessibility/AXRenderObject.cpp b/Source/modules/accessibility/AXRenderObject.cpp
index a7c75232694e36186a0b8961e8e66d5df2346858..0f83e0032cd8a4b74599b9bd04a61661c6e57662 100644
--- a/Source/modules/accessibility/AXRenderObject.cpp
+++ b/Source/modules/accessibility/AXRenderObject.cpp
@@ -896,7 +896,6 @@ String AXRenderObject::stringValue() const
     if (isPasswordFieldAndShouldHideValue())
         return String();
 
-    RenderBoxModelObject* cssBox = renderBoxModelObject();
 
     if (ariaRoleAttribute() == StaticTextRole) {
         String staticText = text();
@@ -908,7 +907,7 @@ String AXRenderObject::stringValue() const
     if (m_renderer->isText())
         return textUnderElement();
 
-    if (cssBox && cssBox->isMenuList()) {
+    if (isHTMLSelectElement(m_renderer->node())) {

[dmazzoni, 2015/02/18 19:01:38]

As I commented before, I don't think we should be making this change, I think
using cssBox->isMenuList is correct for AXRenderObject.cpp.

If I'm wrong, please explain why.

         // RenderMenuList will go straight to the text() of its selected item.
         // This has to be overridden in the case where the selected item has an ARIA label.
         HTMLSelectElement* selectElement = toHTMLSelectElement(m_renderer->node());
@@ -919,7 +918,7 @@ String AXRenderObject::stringValue() const
             if (!overriddenDescription.isNull())
                 return overriddenDescription;
         }
-        return toRenderMenuList(m_renderer)->text();
+        return selectElement->value();
     }
 
     if (m_renderer->isListMarker())