Changed accessibility properies to meet W3C standard.

Source/modules/accessibility/AXNodeObject.cpp

 /*
 * Copyright (C) 2012, Google 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 223 matching lines @@
         if (type == InputTypeNames::range)
             return SliderRole;
         if (type == InputTypeNames::color)
             return ColorWellRole;
         if (type == InputTypeNames::time)
             return TimeRole;
         return TextFieldRole;
     }
     if (isHTMLSelectElement(*node())) {
         HTMLSelectElement& selectElement = toHTMLSelectElement(*node());
-        return selectElement.multiple() ? ListBoxRole : PopUpButtonRole;
+        return (selectElement.multiple() || selectElement.size() > 1) ? ListBoxRole : PopUpButtonRole;
     }
     if (isHTMLTextAreaElement(*node()))
         return TextAreaRole;
     if (headingLevel())
         return HeadingRole;
     if (isHTMLDivElement(*node()))
         return DivRole;
     if (isHTMLMeterElement(*node()))
         return MeterRole;
     if (isHTMLOutputElement(*node()))
@@ skipping 928 matching lines @@
 
     if (isHTMLSelectElement(*node)) {
         HTMLSelectElement& selectElement = toHTMLSelectElement(*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;
         }
-        if (!selectElement.multiple())
-            return selectElement.value();
-        return String();
+        return selectElement.value();
     }
 
     if (isTextControl())
         return text();
 
+    // Handle other HTML input elements that aren't text controls, like date and time
+    // controls, by returning the string value, with the exception of checkboxes
+    // and radio buttons (which would return "on").
+    if (node && isHTMLInputElement(node)) {
+        HTMLInputElement* input = toHTMLInputElement(node);
+        if (input->type() != InputTypeNames::checkbox && input->type() != InputTypeNames::radio)
+            return input->value();

[dmazzoni, 2015/01/29 16:28:29]

Thanks, this change looks good.

+    }
+
     // FIXME: We might need to implement a value here for more types
     // FIXME: It would be better not to advertise a value at all for the types for which we don't implement one;
     // this would require subclassing or making accessibilityAttributeNames do something other than return a
     // single static array.
     return String();
 }
 
 
 const AtomicString& AXNodeObject::textInputType() const
 {
@@ skipping 762 matching lines @@
     float range = maxValueForRange() - minValueForRange();
     float value = valueForRange();
 
     value += range * (percentChange / 100);
     setValue(String::number(value));
 
     axObjectCache()->postNotification(node(), AXObjectCacheImpl::AXValueChanged, true);
 }
 
 } // namespace blink