Web Inspector: AXI: linkified refs to #document and #text are not usually navigable...
authorcommit-queue@webkit.org <commit-queue@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 29 Sep 2016 22:15:57 +0000 (22:15 +0000)
committercommit-queue@webkit.org <commit-queue@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 29 Sep 2016 22:15:57 +0000 (22:15 +0000)
https://bugs.webkit.org/show_bug.cgi?id=130600
<rdar://problem/16391333>

Patch by Aaron Chu <aaron_chu@apple.com> on 2016-09-29
Reviewed by Brian Burg.

.:

Added a manual test to test the node links in the Accessibility Inspector
https://bugs.webkit.org/show_bug.cgi?id=130600

* ManualTests/accessibility/delinkified-non-navigable-links.html: Added.

Source/WebInspectorUI:

Removing link style for non-navigable nodes by first
checking nodeType of the node.

* UserInterface/Base/DOMUtilities.js:
(WebInspector.linkifyNodeReference):

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

ChangeLog
ManualTests/accessibility/delinkified-non-navigable-links.html [new file with mode: 0644]
Source/WebInspectorUI/ChangeLog
Source/WebInspectorUI/UserInterface/Base/DOMUtilities.js

index ee46fbe..5e22fb4 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,16 @@
+2016-09-29  Aaron Chu  <aaron_chu@apple.com>
+
+        Web Inspector: AXI: linkified refs to #document and #text are not usually navigable nodes; consider delinkifying them
+        https://bugs.webkit.org/show_bug.cgi?id=130600
+        <rdar://problem/16391333>
+
+        Reviewed by Brian Burg.
+
+        Added a manual test to test the node links in the Accessibility Inspector
+        https://bugs.webkit.org/show_bug.cgi?id=130600
+
+        * ManualTests/accessibility/delinkified-non-navigable-links.html: Added.
+
 2016-09-28  Michael Catanzaro  <mcatanzaro@igalia.com>
 
         [GTK] Remove obsolete comment from OptionsGTK.cmake
diff --git a/ManualTests/accessibility/delinkified-non-navigable-links.html b/ManualTests/accessibility/delinkified-non-navigable-links.html
new file mode 100644 (file)
index 0000000..22eee30
--- /dev/null
@@ -0,0 +1,23 @@
+<html>
+<body>
+
+
+<p>Test for <a href="https://bugs.webkit.org/show_bug.cgi?id=130600">Bug 130600: Web Inspector: AXI: linkified refs to #document and #text are not usually navigable nodes; consider delinkifying them</a>.</p>
+<ol>
+<li>Open Accessibility Inspector.</li>
+<li>In DOM tree in the Web Inspector, click on the first &lt;html&gt;.</li>
+<li>In the Accessibility Inspector, you should be able to see "#document" as parent, but it should not look like a link.</li>
+<li>In DOM tree in the Web Inspector, click on a &lt;p&gt;.</li>
+<li>In the Accessibility Inspector, you should be able to see "#document" as parent, but it should not look like a link.</li>
+<li>In the Accessibility Inspector, you should be able to see a ":role(text)" under "Children", but it should not look like a link.</li>
+<li>In DOM tree in the Web Inspector, click on the &lt;iframe&gt;.</li>
+<li>In the Accessibility Inspector, you should be able to see a "#document" under "Children", and it <b>should</b> not look like a link.</li>
+
+</ol>
+
+<p>A paragraph</p>
+
+<iframe></iframe>
+
+</body>
+</html>
index 6722094..0fba44c 100644 (file)
@@ -1,3 +1,17 @@
+2016-09-29  Aaron Chu  <aaron_chu@apple.com>
+
+        Web Inspector: AXI: linkified refs to #document and #text are not usually navigable nodes; consider delinkifying them
+        https://bugs.webkit.org/show_bug.cgi?id=130600
+        <rdar://problem/16391333>
+
+        Reviewed by Brian Burg.
+
+        Removing link style for non-navigable nodes by first
+        checking nodeType of the node.
+
+        * UserInterface/Base/DOMUtilities.js:
+        (WebInspector.linkifyNodeReference):
+
 2016-09-27  Matt Baker  <mattbaker@apple.com>
 
         Web Inspector: Refreshing while in Timelines-tab causes negative timestamps in Network-tab
index ba6ccdd..b2a2ca1 100644 (file)
@@ -66,8 +66,12 @@ WebInspector.linkifyNodeReference = function(node, maxLength)
     let link = document.createElement("span");
     link.append(displayName);
     link.setAttribute("role", "link");
-    link.className = "node-link";
+    
     link.title = displayName;
+    
+    let nodeType = node.nodeType();
+    if ((nodeType !== Node.DOCUMENT_NODE || node.parentNode) && nodeType !== Node.TEXT_NODE)
+        link.className = "node-link";
 
     link.addEventListener("click", WebInspector.domTreeManager.inspectElement.bind(WebInspector.domTreeManager, node.id));
     link.addEventListener("mouseover", WebInspector.domTreeManager.highlightDOMNode.bind(WebInspector.domTreeManager, node.id, "all"));