Fixing mapping conflicts for required html attribute and aria-required

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 728 matching lines @@
         HTMLInputElement& input = toHTMLInputElement(*node);
         if (input.isTextField())
             return input.isReadOnly();
     }
 
     return !node->hasEditableStyle();
 }
 
 bool AXNodeObject::isRequired() const
 {
-    if (equalIgnoringCase(getAttribute(aria_requiredAttr), "true"))
-        return true;
-
     Node* n = this->node();
     if (n && (n->isElementNode() && toElement(n)->isFormControlElement()))
         return toHTMLFormControlElement(n)->isRequired();
 
+    if (equalIgnoringCase(getAttribute(aria_requiredAttr), "true"))

[shreeramk, 2014/12/05 07:32:54]

Spec says http://w3c.github.io/aria/aria/aria.html#aria-required

"Unless an exactly equivalent native attribute is available, host languages
should allow authors to use the aria-required attribute on host language form
elements that require input or selection by the user."

As we know html "required" attribute exists, then I think, in that scenario we
should give first priority to required attribute and then to aria-required
attribute for form elements.

Also for form elements we should first check for html required attribute then
check for aria-required attribute.

+        return true;
+
     return false;
 }
 
 bool AXNodeObject::canSetFocusAttribute() const
 {
     Node* node = this->node();
     if (!node)
         return false;
 
     if (isWebArea())
@@ skipping 975 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