Changed accessibility properies to meet W3C standard.

Source/modules/accessibility/AXRenderObject.cpp

 /*
 * Copyright (C) 2008 Apple Inc. All rights reserved.
 *
 * Redistribution and use in source and binary forms, with or without
 * modification, are permitted provided that the following conditions
 * are met:
 *
 * 1.  Redistributions of source code must retain the above copyright
 *     notice, this list of conditions and the following disclaimer.
 * 2.  Redistributions in binary form must reproduce the above copyright
@@ skipping 878 matching lines @@
 }
 
 String AXRenderObject::stringValue() const
 {
     if (!m_renderer)
         return String();
 
     if (isPasswordFieldAndShouldHideValue())
         return String();
 
-    RenderBoxModelObject* cssBox = renderBoxModelObject();
 
     if (ariaRoleAttribute() == StaticTextRole) {
         String staticText = text();
         if (!staticText.length())
             staticText = textUnderElement();
         return staticText;
     }
 
     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());
         int selectedIndex = selectElement->selectedIndex();
         const WillBeHeapVector<RawPtrWillBeMember<HTMLElement>>& listItems = selectElement->listItems();
         if (selectedIndex >= 0 && static_cast<size_t>(selectedIndex) < listItems.size()) {
             const AtomicString& overriddenDescription = listItems[selectedIndex]->fastGetAttribute(aria_labelAttr);
             if (!overriddenDescription.isNull())
                 return overriddenDescription;
         }
-        return toRenderMenuList(m_renderer)->text();
+        return selectElement->value();
     }
 
     if (m_renderer->isListMarker())
         return toRenderListMarker(m_renderer)->text();
 
     if (isWebArea()) {
         // FIXME: Why would a renderer exist when the Document isn't attached to a frame?
         if (m_renderer->frame())
             return String();
 
@@ skipping 1487 matching lines @@
         if (label && label->renderer()) {
             LayoutRect labelRect = axObjectCache()->getOrCreate(label)->elementRect();
             result.unite(labelRect);
         }
     }
 
     return result;
 }
 
 } // namespace blink