Attachment 359148 Details for Bug 193228 – Patch

[patch] Patch

bug-193228-20190115014137.patch (text/plain), 12.10 KB, created by Devin Rousso on 2019-01-15 01:41:38 PST

(hide)

Description:

Filename:

MIME Type:

Creator: Devin Rousso

Created: 2019-01-15 01:41:38 PST

Size: 12.10 KB

patch

obsolete

>diff --git a/Source/WebCore/ChangeLog b/Source/WebCore/ChangeLog
>index 7df1688a0e9beb1d10452ea5a3e0c3f79c18c8b8..6e2455a2aaf4e5df5633e1e04ac7116c555bc728 100644
>--- a/Source/WebCore/ChangeLog
>+++ b/Source/WebCore/ChangeLog
>@@ -1,3 +1,19 @@
>+2019-01-15  Devin Rousso  <drousso@apple.com>
>+
>+        Web Inspector: Audit: provide a way to query for all nodes with a given computed Accessibility role
>+        https://bugs.webkit.org/show_bug.cgi?id=193228
>+        <rdar://problem/46787787>
>+
>+        Reviewed by Joseph Pecoraro.
>+
>+        Test: inspector/audit/run-accessibility.html
>+
>+        * inspector/InspectorAuditAccessibilityObject.idl:
>+        * inspector/InspectorAuditAccessibilityObject.h:
>+        * inspector/InspectorAuditAccessibilityObject.cpp:
>+        (WebCore::accessiblityObjectForNode): Added.
>+        (WebCore::InspectorAuditAccessibilityObject::getElementsByComputedRole): Added.
>+
> 2019-01-11  Antoine Quint  <graouts@apple.com>
> 
>         Support parsing of additional values for the touch-action property
>diff --git a/Source/WebCore/inspector/InspectorAuditAccessibilityObject.cpp b/Source/WebCore/inspector/InspectorAuditAccessibilityObject.cpp
>index fcc59b17042819aec64b15989045fd05cb70efe0..7882a7108de1c790f095ba8ef59500fcfef8cf45 100644
>--- a/Source/WebCore/inspector/InspectorAuditAccessibilityObject.cpp
>+++ b/Source/WebCore/inspector/InspectorAuditAccessibilityObject.cpp
>@@ -27,12 +27,52 @@
> #include "config.h"
> #include "InspectorAuditAccessibilityObject.h"
> 
>+#include "AXObjectCache.h"
>+#include "AccessibilityObject.h"
>+#include "ContainerNode.h"
>+#include "Document.h"
>+#include "ElementDescendantIterator.h"
>+#include "Node.h"
>+#include <wtf/Vector.h>
>+
> namespace WebCore {
> 
> using namespace Inspector;
> 
>-InspectorAuditAccessibilityObject::InspectorAuditAccessibilityObject()
>+#define ERROR_IF_NO_ACTIVE_AUDIT() \
>+    if (!m_auditAgent.hasActiveAudit()) \
>+        return Exception { NotAllowedError, "Cannot be called outside of a Web Inspector Audit"_s };
>+
>+InspectorAuditAccessibilityObject::InspectorAuditAccessibilityObject(InspectorAuditAgent& auditAgent)
>+    : m_auditAgent(auditAgent)
> {
> }
> 
>+static AccessibilityObject* accessiblityObjectForNode(Node& node)
>+{
>+    if (!AXObjectCache::accessibilityEnabled())
>+        AXObjectCache::enableAccessibility();
>+
>+    if (AXObjectCache* axObjectCache = node.document().axObjectCache())
>+        return axObjectCache->getOrCreate(&node);
>+
>+    return nullptr;
>+}
>+
>+ExceptionOr<Vector<Ref<Node>>> InspectorAuditAccessibilityObject::getElementsByComputedRole(Document& document, const String& role, Node* container)
>+{
>+    ERROR_IF_NO_ACTIVE_AUDIT();
>+
>+    Vector<Ref<Node>> nodes;
>+
>+    for (Element& element : elementDescendants(is<ContainerNode>(container) ? downcast<ContainerNode>(*container) : document)) {
>+        if (AccessibilityObject* axObject = accessiblityObjectForNode(element)) {
>+            if (axObject->computedRoleString() == role)
>+                nodes.append(element);
>+        }
>+    }
>+
>+    return nodes;
>+}
>+
> } // namespace WebCore
>diff --git a/Source/WebCore/inspector/InspectorAuditAccessibilityObject.h b/Source/WebCore/inspector/InspectorAuditAccessibilityObject.h
>index 1036fc05b16908b51c39a9839df104fd9af7d85f..88f963a9531b8e6b2cc86198f8b849c537206dcc 100644
>--- a/Source/WebCore/inspector/InspectorAuditAccessibilityObject.h
>+++ b/Source/WebCore/inspector/InspectorAuditAccessibilityObject.h
>@@ -25,21 +25,30 @@
> 
> #pragma once
> 
>+#include "ExceptionOr.h"
> #include <JavaScriptCore/InspectorAuditAgent.h>
>+#include <wtf/Forward.h>
> #include <wtf/Ref.h>
> #include <wtf/RefCounted.h>
> 
> namespace WebCore {
> 
>+class Document;
>+class Node;
>+
> class InspectorAuditAccessibilityObject : public RefCounted<InspectorAuditAccessibilityObject> {
> public:
>-    static Ref<InspectorAuditAccessibilityObject> create(Inspector::InspectorAuditAgent&)
>+    static Ref<InspectorAuditAccessibilityObject> create(Inspector::InspectorAuditAgent& auditAgent)
>     {
>-        return adoptRef(*new InspectorAuditAccessibilityObject());
>+        return adoptRef(*new InspectorAuditAccessibilityObject(auditAgent));
>     }
> 
>+    ExceptionOr<Vector<Ref<Node>>> getElementsByComputedRole(Document&, const String& role, Node* container);
>+
> private:
>-    explicit InspectorAuditAccessibilityObject();
>+    explicit InspectorAuditAccessibilityObject(Inspector::InspectorAuditAgent&);
>+
>+    Inspector::InspectorAuditAgent& m_auditAgent;
> };
> 
> } // namespace WebCore
>diff --git a/Source/WebCore/inspector/InspectorAuditAccessibilityObject.idl b/Source/WebCore/inspector/InspectorAuditAccessibilityObject.idl
>index 24d0398fa4af1faa0b7027743792cd4ed6969286..46f1b52655876a9c0289c40df960b9a74a91932e 100644
>--- a/Source/WebCore/inspector/InspectorAuditAccessibilityObject.idl
>+++ b/Source/WebCore/inspector/InspectorAuditAccessibilityObject.idl
>@@ -28,4 +28,5 @@
>     JSGenerateToJSObject,
>     NoInterfaceObject,
> ] interface InspectorAuditAccessibilityObject {
>+    [CallWith=Document, MayThrowException] sequence<Node> getElementsByComputedRole(DOMString role, optional Node? container);
> };
>diff --git a/LayoutTests/ChangeLog b/LayoutTests/ChangeLog
>index d0d13a9cd5282c985717cb7857ef83014e54b443..7f2122176e80e636bd8fc60991abe7040af439f0 100644
>--- a/LayoutTests/ChangeLog
>+++ b/LayoutTests/ChangeLog
>@@ -1,3 +1,14 @@
>+2019-01-15  Devin Rousso  <drousso@apple.com>
>+
>+        Web Inspector: Audit: provide a way to query for all nodes with a given computed Accessibility role
>+        https://bugs.webkit.org/show_bug.cgi?id=193228
>+        <rdar://problem/46787787>
>+
>+        Reviewed by Joseph Pecoraro.
>+
>+        * inspector/audit/run-accessibility.html: Added.
>+        * inspector/audit/run-accessibility-expected.txt: Added.
>+
> 2019-01-15  Devin Rousso  <drousso@apple.com>
> 
>         Web Inspector: Audit: create new IDL type for exposing special functionality in test context
>diff --git a/LayoutTests/inspector/audit/run-accessibility-expected.txt b/LayoutTests/inspector/audit/run-accessibility-expected.txt
>new file mode 100644
>index 0000000000000000000000000000000000000000..7a663dc9006529c3dfbf610920f6632588d24704
>--- /dev/null
>+++ b/LayoutTests/inspector/audit/run-accessibility-expected.txt
>@@ -0,0 +1,49 @@
>+Tests for the injected WebInspectorAudit.Accessibility functions.
>+
>+
>+
>+== Running test suite: Audit.run.Accessibility
>+-- Running test case: Audit.run.Accessibility.getElementsByComputedRole.FakeRole
>+Audit setup...
>+Audit run `WebInspectorAudit.Accessibility.getElementsByComputedRole("FakeRole")`...
>+Result: []
>+Audit teardown...
>+
>+-- Running test case: Audit.run.Accessibility.getElementsByComputedRole.FakeRole.parent
>+Audit setup...
>+Audit run `WebInspectorAudit.Accessibility.getElementsByComputedRole("FakeRole", document.querySelector("#parent"))`...
>+Result: []
>+Audit teardown...
>+
>+-- Running test case: Audit.run.Accessibility.getElementsByComputedRole.tree
>+Audit setup...
>+Audit run `WebInspectorAudit.Accessibility.getElementsByComputedRole("tree")`...
>+Result: ["#parent"]
>+Audit teardown...
>+
>+-- Running test case: Audit.run.Accessibility.getElementsByComputedRole.tree.parent
>+Audit setup...
>+Audit run `WebInspectorAudit.Accessibility.getElementsByComputedRole("tree", document.querySelector("#parent"))`...
>+Result: []
>+Audit teardown...
>+
>+-- Running test case: Audit.run.Accessibility.getElementsByComputedRole.button
>+Audit setup...
>+Audit run `WebInspectorAudit.Accessibility.getElementsByComputedRole("button")`...
>+Result: ["#button","#link"]
>+Audit teardown...
>+
>+-- Running test case: Audit.run.Accessibility.getElementsByComputedRole.button.parent
>+Audit setup...
>+Audit run `WebInspectorAudit.Accessibility.getElementsByComputedRole("button", document.querySelector("#parent"))`...
>+Result: []
>+Audit teardown...
>+
>+-- Running test case: Audit.run.Accessibility.InvalidCopiedFunctionCall
>+Audit setup...
>+Copying WebInspectorAudit to window...
>+Audit teardown...
>+Testing copied getElementsByComputedRole...
>+PASS: Should produce an exception.
>+Error: NotAllowedError: Cannot be called outside of a Web Inspector Audit
>+
>diff --git a/LayoutTests/inspector/audit/run-accessibility.html b/LayoutTests/inspector/audit/run-accessibility.html
>new file mode 100644
>index 0000000000000000000000000000000000000000..305135f130f9a484a85102ea5893b3d94ee4bbfc
>--- /dev/null
>+++ b/LayoutTests/inspector/audit/run-accessibility.html
>@@ -0,0 +1,97 @@
>+<!DOCTYPE html>
>+<html>
>+<head>
>+<script src="../../http/tests/inspector/resources/inspector-test.js"></script>
>+<script src="resources/audit-utilities.js"></script>
>+<script>
>+
>+function stringify(result) {
>+    if (result instanceof Element)
>+        return "#" + result.id;
>+    if (Array.isArray(result))
>+        return JSON.stringify(result.map(stringify));
>+    return false;
>+}
>+
>+function test()
>+{
>+    let suite = InspectorTest.Audit.createSuite("Audit.run.Accessibility");
>+
>+    function evaluateStringForTest(func, target, role) {
>+        if (target) {
>+            if (role)
>+                return `WebInspectorAudit.Accessibility.${func}("${role}", document.querySelector("#${target}"))`;
>+            return `WebInspectorAudit.Accessibility.${func}(document.querySelector("#${target}"))`;
>+        }
>+
>+        if (role)
>+            return `WebInspectorAudit.Accessibility.${func}("${role}")`;
>+    }
>+
>+    const tests = [
>+        { func: "getElementsByComputedRole", role: "FakeRole" },
>+        { func: "getElementsByComputedRole", role: "FakeRole", target: "parent" },
>+        { func: "getElementsByComputedRole", role: "tree" },
>+        { func: "getElementsByComputedRole", role: "tree", target: "parent" },
>+        { func: "getElementsByComputedRole", role: "button" },
>+        { func: "getElementsByComputedRole", role: "button", target: "parent" },
>+    ];
>+
>+    for (let {func, target, role} of tests) {
>+        suite.addTestCase({
>+            name: "Audit.run.Accessibility." + func + (role ? "." + role : "") + (target ? "." + target : ""),
>+            async test() {
>+                let functionString = evaluateStringForTest(func, target, role);
>+
>+                await InspectorTest.Audit.setupAudit();
>+
>+                InspectorTest.log(`Audit run \`${functionString}\`...`);
>+                let {result, wasThrown} = await AuditAgent.run(`function() { return stringify(${functionString}); }`);
>+                InspectorTest.assert(!wasThrown, "Should not throw an exception.");
>+                if (!wasThrown)
>+                    InspectorTest.log("Result: " + result.value);
>+                else
>+                    InspectorTest.log(result.description);
>+
>+                await InspectorTest.Audit.teardownAudit();
>+            },
>+        });
>+    }
>+
>+    suite.addTestCase({
>+        name: "Audit.run.Accessibility.InvalidCopiedFunctionCall",
>+        description: "Check that WebInspectorAudit.Accessibility functions throw an error when called outside of an audit.",
>+        async test() {
>+            let functions = new Map;
>+            for (let test of tests)
>+                functions.set(test.func, test);
>+
>+            await InspectorTest.Audit.setupAudit();
>+            InspectorTest.log(`Copying WebInspectorAudit to window...`);
>+            let {wasThrown} = await AuditAgent.run(`function() { window.CopiedWebInspectorAudit = WebInspectorAudit; }`);
>+            InspectorTest.assert(!wasThrown, "Should not throw an exception.");
>+            await InspectorTest.Audit.teardownAudit();
>+
>+            for (let {func, target, role} of functions.values()) {
>+                InspectorTest.log(`Testing copied ${func}...`);
>+                await InspectorTest.expectException(async function() {
>+                    await InspectorTest.evaluateInPage("window.Copied" + evaluateStringForTest(func, target, role));
>+                });
>+            }
>+        },
>+    });
>+
>+     suite.runTestCasesAndFinish();
>+}
>+</script>
>+</head>
>+<body onload="runTest()">
>+    <p>Tests for the injected WebInspectorAudit.Accessibility functions.</p>
>+    <ul id="parent" role="tree" aria-activedescendant="child" aria-controls="child" aria-flowto="child" aria-owns="child" onclick="void(0);">
>+        <li id="child" role="treeitem" aria-selected="true"></li>
>+    </ul>
>+    <button id="button"></button>
>+    <a id="link" role="button"></a>
>+    <div id="fakeRole" role="fakeRole"></div>
>+</body>
>+</html>

Actions: View | Formatted Diff | Diff

Attachments on bug 193228: 358572 | 358850 | 359148