2011-04-12 Ilya Tikhonovsky <loislo@chromium.org>
authorloislo@chromium.org <loislo@chromium.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 12 Apr 2011 13:03:49 +0000 (13:03 +0000)
committerloislo@chromium.org <loislo@chromium.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 12 Apr 2011 13:03:49 +0000 (13:03 +0000)
        Reviewed by Yury Semikhatsky.

        Web Inspector: InspectorTest.addObject dumps inner array properties as objects.
        https://bugs.webkit.org/show_bug.cgi?id=58310

        * http/tests/inspector/inspector-test.js:
        (initialize_InspectorTest.InspectorTest.addObject):
        (initialize_InspectorTest.InspectorTest.addArray):
        (initialize_InspectorTest.InspectorTest.dump):
        * http/tests/inspector/resource-har-conversion-expected.txt:
        * http/tests/inspector/resource-parameters-expected.txt:
        * inspector/protocol/console-agent-expected.txt:
        * inspector/protocol/runtime-agent-expected.txt:
        * inspector/report-protocol-errors-expected.txt:

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

LayoutTests/ChangeLog
LayoutTests/http/tests/inspector/inspector-test.js
LayoutTests/http/tests/inspector/resource-har-conversion-expected.txt
LayoutTests/http/tests/inspector/resource-parameters-expected.txt
LayoutTests/inspector/protocol/console-agent-expected.txt
LayoutTests/inspector/protocol/runtime-agent-expected.txt
LayoutTests/inspector/report-protocol-errors-expected.txt

index 20fe735..b9f5ce2 100644 (file)
@@ -1,3 +1,20 @@
+2011-04-12  Ilya Tikhonovsky  <loislo@chromium.org>
+
+        Reviewed by Yury Semikhatsky.
+
+        Web Inspector: InspectorTest.addObject dumps inner array properties as objects.
+        https://bugs.webkit.org/show_bug.cgi?id=58310
+
+        * http/tests/inspector/inspector-test.js:
+        (initialize_InspectorTest.InspectorTest.addObject):
+        (initialize_InspectorTest.InspectorTest.addArray):
+        (initialize_InspectorTest.InspectorTest.dump):
+        * http/tests/inspector/resource-har-conversion-expected.txt:
+        * http/tests/inspector/resource-parameters-expected.txt:
+        * inspector/protocol/console-agent-expected.txt:
+        * inspector/protocol/runtime-agent-expected.txt:
+        * inspector/report-protocol-errors-expected.txt:
+
 2011-04-11  Stephen White  <senorblanco@chromium.org>
 
         Reviewed by James Robinson.
index b7dfbb5..66d0bfd 100644 (file)
@@ -109,22 +109,41 @@ InspectorTest.addObject = function(object, nondeterministicProps, prefix, firstL
     for (var prop in object) {
         if (typeof object.hasOwnProperty === "function" && !object.hasOwnProperty(prop))
             continue;
-        var prefixWithName = prefix + "    " + prop + " : ";
+        var prefixWithName = "    " + prefix + prop + " : ";
         var propValue = object[prop];
         if (nondeterministicProps && prop in nondeterministicProps)
             InspectorTest.addResult(prefixWithName + "<" + typeof propValue + ">");
-        else if (propValue === null)
-            InspectorTest.addResult(prefixWithName + "null");
-        else if (typeof propValue === "object")
-            InspectorTest.addObject(propValue, nondeterministicProps, prefix + "    ", prefixWithName);
-        else if (typeof propValue === "string")
-            InspectorTest.addResult(prefixWithName + "\"" + propValue + "\"");
         else
-            InspectorTest.addResult(prefixWithName + propValue);
+            InspectorTest.dump(propValue, nondeterministicProps, "    " + prefix, prefixWithName);
     }
     InspectorTest.addResult(prefix + "}");
 }
 
