IA2 role is not correct for input type attribute in button & checkbox state with aria-role menu as …

Source/core/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 186 matching lines @@
     if (isHTMLSummaryElement(*node())) {
         if (node()->parentNode() && isHTMLDetailsElement(node()->parentNode()))
             return DisclosureTriangleRole;
         return UnknownRole;
     }
 
     if (isHTMLInputElement(*node())) {
         HTMLInputElement& input = toHTMLInputElement(*node());
         const AtomicString& type = input.type();
         if (type == InputTypeNames::button) {
-            if (node()->parentNode() && isHTMLMenuElement(node()->parentNode()))
+            if (node()->parentNode() && (isHTMLMenuElement(node()->parentNode()) || parentObject()->roleValue() == MenuRole))

[dmazzoni, 2014/10/06 15:25:43]

It's possible parentObject() could return null, so you should check that too

                 return MenuItemRole;
             return buttonRoleType();
         }
         if (type == InputTypeNames::checkbox) {
-            if (node()->parentNode() && isHTMLMenuElement(node()->parentNode()))
+            if (node()->parentNode() && (isHTMLMenuElement(node()->parentNode()) || parentObject()->roleValue() == MenuRole))
                 return CheckBoxMenuItemRole;
             return CheckBoxRole;
         }
         if (type == InputTypeNames::date)
             return DateRole;
         if (type == InputTypeNames::datetime
             || type == InputTypeNames::datetime_local
             || type == InputTypeNames::month
             || type == InputTypeNames::week)
             return DateTimeRole;
@@ skipping 1514 matching lines @@
     float range = maxValueForRange() - minValueForRange();
     float value = valueForRange();
 
     value += range * (percentChange / 100);
     setValue(String::number(value));
 
     axObjectCache()->postNotification(node(), AXObjectCache::AXValueChanged, true);
 }
 
 } // namespace blink