Changing slider values do not work properly when percent change of a step is less than one.

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 110 matching lines @@
             builder.append(' ');
     }
     return builder.toString();
 }
 
 void AXNodeObject::alterSliderValue(bool increase)
 {
     if (roleValue() != SliderRole)
         return;
 
-    if (!getAttribute(stepAttr).isEmpty())
-        changeValueByStep(increase);
-    else
-        changeValueByPercent(increase ? 5 : -5);
+    float value = valueForRange();
+    float step = stepValueForRange();
+
+    value += increase ? step : -step;
+
+    setValue(String::number(value));
+    axObjectCache()->postNotification(node(), AXObjectCacheImpl::AXValueChanged);
 }
 
 String AXNodeObject::ariaAccessibilityDescription() const
 {
     String ariaLabeledBy = ariaLabeledByAttribute();
     if (!ariaLabeledBy.isEmpty())
         return ariaLabeledBy;
 
     const AtomicString& ariaLabel = getAttribute(aria_labelAttr);
     if (!ariaLabel.isEmpty())
         return ariaLabel;
 
     return String();
 }
 
 
 void AXNodeObject::ariaLabeledByElements(WillBeHeapVector<RawPtrWillBeMember<Element>>& elements) const
 {
     elementsFromAttribute(elements, aria_labeledbyAttr);
     if (!elements.size())
         elementsFromAttribute(elements, aria_labelledbyAttr);
 }
 
-void AXNodeObject::changeValueByStep(bool increase)
-{
-    float step = stepValueForRange();
-    float value = valueForRange();
-
-    value += increase ? step : -step;
-
-    setValue(String::number(value));
-
-    axObjectCache()->postNotification(node(), AXObjectCacheImpl::AXValueChanged);
-}
-
 bool AXNodeObject::computeAccessibilityIsIgnored() const
 {
 #if ENABLE(ASSERT)
     // Double-check that an AXObject is never accessed before
     // it's been initialized.
     ASSERT(m_initialized);
 #endif
 
     // If this element is within a parent that cannot have children, it should not be exposed.
     if (isDescendantOfBarrenParent())
@@ skipping 583 matching lines @@
 bool AXNodeObject::isProgressIndicator() const
 {
     return roleValue() == ProgressIndicatorRole;
 }
 
 bool AXNodeObject::isSlider() const
 {
     return roleValue() == SliderRole;
 }
 
+bool AXNodeObject::isNativeSlider() const
+{
+    Node* node = this->node();
+    if (!node)
+        return false;
+
+    if (!isHTMLInputElement(node))
+        return false;
+
+    return toHTMLInputElement(node)->type() == InputTypeNames::range;
+}
+
 bool AXNodeObject::isChecked() const
 {
     Node* node = this->node();
     if (!node)
         return false;
 
     // First test for native checkedness semantics
     if (isHTMLInputElement(*node))
         return toHTMLInputElement(*node).shouldAppearChecked();
 
@@ skipping 420 matching lines @@
         return toHTMLMeterElement(*node()).value();
 
     return 0.0;
 }
 
 float AXNodeObject::maxValueForRange() const
 {
     if (hasAttribute(aria_valuemaxAttr))
         return getAttribute(aria_valuemaxAttr).toFloat();
 
-    if (isHTMLInputElement(node())) {
-        HTMLInputElement& input = toHTMLInputElement(*node());
-        if (input.type() == InputTypeNames::range)
-            return input.maximum();
-    }
+    if (isNativeSlider())
+        return toHTMLInputElement(*node()).maximum();
 
     if (isHTMLMeterElement(node()))
         return toHTMLMeterElement(*node()).max();
 
     return 0.0;
 }
 
 float AXNodeObject::minValueForRange() const
 {
     if (hasAttribute(aria_valueminAttr))
         return getAttribute(aria_valueminAttr).toFloat();
 
-    if (isHTMLInputElement(node())) {
-        HTMLInputElement& input = toHTMLInputElement(*node());
-        if (input.type() == InputTypeNames::range)
-            return input.minimum();
-    }
+    if (isNativeSlider())
+        return toHTMLInputElement(*node()).minimum();
 
     if (isHTMLMeterElement(node()))
         return toHTMLMeterElement(*node()).min();
 
     return 0.0;
 }
 
 float AXNodeObject::stepValueForRange() const
 {
-    return getAttribute(stepAttr).toFloat();
+    if (!isNativeSlider())
+        return 0.0;
+
+    Decimal step = toHTMLInputElement(*node()).createStepRange(RejectAny).step();
+    return step.toString().toFloat();
 }
 
 String AXNodeObject::stringValue() const
 {
     Node* node = this->node();
     if (!node)
         return String();
 
     if (ariaRoleAttribute() == StaticTextRole) {
         String staticText = text();
@@ skipping 815 matching lines @@
         WillBeHeapVector<RawPtrWillBeMember<Element>> elements;
         ariaLabeledByElements(elements);
 
         for (const auto& element : elements) {
             RefPtr<AXObject> axElement = axObjectCache()->getOrCreate(element);
             textOrder.append(AccessibilityText(ariaLabeledBy, AlternativeText, axElement));
         }
     }
 }
 
-void AXNodeObject::changeValueByPercent(float percentChange)
-{
-    float range = maxValueForRange() - minValueForRange();
-    float value = valueForRange();
-
-    value += range * (percentChange / 100);
-    setValue(String::number(value));
-
-    axObjectCache()->postNotification(node(), AXObjectCacheImpl::AXValueChanged);
-}
-
 } // namespace blink