+InspectorTest.addArray = function(array, nondeterministicProps, prefix, firstLinePrefix)
+{
+    prefix = prefix || "";
+    firstLinePrefix = firstLinePrefix || prefix;
+    InspectorTest.addResult(firstLinePrefix + "[");
+    for (var i = 0; i < array.length; ++i)
+        InspectorTest.dump(array[i], nondeterministicProps, prefix + "    ");
+    InspectorTest.addResult(prefix + "]");
+}
+
+InspectorTest.dump = function(value, nondeterministicProps, prefix, prefixWithName)
+{
+    prefixWithName = prefixWithName || prefix;
+    if (value === null)
+        InspectorTest.addResult(prefixWithName + "null");
+    else if (value instanceof Array)
+        InspectorTest.addArray(value, nondeterministicProps, prefix, prefixWithName);
+    else if (typeof value === "object")
+        InspectorTest.addObject(value, nondeterministicProps, prefix, prefixWithName);
+    else if (typeof value === "string")
+        InspectorTest.addResult(prefixWithName + "\"" + value + "\"");
+    else
+        InspectorTest.addResult(prefixWithName + value);
+}
+
 InspectorTest.assertGreaterOrEqual = function(expected, actual, message)
 {
     if (actual < expected)
index a2a6cc4..2c56c88 100644 (file)
@@ -7,8 +7,8 @@ Page reloaded.
         name : "WebInspector"
         version : <string>
     }
