DevTools: introduce dom markers, decorate hidden, forced state and breakpoint elements using marker…

LayoutTests/inspector/elements/hide-shortcut.html

Index: LayoutTests/inspector/elements/hide-shortcut.html
diff --git a/LayoutTests/inspector/elements/hide-shortcut.html b/LayoutTests/inspector/elements/hide-shortcut.html
index 806a00fdf9f89000d5460737b38fb612156bf93e..c4e6bc041ce24ab90bf6817eebfa957e7e229512 100644
--- a/LayoutTests/inspector/elements/hide-shortcut.html
+++ b/LayoutTests/inspector/elements/hide-shortcut.html
@@ -4,6 +4,18 @@
 <script src="../../http/tests/inspector/elements-test.js"></script>
 <script>
 
+function pseudoVisibility(resolve, reject, pseudo)
+{
+    var parentNode = document.getElementById("parent-node");
+    resolve(getComputedStyle(parentNode, ":" + pseudo).visibility);
+}
+
+function pseudoIframeVisibility(resolve, reject, pseudo)

[dgozman, 2015/08/07 23:35:05]

You never pass |pseudo| here. Remove?

[pfeldman, 2015/08/10 21:25:16]

Done.

+{
+    var parentNode = frames[0].document.getElementById("frame-node");
+    resolve(getComputedStyle(parentNode, ":" + pseudo).visibility);
+}
+
 function test()
 {
     var treeOutline;
@@ -113,15 +125,12 @@ function test()
 
             function callback()
             {
-                InspectorTest.addResult("=== Added hide shortcut in frame ===");
-                InspectorTest.cssModel.computedStylePromise(frameNode.id).then(callback2);
-            }
-
-            function callback2(style)
-            {
-                InspectorTest.addResult("=== Frame node is hidden ===");
-                InspectorTest.addResult(style.getLiveProperty("visibility").propertyText);
-                next();
+                InspectorTest.invokePageFunctionPromise("pseudoIframeVisibility", []).then(function(result) {
+                    InspectorTest.addResult("=== Added hide shortcut in frame ===");
+                    InspectorTest.addResult("=== Frame node is hidden ===");
+                    InspectorTest.addResult("visibility: " + result + ";");
+                    next();
+                });
             }
         }
     ]);
@@ -129,16 +138,12 @@ function test()
     function testPseudoToggle(pseudoNode, next)
     {
         treeOutline.toggleHideElement(pseudoNode, callback);
-
         function callback()
         {
-            InspectorTest.cssModel.computedStylePromise(pseudoNode.id).then(callback2);
-        }
-
-        function callback2(style)
-        {
-            InspectorTest.addResult("::" + pseudoNode.pseudoType() + " node visibility: '" + style.getLiveProperty("visibility").value + "'");
-            next();
+            InspectorTest.invokePageFunctionPromise("pseudoVisibility", [pseudoNode.pseudoType()]).then(function(result) {
+                InspectorTest.addResult("::" + pseudoNode.pseudoType() + " node visibility: '" + result + "'");
+                next();
+            });
         }
     }
 }