<meter> element AXValue is listed as a writable value
authorcommit-queue@webkit.org <commit-queue@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 12 Sep 2013 22:41:43 +0000 (22:41 +0000)
committercommit-queue@webkit.org <commit-queue@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 12 Sep 2013 22:41:43 +0000 (22:41 +0000)
https://bugs.webkit.org/show_bug.cgi?id=117650

Patch by Samuel White <samuel_white@apple.com> on 2013-09-12
Reviewed by Chris Fleizach.

Source/WebCore:

No new tests, updated existing accessibility/meter-element.html test
to check writability. Changed meter element value to be unwritable.

* accessibility/AccessibilityRenderObject.cpp:
(WebCore::AccessibilityRenderObject::canSetValueAttribute):

LayoutTests:

Added writability check to meter element test.

* accessibility/meter-element.html:
* platform/mac/accessibility/meter-element-expected.txt:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@155653 268f45cc-cd09-0410-ab3c-d52691b4dbfc

LayoutTests/ChangeLog
LayoutTests/accessibility/meter-element.html
LayoutTests/platform/mac/accessibility/meter-element-expected.txt
Source/WebCore/ChangeLog
Source/WebCore/accessibility/AccessibilityRenderObject.cpp

index aacf84a..c9b7f02 100644 (file)
@@ -1,3 +1,15 @@
+2013-09-12  Samuel White  <samuel_white@apple.com>
+
+        <meter> element AXValue is listed as a writable value
+        https://bugs.webkit.org/show_bug.cgi?id=117650
+
+        Reviewed by Chris Fleizach.
+
+        Added writability check to meter element test.
+
+        * accessibility/meter-element.html:
+        * platform/mac/accessibility/meter-element-expected.txt:
+
 2013-09-12  Renata Hodovan  <reni@webkit.org>
 
         [Qt] SHOULD NEVER BE REACHED is touched WebCore::InputType::createStepRange
index ef68f59..9469073 100644 (file)
@@ -26,6 +26,7 @@ if (window.testRunner && window.accessibilityController) {
         debug(meter.title);
         debug(meter.description);
         debug(meter.valueDescription);
+        debug("AXValueSettable: " + meter.isAttributeSettable("AXValue"));
         debug("\n");
     }
 
index 3010e2a..3de3774 100644 (file)
@@ -8,6 +8,7 @@ AXRole: AXProgressIndicator
 AXTitle: 
 AXDescription: 
 AXValueDescription: 6 blocks used (out of 8 total)
+AXValueSettable: false
 
 
 Meter2
@@ -15,6 +16,7 @@ AXRole: AXProgressIndicator
 AXTitle: 
 AXDescription: 
 AXValueDescription: 75%
+AXValueSettable: false
 
 
 Meter3
@@ -22,6 +24,7 @@ AXRole: AXProgressIndicator
 AXTitle: 
 AXDescription: 
 AXValueDescription: 
+AXValueSettable: false
 
 
 Meter4
@@ -29,6 +32,7 @@ AXRole: AXProgressIndicator
 AXTitle: 
 AXDescription: 
 AXValueDescription: 12cm
+AXValueSettable: false
 
 
 Meter5
@@ -36,6 +40,7 @@ AXRole: AXProgressIndicator
 AXTitle: 
 AXDescription: 
 AXValueDescription: 2cm
+AXValueSettable: false
 
 
 Meter6
@@ -43,6 +48,7 @@ AXRole: AXProgressIndicator
 AXTitle: 
 AXDescription: 
 AXValueDescription: 12cm
+AXValueSettable: false
 
 
 Meter7
@@ -50,6 +56,7 @@ AXRole: AXProgressIndicator
 AXTitle: 
 AXDescription: 
 AXValueDescription: 2cm
+AXValueSettable: false
 
 
 PASS successfullyParsed is true
index 013282c..a72a87f 100644 (file)
@@ -1,3 +1,16 @@
+2013-09-12  Samuel White  <samuel_white@apple.com>
+
+        <meter> element AXValue is listed as a writable value
+        https://bugs.webkit.org/show_bug.cgi?id=117650
+
+        Reviewed by Chris Fleizach.
+
+        No new tests, updated existing accessibility/meter-element.html test
+        to check writability. Changed meter element value to be unwritable.
+
+        * accessibility/AccessibilityRenderObject.cpp:
+        (WebCore::AccessibilityRenderObject::canSetValueAttribute):
+
 2013-09-12  Renata Hodovan  <reni@webkit.org>
 
         [Qt] SHOULD NEVER BE REACHED is touched WebCore::InputType::createStepRange
index cac17b6..c1dc41b 100644 (file)
@@ -2670,6 +2670,9 @@ bool AccessibilityRenderObject::canSetValueAttribute() const
     if (isNativeTextControl())
         return !isReadOnly();
 
+    if (isMeter())
+        return false;
+
     if (equalIgnoringCase(getAttribute(aria_readonlyAttr), "true"))
         return false;