-    pages : {
-        0 : {
+    pages : [
+        {
             startedDateTime : <object>
             id : <string>
             title : ""
@@ -17,9 +17,9 @@ Page reloaded.
                 onLoad : <number>
             }
         }
-    }
-    entries : {
-        0 : {
+    ]
+    entries : [
+        {
             pageref : "http://127.0.0.1:8000/inspector/resource-har-conversion.html"
             startedDateTime : <object>
             time : <number>
@@ -27,10 +27,10 @@ Page reloaded.
                 method : "GET"
                 url : "http://127.0.0.1:8000/inspector/inspector-test.js"
                 headers : <object>
-                queryString : {
-                }
-                cookies : {
-                    0 : {
+                queryString : [
+                ]
+                cookies : [
+                    {
                         name : "a"
                         value : "b"
                         path : "/path"
@@ -39,7 +39,7 @@ Page reloaded.
                         httpOnly : false
                         secure : false
                     }
-                    1 : {
+                    {
                         name : "a1"
                         value : "b1"
                         path : undefined
@@ -48,7 +48,7 @@ Page reloaded.
                         httpOnly : false
                         secure : false
                     }
-                    2 : {
+                    {
                         name : "c1"
                         value : "d1"
                         path : undefined
@@ -57,7 +57,7 @@ Page reloaded.
                         httpOnly : false
                         secure : false
                     }
-                }
+                ]
                 headersSize : -1
                 bodySize : <number>
             }
@@ -65,8 +65,8 @@ Page reloaded.
                 status : 304
                 statusText : "Not Modified"
                 headers : <object>
-                cookies : {
-                    0 : {
+                cookies : [
+                    {
                         name : "x"
                         value : "y"
                         path : "/path"
@@ -75,7 +75,7 @@ Page reloaded.
                         httpOnly : true
                         secure : true
                     }
-                    1 : {
+                    {
                         name : "x1"
                         value : "y1"
                         path : undefined
@@ -84,7 +84,7 @@ Page reloaded.
                         httpOnly : false
                         secure : false
                     }
-                    2 : {
+                    {
                         name : "z2"
                         value : "y2"
                         path : undefined
@@ -93,7 +93,7 @@ Page reloaded.
                         httpOnly : false
                         secure : false
                     }
-                }
+                ]
                 content : {
                     size : <number>
                     mimeType : "application/x-javascript"
@@ -106,7 +106,7 @@ Page reloaded.
             }
             timings : <object>
         }
-        1 : {
+        {
             pageref : "http://127.0.0.1:8000/inspector/resource-har-conversion.html"
             startedDateTime : <object>
             time : <number>
@@ -114,10 +114,10 @@ Page reloaded.
                 method : "GET"
                 url : "http://127.0.0.1:8000/inspector/resource-har-conversion.html"
                 headers : <object>
-                queryString : {
-                }
-                cookies : {
-                }
+                queryString : [
+                ]
+                cookies : [
+                ]
                 headersSize : -1
                 bodySize : <number>
             }
@@ -125,8 +125,8 @@ Page reloaded.
                 status : 200
                 statusText : "OK"
                 headers : <object>
-                cookies : {
-                }
+                cookies : [
+                ]
                 content : {
                     size : <number>
                     mimeType : "text/html"
@@ -139,7 +139,7 @@ Page reloaded.
             }
             timings : <object>
         }
-        2 : {
+        {
             pageref : "http://127.0.0.1:8000/inspector/resource-har-conversion.html"
             startedDateTime : <object>
             time : <number>
@@ -147,10 +147,10 @@ Page reloaded.
                 method : "GET"
                 url : "http://127.0.0.1:8000/inspector/resources-test.js"
                 headers : <object>
-                queryString : {
-                }
-                cookies : {
-                }
+                queryString : [
+                ]
+                cookies : [
+                ]
                 headersSize : -1
                 bodySize : <number>
             }
@@ -158,8 +158,8 @@ Page reloaded.
                 status : 304
                 statusText : "Not Modified"
                 headers : <object>
-                cookies : {
-                }
+                cookies : [
+                ]
                 content : {
                     size : <number>
                     mimeType : "application/x-javascript"
@@ -172,7 +172,7 @@ Page reloaded.
             }
             timings : <object>
         }
-        3 : {
+        {
             pageref : "http://127.0.0.1:8000/inspector/resource-har-conversion.html"
             startedDateTime : <object>
             time : <number>
@@ -180,10 +180,10 @@ Page reloaded.
                 method : "POST"
                 url : "http://127.0.0.1:8000/inspector/resources/post-target.cgi"
                 headers : <object>
-                queryString : {
-                }
-                cookies : {
-                }
+                queryString : [
+                ]
+                cookies : [
+                ]
                 headersSize : -1
                 bodySize : <number>
                 postData : {
@@ -195,8 +195,8 @@ Page reloaded.
                 status : 200
                 statusText : "OK"
                 headers : <object>
-                cookies : {
-                }
+                cookies : [
+                ]
                 content : {
                     size : <number>
                     mimeType : "application/xml"
@@ -209,6 +209,6 @@ Page reloaded.
             }
             timings : <object>
         }
-    }
+    ]
 }
 
index 61eb264..aaaa24a 100644 (file)
@@ -10,41 +10,41 @@ Tests that resources panel shows form data parameters.
         method : "POST"
         url : "http://localhost:8000/inspector/resources/post-target.cgi?queryParam1=queryValue1&queryParam2="
         headers : <object>
-        queryString : {
-            0 : {
+        queryString : [
+            {
                 name : "queryParam1"
                 value : "queryValue1"
             }
-            1 : {
+            {
                 name : "queryParam2"
                 value : ""
             }
-        }
-        cookies : {
-        }
+        ]
+        cookies : [
+        ]
         headersSize : -1
         bodySize : -1
         postData : {
             mimeType : "application/x-www-form-urlencoded"
             text : "formParam1=formValue1&formParam2="
-            params : {
-                0 : {
+            params : [
+                {
                     name : "formParam1"
                     value : "formValue1"
                 }
-                1 : {
+                {
                     name : "formParam2"
                     value : ""
                 }
-            }
+            ]
         }
     }
     response : {
         status : 200
         statusText : "OK"
         headers : <object>
-        cookies : {
-        }
+        cookies : [
+        ]
         content : {
             size : 14
             mimeType : "application/xml"
index 412d82d..86fc88d 100644 (file)
@@ -61,12 +61,12 @@ event ConsoleAgent.messageAdded
             url : ""
             repeatCount : 1
             text : "test"
-            parameters : {
-                0 : {
+            parameters : [
+                {
                     type : "string"
                     description : "test"
                 }
-            }
+            ]
             stackTrace : <object>
         }
     }
index edaa58d..c059190 100644 (file)
@@ -135,22 +135,22 @@ request:
 response:
 {
     result : {
-        result : {
-            0 : {
+        result : [
+            {
                 name : "assignedByEvaluateOn"
                 value : {
                     type : "string"
                     description : "evaluateOn function works fine"
                 }
             }
-            1 : {
+            {
                 name : "assignedBySetPropertyValue"
                 value : {
                     type : "boolean"
                     description : "true"
                 }
             }
-            2 : {
+            {
                 name : "__proto__"
                 value : {
                     objectId : <string>
@@ -159,7 +159,7 @@ response:
                     description : "TestObject"
                 }
             }
-        }
+        ]
     }
     id : <number>
 }
index 0779232..3b8c289 100644 (file)
@@ -4,9 +4,9 @@ Tests that InspectorBackendDispatcher is catching incorrect messages.
     error : {
         code : -32700
         message : "Parse error."
-        data : {
-            0 : "Message should be in JSON format."
-        }
+        data : [
+            "Message should be in JSON format."
+        ]
     }
     id : null
 }
@@ -14,9 +14,9 @@ Tests that InspectorBackendDispatcher is catching incorrect messages.
     error : {
         code : -32600
         message : "Invalid Request."
-        data : {
-            0 : "Invalid message format. 'id' property was not found in the request."
-        }
+        data : [
+            "Invalid message format. 'id' property was not found in the request."
+        ]
     }
     id : null
 }
@@ -24,9 +24,9 @@ Tests that InspectorBackendDispatcher is catching incorrect messages.
     error : {
         code : -32600
         message : "Invalid Request."
-        data : {
-            0 : "Invalid message format. The type of 'id' property should be number."
-        }
+        data : [
+            "Invalid message format. The type of 'id' property should be number."
+        ]
     }
     id : null
 }
@@ -34,9 +34,9 @@ Tests that InspectorBackendDispatcher is catching incorrect messages.
     error : {
         code : -32600
         message : "Invalid Request."
-        data : {
-            0 : "Invalid message format. The type of 'method' property should be string."
-        }
+        data : [
+            "Invalid message format. The type of 'method' property should be string."
+        ]
     }
     id : 1
 }
@@ -44,9 +44,9 @@ Tests that InspectorBackendDispatcher is catching incorrect messages.
     error : {
         code : -32601
         message : "Method not found."
-        data : {
-            0 : "Invalid method name was received. 'resourceContent' wasn't found."
-        }
+        data : [
+            "Invalid method name was received. 'resourceContent' wasn't found."
+        ]
     }
     id : 2
 }
@@ -54,9 +54,9 @@ Tests that InspectorBackendDispatcher is catching incorrect messages.
     error : {
         code : -32601
         message : "Method not found."
-        data : {
-            0 : "Invalid method name was received. 'DOM.test' wasn't found."
-        }
+        data : [
+            "Invalid method name was received. 'DOM.test' wasn't found."
+        ]
     }
     id : 3
 }
@@ -64,9 +64,9 @@ Tests that InspectorBackendDispatcher is catching incorrect messages.
     error : {
         code : -32602
         message : "Invalid params."
-        data : {
-            0 : "'params' property with type 'object' was not found."
-        }
+        data : [
+            "'params' property with type 'object' was not found."
+        ]
     }
     id : 4
 }
@@ -74,9 +74,9 @@ Tests that InspectorBackendDispatcher is catching incorrect messages.
     error : {
         code : -32602
         message : "Invalid params."
-        data : {
-            0 : "'params' property with type 'object' was not found."
-        }
+        data : [
+            "'params' property with type 'object' was not found."
+        ]
     }
     id : 5
 }
@@ -84,10 +84,10 @@ Tests that InspectorBackendDispatcher is catching incorrect messages.
     error : {
         code : -32602
         message : "Invalid params."
-        data : {
-            0 : "Parameter 'frameId' with type 'String' was not found."
-            1 : "Parameter 'url' with type 'String' was not found."
-        }
+        data : [
+            "Parameter 'frameId' with type 'String' was not found."
+            "Parameter 'url' with type 'String' was not found."
+        ]
     }
     id : 6
 }
@@ -95,9 +95,10 @@ Tests that InspectorBackendDispatcher is catching incorrect messages.
     error : {
         code : -32602
         message : "Invalid params."
-        data : {
-            0 : "Parameter 'url' with type 'String' was not found."
-        }
+        data : [
+            "Parameter 'url' with type 'String' was not found."
+        ]
     }
     id : 7
 }
+