Attachment 358569 Details for Bug 193227 – Patch

[patch] Patch

193227.diff (text/plain), 10.04 KB, created by Devin Rousso on 2019-01-07 21:32:20 PST

(hide)

Description:

Filename:

MIME Type:

Creator: Devin Rousso

Created: 2019-01-07 21:32:20 PST

Size: 10.04 KB

patch

obsolete

>diff --git a/Source/WebCore/ChangeLog b/Source/WebCore/ChangeLog
>index 4fd67da18d9..db194a0fea8 100644
>--- a/Source/WebCore/ChangeLog
>+++ b/Source/WebCore/ChangeLog
>@@ -1,3 +1,19 @@
>+2019-01-07  Devin Rousso  <drousso@apple.com>
>+
>+        Web Inspector: Audit: provide a way to get related Accessibility properties for a given node
>+        https://bugs.webkit.org/show_bug.cgi?id=193227
>+        <rdar://problem/46787862>
>+
>+        Reviewed by NOBODY (OOPS!).
>+
>+        No new tests (OOPS!).
>+
>+        * inspector/InspectorAuditAccessibilityUtilities.idl:
>+        * inspector/InspectorAuditAccessibilityUtilities.h:
>+        * inspector/InspectorAuditAccessibilityUtilities.cpp:
>+        (WebCore::accessiblityObjectForNode): Added.
>+        (WebCore::InspectorAuditAccessibilityUtilities::getComputedProperties): Added.
>+
> 2019-01-06  Devin Rousso  <drousso@apple.com>
> 
>         Web Inspector: Audit: create new IDL type for exposing special functionality in test context
>diff --git a/Source/WebCore/inspector/InspectorAuditAccessibilityUtilities.cpp b/Source/WebCore/inspector/InspectorAuditAccessibilityUtilities.cpp
>index 0e4ebf85693..0d9fa21859f 100644
>--- a/Source/WebCore/inspector/InspectorAuditAccessibilityUtilities.cpp
>+++ b/Source/WebCore/inspector/InspectorAuditAccessibilityUtilities.cpp
>@@ -27,10 +27,147 @@
> #include "config.h"
> #include "InspectorAuditAccessibilityUtilities.h"
> 
>+#include "AXObjectCache.h"
>+#include "AccessibilityObject.h"
>+#include "Element.h"
>+#include "Node.h"
>+#include "SpaceSplitString.h"
>+#include <wtf/Vector.h>
>+#include <wtf/text/WTFString.h>
>+
> namespace WebCore {
> 
> InspectorAuditAccessibilityUtilities::InspectorAuditAccessibilityUtilities()
> {
> }
> 
>+static AccessibilityObject* accessiblityObjectForNode(Node& node)
>+{
>+    if (!AXObjectCache::accessibilityEnabled())
>+        AXObjectCache::enableAccessibility();
>+
>+    if (AXObjectCache* axObjectCache = node.document().axObjectCache())
>+        return axObjectCache->getOrCreate(&node);
>+    return nullptr;
>+}
>+
>+Optional<InspectorAuditAccessibilityUtilities::ComputedProperties> InspectorAuditAccessibilityUtilities::getComputedProperties(Node& node)
>+{
>+    if (AccessibilityObject* axObject = accessiblityObjectForNode(node)) {
>+        ComputedProperties computedProperties;
>+
>+        AccessibilityObject* current = axObject;
>+        while (current && (!computedProperties.busy || !computedProperties.busy.value())) {
>+            computedProperties.busy = current->isBusy();
>+            current = current->parentObject();
>+        }
>+
>+        if (axObject->supportsChecked()) {
>+            AccessibilityButtonState checkValue = axObject->checkboxOrRadioValue();
>+            if (checkValue == AccessibilityButtonState::On)
>+                computedProperties.checked = "true"_s;
>+            else if (checkValue == AccessibilityButtonState::Mixed)
>+                computedProperties.checked = "mixed"_s;
>+            else if (axObject->isChecked())
>+                computedProperties.checked = "true"_s;
>+            else
>+                computedProperties.checked = "false"_s;
>+        }
>+
>+        switch (axObject->currentState()) {
>+        case AccessibilityCurrentState::False:
>+            computedProperties.currentState = "false"_s;
>+            break;
>+        case AccessibilityCurrentState::True:
>+            computedProperties.currentState = "true"_s;
>+            break;
>+        case AccessibilityCurrentState::Page:
>+            computedProperties.currentState = "page"_s;
>+            break;
>+        case AccessibilityCurrentState::Step:
>+            computedProperties.currentState = "step"_s;
>+            break;
>+        case AccessibilityCurrentState::Location:
>+            computedProperties.currentState = "location"_s;
>+            break;
>+        case AccessibilityCurrentState::Date:
>+            computedProperties.currentState = "date"_s;
>+            break;
>+        case AccessibilityCurrentState::Time:
>+            computedProperties.currentState = "time"_s;
>+            break;
>+        }
>+
>+        computedProperties.disabled = !axObject->isEnabled();
>+
>+        if (axObject->supportsExpanded())
>+            computedProperties.expanded = axObject->isExpanded();
>+
>+        if (is<Element>(node) && axObject->canSetFocusAttribute())
>+            computedProperties.focused = axObject->isFocused();
>+
>+        computedProperties.headingLevel = axObject->headingLevel();
>+        computedProperties.hidden = axObject->isAXHidden() || axObject->isDOMHidden();
>+        computedProperties.hierarchicalLevel = axObject->hierarchicalLevel();
>+        computedProperties.ignored = axObject->accessibilityIsIgnored();
>+        computedProperties.ignoredByDefault = axObject->accessibilityIsIgnoredByDefault();
>+
>+        String invalidValue = axObject->invalidStatus();
>+        if (invalidValue == "false")
>+            computedProperties.invalidStatus = "false"_s;
>+        else if (invalidValue == "grammar")
>+            computedProperties.invalidStatus = "grammar"_s;
>+        else if (invalidValue == "spelling")
>+            computedProperties.invalidStatus = "spelling"_s;
>+        else
>+            computedProperties.invalidStatus = "true"_s;
>+
>+        computedProperties.isPopUpButton = axObject->isPopUpButton() || axObject->hasPopup();
>+        computedProperties.label = axObject->computedLabel();
>+
>+        if (axObject->supportsLiveRegion()) {
>+            computedProperties.liveRegionAtomic = axObject->liveRegionAtomic();
>+
>+            String ariaRelevantAttrValue = axObject->liveRegionRelevant();
>+            if (!ariaRelevantAttrValue.isEmpty()) {
>+                Vector<String> liveRegionRelevant;
>+                String ariaRelevantAdditions = "additions";
>+                String ariaRelevantRemovals = "removals";
>+                String ariaRelevantText = "text";
>+
>+                const auto& values = SpaceSplitString(ariaRelevantAttrValue, true);
>+                if (values.contains("all")) {
>+                    liveRegionRelevant.append(ariaRelevantAdditions);
>+                    liveRegionRelevant.append(ariaRelevantRemovals);
>+                    liveRegionRelevant.append(ariaRelevantText);
>+                } else {
>+                    if (values.contains(ariaRelevantAdditions))
>+                        liveRegionRelevant.append(ariaRelevantAdditions);
>+                    if (values.contains(ariaRelevantRemovals))
>+                        liveRegionRelevant.append(ariaRelevantRemovals);
>+                    if (values.contains(ariaRelevantText))
>+                        liveRegionRelevant.append(ariaRelevantText);
>+                }
>+                computedProperties.liveRegionRelevant = liveRegionRelevant;
>+            }
>+
>+            computedProperties.liveRegionStatus = axObject->liveRegionStatus();
>+        }
>+
>+        computedProperties.pressed = axObject->pressedIsPresent() && axObject->isPressed();
>+
>+        if (axObject->isTextControl())
>+            computedProperties.readonly = !axObject->canSetValueAttribute();
>+
>+        if (axObject->supportsRequiredAttribute())
>+            computedProperties.required = axObject->isRequired();
>+
>+        computedProperties.role = axObject->computedRoleString();
>+        computedProperties.selected = axObject->isSelected();
>+
>+        return computedProperties;
>+    }
>+    return WTF::nullopt;
>+}
>+
> } // namespace WebCore
>diff --git a/Source/WebCore/inspector/InspectorAuditAccessibilityUtilities.h b/Source/WebCore/inspector/InspectorAuditAccessibilityUtilities.h
>index a770a8087a4..c208de8e5b0 100644
>--- a/Source/WebCore/inspector/InspectorAuditAccessibilityUtilities.h
>+++ b/Source/WebCore/inspector/InspectorAuditAccessibilityUtilities.h
>@@ -25,11 +25,15 @@
> 
> #pragma once
> 
>+#include <wtf/Forward.h>
>+#include <wtf/Optional.h>
> #include <wtf/Ref.h>
> #include <wtf/RefCounted.h>
> 
> namespace WebCore {
> 
>+class Node;
>+
> class InspectorAuditAccessibilityUtilities : public RefCounted<InspectorAuditAccessibilityUtilities> {
> public:
>     static Ref<InspectorAuditAccessibilityUtilities> create()
>@@ -37,6 +41,32 @@ public:
>         return adoptRef(*new InspectorAuditAccessibilityUtilities());
>     }
> 
>+    struct ComputedProperties {
>+        Optional<bool> busy;
>+        String checked;
>+        String currentState;
>+        Optional<bool> disabled;
>+        Optional<bool> expanded;
>+        Optional<bool> focused;
>+        Optional<int> headingLevel;
>+        Optional<bool> hidden;
>+        Optional<int> hierarchicalLevel;
>+        Optional<bool> ignored;
>+        Optional<bool> ignoredByDefault;
>+        String invalidStatus;
>+        Optional<bool> isPopUpButton;
>+        String label;
>+        Optional<bool> liveRegionAtomic;
>+        Optional<Vector<String>> liveRegionRelevant;
>+        String liveRegionStatus;
>+        Optional<bool> pressed;
>+        Optional<bool> readonly;
>+        Optional<bool> required;
>+        String role;
>+        Optional<bool> selected;
>+    };
>+    Optional<ComputedProperties> getComputedProperties(Node&);
>+
> private:
>     InspectorAuditAccessibilityUtilities();
> };
>diff --git a/Source/WebCore/inspector/InspectorAuditAccessibilityUtilities.idl b/Source/WebCore/inspector/InspectorAuditAccessibilityUtilities.idl
>index 7ccd72bb093..739be98ae4a 100644
>--- a/Source/WebCore/inspector/InspectorAuditAccessibilityUtilities.idl
>+++ b/Source/WebCore/inspector/InspectorAuditAccessibilityUtilities.idl
>@@ -28,4 +28,32 @@
>     JSGenerateToJSObject,
>     NoInterfaceObject,
> ] interface InspectorAuditAccessibilityUtilities {
>+    ComputedProperties? getComputedProperties(Node node);
>+};
>+
>+[
>+    JSGenerateToJSObject,
>+] dictionary ComputedProperties {
>+    boolean? busy;
>+    DOMString? checked;
>+    DOMString? currentState;
>+    boolean? disabled;
>+    boolean? expanded;
>+    boolean? focused;
>+    long? headingLevel;
>+    boolean? hidden;
>+    long? hierarchicalLevel;
>+    boolean? ignored;
>+    boolean? ignoredByDefault;
>+    DOMString? invalidStatus;
>+    boolean? isPopUpButton;
>+    DOMString? label;
>+    boolean? liveRegionAtomic;
>+    sequence<DOMString>? liveRegionRelevant;
>+    DOMString? liveRegionStatus;
>+    boolean? pressed;
>+    boolean? readonly;
>+    boolean? required;
>+    DOMString? role;
>+    boolean? selected;
> };

Flags:

hi: commit-queue-

Actions: View | Formatted Diff | Diff

Attachments on bug 193227: 358569 | 358853 | 359775 | 359783