Web Inspector: Fix some typos and style in LayoutTests/inspector
authorcommit-queue@webkit.org <commit-queue@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 21 Dec 2016 18:52:28 +0000 (18:52 +0000)
committercommit-queue@webkit.org <commit-queue@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 21 Dec 2016 18:52:28 +0000 (18:52 +0000)
https://bugs.webkit.org/show_bug.cgi?id=166273

Patch by Joseph Pecoraro <pecoraro@apple.com> on 2016-12-21
Reviewed by Brian Burg.

* inspector/*

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

130 files changed:
LayoutTests/ChangeLog
LayoutTests/inspector/codemirror/resources/prettyprinting/utilities.js
LayoutTests/inspector/console/addInspectedNode.html
LayoutTests/inspector/console/command-line-api-copy.html
LayoutTests/inspector/console/console-count.html
LayoutTests/inspector/console/heapSnapshot.html
LayoutTests/inspector/console/js-isLikelyStackTrace.html
LayoutTests/inspector/controller/runtime-controller.html
LayoutTests/inspector/css/generate-css-rule-string.html
LayoutTests/inspector/css/get-system-fonts-expected.txt
LayoutTests/inspector/css/get-system-fonts.html
LayoutTests/inspector/css/getSupportedCSSProperties.html
LayoutTests/inspector/css/matched-style-properties.html
LayoutTests/inspector/css/modify-rule-selector.html
LayoutTests/inspector/css/pseudo-element-matches-expected.txt
LayoutTests/inspector/css/pseudo-element-matches.html
LayoutTests/inspector/css/selector-dynamic-specificity.html
LayoutTests/inspector/css/selector-specificity.html
LayoutTests/inspector/css/stylesheet-events-imports.html
LayoutTests/inspector/css/stylesheet-with-mutations.html
LayoutTests/inspector/debugger/breakpoint-action-eval-expected.txt
LayoutTests/inspector/debugger/breakpoint-action-eval.html
LayoutTests/inspector/debugger/breakpoint-columns.html
LayoutTests/inspector/debugger/breakpoint-scope.html
LayoutTests/inspector/debugger/continueUntilNextRunLoop.html
LayoutTests/inspector/debugger/debugger-stack-overflow.html
LayoutTests/inspector/debugger/pause-reason.html
LayoutTests/inspector/debugger/probe-manager-add-remove-actions.html
LayoutTests/inspector/debugger/search-scripts.html
LayoutTests/inspector/debugger/tail-deleted-frames-from-vm-entry.html
LayoutTests/inspector/debugger/tail-deleted-frames-this-value.html
LayoutTests/inspector/debugger/tail-deleted-frames.html
LayoutTests/inspector/debugger/tail-recursion.html
LayoutTests/inspector/dom-debugger/node-removed.html
LayoutTests/inspector/dom/content-flow-content-nodes.html
LayoutTests/inspector/dom/content-flow-content-removal.html
LayoutTests/inspector/dom/content-flow-list.html
LayoutTests/inspector/dom/content-node-region-info.html
LayoutTests/inspector/dom/dom-remove-events.html
LayoutTests/inspector/dom/dom-search-crash.html
LayoutTests/inspector/dom/dom-search-with-context.html
LayoutTests/inspector/dom/dom-search.html
LayoutTests/inspector/dom/focus.html
LayoutTests/inspector/dom/getAccessibilityPropertiesForNode.html
LayoutTests/inspector/dom/getAccessibilityPropertiesForNode_ariaCurrent.html
LayoutTests/inspector/dom/getAccessibilityPropertiesForNode_liveRegion.html
LayoutTests/inspector/dom/getAccessibilityPropertiesForNode_mouseEventNodeId.html
LayoutTests/inspector/dom/getOuterHTML.html
LayoutTests/inspector/dom/hideHighlight.html
LayoutTests/inspector/dom/highlight-flow-with-no-region.html
LayoutTests/inspector/dom/highlight-shape-outside-margin.html
LayoutTests/inspector/dom/highlight-shape-outside.html
LayoutTests/inspector/dom/highlightQuad.html
LayoutTests/inspector/dom/highlightRect.html
LayoutTests/inspector/dom/remove-multiple-nodes.html
LayoutTests/inspector/dom/request-child-nodes-depth.html
LayoutTests/inspector/dom/setOuterHTML.html
LayoutTests/inspector/dom/shadowRootType.html
LayoutTests/inspector/formatting/formatting-json.html
LayoutTests/inspector/formatting/is-text-likely-minified.html
LayoutTests/inspector/formatting/resources/utilities.js
LayoutTests/inspector/heap/garbageCollected.html
LayoutTests/inspector/heap/gc.html
LayoutTests/inspector/heap/getPreview.html
LayoutTests/inspector/heap/getRemoteObject.html
LayoutTests/inspector/heap/snapshot.html
LayoutTests/inspector/heap/tracking.html
LayoutTests/inspector/indexeddb/requestData-expected.txt
LayoutTests/inspector/indexeddb/requestData.html
LayoutTests/inspector/indexeddb/requestDatabase.html
LayoutTests/inspector/indexeddb/resources/utilities.js
LayoutTests/inspector/layers/layers-anonymous.html
LayoutTests/inspector/layers/layers-blending-compositing-reasons.html
LayoutTests/inspector/layers/layers-compositing-reasons.html
LayoutTests/inspector/layers/layers-for-node.html
LayoutTests/inspector/layers/layers-generated-content.html
LayoutTests/inspector/layers/layers-reflected-content.html
LayoutTests/inspector/memory/tracking.html
LayoutTests/inspector/model/color.html
LayoutTests/inspector/model/parse-script-syntax-tree.html
LayoutTests/inspector/model/remote-object-get-properties.html
LayoutTests/inspector/model/remote-object-weak-collection.html
LayoutTests/inspector/model/remote-object.html
LayoutTests/inspector/network/client-blocked-load.html
LayoutTests/inspector/network/xhr-json-blob-has-content.html
LayoutTests/inspector/page/archive.html
LayoutTests/inspector/page/filter-cookies-for-domain.html
LayoutTests/inspector/page/frameScheduledNavigation.html
LayoutTests/inspector/page/frameStartedLoading.html
LayoutTests/inspector/page/main-frame-resource.html
LayoutTests/inspector/protocol/backend-dispatcher-argument-errors.html
LayoutTests/inspector/protocol/backend-dispatcher-malformed-message-errors.html
LayoutTests/inspector/protocol/inspector-backend-invocation-return-value.html
LayoutTests/inspector/replay/javascript-date-now.html
LayoutTests/inspector/replay/javascript-random-seed.html
LayoutTests/inspector/replay/window-navigator-plugins-memoized.html
LayoutTests/inspector/runtime/evaluate-CommandLineAPI.html
LayoutTests/inspector/runtime/getProperties.html
LayoutTests/inspector/runtime/parse.html
LayoutTests/inspector/runtime/saveResult.html
LayoutTests/inspector/sampling-profiler/basic.html
LayoutTests/inspector/sampling-profiler/call-frame-with-dom-functions.html
LayoutTests/inspector/sampling-profiler/eval-source-url.html
LayoutTests/inspector/sampling-profiler/expression-location-info.html
LayoutTests/inspector/sampling-profiler/many-call-frames.html
LayoutTests/inspector/sampling-profiler/named-function-expression.html
LayoutTests/inspector/script-profiler/event-type-API.html
LayoutTests/inspector/script-profiler/event-type-Microtask.html
LayoutTests/inspector/script-profiler/tracking.html
LayoutTests/inspector/timeline/debugger-paused-while-recording.html
LayoutTests/inspector/timeline/exception-in-injected-script-while-recording.html
LayoutTests/inspector/timeline/setInstruments-errors.html
LayoutTests/inspector/timeline/setInstruments-programmatic-capture.html
LayoutTests/inspector/unit-tests/array-utilities-expected.txt
LayoutTests/inspector/unit-tests/array-utilities.html
LayoutTests/inspector/unit-tests/async-test-suite.html
LayoutTests/inspector/unit-tests/debounce-expected.txt
LayoutTests/inspector/unit-tests/debounce.html
LayoutTests/inspector/unit-tests/event-listener-set-expected.txt
LayoutTests/inspector/unit-tests/event-listener-set.html
LayoutTests/inspector/unit-tests/heap-snapshot-collection-event.html
LayoutTests/inspector/unit-tests/heap-snapshot.html
LayoutTests/inspector/unit-tests/linked-list.html
LayoutTests/inspector/unit-tests/list-multimap.html
LayoutTests/inspector/unit-tests/number-utilities.html
LayoutTests/inspector/unit-tests/object-utilities-expected.txt
LayoutTests/inspector/unit-tests/object-utilities.html
LayoutTests/inspector/unit-tests/string-utilities.html
LayoutTests/inspector/unit-tests/sync-test-suite.html
LayoutTests/inspector/unit-tests/yieldable-task.html

index 45b50f9..11bc20e 100644 (file)
@@ -1,3 +1,12 @@
+2016-12-21  Joseph Pecoraro  <pecoraro@apple.com>
+
+        Web Inspector: Fix some typos and style in LayoutTests/inspector
+        https://bugs.webkit.org/show_bug.cgi?id=166273
+
+        Reviewed by Brian Burg.
+
+        * inspector/*
+
 2016-12-21  Simon Fraser  <simon.fraser@apple.com>
 
         Fixed bars are positioned incorrectly when there are header and footer banners
index 2829517..5a3334b 100644 (file)
@@ -53,7 +53,7 @@ TestPage.registerInitializer(function() {
             let testURL = testPageResourcesURL + "/" + test;
             suite.addTestCase({
                 name: suite.name + "." + testName,
-                test: (resolve, reject) => { 
+                test(resolve, reject) { 
                     runPrettyPrintingTest(mode, testName, testURL).then(resolve).catch(reject);
                 }
             });
index c496319..8cba69e 100644 (file)
@@ -16,7 +16,7 @@ function test()
     suite.addTestCase({
         name: "Check$0IsUndefined",
         description: "No inspected node yet, $0 should be undefined.",
-        test: (resolve, reject) => {
+        test(resolve, reject) {
             evaluate$0((remoteObject) => {
                 InspectorTest.expectThat(remoteObject.isUndefined(), "$0 should be undefined.");
                 resolve();
@@ -27,7 +27,7 @@ function test()
     suite.addTestCase({
         name: "SetAndCheckNodeA",
         description: "Set the inspected node to be Node A and check that $0 is Node A.",
-        test: (resolve, reject) => {
+        test(resolve, reject) {
             ConsoleAgent.addInspectedNode(nodeA.id);
             evaluate$0((remoteObject) => {
                 remoteObject.pushNodeToFrontend((nodeId) => {
@@ -42,7 +42,7 @@ function test()
     suite.addTestCase({
         name: "SetBadNodeId",
         description: "If an invalid inspected node is set, it should not be bound to $0.",
-        test: (resolve, reject) => {
+        test(resolve, reject) {
             ConsoleAgent.addInspectedNode(999999, (error) => {
                 InspectorTest.expectThat(error, "Should be an error");
             });
@@ -59,7 +59,7 @@ function test()
     suite.addTestCase({
         name: "SetAndCheckNodeB",
         description: "Set the inspected node to be Node B and check that $0 is Node B.",
-        test: (resolve, reject) => {
+        test(resolve, reject) {
             ConsoleAgent.addInspectedNode(nodeB.id);
             evaluate$0((remoteObject) => {
                 remoteObject.pushNodeToFrontend((nodeId) => {
index 6d93f57..26c4a15 100644 (file)
@@ -63,7 +63,7 @@ function test()
         suite.addTestCase({
             name: `Console.CommandLineAPI.Copy.${name}`,
             description: "Should be able to copy values.",
-            test: (resolve, reject) => {
+            test(resolve, reject) {
                 InspectorTest.log(`TEST: copy(${expression})`);
                 commandLineAPICopyAndPaste(expression, (result) => {
                     if (result !== expected) {
index 7cb6dc8..914f9a5 100644 (file)
@@ -28,7 +28,7 @@ function test()
     suite.addTestCase({
         name: "NoArguments",
         description: "No arguments increments a shared global counter.",
-        test: (resolve, reject) => {
+        test(resolve, reject) {
             let seen = 0;
             const expected = 16;
             WebInspector.logManager.addEventListener(WebInspector.LogManager.Event.MessageAdded, handler);
@@ -49,7 +49,7 @@ function test()
     suite.addTestCase({
         name: "WithLabel",
         description: "Labeled counters increment the label.",
-        test: (resolve, reject) => {
+        test(resolve, reject) {
             let seen = 0;
             const expected = 20;
             WebInspector.logManager.addEventListener(WebInspector.LogManager.Event.MessageAdded, handler);
index 911c444..92be69d 100644 (file)
@@ -17,7 +17,7 @@ function test()
 
     suite.addTestCase({
         name: "TriggerHeapSnapshotNoTitle",
-        test: function(resolve, reject) {
+        test(resolve, reject) {
             InspectorProtocol.awaitEvent({event: "Console.heapSnapshot"}).then((messageObject) => {
                 ProtocolTest.log("Console.heapSnapshot");
                 ProtocolTest.expectThat(typeof messageObject.params.timestamp === "number", "Should have a timestamp.");
@@ -32,7 +32,7 @@ function test()
 
     suite.addTestCase({
         name: "TriggerHeapSnapshotWithTitle",
-        test: function(resolve, reject) {
+        test(resolve, reject) {
             InspectorProtocol.awaitEvent({event: "Console.heapSnapshot"}).then((messageObject) => {
                 ProtocolTest.log("Console.heapSnapshot");
                 ProtocolTest.expectThat(typeof messageObject.params.timestamp === "number", "Should have a timestamp.");
index 2082203..6bd12c5 100644 (file)
@@ -45,7 +45,7 @@ function test()
 
     suite.addTestCase({
         name: "notStacktrace",
-        test: function(resolve, reject) {
+        test(resolve, reject) {
             var result = WebInspector.StackTrace.isLikelyStackTrace("Messages:42\nUnread:16");
             InspectorTest.assert(result, "Should NOT be a stacktrace.");
             resolve();
@@ -54,7 +54,7 @@ function test()
 
     suite.addTestCase({
         name: "typeErrorWrap",
-        test: function(resolve, reject) {
+        test(resolve, reject) {
             InspectorTest.evaluateInPage("typeErrorWrap()", function(error, result) {
                 if (error)
                     reject(error);
@@ -68,7 +68,7 @@ function test()
 
     suite.addTestCase({
         name: "getAnonymousFunctionError",
-        test: function(resolve, reject) {
+        test(resolve, reject) {
             InspectorTest.evaluateInPage("getAnonymousFunctionError()", function(error, result) {
                 if (error)
                     reject(error);
@@ -82,7 +82,7 @@ function test()
 
     suite.addTestCase({
         name: "testWithNativeCallInBetween",
-        test: function(resolve, reject) {
+        test(resolve, reject) {
             InspectorTest.evaluateInPage("testWithNativeCallInBetween()", function(error, result) {
                 if (error)
                     reject(error);
index 4808bd6..4ca23f3 100644 (file)
@@ -104,7 +104,7 @@ function test()
 
             InspectorTest.log("");
 
-            // The convenience will not apply to these noexpressions.
+            // The convenience will not apply to these expressions.
             testSource("return 10", expectException);
             testSource("await 10; 1", expectException);
             testSource("1; await 10;", expectException);
index 68b4e51..56effd8 100644 (file)
@@ -12,7 +12,7 @@
     }
 }
 </style>
-<script type="text/javascript" src="../../http/tests/inspector/resources/inspector-test.js"></script>
+<script src="../../http/tests/inspector/resources/inspector-test.js"></script>
 <script>
 function test() {
     let nodeStyles;
index 2539752..1f2b780 100644 (file)
@@ -1,4 +1,4 @@
-This test ensures that the inspector can enumerate system font families, and checks for the existance of common fonts.
+This test ensures that the inspector can enumerate system font families, and checks for the existence of common fonts.
 
 Includes Arial
 Includes Times
index 288499e..75e9c15 100644 (file)
@@ -1,7 +1,7 @@
 <!DOCTYPE html>
 <html>
 <head>
-<script type="text/javascript" src="../../http/tests/inspector/resources/inspector-test.js"></script>
+<script src="../../http/tests/inspector/resources/inspector-test.js"></script>
 <script>
 function test() {
     function hasFontFamily(fontFamilies, fontFamily) {
@@ -29,6 +29,6 @@ function test() {
 </head>
 <body onload="runTest()">
   <p>This test ensures that the inspector can enumerate system font families, and checks for the
-     existance of common fonts.</p>
+     existence of common fonts.</p>
 </body>
 </html>
index ae09c86..cc2350a 100644 (file)
@@ -1,6 +1,6 @@
 <html>
 <head>
-<script type="text/javascript" src="../../http/tests/inspector/resources/protocol-test.js"></script>
+<script src="../../http/tests/inspector/resources/protocol-test.js"></script>
 <script>
 function test()
 {
index 9d81e63..62772d6 100644 (file)
@@ -12,7 +12,7 @@ div#node1 {
     lEfT:0;
 }
 </style>
-<script type="text/javascript" src="../../http/tests/inspector/resources/inspector-test.js"></script>
+<script src="../../http/tests/inspector/resources/inspector-test.js"></script>
 <script>
 function test() {
     var nodeStyles;
index 3dd169c..fee89bb 100644 (file)
@@ -4,7 +4,7 @@
 <style>
     .foo {}
 </style>
-<script type="text/javascript" src="../../http/tests/inspector/resources/inspector-test.js"></script>
+<script src="../../http/tests/inspector/resources/inspector-test.js"></script>
 <script>
 function test() {
     // FIXME: remove this once we know why this test is sometimes flaky.
index 16ff7ed..d647bd4 100644 (file)
@@ -1,4 +1,4 @@
-Testing that psuedo element selectors correctly show matching selector information.
+Testing that pseudo element selectors correctly show matching selector information.
 
 
 video::-webkit-media-text-track-container
index 539ad22..9565177 100644 (file)
@@ -1,7 +1,7 @@
 <!DOCTYPE html>
 <html>
 <head>
-<script type="text/javascript" src="../../http/tests/inspector/resources/inspector-test.js"></script>
+<script src="../../http/tests/inspector/resources/inspector-test.js"></script>
 <script>
 function test() {
     var nodeStyles;
@@ -76,7 +76,7 @@ function test() {
 </script>
 </head>
 <body onload="runTest()">
-    <p>Testing that psuedo element selectors correctly show matching selector information.</p>
+    <p>Testing that pseudo element selectors correctly show matching selector information.</p>
 
     <audio id="node1" controls>
 </body>
index 70ba73e..43cfa33 100644 (file)
@@ -15,7 +15,7 @@ div:nth-child(odd),                             /* static, no selector list */
     color: green;
 }
 </style>
-<script type="text/javascript" src="../../http/tests/inspector/resources/inspector-test.js"></script>
+<script src="../../http/tests/inspector/resources/inspector-test.js"></script>
 <script>
 function test() {
     var nodeStyles;
index a671515..f71d1af 100644 (file)
@@ -16,7 +16,7 @@ body.a h1.b::before {
     color: green;
 }
 </style>
-<script type="text/javascript" src="../../http/tests/inspector/resources/inspector-test.js"></script>
+<script src="../../http/tests/inspector/resources/inspector-test.js"></script>
 <script>
 function test() {
     var nodeStyles;
index ca39a59..50cf3dd 100644 (file)
@@ -45,7 +45,7 @@ function test()
     suite.addTestCase({
         name: "CheckStyleSheets",
         description: "Ensure there are currently two stylesheets.",
-        test: (resolve, reject) => {
+        test(resolve, reject) {
             let styleSheets = WebInspector.cssStyleManager.styleSheets;
             InspectorTest.expectThat(styleSheets.length === 3, "Should be 3 stylesheets.");
             for (let styleSheet of styleSheets)
@@ -57,7 +57,7 @@ function test()
     suite.addTestCase({
         name: "DisablePropagatesThroughImports",
         description: "Disabling the top stylesheet containing imports will remove all stylesheets.",
-        test: (resolve, reject) => {
+        test(resolve, reject) {
             currentStyleSheetRemovedHandler = collectEvents(3, function(events) {
                 events.map((e) => sanitizeURL(e.data.styleSheet.url))
                     .sort()
@@ -72,7 +72,7 @@ function test()
     suite.addTestCase({
         name: "EnablePropagatesThroughImports",
         description: "Enabling the top stylesheet containing imports will add all stylesheets.",
-        test: (resolve, reject) => {
+        test(resolve, reject) {
             currentStyleSheetAddedHandler = collectEvents(3, function(events) {
                 events.map((e) => sanitizeURL(e.data.styleSheet.url))
                     .sort()
@@ -87,7 +87,7 @@ function test()
     suite.addTestCase({
         name: "DeleteImportRemovesStyleSheet",
         description: "Deleting the @import rule should remove that stylesheet and its imports.",
-        test: (resolve, reject) => {
+        test(resolve, reject) {
             currentStyleSheetRemovedHandler = collectEvents(2, function(events) {
                 events.map((e) => sanitizeURL(e.data.styleSheet.url))
                     .sort()
index bff1149..944685d 100644 (file)
@@ -4,7 +4,7 @@
 <style>
 .foo, .bar { color: red; }
 </style>
-<script type="text/javascript" src="../../http/tests/inspector/resources/inspector-test.js"></script>
+<script src="../../http/tests/inspector/resources/inspector-test.js"></script>
 <script>
 // Insert a rule at the beginning of the inline stylesheet above that would shift all rule ordinals down.
 document.styleSheets[0].insertRule("body { padding: 10px; }", 0);
index c9829da..0c09361 100644 (file)
@@ -1,4 +1,4 @@
-Testing that "Evaluate JavaScript" breakpoint actions work correctly in the presense of a Content Security Policy that doesn't include 'unsafe-eval'.
+Testing that "Evaluate JavaScript" breakpoint actions work correctly in the presence of a Content Security Policy that doesn't include 'unsafe-eval'.
 
 inside breakpointActions a:(42) b:([object Object])
 Breakpoint action evaluated. a:(42) b:([object Object])
index 5188912..8fac68b 100644 (file)
@@ -41,6 +41,6 @@ function test()
 </script>
 </head>
 <body onload="runTest()">
-    <p>Testing that "Evaluate JavaScript" breakpoint actions work correctly in the presense of a Content Security Policy that doesn't include 'unsafe-eval'.</p>
+    <p>Testing that "Evaluate JavaScript" breakpoint actions work correctly in the presence of a Content Security Policy that doesn't include 'unsafe-eval'.</p>
 </body>
 </html>
index 65904ae..e96de75 100644 (file)
@@ -1,10 +1,10 @@
 <!doctype html>
 <html>
 <head>
-<script type="text/javascript" src="../../http/tests/inspector/resources/inspector-test.js"></script>
-<script type="text/javascript" src="../../http/tests/inspector/debugger/debugger-test.js"></script>
-<script type="text/javascript" src="./resources/column-breakpoints-1.js"></script>
-<script type="text/javascript" src="./resources/column-breakpoints-2.js"></script>
+<script src="../../http/tests/inspector/resources/inspector-test.js"></script>
+<script src="../../http/tests/inspector/debugger/debugger-test.js"></script>
+<script src="./resources/column-breakpoints-1.js"></script>
+<script src="./resources/column-breakpoints-2.js"></script>
 <script>
 
 function test()
index 76a26bd..f78a1d7 100644 (file)
@@ -1,9 +1,9 @@
 <!doctype html>
 <html>
 <head>
-<script type="text/javascript" src="../../http/tests/inspector/resources/inspector-test.js"></script>
-<script type="text/javascript" src="../../http/tests/inspector/debugger/debugger-test.js"></script>
-<script type="text/javascript" src="./resources/scope.js"></script>
+<script src="../../http/tests/inspector/resources/inspector-test.js"></script>
+<script src="../../http/tests/inspector/debugger/debugger-test.js"></script>
+<script src="./resources/scope.js"></script>
 <script>
 
 function test()
index 46663db..e6cae51 100644 (file)
@@ -18,8 +18,6 @@ function pause2() {
 
 function test()
 {
-    InspectorTest.debug();
-
     function topCallFrameName() {
         let targetData = WebInspector.debuggerManager.dataForTarget(WebInspector.mainTarget);
         return targetData.callFrames[0].functionName;
index 305384b..a0d04ce 100644 (file)
@@ -1,9 +1,9 @@
 <!doctype html>
 <html>
 <head>
-<script type="text/javascript" src="../../http/tests/inspector/resources/inspector-test.js"></script>
-<script type="text/javascript" src="../../http/tests/inspector/debugger/debugger-test.js"></script>
-<script type="text/javascript" src="./resources/stack-overflow.js"></script>
+<script src="../../http/tests/inspector/resources/inspector-test.js"></script>
+<script src="../../http/tests/inspector/debugger/debugger-test.js"></script>
+<script src="./resources/stack-overflow.js"></script>
 <script>
 
 function test()
index fc427e0..13b9a4e 100644 (file)
@@ -1,8 +1,8 @@
 <!doctype html>
 <html>
 <head>
-<script type="text/javascript" src="../../http/tests/inspector/resources/inspector-test.js"></script>
-<script type="text/javascript" src="./resources/pause-reasons.js"></script>
+<script src="../../http/tests/inspector/resources/inspector-test.js"></script>
+<script src="./resources/pause-reasons.js"></script>
 <script>
 TestPage.allowUncaughtExceptions = true;
 TestPage.needToSanitizeUncaughtExceptionURLs = true;
index f3138a2..59889d9 100644 (file)
@@ -1,9 +1,9 @@
 <!doctype html>
 <html>
 <head>
-<script type="text/javascript" src="../../http/tests/inspector/resources/inspector-test.js"></script>
-<script type="text/javascript" src="../../http/tests/inspector/debugger/debugger-test.js"></script>
-<script type="text/javascript" src="resources/script-for-breakpoint-actions.js"></script>
+<script src="../../http/tests/inspector/resources/inspector-test.js"></script>
+<script src="../../http/tests/inspector/debugger/debugger-test.js"></script>
+<script src="resources/script-for-breakpoint-actions.js"></script>
 <script>
 function test()
 {
index 93d396b..c11be3b 100644 (file)
@@ -1,7 +1,7 @@
 <!doctype html>
 <html>
 <head>
-<script type="text/javascript" src="../../http/tests/inspector/resources/inspector-test.js"></script>
+<script src="../../http/tests/inspector/resources/inspector-test.js"></script>
 <script>
 // SEARCHTEST: Comment in inline <script>.
 function performEvals()
index c44a80a..09bef2f 100644 (file)
@@ -1,9 +1,9 @@
 <!doctype html>
 <html>
 <head>
-<script type="text/javascript" src="../../http/tests/inspector/resources/inspector-test.js"></script>
-<script type="text/javascript" src="../../http/tests/inspector/debugger/debugger-test.js"></script>
-<script type="text/javascript" src="./resources/tail-deleted-frames-from-vm-entry.js"></script>
+<script src="../../http/tests/inspector/resources/inspector-test.js"></script>
+<script src="../../http/tests/inspector/debugger/debugger-test.js"></script>
+<script src="./resources/tail-deleted-frames-from-vm-entry.js"></script>
 <script>
 
 function test()
index f2098bc..605e4e2 100644 (file)
@@ -1,9 +1,9 @@
 <!doctype html>
 <html>
 <head>
-<script type="text/javascript" src="../../http/tests/inspector/resources/inspector-test.js"></script>
-<script type="text/javascript" src="../../http/tests/inspector/debugger/debugger-test.js"></script>
-<script type="text/javascript" src="./resources/tail-deleted-frames-this-value.js"></script>
+<script src="../../http/tests/inspector/resources/inspector-test.js"></script>
+<script src="../../http/tests/inspector/debugger/debugger-test.js"></script>
+<script src="./resources/tail-deleted-frames-this-value.js"></script>
 <script>
 
 function test()
index 3ac1d94..b61d7aa 100644 (file)
@@ -1,9 +1,9 @@
 <!doctype html>
 <html>
 <head>
-<script type="text/javascript" src="../../http/tests/inspector/resources/inspector-test.js"></script>
-<script type="text/javascript" src="../../http/tests/inspector/debugger/debugger-test.js"></script>
-<script type="text/javascript" src="./resources/tail-deleted-frames.js"></script>
+<script src="../../http/tests/inspector/resources/inspector-test.js"></script>
+<script src="../../http/tests/inspector/debugger/debugger-test.js"></script>
+<script src="./resources/tail-deleted-frames.js"></script>
 <script>
 
 function test()
index bb94f08..d3dfb25 100644 (file)
@@ -1,9 +1,9 @@
 <!doctype html>
 <html>
 <head>
-<script type="text/javascript" src="../../http/tests/inspector/resources/inspector-test.js"></script>
-<script type="text/javascript" src="../../http/tests/inspector/debugger/debugger-test.js"></script>
-<script type="text/javascript" src="./resources/tail-recursion.js"></script>
+<script src="../../http/tests/inspector/resources/inspector-test.js"></script>
+<script src="../../http/tests/inspector/debugger/debugger-test.js"></script>
+<script src="./resources/tail-recursion.js"></script>
 <script>
 
 function test()
index 2a2b86e..1a33837 100644 (file)
@@ -1,6 +1,6 @@
 <html>
 <head>
-<script type="text/javascript" src="../../http/tests/inspector/resources/protocol-test.js"></script>
+<script src="../../http/tests/inspector/resources/protocol-test.js"></script>
 <script>
 window.onload = runTest;
 
index 50c3c80..22a0702 100644 (file)
@@ -11,7 +11,7 @@
     -webkit-flow-into: none;
 }
 </style>
-<script type="text/javascript" src="../../http/tests/inspector/resources/inspector-test.js"></script>
+<script src="../../http/tests/inspector/resources/inspector-test.js"></script>
 <script>
 function changeFlowContent()
 {
index 6a570aa..1668716 100644 (file)
@@ -7,7 +7,7 @@
     -webkit-flow-into: flow;
 }
 </style>
-<script type="text/javascript" src="../../http/tests/inspector/resources/inspector-test.js"></script>
+<script src="../../http/tests/inspector/resources/inspector-test.js"></script>
 <script>
 function changeFlowContent()
 {
index ff0628a..846920c 100644 (file)
@@ -7,7 +7,7 @@
     -webkit-flow-into: flow1;
 }
 </style>
-<script type="text/javascript" src="../../http/tests/inspector/resources/inspector-test.js"></script>
+<script src="../../http/tests/inspector/resources/inspector-test.js"></script>
 <script>
 function removeFlow()
 {
index 6018957..a2ee684 100644 (file)
@@ -23,7 +23,7 @@
     height: 100px;
 }
 </style>
-<script type="text/javascript" src="../../http/tests/inspector/resources/inspector-test.js"></script>
+<script src="../../http/tests/inspector/resources/inspector-test.js"></script>
 <script>
 function test()
 {
index e01df04..4817ee7 100644 (file)
@@ -1,6 +1,6 @@
 <html>
 <head>
-<script type="text/javascript" src="../../http/tests/inspector/resources/protocol-test.js"></script>
+<script src="../../http/tests/inspector/resources/protocol-test.js"></script>
 <script>
 function moveNode()
 {
index efe2aa4..f0218f4 100644 (file)
@@ -1,6 +1,6 @@
 <html>
 <head>
-<script type="text/javascript" src="../../http/tests/inspector/resources/protocol-test.js"></script>
+<script src="../../http/tests/inspector/resources/protocol-test.js"></script>
 <script>
 function test()
 {
index e8f9b67..45af6f2 100644 (file)
@@ -1,7 +1,7 @@
 <html>
 <head>
-<script type="text/javascript" src="../../http/tests/inspector/resources/protocol-test.js"></script>
-<script type="text/javascript" src="../../http/tests/inspector/dom/resources/InspectorDOMListener.js"></script>
+<script src="../../http/tests/inspector/resources/protocol-test.js"></script>
+<script src="../../http/tests/inspector/dom/resources/InspectorDOMListener.js"></script>
 <script>
 function test()
 {
index bb1bdf3..4cc077e 100644 (file)
@@ -1,9 +1,9 @@
 <html>
 <head>
-<script type="text/javascript" src="../../http/tests/inspector/resources/protocol-test.js"></script>
-<script type="text/javascript" src="../../http/tests/inspector/dom/resources/InspectorDOMListener.js"></script>
+<script src="../../http/tests/inspector/resources/protocol-test.js"></script>
+<script src="../../http/tests/inspector/dom/resources/InspectorDOMListener.js"></script>
 <!-- Loading the queries from external file to avoid having them show up in the results. -->
-<script type="text/javascript" src="resources/dom-search-queries.js"></script>
+<script src="resources/dom-search-queries.js"></script>
 <script>
 function test()
 {
index 21fe199..8194ca0 100644 (file)
@@ -1,6 +1,6 @@
 <html>
 <head>
-<script type="text/javascript" src="../../http/tests/inspector/resources/protocol-test.js"></script>
+<script src="../../http/tests/inspector/resources/protocol-test.js"></script>
 <script>
 window.addEventListener("load", function() {
     document.querySelector("#second").addEventListener("focus", function() {
index 5e4853e..db33416 100644 (file)
@@ -1,6 +1,6 @@
 <html>
 <head>
-<script type="text/javascript" src="../../http/tests/inspector/resources/protocol-test.js"></script>
+<script src="../../http/tests/inspector/resources/protocol-test.js"></script>
 </head>
 <body onLoad="runTest()">
 
 
 </div>
 
-<script type="text/javascript">
+<script>
 
 function $(selector) {
     return document.querySelectorAll(selector);
index 3bf4ff5..4bf4d59 100644 (file)
@@ -1,6 +1,6 @@
 <html>
 <head>
-<script type="text/javascript" src="../../http/tests/inspector/resources/protocol-test.js"></script>
+<script src="../../http/tests/inspector/resources/protocol-test.js"></script>
 </head>
 <body onLoad="runTest()">
 
@@ -33,7 +33,7 @@
 
 </div>
 
-<script type="text/javascript">
+<script>
 
 function $(id) {
     return document.getElementById(id);
index bb1491f..d56692d 100644 (file)
@@ -1,6 +1,6 @@
 <html>
 <head>
-<script type="text/javascript" src="../../http/tests/inspector/resources/protocol-test.js"></script>
+<script src="../../http/tests/inspector/resources/protocol-test.js"></script>
 </head>
 <body onLoad="runTest()">
 
@@ -44,7 +44,7 @@
 
 </div>
 
-<script type="text/javascript">
+<script>
 
 function $(id) {
     return document.getElementById(id);
index ce6f210..7441c52 100644 (file)
@@ -1,6 +1,6 @@
 <html>
 <head>
-<script type="text/javascript" src="../../http/tests/inspector/resources/protocol-test.js"></script>
+<script src="../../http/tests/inspector/resources/protocol-test.js"></script>
 </head>
 <body onLoad="runTest()" onclick="void(0);">
 
@@ -38,7 +38,7 @@
 
 </div>
 
-<script type="text/javascript">
+<script>
 
 function $(id) {
     return document.getElementById(id);
index d0f930e..816ff54 100644 (file)
@@ -10,7 +10,7 @@ function test()
     suite.addTestCase({
         name: "GetOuterHTML",
         description: "Get the OuterHTML of the target element.",
-        test: (resolve, reject) => {
+        test(resolve, reject) {
             WebInspector.domTreeManager.requestDocument(function(documentNode) {
                 WebInspector.domTreeManager.querySelector(documentNode.id, "#x", function(nodeId) {
                     let domNode = WebInspector.domTreeManager.nodeForId(nodeId);
index 99f9a7d..355491c 100644 (file)
@@ -46,7 +46,7 @@ function test()
         suite.addTestCase({
             name: "HideHighlight",
             description: "Calling hideHighlight should hide the highlight.",
-            test: (resolve, reject) => {
+            test(resolve, reject) {
                 DOMAgent.hideHighlight(() => {
                     getHighlightRects((highlightRects) => {
                         InspectorTest.expectThat(highlightRects.length === 0, "Should be no highlight.");
@@ -60,7 +60,7 @@ function test()
     suite.addTestCase({
         name: "CheckEmptyHighlight",
         description: "Should not be a highlight yet.",
-        test: (resolve, reject) => {
+        test(resolve, reject) {
             getHighlightRects((highlightRects) => {
                 InspectorTest.expectThat(highlightRects.length === 0, "Should not be a highlight yet.");
                 resolve();
@@ -71,7 +71,7 @@ function test()
     suite.addTestCase({
         name: "HighlightRect",
         description: "Call highlightRect to create a highlight.",
-        test: (resolve, reject) => {
+        test(resolve, reject) {
             let rect = new WebInspector.Rect(0, 0, 100, 100);
             DOMAgent.highlightRect(rect.origin.x, rect.origin.y, rect.size.width, rect.size.height, color, outlineColor, (error) => {
                 dumpHighlightRects(resolve);
@@ -84,7 +84,7 @@ function test()
     suite.addTestCase({
         name: "HighlightQuad",
         description: "Call highlightQuad to create a highlight.",
-        test: (resolve, reject) => {
+        test(resolve, reject) {
             let quad = new WebInspector.Quad([100, 100, 150, 150, 100, 200, 50, 150]);
             DOMAgent.highlightQuad(quad.toProtocol(), color, outlineColor, (error) => {
                 dumpHighlightRects(resolve);
@@ -97,7 +97,7 @@ function test()
     suite.addTestCase({
         name: "HighlightNode",
         description: "Call highlightNode to create a highlight.",
-        test: (resolve, reject) => {
+        test(resolve, reject) {
             const highlightConfig = {
                 showInfo: true,
                 contentColor: {r: 255, g: 255, b: 255},
index 09c346d..a0bcbe2 100644 (file)
@@ -1,6 +1,6 @@
 <html>
 <head>
-<script type="text/javascript" src="../../http/tests/inspector/resources/protocol-test.js"></script>
+<script src="../../http/tests/inspector/resources/protocol-test.js"></script>
 <style>
 #flow {
     -webkit-flow-into: flow;
index 6b1d284..3e98c58 100644 (file)
@@ -24,8 +24,8 @@ body {
     -webkit-shape-margin: 1px;
 }
 </style>
-<script type="text/javascript" src="../../http/tests/inspector/resources/inspector-test.js"></script>
-<script type="text/javascript" src="../../http/tests/inspector/dom/shapes-test.js"></script>
+<script src="../../http/tests/inspector/resources/inspector-test.js"></script>
+<script src="../../http/tests/inspector/dom/shapes-test.js"></script>
 <script>
 function test() {
     var testcases = [
index a6d071a..981b3bd 100644 (file)
@@ -52,8 +52,8 @@ body {
    -webkit-shape-outside: url("data:image/png;base64,iVBORw0KGgoAAAANSUhEUgAAAGQAAABkCAMAAABHPGVmAAAAGXRFWHRTb2Z0d2FyZQBBZG9iZSBJbWFnZVJlYWR5ccllPAAAAAZQTFRFAAD/////e9yZLAAAAAJ0Uk5T/wDltzBKAAAASUlEQVR42uzYsQ0AMAgEsWf/pTMDBURIvgXcX2qhQCAQCAQCgUA+I2kHgUAgEAgEAoFAIBAIBAKBQOYRoxMCgUAgEAjkHPIEGAA2ih1NVnaDGgAAAABJRU5ErkJggg==");
 }
 </style>
-<script type="text/javascript" src="../../http/tests/inspector/resources/inspector-test.js"></script>
-<script type="text/javascript" src="../../http/tests/inspector/dom/shapes-test.js"></script>
+<script src="../../http/tests/inspector/resources/inspector-test.js"></script>
+<script src="../../http/tests/inspector/dom/shapes-test.js"></script>
 <script>
 function test() {
     var testcases = [
index 0a0331d..0a1b18e 100644 (file)
@@ -31,7 +31,7 @@ function test()
     suite.addTestCase({
         name: "CheckEmptyHighlight",
         description: "Should not be a highlight yet.",
-        test: (resolve, reject) => {
+        test(resolve, reject) {
             getHighlightRects((highlightRects) => {
                 InspectorTest.expectThat(highlightRects.length === 0, "Should not be a highlight yet.");
                 resolve();
@@ -42,7 +42,7 @@ function test()
     suite.addTestCase({
         name: "HighlightSmallRectWithQuad",
         description: "Should create a highlight using viewport coordinates.",
-        test: (resolve, reject) => {
+        test(resolve, reject) {
             let quad = new WebInspector.Quad([0, 0, 100, 0, 100, 100, 100, 0]);
             DOMAgent.highlightQuad(quad.toProtocol(), color, outlineColor, (error) => {
                 dumpHighlightRects(resolve);
@@ -53,7 +53,7 @@ function test()
     suite.addTestCase({
         name: "HighlightSmallQuadViewportCoordinates",
         description: "Should create a highlight using viewport coordinates.",
-        test: (resolve, reject) => {
+        test(resolve, reject) {
             let quad = new WebInspector.Quad([100, 100, 150, 150, 100, 200, 50, 150]);
             DOMAgent.highlightQuad(quad.toProtocol(), color, outlineColor, useViewportCoordinates, (error) => {
                 dumpHighlightRects(resolve);
@@ -64,7 +64,7 @@ function test()
     suite.addTestCase({
         name: "HighlightSmallQuadPageCoordinates",
         description: "Should create a highlight using page coordinates.",
-        test: (resolve, reject) => {
+        test(resolve, reject) {
             let quad = new WebInspector.Quad([100, 100, 150, 150, 100, 200, 50, 150]);
             DOMAgent.highlightQuad(quad.toProtocol(), color, outlineColor, usePageCoordinates, (error) => {
                 dumpHighlightRects(resolve);
@@ -75,7 +75,7 @@ function test()
     suite.addTestCase({
         name: "HighlightSmallUnspecifiedCoordinatesUsesViewportCoordinates",
         description: "Should create a highlight using page coordinates.",
-        test: (resolve, reject) => {
+        test(resolve, reject) {
             let quad = new WebInspector.Quad([100, 100, 150, 150, 100, 200, 50, 150]);
             DOMAgent.highlightQuad(quad.toProtocol(), color, outlineColor, (error) => {
                 dumpHighlightRects(resolve);
@@ -86,7 +86,7 @@ function test()
     suite.addTestCase({
         name: "HighlightLargeQuadViewportCoordinates",
         description: "Should create a highlight using viewport coordinates.",
-        test: (resolve, reject) => {
+        test(resolve, reject) {
             let quad = new WebInspector.Quad([0, 0, 500, 0, 1000, 1000, 0, 2000]);
             DOMAgent.highlightQuad(quad.toProtocol(), color, outlineColor, useViewportCoordinates, (error) => {
                 dumpHighlightRects(resolve);
@@ -97,7 +97,7 @@ function test()
     suite.addTestCase({
         name: "HighlightLargeQuadPageCoordinates",
         description: "Should create a highlight using viewport coordinates.",
-        test: (resolve, reject) => {
+        test(resolve, reject) {
             let quad = new WebInspector.Quad([0, 0, 500, 0, 1000, 1000, 0, 2000]);
             DOMAgent.highlightQuad(quad.toProtocol(), color, outlineColor, usePageCoordinates, (error) => {
                 dumpHighlightRects(resolve);
@@ -110,7 +110,7 @@ function test()
     suite.addTestCase({
         name: "BadQuadShouldError",
         description: "Should get an error when specifying a quad with too few points.",
-        test: (resolve, reject) => {
+        test(resolve, reject) {
             let badQuadArray = [1, 2, 3, 4];
             DOMAgent.highlightQuad(badQuadArray, color, outlineColor, usePageCoordinates, (error) => {
                 InspectorTest.expectThat(error, "Should produce an error.");
index 6f9032e..f9ccc89 100644 (file)
@@ -31,7 +31,7 @@ function test()
     suite.addTestCase({
         name: "CheckEmptyHighlight",
         description: "Should not be a highlight yet.",
-        test: (resolve, reject) => {
+        test(resolve, reject) {
             getHighlightRects((highlightRects) => {
                 InspectorTest.expectThat(highlightRects.length === 0, "Should not be a highlight yet.");
                 resolve();
@@ -42,7 +42,7 @@ function test()
     suite.addTestCase({
         name: "HighlightSmallRectViewportCoordinates",
         description: "Should create a highlight using viewport coordinates.",
-        test: (resolve, reject) => {
+        test(resolve, reject) {
             let rect = new WebInspector.Rect(0, 0, 100, 200);
             DOMAgent.highlightRect(rect.origin.x, rect.origin.y, rect.size.width, rect.size.height, color, outlineColor, useViewportCoordinates, (error) => {
                 dumpHighlightRects(resolve);
@@ -53,7 +53,7 @@ function test()
     suite.addTestCase({
         name: "HighlightSmallRectPageCoordinates",
         description: "Should create a highlight using page coordinates.",
-        test: (resolve, reject) => {
+        test(resolve, reject) {
             let rect = new WebInspector.Rect(0, 0, 100, 200);
             DOMAgent.highlightRect(rect.origin.x, rect.origin.y, rect.size.width, rect.size.height, color, outlineColor, usePageCoordinates, (error) => {
                 dumpHighlightRects(resolve);
@@ -63,8 +63,8 @@ function test()
 
     suite.addTestCase({
         name: "HighlightSmallRectUnspecifiedCoordinatesUsesViewportCoordinates",
-        description: "Should create a highlight using viewport coordinates if unspecificed.",
-        test: (resolve, reject) => {
+        description: "Should create a highlight using viewport coordinates if unspecified.",
+        test(resolve, reject) {
             let rect = new WebInspector.Rect(0, 0, 100, 200);
             DOMAgent.highlightRect(rect.origin.x, rect.origin.y, rect.size.width, rect.size.height, color, outlineColor, (error) => {
                 dumpHighlightRects(resolve);
@@ -75,7 +75,7 @@ function test()
     suite.addTestCase({
         name: "HighlightLargeRectViewportCoordinates",
         description: "Should create a large highlight using viewport coordinates.",
-        test: (resolve, reject) => {
+        test(resolve, reject) {
             let rect = new WebInspector.Rect(0, 0, 100000, 200000);
             DOMAgent.highlightRect(rect.origin.x, rect.origin.y, rect.size.width, rect.size.height, color, outlineColor, useViewportCoordinates, (error) => {
                 dumpHighlightRects(resolve);
@@ -86,7 +86,7 @@ function test()
     suite.addTestCase({
         name: "HighlightLargeRectPageCoordinates",
         description: "Should create a large highlight using page coordinates.",
-        test: (resolve, reject) => {
+        test(resolve, reject) {
             let rect = new WebInspector.Rect(0, 0, 100000, 200000);
             DOMAgent.highlightRect(rect.origin.x, rect.origin.y, rect.size.width, rect.size.height, color, outlineColor, usePageCoordinates, (error) => {
                 dumpHighlightRects(resolve);
@@ -97,7 +97,7 @@ function test()
     suite.addTestCase({
         name: "HighlightEmptyRect",
         description: "Should create an empty highlight.",
-        test: (resolve, reject) => {
+        test(resolve, reject) {
             let rect = new WebInspector.Rect(0, 0, 0, 0);
             DOMAgent.highlightRect(rect.origin.x, rect.origin.y, rect.size.width, rect.size.height, color, outlineColor, false, (error) => {
                 dumpHighlightRects(resolve);
index 511ad48..6e60c65 100644 (file)
@@ -1,6 +1,6 @@
 <html>
 <head>
-<script type="text/javascript" src="../../http/tests/inspector/resources/protocol-test.js"></script>
+<script src="../../http/tests/inspector/resources/protocol-test.js"></script>
 <script>
 function moveNode()
 {
index 4d3dfdf..49357db 100644 (file)
@@ -1,6 +1,6 @@
 <html>
 <head>
-<script type="text/javascript" src="../../http/tests/inspector/resources/protocol-test.js"></script>
+<script src="../../http/tests/inspector/resources/protocol-test.js"></script>
 <script>
 function test()
 {
index aec6ffd..3ecc370 100644 (file)
@@ -11,7 +11,7 @@ function test()
     suite.addTestCase({
         name: "OuterHTMLBefore",
         description: "Log the initial outerHTML of the target element.",
-        test: (resolve, reject) => {
+        test(resolve, reject) {
             domNode.getOuterHTML(function(error, outerHTML) {
                 InspectorTest.expectNoError(error);
                 InspectorTest.log(outerHTML);
@@ -49,7 +49,7 @@ function test()
         suite.addTestCase({
             name: `SetOuterHTML${title}`,
             description: "Change the outerHTML.",
-            test: (resolve, reject) => {
+            test(resolve, reject) {
                 domNode.setOuterHTML(replacementOuterHTML, function(error, outerHTML) {
                     InspectorTest.expectNoError(error);
                     resolve();
@@ -60,7 +60,7 @@ function test()
         suite.addTestCase({
             name: `CheckOuterHTMLAfter${title}`,
             description: "Log the outerHTML of the target element after changes.",
-            test: (resolve, reject) => {
+            test(resolve, reject) {
                 domNode.getOuterHTML(function(error, outerHTML) {
                     InspectorTest.expectNoError(error);
                     InspectorTest.expectThat(outerHTML === replacementOuterHTML, "The outerHTML should be what was just set.");
index d6a9848..07de2cc 100644 (file)
@@ -11,7 +11,7 @@ function test()
 
     suite.addTestCase({
         name: "ShadowRootType.UserAgent",
-        test: (resolve, reject) => {
+        test(resolve, reject) {
             WebInspector.domTreeManager.querySelector(documentNode.id, "#host-user-agent", (nodeId) => {
                 let domNode = WebInspector.domTreeManager.nodeForId(nodeId);
                 InspectorTest.assert(domNode, "Got DOMNode for #host-user-agent");
@@ -25,7 +25,7 @@ function test()
 
     suite.addTestCase({
         name: "ShadowRootType.Closed",
-        test: (resolve, reject) => {
+        test(resolve, reject) {
             WebInspector.domTreeManager.querySelector(documentNode.id, "#host-closed", (nodeId) => {
                 let domNode = WebInspector.domTreeManager.nodeForId(nodeId);
                 InspectorTest.assert(domNode, "Got DOMNode for #host-closed");
@@ -39,7 +39,7 @@ function test()
 
     suite.addTestCase({
         name: "ShadowRootType.Open",
-        test: (resolve, reject) => {
+        test(resolve, reject) {
             WebInspector.domTreeManager.querySelector(documentNode.id, "#host-open", (nodeId) => {
                 let domNode = WebInspector.domTreeManager.nodeForId(nodeId);
                 InspectorTest.assert(domNode, "Got DOMNode for #host-open");
@@ -53,7 +53,7 @@ function test()
 
     suite.addTestCase({
         name: "NoShadowRootType",
-        test: (resolve, reject) => {
+        test(resolve, reject) {
             WebInspector.domTreeManager.querySelector(documentNode.id, "#no-host", (nodeId) => {
                 let domNode = WebInspector.domTreeManager.nodeForId(nodeId);
                 InspectorTest.assert(domNode, "Got DOMNode for #no-host");
index 3745c67..ad9a435 100644 (file)
@@ -41,7 +41,7 @@ function test()
     suite.addTestCase({
         name: "EsprimaFormatter.ValidJSON",
         description: "Format valid JSON",
-        test: (resolve, reject) => {
+        test(resolve, reject) {
             let validJSON = JSON.stringify({"a":123,"b":[1,2,3],"c":{"d":"e"}});
             InspectorTest.log("JSON: " + doubleQuotedString(validJSON));
             ensureJSON(validJSON, "valid");
@@ -56,7 +56,7 @@ function test()
     suite.addTestCase({
         name: "EsprimaFormatter.InvalidButObject",
         description: "Format invalid JSON due to literal keys, but it can be evaluated as a valid JavaScript Object",
-        test: (resolve, reject) => {
+        test(resolve, reject) {
             let invalidJSON = `{a:123,b:[1,2,3],c:{d:"e"}}`;
             InspectorTest.log("JSON: " + doubleQuotedString(invalidJSON));
             ensureJSON(invalidJSON, "invalid");
@@ -72,7 +72,7 @@ function test()
     suite.addTestCase({
         name: "EsprimaFormatter.InvalidWithComments",
         description: "Format invalid JSON due to comments, but it can be evaluated as a valid JavaScript Object",
-        test: (resolve, reject) => {
+        test(resolve, reject) {
             let invalidJSON = JSON.stringify({"a":123,"b":[1,2,3],"c":{"d":"e"}}) + "/*Comment*/";
             InspectorTest.log("JSON: " + doubleQuotedString(invalidJSON));
             ensureJSON(invalidJSON, "invalid");
@@ -88,7 +88,7 @@ function test()
     suite.addTestCase({
         name: "EsprimaFormatter.Invalid",
         description: "Format invalid JSON, that is not a valid JavaScript Object",
-        test: (resolve, reject) => {
+        test(resolve, reject) {
             let invalid = `{a:1 b:2}`;
             InspectorTest.log("INPUT: " + doubleQuotedString(invalid));
             ensureJSON(invalid, "invalid");
index 7abfb90..42925af 100644 (file)
@@ -9,7 +9,7 @@ function test()
 
     suite.addTestCase({
         name: "JavaScriptShortSnippet.Unminified",
-        test: (resolve, reject) => {
+        test(resolve, reject) {
             InspectorTest.expectThat(!isTextLikelyMinified("var x = 42;\nvar y = 24;\n"), "This small JavaScript snippet should be classified as unminified.");
             resolve();
         }
@@ -91,7 +91,7 @@ var support = {};`;
 
     suite.addTestCase({
         name: "JQuerySnippet.Unminified",
-        test: (resolve, reject) => {
+        test(resolve, reject) {
             InspectorTest.expectThat(!isTextLikelyMinified(jquerySourceFragment), "jQuery snippet should be classified as unminified.");
             resolve();
         }
@@ -102,7 +102,7 @@ var support = {};`;
 
     suite.addTestCase({
         name: "JQuerySnippet.Minified",
-        test: (resolve, reject) => {
+        test(resolve, reject) {
             InspectorTest.expectThat(isTextLikelyMinified(jQueryMinifiedSourceFragment), "jQuery snippet should be classified as minified.");
             resolve();
         }
index b164e9d..9871d40 100644 (file)
@@ -50,7 +50,7 @@ TestPage.registerInitializer(function() {
             let testURL = testPageResourcesURL + "/" + test;
             suite.addTestCase({
                 name: suite.name + "." + testName,
-                test: (resolve, reject) => {
+                test(resolve, reject) {
                     runFormattingTest(mode, testName, testURL).then(resolve).catch(reject);
                 }
             });
index 52ee66f..d0de959 100644 (file)
@@ -36,7 +36,7 @@ function test()
     suite.addTestCase({
         name: "TriggerGCs",
         description: "Evaluating allocation-heavy JavaScript should trigger Heap.garbageCollected events.",
-        test: (resolve, reject) => {
+        test(resolve, reject) {
             InspectorTest.evaluateInPage("periodicallyTriggerGCs()");
 
             let seenFull = false;
index 65dcf91..ea58b59 100644 (file)
@@ -10,7 +10,7 @@ function test()
     suite.addTestCase({
         name: "TriggerGCShouldCreateGCEvent",
         description: "Calling Heap.gc should trigger Heap.garbageCollected event if enabled.",
-        test: (resolve, reject) => {
+        test(resolve, reject) {
             HeapAgent.gc();
             let resolving = false;
             WebInspector.heapManager.addEventListener(WebInspector.HeapManager.Event.GarbageCollected, (event) => {
index 1061386..4257f4b 100644 (file)
@@ -35,7 +35,7 @@ function test()
     suite.addTestCase({
         name: "GetPreviewNoSnapshot",
         description: "Calling Heap.getPreview when no snapshot exists should result in an error.",
-        test: (resolve, reject) => {
+        test(resolve, reject) {
             HeapAgent.getPreview(1, (error, string, functionDetails, objectPreviewPayload) => {
                 InspectorTest.expectThat(error, "Should get an error when no snapshot exists.");
                 InspectorTest.pass(error);
@@ -47,7 +47,7 @@ function test()
     suite.addTestCase({
         name: "GetPreviewForString",
         description: "Calling Heap.getPreview for a live string should return the value of that string.",
-        test: (resolve, reject) => {
+        test(resolve, reject) {
             HeapAgent.snapshot((error, timestamp, snapshotStringData) => { // All pre-existing objects.
                 InspectorTest.evaluateInPage("triggerCreateStringObject()");
                 HeapAgent.snapshot((error, timestamp, snapshotStringData) => { // Newly created objects.
@@ -72,7 +72,7 @@ function test()
     suite.addTestCase({
         name: "GetPreviewForFunction",
         description: "Calling Heap.getPreview for a live Function should return function details for that value.",
-        test: (resolve, reject) => {
+        test(resolve, reject) {
             HeapAgent.snapshot((error, timestamp, snapshotStringData) => { // All pre-existing objects.
                 InspectorTest.evaluateInPage("triggerCreateFunctionObject()");
                 HeapAgent.snapshot((error, timestamp, snapshotStringData) => { // Newly created objects.
@@ -97,7 +97,7 @@ function test()
     suite.addTestCase({
         name: "GetPreviewForObject",
         description: "Calling Heap.getPreview for a live Object should return an object preview for that value.",
-        test: (resolve, reject) => {
+        test(resolve, reject) {
             HeapAgent.snapshot((error, timestamp, snapshotStringData) => { // All pre-existing objects.
                 InspectorTest.evaluateInPage("triggerCreateMapObject()");
                 HeapAgent.snapshot((error, timestamp, snapshotStringData) => { // Newly created objects.
@@ -122,7 +122,7 @@ function test()
     suite.addTestCase({
         name: "GetPreviewBadIdentifier",
         description: "Calling Heap.getPreview with a bad identifier should result in an error.",
-        test: (resolve, reject) => {
+        test(resolve, reject) {
             HeapAgent.snapshot((error, timestamp, snapshotStringData) => {
                 InspectorTest.expectThat(!error, "Should not have an error creating a snapshot.");
                 HeapAgent.getPreview(9999999, (error, string, functionDetails, objectPreviewPayload) => {
@@ -137,7 +137,7 @@ function test()
     suite.addTestCase({
         name: "GetRemoteObjectCollectedObject",
         description: "Calling Heap.getRemoteObject for an object that has been collected should result in an error.",
-        test: (resolve, reject) => {
+        test(resolve, reject) {
             HeapAgent.snapshot((error, timestamp, snapshotStringData) => { // All pre-existing objects.
                 InspectorTest.evaluateInPage("triggerCreateMapObject()");
                 HeapAgent.snapshot((error, timestamp, snapshotStringData) => { // Newly created objects.
index 1edd1f8..bc9c035 100644 (file)
@@ -18,7 +18,7 @@ function test()
     suite.addTestCase({
         name: "GetRemoteObjectNoSnapshot",
         description: "Calling Heap.getRemoteObject when no snapshot exists should result in an error.",
-        test: (resolve, reject) => {
+        test(resolve, reject) {
             HeapAgent.getRemoteObject(1, "test", (error, remoteObjectPayload) => {
                 InspectorTest.expectThat(error, "Should get an error when no snapshot exists.");
                 InspectorTest.pass(error);
@@ -30,7 +30,7 @@ function test()
     suite.addTestCase({
         name: "GetRemoteObjectForWindow",
         description: "Calling Heap.getRemoteObject for a live value should return a remote object for that value.",
-        test: (resolve, reject) => {
+        test(resolve, reject) {
             HeapAgent.snapshot((error, timestamp, snapshotStringData) => {
                 InspectorTest.expectThat(!error, "Should not have an error creating a snapshot.");
                 let workerProxy = WebInspector.HeapSnapshotWorkerProxy.singleton();
@@ -55,7 +55,7 @@ function test()
     suite.addTestCase({
         name: "GetRemoteObjectBadIdentifier",
         description: "Calling Heap.getRemoteObject with a bad identifier should result in an error.",
-        test: (resolve, reject) => {
+        test(resolve, reject) {
             HeapAgent.snapshot((error, timestamp, snapshotStringData) => {
                 InspectorTest.expectThat(!error, "Should not have an error creating a snapshot.");
                 HeapAgent.getRemoteObject(9999999, "test", (error, remoteObjectPayload) => {
@@ -70,7 +70,7 @@ function test()
     suite.addTestCase({
         name: "GetRemoteObjectCollectedObject",
         description: "Calling Heap.getRemoteObject for an object that has been collected should result in an error.",
-        test: (resolve, reject) => {
+        test(resolve, reject) {
             HeapAgent.snapshot((error, timestamp, snapshotStringData) => { // All pre-existing objects.
                 InspectorTest.evaluateInPage("triggerCreateMapObject()");
                 HeapAgent.snapshot((error, timestamp, snapshotStringData) => { // Newly created objects.
index 2248913..96072ae 100644 (file)
@@ -10,7 +10,7 @@ function test()
     suite.addTestCase({
         name: "TriggerSnapshot",
         description: "Calling Heap.snapshot should create a heap snapshot.",
-        test: (resolve, reject) => {
+        test(resolve, reject) {
             HeapAgent.snapshot((error, timestamp, snapshotStringData) => {
                 InspectorTest.expectThat(!error, "Should not have an error creating a snapshot.");
                 let workerProxy = WebInspector.HeapSnapshotWorkerProxy.singleton();
index 205b3cb..2bb83d0 100644 (file)
@@ -9,7 +9,7 @@ function test()
 
     suite.addTestCase({
         name: "StartAndStopTrackingIncludeSnapshots",
-        test: function(resolve, reject) {
+        test(resolve, reject) {
             InspectorProtocol.awaitEvent({event: "Heap.trackingStart"}).then((messageObject) => {
                 ProtocolTest.log("Heap.trackingStart");
                 ProtocolTest.expectThat(typeof messageObject.params.timestamp === "number", "Should have a timestamp when starting.");
index ec745a2..943bf82 100644 (file)
@@ -9,7 +9,7 @@ CONSOLE MESSAGE: line 19: Created Database 'Database3'
 PASS: Should be 4 entries.
 PASS: Should not have more entries.
 PASS: Primary key should be ordered by primary key: 1
-PASS: Value should be a remote object for: 'Thirsty Hampster'
+PASS: Value should be a remote object for: 'Thirsty Hamster'
 PASS: Primary key should be ordered by primary key: 2
 PASS: Value should be a remote object for: 'Jamming Peacock'
 PASS: Primary key should be ordered by primary key: 3
@@ -29,8 +29,8 @@ PASS: Primary key should be ordered by name: 4
 PASS: Key should be ordered by name: 'Monstrous Beaver'
 PASS: Value should be a remote object for: 'Monstrous Beaver'
 PASS: Primary key should be ordered by name: 1
-PASS: Key should be ordered by name: 'Thirsty Hampster'
-PASS: Value should be a remote object for: 'Thirsty Hampster'
+PASS: Key should be ordered by name: 'Thirsty Hamster'
+PASS: Value should be a remote object for: 'Thirsty Hamster'
 -- Email Index
 PASS: Should be 4 entries.
 PASS: Should not have more entries.
@@ -43,8 +43,8 @@ PASS: Value should be a remote object for: 'Monstrous Beaver'
 FAIL: Primary key should be ordered by email: 2
     Expected: truthy
     Actual: false
-PASS: Key should be ordered by email: 'hampster@webkit.org'
-PASS: Value should be a remote object for: 'Thirsty Hampster'
+PASS: Key should be ordered by email: 'hamster@webkit.org'
+PASS: Value should be a remote object for: 'Thirsty Hamster'
 FAIL: Primary key should be ordered by email: 1
     Expected: truthy
     Actual: false
@@ -56,7 +56,7 @@ PASS: Value should be a remote object for: 'Jamming Peacock'
 PASS: Entries length should match page size.
 PASS: Should have more entries.
 PASS: Primary key should be ordered by primary key: 1
-PASS: Value should be a remote object for: 'Thirsty Hampster'
+PASS: Value should be a remote object for: 'Thirsty Hamster'
 PASS: Primary key should be ordered by primary key: 2
 PASS: Value should be a remote object for: 'Jamming Peacock'
 -- Page 2
@@ -76,7 +76,7 @@ PASS: Should not have more entries.
 PASS: Primary key should be ordered by primary key: 4
 PASS: Value should be a remote object for: 'Monstrous Beaver'
 PASS: Primary key should be ordered by primary key: 1
-PASS: Value should be a remote object for: 'Thirsty Hampster'
+PASS: Value should be a remote object for: 'Thirsty Hamster'
 -- Keys > 'M'
 -- Lower Bound: 'M' (open)
 -- Upper Bound: -
@@ -85,14 +85,14 @@ PASS: Should not have more entries.
 PASS: Primary key should be ordered by primary key: 4
 PASS: Value should be a remote object for: 'Monstrous Beaver'
 PASS: Primary key should be ordered by primary key: 1
-PASS: Value should be a remote object for: 'Thirsty Hampster'
+PASS: Value should be a remote object for: 'Thirsty Hamster'
 -- Keys > 'Monstrous Beaver'
 -- Lower Bound: 'Monstrous Beaver' (open)
 -- Upper Bound: -
 PASS: Should be 1 entry.
 PASS: Should not have more entries.
 PASS: Primary key should be ordered by primary key: 1
-PASS: Value should be a remote object for: 'Thirsty Hampster'
+PASS: Value should be a remote object for: 'Thirsty Hamster'
 -- Keys <= 'M'
 -- Lower Bound: -
 -- Upper Bound: 'M' (closed)
index bc6a0ed..6eefa6e 100644 (file)
@@ -19,7 +19,7 @@ function test()
     suite.addTestCase({
         name: "ClearDatabases",
         description: "Remove any existing IndexedDB databases.",
-        test: (resolve, reject) => {
+        test(resolve, reject) {
             // FIXME: Remove any existing IndexedDB databases that might exist to workaround:
             // <https://webkit.org/b/148006> Each test should run with its own clean data store
             IndexedDBAgent.requestDatabaseNames(WebInspector.frameResourceManager.mainFrame.securityOrigin, (error, names) => {
@@ -31,7 +31,7 @@ function test()
     suite.addTestCase({
         name: "RequestDataBasic",
         description: "Request data for an object store with different indexes.",
-        test: (resolve, reject) => {
+        test(resolve, reject) {
             InspectorTest.evaluateInPage("createDatabaseWithStores('Database1', 1)");
             IndexedDBAgent.requestDatabase(WebInspector.frameResourceManager.mainFrame.securityOrigin, "Database1", (error, database) => {
                 InspectorTest.expectNoError(error);
@@ -49,7 +49,7 @@ function test()
                     InspectorTest.expectThat(!hasMore, "Should not have more entries.");
 
                     let expectedPrimaryKeyOrder = [1, 2, 3, 4];
-                    let expectedNamesOrderedByPrimaryKey = ["Thirsty Hampster", "Jamming Peacock", "Bustling Badger", "Monstrous Beaver"];
+                    let expectedNamesOrderedByPrimaryKey = ["Thirsty Hamster", "Jamming Peacock", "Bustling Badger", "Monstrous Beaver"];
                     for (let i = 0; i < entries.length; ++i) {
                         let entryPayload = entries[i];
                         let primaryKey = WebInspector.RemoteObject.fromPayload(entryPayload.primaryKey);
@@ -67,7 +67,7 @@ function test()
                     InspectorTest.expectThat(!hasMore, "Should not have more entries.");
 
                     let expectedPrimaryKeyOrder = [3, 2, 4, 1];
-                    let expectedKeysOrderedByName = ["Bustling Badger", "Jamming Peacock", "Monstrous Beaver", "Thirsty Hampster"];
+                    let expectedKeysOrderedByName = ["Bustling Badger", "Jamming Peacock", "Monstrous Beaver", "Thirsty Hamster"];
                     let expectedNamesOrderedByName = expectedKeysOrderedByName;
                     for (let i = 0; i < entries.length; ++i) {
                         let entryPayload = entries[i];
@@ -88,8 +88,8 @@ function test()
                     InspectorTest.expectThat(!hasMore, "Should not have more entries.");
 
                     let expectedPrimaryKeyOrder = [3, 4, 2, 1];
-                    let expectedKeysOrderedByEmail = ["badger@webkit.org", "beaver@webkit.org", "hampster@webkit.org", "peacock@webkit.org"];
-                    let expectedNamesOrderedByEmail = ["Bustling Badger", "Monstrous Beaver", "Thirsty Hampster", "Jamming Peacock"];
+                    let expectedKeysOrderedByEmail = ["badger@webkit.org", "beaver@webkit.org", "hamster@webkit.org", "peacock@webkit.org"];
+                    let expectedNamesOrderedByEmail = ["Bustling Badger", "Monstrous Beaver", "Thirsty Hamster", "Jamming Peacock"];
                     for (let i = 0; i < entries.length; ++i) {
                         let entryPayload = entries[i];
                         let primaryKey = WebInspector.RemoteObject.fromPayload(entryPayload.primaryKey);
@@ -109,7 +109,7 @@ function test()
     suite.addTestCase({
         name: "RequestDataPagination",
         description: "Request data for an object store with paginated requests.",
-        test: (resolve, reject) => {
+        test(resolve, reject) {
             InspectorTest.evaluateInPage("createDatabaseWithStores('Database2', 1)");
             IndexedDBAgent.requestDatabase(WebInspector.frameResourceManager.mainFrame.securityOrigin, "Database2", (error, database) => {
                 InspectorTest.expectNoError(error);
@@ -127,7 +127,7 @@ function test()
                     InspectorTest.expectThat(hasMore, "Should have more entries.");
 
                     let expectedPrimaryKeyOrder = [1, 2];
-                    let expectedNamesOrderedByPrimaryKey = ["Thirsty Hampster", "Jamming Peacock"];
+                    let expectedNamesOrderedByPrimaryKey = ["Thirsty Hamster", "Jamming Peacock"];
                     for (let i = 0; i < entries.length; ++i) {
                         let entryPayload = entries[i];
                         let primaryKey = WebInspector.RemoteObject.fromPayload(entryPayload.primaryKey);
@@ -163,7 +163,7 @@ function test()
     suite.addTestCase({
         name: "RequestDataKeyRangeBounds",
         description: "Request key range bounded data for an object store.",
-        test: (resolve, reject) => {
+        test(resolve, reject) {
             InspectorTest.evaluateInPage("createDatabaseWithStores('Database3', 1)");
             IndexedDBAgent.requestDatabase(WebInspector.frameResourceManager.mainFrame.securityOrigin, "Database3", (error, database) => {
                 InspectorTest.expectNoError(error);
@@ -180,7 +180,7 @@ function test()
 
                 // Index: Name Index
                 // Keys: [3, 2, 4, 1]
-                // Values: ["Bustling Badger", "Jamming Peacock", "Monstrous Beaver", "Thirsty Hampster"];
+                // Values: ["Bustling Badger", "Jamming Peacock", "Monstrous Beaver", "Thirsty Hamster"];
 
                 // Keys >= "M"
                 let keyRange = {lower: stringKeyWithString("M"), lowerOpen: false, upperOpen: true};
@@ -193,7 +193,7 @@ function test()
                     InspectorTest.expectThat(!hasMore, "Should not have more entries.");
 
                     let expectedPrimaryKeyOrder = [4, 1];
-                    let expectedNamesOrderedByPrimaryKey = ["Monstrous Beaver", "Thirsty Hampster"];
+                    let expectedNamesOrderedByPrimaryKey = ["Monstrous Beaver", "Thirsty Hamster"];
                     for (let i = 0; i < entries.length; ++i) {
                         let entryPayload = entries[i];
                         let primaryKey = WebInspector.RemoteObject.fromPayload(entryPayload.primaryKey);
@@ -214,7 +214,7 @@ function test()
                     InspectorTest.expectThat(!hasMore, "Should not have more entries.");
 
                     let expectedPrimaryKeyOrder = [4, 1];
-                    let expectedNamesOrderedByPrimaryKey = ["Monstrous Beaver", "Thirsty Hampster"];
+                    let expectedNamesOrderedByPrimaryKey = ["Monstrous Beaver", "Thirsty Hamster"];
                     for (let i = 0; i < entries.length; ++i) {
                         let entryPayload = entries[i];
                         let primaryKey = WebInspector.RemoteObject.fromPayload(entryPayload.primaryKey);
@@ -235,7 +235,7 @@ function test()
                     InspectorTest.expectThat(!hasMore, "Should not have more entries.");
 
                     let expectedPrimaryKeyOrder = [1];
-                    let expectedNamesOrderedByPrimaryKey = ["Thirsty Hampster"];
+                    let expectedNamesOrderedByPrimaryKey = ["Thirsty Hamster"];
                     for (let i = 0; i < entries.length; ++i) {
                         let entryPayload = entries[i];
                         let primaryKey = WebInspector.RemoteObject.fromPayload(entryPayload.primaryKey);
index 1247557..e2892fd 100644 (file)
@@ -82,7 +82,7 @@ function test()
     // suite.addTestCase({
     //     name: "NoSuchDatabase",
     //     description: "Request a database that does not exist.",
-    //     test: (resolve, reject) => {
+    //     test(resolve, reject) {
     //         IndexedDBAgent.requestDatabase(WebInspector.frameResourceManager.mainFrame.securityOrigin, "NameDoesNotExist", (error, databaseWithObjectStores) => {
     //             InspectorTest.expectThat(error, "Should be an error trying to request a database that does not exist.");
     //             InspectorTest.log(JSON.stringify(databaseWithObjectStores));
index 3f06fb3..93d5e37 100644 (file)
@@ -39,7 +39,7 @@ function createDatabaseWithStores(name, version) {
         // Populate once stores are created.
         lastObjectStore.transaction.oncomplete = (event) => {
             let reviewerObjectStore = db.transaction("Reviewers", "readwrite").objectStore("Reviewers");
-            reviewerObjectStore.add({name: "Thirsty Hampster", email: "hampster@webkit.org"});
+            reviewerObjectStore.add({name: "Thirsty Hamster", email: "hamster@webkit.org"});
             reviewerObjectStore.add({name: "Jamming Peacock", email: "peacock@webkit.org"});
             reviewerObjectStore.add({name: "Bustling Badger", email: "badger@webkit.org"});
             reviewerObjectStore.add({name: "Monstrous Beaver", email: "beaver@webkit.org"});
index 004227d..e8852ad 100644 (file)
@@ -1,7 +1,7 @@
 <html>
 <head>
-<script type="text/javascript" src="../../http/tests/inspector/resources/protocol-test.js"></script>
-<script type="text/javascript">
+<script src="../../http/tests/inspector/resources/protocol-test.js"></script>
+<script>
 
 function test()
 {
index fc887b7..fdd16e3 100644 (file)
@@ -1,7 +1,7 @@
 <html>
 <head>
-<script type="text/javascript" src="../../http/tests/inspector/resources/protocol-test.js"></script>
-<script type="text/javascript">
+<script src="../../http/tests/inspector/resources/protocol-test.js"></script>
+<script>
 
 function test()
 {
index bd2cff4..bc7039a 100644 (file)
@@ -1,7 +1,7 @@
 <html>
 <head>
-<script type="text/javascript" src="../../http/tests/inspector/resources/protocol-test.js"></script>
-<script type="text/javascript">
+<script src="../../http/tests/inspector/resources/protocol-test.js"></script>
+<script>
 
 function test()
 {
index d5fed01..53b9e20 100644 (file)
@@ -1,7 +1,7 @@
 <html>
 <head>
-<script type="text/javascript" src="../../http/tests/inspector/resources/protocol-test.js"></script>
-<script type="text/javascript">
+<script src="../../http/tests/inspector/resources/protocol-test.js"></script>
+<script>
 
 function addCompositedLayer()
 {
index ee9da7d..5fae740 100644 (file)
@@ -1,7 +1,7 @@
 <html>
 <head>
-<script type="text/javascript" src="../../http/tests/inspector/resources/protocol-test.js"></script>
-<script type="text/javascript">
+<script src="../../http/tests/inspector/resources/protocol-test.js"></script>
+<script>
 
 function test()
 {
index b2fc90f..c607027 100644 (file)
@@ -1,7 +1,7 @@
 <html>
 <head>
-<script type="text/javascript" src="../../http/tests/inspector/resources/protocol-test.js"></script>
-<script type="text/javascript">
+<script src="../../http/tests/inspector/resources/protocol-test.js"></script>
+<script>
 
 function test()
 {
index 3af006b..bee202a 100644 (file)
@@ -9,7 +9,7 @@ function test()
 
     suite.addTestCase({
         name: "StartAndStopTrackingWithEvent",
-        test: function(resolve, reject) {
+        test(resolve, reject) {
             InspectorProtocol.awaitEvent({event: "Memory.trackingStart"}).then((messageObject) => {
                 ProtocolTest.log("Memory.trackingStart");
                 ProtocolTest.expectThat(typeof messageObject.params.timestamp === "number", "Should have a timestamp when starting.");
index ba669be..902787a 100644 (file)
@@ -37,7 +37,7 @@ function test()
     suite.addTestCase({
         name: "WebInspector.Color.fromString",
         description: "Test we can detect colors from strings.",
-        test: (resolve, reject) => {
+        test(resolve, reject) {
             function testGood(string, expectedFormat) {
                 let color = WebInspector.Color.fromString(string);
                 InspectorTest.expectThat(color instanceof WebInspector.Color, `'${string}' should be detected`);
@@ -114,7 +114,7 @@ function test()
     suite.addTestCase({
         name: "WebInspector.Color properties",
         description: "Test different color properties.",
-        test: (resolve, reject) => {
+        test(resolve, reject) {
             function shallowEqual(arr1, arr2) {
                 if (arr1.length !== arr2.length)
                     return false;
@@ -225,7 +225,7 @@ function test()
     suite.addTestCase({
         name: "WebInspector.Color.prototype.toString",
         description: "Test the different toString outputs.",
-        test: (resolve, reject) => {
+        test(resolve, reject) {
             let color;
             function test(expected, format) {
                 let pass = color.toString(format) === expected;
index 76d112d..ca1144d 100644 (file)
@@ -1,7 +1,7 @@
 <!doctype html>
 <html>
 <head>
-<script type="text/javascript" src="../../http/tests/inspector/resources/inspector-test.js"></script>
+<script src="../../http/tests/inspector/resources/inspector-test.js"></script>
 <script>
 function test()
 {
index a0ee541..634bee6 100644 (file)
@@ -1,7 +1,7 @@
 <!doctype html>
 <html>
 <head>
-<script type="text/javascript" src="../../http/tests/inspector/resources/inspector-test.js"></script>
+<script src="../../http/tests/inspector/resources/inspector-test.js"></script>
 <script>
 // ------------------------------
 //   Setup Test Prototype Chain
index bc286f6..650a211 100644 (file)
@@ -2,7 +2,7 @@
 <html>
 <head>
 <meta http-equiv="Content-Type" content="text/html; charset=utf-8">
-<script type="text/javascript" src="../../http/tests/inspector/resources/inspector-test.js"></script>
+<script src="../../http/tests/inspector/resources/inspector-test.js"></script>
 <script>
 strongKey1 = {id:1};
 strongKey2 = {id:2};
index 7aabd2b..7a356e0 100644 (file)
@@ -2,7 +2,7 @@
 <html>
 <head>
 <meta http-equiv="Content-Type" content="text/html; charset=utf-8">
-<script type="text/javascript" src="../../http/tests/inspector/resources/inspector-test.js"></script>
+<script src="../../http/tests/inspector/resources/inspector-test.js"></script>
 <script>
 function test()
 {
index a384a5e..f1b1374 100644 (file)
@@ -18,7 +18,7 @@ function test()
     suite.addTestCase({
         name: "TriggerBlockedResourceLoad",
         description: "Trigger a blocked resource load and ensure we get notified of the request.",
-        test: (resolve, reject) => {
+        test(resolve, reject) {
             InspectorTest.evaluateInPage("createBlockedResourceLoad()");
             WebInspector.Frame.singleFireEventListener(WebInspector.Frame.Event.ResourceWasAdded, (event) => {
                 let resource = event.data.resource;
index e407d91..010cc59 100644 (file)
@@ -25,7 +25,7 @@ function test()
     suite.addTestCase({
         name: "XHR.JSONContent",
         description: "Ensure an XMLHttpRequest with JSON content still gives us text.",
-        test: (resolve, reject) => {
+        test(resolve, reject) {
             InspectorTest.evaluateInPage("createJSONXHR()");
             WebInspector.Frame.singleFireEventListener(WebInspector.Frame.Event.ResourceWasAdded, (event) => {
                 let resource = event.data.resource;
@@ -43,7 +43,7 @@ function test()
     suite.addTestCase({
         name: "XHR.JSONContent.Blob",
         description: "Ensure an XMLHttpRequest with JSON content and a responseType of blob still gives us text.",
-        test: (resolve, reject) => {
+        test(resolve, reject) {
             InspectorTest.evaluateInPage("createJSONBlobXHR()");
             WebInspector.Frame.singleFireEventListener(WebInspector.Frame.Event.ResourceWasAdded, (event) => {
                 let resource = event.data.resource;
index 2bcb2a4..80691d0 100644 (file)
@@ -1,6 +1,6 @@
 <html>
 <head>
-<script type="text/javascript" src="../../http/tests/inspector/resources/protocol-test.js"></script>
+<script src="../../http/tests/inspector/resources/protocol-test.js"></script>
 <script>
 function test()
 {
index c81e7ff..d2938aa 100644 (file)
@@ -10,7 +10,7 @@ function test()
     suite.addTestCase({
         name: "MatchCookiesAgainstResourceDomains",
         description: "Test that Inspector properly detects whether a cookie matches a resource's domain or not.",
-        test: (resolve, reject) => {
+        test(resolve, reject) {
             let testcasesByCookieDomain = {
                 ".example.com": [
                     // Unrelated top-level domain.
index 44d8bae..f149eb6 100644 (file)
@@ -1,6 +1,6 @@
 <html>
 <head>
-<script type="text/javascript" src="../../http/tests/inspector/resources/protocol-test.js"></script>
+<script src="../../http/tests/inspector/resources/protocol-test.js"></script>
 <script>
 
 function load()
index 1fb2d03..3f7280c 100644 (file)
@@ -1,6 +1,6 @@
 <html>
 <head>
-<script type="text/javascript" src="../../http/tests/inspector/resources/protocol-test.js"></script>
+<script src="../../http/tests/inspector/resources/protocol-test.js"></script>
 <script>
 
 function load()
index 9470c9f..db17a92 100644 (file)
@@ -1,7 +1,7 @@
 <!doctype html>
 <html>
 <head>
-<script type="text/javascript" src="../../http/tests/inspector/resources/inspector-test.js"></script>
+<script src="../../http/tests/inspector/resources/inspector-test.js"></script>
 <script>
 function test()
 {
index 05aa7e4..ad426eb 100644 (file)
@@ -23,7 +23,7 @@ function test()
         suite.addTestCase({
             name,
             description,
-            test: (resolve, reject) => {
+            test(resolve, reject) {
                 let stringifiedMessage = (typeof message !== "string") ? JSON.stringify(message) : message;
                 ProtocolTest.log("Sending message: " + stringifiedMessage);
                 InspectorProtocol.awaitMessage(message)
index 4552120..ce0d77a 100644 (file)
@@ -23,7 +23,7 @@ function test()
         suite.addTestCase({
             name,
             description,
-            test: (resolve, reject) => {
+            test(resolve, reject) {
                 let stringifiedMessage = (typeof message !== "string") ? JSON.stringify(message) : message;
                 ProtocolTest.log("Sending message: " + stringifiedMessage);
                 InspectorProtocol.awaitMessage(message)
index 6f9e90b..09877e6 100644 (file)
@@ -13,7 +13,7 @@ function test()
     suite.addTestCase({
         name: "ResolveCommandPromiseOnSuccess",
         description: "Backend command's returned promise should be resolved if the command is successful.",
-        test: (resolve, reject) => {
+        test(resolve, reject) {
             let returnValue = RuntimeAgent.evaluate("42");
 
             InspectorTest.expectThat(returnValue instanceof Promise, "A backend command should return a Promise when invoked without a callback.");
@@ -34,7 +34,7 @@ function test()
     suite.addTestCase({
         name: "RejectCommandPromiseWithInvalidArguments",
         description: "Backend command's returned promise should be rejected if the command has invalid arguments.",
-        test: (resolve, reject) => {
+        test(resolve, reject) {
             let result = RuntimeAgent.evaluate(42);
 
             InspectorTest.expectThat(result instanceof Promise, "A backend command should return a Promise when invoked without a callback.");
@@ -55,7 +55,7 @@ function test()
     suite.addTestCase({
         name: "RejectCommandPromiseWithMissingArguments",
         description: "Backend command's returned promise should be rejected if the command lacks required arguments.",
-        test: (resolve, reject) => {
+        test(resolve, reject) {
             let result = RuntimeAgent.evaluate();
 
             InspectorTest.expectThat(result instanceof Promise, "A backend command should return a Promise when invoked without a callback.");
@@ -76,7 +76,7 @@ function test()
     suite.addTestCase({
         name: "RejectCommandPromiseWithExtraArgumentThatIsNotCallback",
         description: "Backend command's returned promise should be rejected if the command has an extra argument that is not a callback.",
-        test: (resolve, reject) => {
+        test(resolve, reject) {
             let result = RuntimeAgent.enable(1);
 
             InspectorTest.expectThat(result instanceof Promise, "A backend command should return a Promise when invoked without a callback.");
@@ -99,7 +99,7 @@ function test()
     suite.addTestCase({
         name: "ReturnNothingIfCallback",
         description: "Backend commands should not return anything if a callback is supplied.",
-        test: (resolve, reject) => {
+        test(resolve, reject) {
             let returnValue = RuntimeAgent.evaluate("41", dummyCallback);
             InspectorTest.expectThat(returnValue === undefined, "A backend command should not have a return value when invoked with a callback.");
             resolve();
@@ -109,7 +109,7 @@ function test()
     suite.addTestCase({
         name: "InvokeCallbackWithResultOnSuccess",
         description: "Backend command callback should be invoked with a result if the command is successful.",
-        test: (resolve, reject) => {
+        test(resolve, reject) {
             let initialState = 1;
             let returnValue = RuntimeAgent.evaluate("42", function(error, result) {
                 InspectorTest.expectThat(error === null, "A successful command should invoke the callback with a 'null' first parameter.");
@@ -126,7 +126,7 @@ function test()
     suite.addTestCase({
         name: "InvokeCallbackWithErrorForInvalidArguments",
         description: "Backend command callback should be invoked with an error if the command has invalid arguments.",
-        test: (resolve, reject) => {
+        test(resolve, reject) {
             let initialState = 1;
             let returnValue = RuntimeAgent.evaluate(42, function callback(error) {
                 InspectorTest.expectThat(typeof error === "string", "A failed command should invoke the callback with a string error message as its first parameter.");
@@ -143,7 +143,7 @@ function test()
     suite.addTestCase({
         name: "InvokeCallbackWithErrorForMissingArguments",
         description: "Backend command callback should be invoked with an error if the command lacks required arguments.",
-        test: (resolve, reject) => {
+        test(resolve, reject) {
             let initialState = 1;
             let returnValue = RuntimeAgent.evaluate(function callback(error) {
                 InspectorTest.expectThat(typeof error === "string", "A failed command should invoke the callback with a string error message as its first parameter.");
index 6d6b49b..4044e71 100644 (file)
@@ -1,7 +1,7 @@
 <html>
 <head>
-<script type="text/javascript" src="../../http/tests/inspector/resources/inspector-test.js"></script>
-<script type="text/javascript" src="../../http/tests/inspector/replay/replay-test.js"></script>
+<script src="../../http/tests/inspector/resources/inspector-test.js"></script>
+<script src="../../http/tests/inspector/replay/replay-test.js"></script>
 <script>
 
 window._times = [Date.now(), new Date().toString()];
index 9391f52..4456f22 100644 (file)
@@ -1,7 +1,7 @@
 <html>
 <head>
-<script type="text/javascript" src="../../http/tests/inspector/resources/inspector-test.js"></script>
-<script type="text/javascript" src="../../http/tests/inspector/replay/replay-test.js"></script>
+<script src="../../http/tests/inspector/resources/inspector-test.js"></script>
+<script src="../../http/tests/inspector/replay/replay-test.js"></script>
 <script>
 
 window._randomNumber = Math.random();
index 3bb2b39..04fc339 100644 (file)
@@ -1,7 +1,7 @@
 <html>
 <head>
-<script type="text/javascript" src="../../http/tests/inspector/resources/inspector-test.js"></script>
-<script type="text/javascript" src="../../http/tests/inspector/replay/replay-test.js"></script>
+<script src="../../http/tests/inspector/resources/inspector-test.js"></script>
+<script src="../../http/tests/inspector/replay/replay-test.js"></script>
 <script>
 function shallowCopy(object, properties) {
     var copiedObject = {};
index 724f811..9f05285 100644 (file)
@@ -36,7 +36,7 @@ function test()
     suite.addTestCase({
         name: "AccessCommandLineAPI",
         description: "Test evaluate can access CommandLineAPI methods.",
-        test: (resolve, reject) => {
+        test(resolve, reject) {
             testEvaluate("keys.toString()", (resultValue) => {
                 InspectorTest.expectThat(resultValue.includes("[Command Line API]"), "CommandLineAPI `keys` can be accessed.");
             });
@@ -50,7 +50,7 @@ function test()
     suite.addTestCase({
         name: "AccessGlobalVariable",
         description: "Test evaluate can access global variables.",
-        test: (resolve, reject) => {
+        test(resolve, reject) {
             testEvaluate("varGlobalVariable", (resultValue) => {
                 InspectorTest.expectThat(resultValue === 1, "Should be able to access var in global scope.");
             });
@@ -67,7 +67,7 @@ function test()
     suite.addTestCase({
         name: "CreateGlobalVariable",
         description: "Test evaluate can create global variables.",
-        test: (resolve, reject) => {
+        test(resolve, reject) {
             testEvaluate(`
                 var createdVarGlobalVariable = 1;
                 let createdLetGlobalVariable = 2;
@@ -89,7 +89,7 @@ function test()
     suite.addTestCase({
         name: "CreateGlobalClass",
         description: "Test evaluate can create a class.",
-        test: (resolve, reject) => {
+        test(resolve, reject) {
             testEvaluate(`
                 class Foo {
                     static staticMethod() {
@@ -107,7 +107,7 @@ function test()
     suite.addTestCase({
         name: "ExpectedExceptionCreatingDuplicateLexicalGlobalVariables",
         description: "Test evaluate produces expected exception creating duplicate lexical global variables across evaluations.",
-        test: (resolve, reject) => {
+        test(resolve, reject) {
             testEvaluate(`let duplicateGlobalVariable = 1`);
             testEvaluateThrow(`let duplicateGlobalVariable = 1`, (wasThrown) => {
                 InspectorTest.expectThat(wasThrown, "Should be an exception defining a lexical global multiple times.");
@@ -119,7 +119,7 @@ function test()
     suite.addTestCase({
         name: "NonStrictAndStrictEvaluations",
         description: "Test evaluate can run strict and non-strict programs.",
-        test: (resolve, reject) => {
+        test(resolve, reject) {
             testEvaluate(`
                 // Not strict, this is okay.
                 (function f() {
@@ -143,7 +143,7 @@ function test()
     suite.addTestCase({
         name: "CommandLineAPIDoesNotShadowVariables",
         description: "Test CommandLineAPI does not shadow global variables.",
-        test: (resolve, reject) => {
+        test(resolve, reject) {
             testEvaluate("keys.toString()", (resultValue) => {
                 InspectorTest.expectThat(resultValue.includes("[Command Line API]"), "`keys` is currently CommandLineAPI function.");
             });
@@ -160,7 +160,7 @@ function test()
     suite.addTestCase({
         name: "CommandLineAPIDoesNotShadowGlobalObjectProperties",
         description: "Test CommandLineAPI does not shadow global object properties.",
-        test: (resolve, reject) => {
+        test(resolve, reject) {
             testEvaluate("values.toString()", (resultValue) => {
                 InspectorTest.expectThat(resultValue === "[object HTMLDivElement]", "`values` should be `window.values` and not shadowed by CommandLineAPI `values` function.");
                 resolve();
index 1fe6e22..a8d971b 100644 (file)
@@ -1,6 +1,6 @@
 <html>
 <head>
-<script type="text/javascript" src="../../http/tests/inspector/resources/protocol-test.js"></script>
+<script src="../../http/tests/inspector/resources/protocol-test.js"></script>
 <script>
 function test()
 {
@@ -38,7 +38,7 @@ function test()
         suite.addTestCase({
             name,
             description,
-            test: function(resolve, reject) {
+            test(resolve, reject) {
                 ProtocolTest.log("Evaluating expression: " + expression);
                 InspectorProtocol.awaitCommand({
                     method: "Runtime.evaluate",
index 285eafb..150c56d 100644 (file)
@@ -18,7 +18,7 @@ function test()
     suite.addTestCase({
         name: "SyntaxErrorType.None",
         description: "Test source without an issue.",
-        test: (resolve, reject) => {
+        test(resolve, reject) {
             function testSource(source) {
                 RuntimeAgent.parse(source, (error, result, message, range) => {
                     InspectorTest.assert(!error, "Should not be a protocol error.");
@@ -46,7 +46,7 @@ function test()
     suite.addTestCase({
         name: "SyntaxErrorType.UnterminatedLiteral",
         description: "Test source with an unterminated literal.",
-        test: (resolve, reject) => {
+        test(resolve, reject) {
             function testSource(source) {
                 RuntimeAgent.parse(source, (error, result, message, range) => {
                     InspectorTest.assert(!error, "Should not be a protocol error.");
@@ -75,7 +75,7 @@ function test()
     suite.addTestCase({
         name: "SyntaxErrorType.Recoverable",
         description: "Test source with a recoverable error.",
-        test: (resolve, reject) => {
+        test(resolve, reject) {
             function testSource(source) {
                 RuntimeAgent.parse(source, (error, result, message, range) => {
                     InspectorTest.assert(!error, "Should not be a protocol error.");
@@ -155,7 +155,7 @@ function test()
     suite.addTestCase({
         name: "SyntaxErrorType.Irrecoverable",
         description: "Test source with an irrecoverable issue.",
-        test: (resolve, reject) => {
+        test(resolve, reject) {
             function testSource(source) {
                 RuntimeAgent.parse(source, (error, result, message, range) => {
                     InspectorTest.assert(!error, "Should not be a protocol error.");
index 00c8c7c..731ac25 100644 (file)
@@ -25,7 +25,7 @@ function test()
     suite.addTestCase({
         name: "SavePrimitiveValue1",
         description: "Saving a new value should produce a new $n value.",
-        test: (resolve, reject) => {
+        test(resolve, reject) {
             let value = 123;
             RuntimeAgent.saveResult(createCallArgumentWithValue(value), contextId, (error, savedResultIndex) => {
                 InspectorTest.assert(!error, "Should not be a protocol error.");
@@ -38,7 +38,7 @@ function test()
     suite.addTestCase({
         name: "SavePrimitiveValue2",
         description: "Saving a new value should produce a new $n value.",
-        test: (resolve, reject) => {
+        test(resolve, reject) {
             let value = true;
             RuntimeAgent.saveResult(createCallArgumentWithValue(value), contextId, (error, savedResultIndex) => {
                 InspectorTest.assert(!error, "Should not be a protocol error.");
@@ -51,7 +51,7 @@ function test()
     suite.addTestCase({
         name: "RepeatPrimitiveValue1",
         description: "Saving a previously saved value should produce the original $n value.",
-        test: (resolve, reject) => {
+        test(resolve, reject) {
             let value = 123;
             RuntimeAgent.saveResult(createCallArgumentWithValue(value), contextId, (error, savedResultIndex) => {
                 InspectorTest.assert(!error, "Should not be a protocol error.");
@@ -64,7 +64,7 @@ function test()
     suite.addTestCase({
         name: "$1MatchesValue1",
         description: "Evaluating $1 should get us the value we saved previously.",
-        test: (resolve, reject) => {
+        test(resolve, reject) {
             RuntimeAgent.evaluate.invoke({expression: "$1", objectGroup: "test", includeCommandLineAPI: true}, (error, remoteObjectPayload, wasThrown, savedResultIndex) => {
                 InspectorTest.assert(!error, "Should not be a protocol error.");
                 let remoteObject = WebInspector.RemoteObject.fromPayload(remoteObjectPayload);
@@ -78,7 +78,7 @@ function test()
     suite.addTestCase({
         name: "SaveObject1",
         description: "Saving a new object should produce a new $n value.",
-        test: (resolve, reject) => {
+        test(resolve, reject) {
             let object = {x:100, y:200};
             remoteObjectForSimpleObject(object, (error, remoteObject) => {
                 remoteObject1 = remoteObject;
@@ -94,7 +94,7 @@ function test()
     suite.addTestCase({
         name: "SaveObject2",
         description: "Saving a new object should produce a new $n value.",
-        test: (resolve, reject) => {
+        test(resolve, reject) {
             let object = {x:100, y:200};
             remoteObjectForSimpleObject(object, (error, remoteObject) => {
                 RuntimeAgent.saveResult(createCallArgumentWithRemoteObject(remoteObject), (error, savedResultIndex) => {
@@ -109,7 +109,7 @@ function test()
     suite.addTestCase({
         name: "RepeatSaveObject1",
         description: "Saving a previously saved value should produce the original $n value.",
-        test: (resolve, reject) => {
+        test(resolve, reject) {
             RuntimeAgent.saveResult(createCallArgumentWithRemoteObject(remoteObject1), (error, savedResultIndex) => {
                 InspectorTest.assert(!error, "Should not be a protocol error.");
                 InspectorTest.expectThat(savedResultIndex === 3, `Repeat Object should have existed as $3.`);
@@ -123,7 +123,7 @@ function test()
     suite.addTestCase({
         name: "NoContextWithPrimitiveShouldUseMainContext",
         description: "Saving a primitive should specify the context.",
-        test: (resolve, reject) => {
+        test(resolve, reject) {
             let value = 999;
             RuntimeAgent.saveResult(createCallArgumentWithValue(value), (error, savedResultIndex) => {
                 InspectorTest.assert(!error, "Should not be a protocol error.");
@@ -142,7 +142,7 @@ function test()
     suite.addTestCase({
         name: "ClearConsoleShouldResetValues",
         description: "Clearing the console should reset the $n values.",
-        test: (resolve, reject) => {
+        test(resolve, reject) {
             let value = 999;
             ConsoleAgent.clearMessages();
             RuntimeAgent.saveResult(createCallArgumentWithValue(value), (error, savedResultIndex) => {
@@ -158,7 +158,7 @@ function test()
     suite.addTestCase({
         name: "EvaluateWithSaveIndex",
         description: "Evalute with the saveIndex option should affect $n.",
-        test: (resolve, reject) => {
+        test(resolve, reject) {
             RuntimeAgent.evaluate.invoke({expression: "900 + 90", objectGroup: "test", includeCommandLineAPI: true, saveResult: true}, (error, remoteObjectPayload, wasThrown, savedResultIndex) => {
                 InspectorTest.assert(!error, "Should not be a protocol error.");
                 InspectorTest.expectThat(savedResultIndex === 2, "Evaluated result 990 should become $2.");
index 477995d..1d9a1bc 100644 (file)
@@ -27,7 +27,7 @@ function test()
     suite.addTestCase({
         name: "Sampling Profiler basic",
         description: "Sample some basic code.",
-        test: (resolve, reject) => {
+        test(resolve, reject) {
             InspectorProtocol.awaitEvent({event: "ScriptProfiler.trackingComplete"}).then((messageObject) => {
                 let tree = WebInspector.CallingContextTree.__test_makeTreeFromProtocolMessageObject(messageObject);
                 let trace = [
index afb113d..3a19146 100644 (file)
@@ -28,7 +28,7 @@ function test()
     suite.addTestCase({
         name: "Sampling Profiler sourceURL directive.",
         description: "Sample some basic code.",
-        test: (resolve, reject) => {
+        test(resolve, reject) {
             InspectorProtocol.awaitEvent({event: "ScriptProfiler.trackingComplete"}).then((messageObject) => {
                 let tree = WebInspector.CallingContextTree.__test_makeTreeFromProtocolMessageObject(messageObject);
 
index fbed040..f66d024 100644 (file)
@@ -32,7 +32,7 @@ function test()
     suite.addTestCase({
         name: "Sampling Profiler sourceURL directive.",
         description: "Sample some basic code.",
-        test: (resolve, reject) => {
+        test(resolve, reject) {
             InspectorProtocol.awaitEvent({event: "ScriptProfiler.trackingComplete"}).then((messageObject) => {
                 let tree = WebInspector.CallingContextTree.__test_makeTreeFromProtocolMessageObject(messageObject);
 
index 0564d55..2e100ca 100644 (file)
@@ -35,7 +35,7 @@ function test()
     suite.addTestCase({
         name: "Sampling Profiler Expression Location",
         description: "Make sure we properly collect location information.",
-        test: (resolve, reject) => {
+        test(resolve, reject) {
             InspectorProtocol.awaitEvent({event: "ScriptProfiler.trackingComplete"}).then((messageObject) => {
                 let tree = WebInspector.CallingContextTree.__test_makeTreeFromProtocolMessageObject(messageObject);
                 let foundLine19Column14 = false;
index 6e21848..38f46ef 100644 (file)
@@ -34,7 +34,7 @@ function test()
     suite.addTestCase({
         name: "Sampling Profiler sourceURL directive.",
         description: "Sample some basic code.",
-        test: (resolve, reject) => {
+        test(resolve, reject) {
             InspectorProtocol.awaitEvent({event: "ScriptProfiler.trackingComplete"}).then((messageObject) => {
                 let tree = WebInspector.CallingContextTree.__test_makeTreeFromProtocolMessageObject(messageObject);
 
index ed5a876..5c23c4b 100644 (file)
@@ -31,7 +31,7 @@ function test()
     suite.addTestCase({
         name: "Sampling Profiler basic",
         description: "Sample some basic code.",
-        test: (resolve, reject) => {
+        test(resolve, reject) {
             InspectorProtocol.awaitEvent({event: "ScriptProfiler.trackingComplete"}).then((messageObject) => {
                 let tree = WebInspector.CallingContextTree.__test_makeTreeFromProtocolMessageObject(messageObject);
                 let trace = [
index 95a6b81..13461c6 100644 (file)
@@ -14,7 +14,7 @@ function test()
     suite.addTestCase({
         name: "TrackingSession",
         description: "Start a tracking session that includes an API event type.",
-        test: function(resolve, reject) {
+        test(resolve, reject) {
             
             InspectorProtocol.awaitEvent({event: "ScriptProfiler.trackingStart"}).then((messageObject) => {
                 ProtocolTest.log("ScriptProfiler.trackingStart");
index 737189c..c5367b5 100644 (file)
@@ -16,7 +16,7 @@ function test()
     suite.addTestCase({
         name: "TrackingSession",
         description: "Start a tracking session that includes a Microtask event type.",
-        test: function(resolve, reject) {
+        test(resolve, reject) {
             InspectorProtocol.awaitEvent({event: "ScriptProfiler.trackingStart"}).then((messageObject) => {
                 ProtocolTest.log("ScriptProfiler.trackingStart");
             });
index 2703f24..dfa097f 100644 (file)
@@ -10,7 +10,7 @@ function test()
     suite.addTestCase({
         name: "StartAndStopTracking",
         description: "Start and stop the ScriptProfiler should cause trackingStart and trackingComplete events.",
-        test: function(resolve, reject) {
+        test(resolve, reject) {
             InspectorProtocol.awaitEvent({event: "ScriptProfiler.trackingStart"}).then((messageObject) => {
                 ProtocolTest.log("ScriptProfiler.trackingStart");
                 ProtocolTest.expectThat(typeof messageObject.params.timestamp === "number", "Should have a timestamp when starting.");
index e9ac9b3..3589018 100644 (file)
@@ -2,8 +2,8 @@
 <html>
 <head>
 <meta http-equiv="Content-Security-Policy" content="script-src 'self' 'unsafe-inline'">
-<script type="text/javascript" src="../../http/tests/inspector/resources/inspector-test.js"></script>
-<script type="text/javascript" src="./resources/timeline-helper.js"></script>
+<script src="../../http/tests/inspector/resources/inspector-test.js"></script>
+<script src="./resources/timeline-helper.js"></script>
 <script>
 function installTimer()
 {
index 3e25c1c..7e275b1 100644 (file)
@@ -2,8 +2,8 @@
 <html>
 <head>
 <meta http-equiv="Content-Security-Policy" content="script-src 'self' 'unsafe-inline'">
-<script type="text/javascript" src="../../http/tests/inspector/resources/inspector-test.js"></script>
-<script type="text/javascript" src="./resources/timeline-helper.js"></script>
+<script src="../../http/tests/inspector/resources/inspector-test.js"></script>
+<script src="./resources/timeline-helper.js"></script>
 <script>
 function installTimer()
 {
index fcd59ac..fa9faab 100644 (file)
@@ -9,7 +9,7 @@ function test()
 
     suite.addTestCase({
         name: "MissingRequiredArgument",
-        test: (resolve, reject) => {
+        test(resolve, reject) {
             InspectorProtocol.sendCommand("Timeline.setInstruments", {}, (messageObject) => {
                 ProtocolTest.expectThat(messageObject.error, `Should be an error: ${messageObject.error ? messageObject.error.message : ""}`);
                 resolve();
@@ -19,7 +19,7 @@ function test()
 
     suite.addTestCase({
         name: "InvalidTypeInInstrumentsList",
-        test: (resolve, reject) => {
+        test(resolve, reject) {
             InspectorProtocol.sendCommand("Timeline.setInstruments", {"instruments": [123]}, (messageObject) => {
                 ProtocolTest.expectThat(messageObject.error, `Should be an error: ${messageObject.error ? messageObject.error.message : ""}`);
                 resolve();
@@ -29,7 +29,7 @@ function test()
 
     suite.addTestCase({
         name: "InvalidInstrumentInInstrumentsList",
-        test: (resolve, reject) => {
+        test(resolve, reject) {
             InspectorProtocol.sendCommand("Timeline.setInstruments", {"instruments": ["NoSuchInstrument"]}, (messageObject) => {
                 ProtocolTest.expectThat(messageObject.error, `Should be an error: ${messageObject.error ? messageObject.error.message : ""}`);
                 resolve();
@@ -39,7 +39,7 @@ function test()
 
     suite.addTestCase({
         name: "ValidInstrumentInInstrumentsList",
-        test: (resolve, reject) => {
+        test(resolve, reject) {
             InspectorProtocol.sendCommand("Timeline.setInstruments", {"instruments": ["ScriptProfiler", "Heap", "Timeline", "Memory"]}, (messageObject) => {
                 ProtocolTest.expectThat(!messageObject.error, "Should not be an error setting valid instruments.");
                 resolve();
index a81dae4..42c4f86 100644 (file)
@@ -37,7 +37,7 @@ function test()
 
     suite.addTestCase({
         name: "NoInstrumentsProgrammaticCapture",
-        test: (resolve, reject) => {
+        test(resolve, reject) {
             heapExpected = false;
             programmaticCaptureStopped = resolve;
 
@@ -50,7 +50,7 @@ function test()
 
     suite.addTestCase({
         name: "MultipleInstrumentsProgrammaticCapture",
-        test: (resolve, reject) => {
+        test(resolve, reject) {
             heapExpected = true;
             programmaticCaptureStopped = resolve;
 
@@ -63,7 +63,7 @@ function test()
 
     suite.addTestCase({
         name: "NoInstrumentsProgrammaticCaptureAgain",
-        test: (resolve, reject) => {
+        test(resolve, reject) {
             heapExpected = true;
             programmaticCaptureStopped = resolve;
 
index a799a9f..fd0ff99 100644 (file)
@@ -38,8 +38,8 @@ PASS: shallowEqual of empty arrays should be true.
 PASS: shallowEqual of an array with itself should be true.
 PASS: shallowEqual of equal arrays should be true.
 PASS: shallowEqual of equal arrays should be true.
-PASS: shallowEqual of inequal arrays should be false.
-PASS: shallowEqual of inequal arrays should be false.
+PASS: shallowEqual of unequal arrays should be false.
+PASS: shallowEqual of unequal arrays should be false.
 PASS: shallowEqual of an array and null should be false.
 PASS: shallowEqual of an array and non-array should be false.
 PASS: shallowEqual of a non-array with itself should be false.
index e3f08f0..7ee30c2 100644 (file)
@@ -98,8 +98,8 @@ function test()
             InspectorTest.expectThat(Array.shallowEqual(arr2, arr1), "shallowEqual of equal arrays should be true.");
 
             let arr3 = [1, 2, 3, 4, 5];
-            InspectorTest.expectFalse(Array.shallowEqual(arr1, arr3), "shallowEqual of inequal arrays should be false.");
-            InspectorTest.expectFalse(Array.shallowEqual(arr3, arr1), "shallowEqual of inequal arrays should be false.");
+            InspectorTest.expectFalse(Array.shallowEqual(arr1, arr3), "shallowEqual of unequal arrays should be false.");
+            InspectorTest.expectFalse(Array.shallowEqual(arr3, arr1), "shallowEqual of unequal arrays should be false.");
 
             InspectorTest.expectFalse(Array.shallowEqual([], null), "shallowEqual of an array and null should be false.");
             InspectorTest.expectFalse(Array.shallowEqual([], 1.23), "shallowEqual of an array and non-array should be false.");
index 6c18d34..51b302c 100644 (file)
@@ -49,7 +49,7 @@ function test()
     try {
         badArgsSuite.addTestCase({
             name: {},
-            test: () => {},
+            test() {},
         });
         ProtocolTest.log("FAIL: test case should require string name.");
     } catch (e) {
@@ -58,7 +58,7 @@ function test()
     try {
         badArgsSuite.addTestCase({
             name: "        ",
-            test: () => {},
+            test() {},
         });
         ProtocolTest.log("FAIL: test case should require non-whitespace name.");
     } catch (e) {
@@ -76,7 +76,7 @@ function test()
     try {
         badArgsSuite.addTestCase({
             name: "foo",
-            test: () => {},
+            test() {},
             setup: "astd"
         });
         ProtocolTest.log("FAIL: should not be able to specify non-Function `setup` parameter.");
@@ -86,7 +86,7 @@ function test()
     try {
         badArgsSuite.addTestCase({
             name: "foo",
-            test: () => {},
+            test() {},
             setup: 123
         });
         ProtocolTest.log("FAIL: should not be able to specify non-Function `setup` parameter.");
@@ -96,7 +96,7 @@ function test()
     try {
         badArgsSuite.addTestCase({
             name: "foo",
-            test: () => {},
+            test() {},
             setup: {}
         });
         ProtocolTest.log("FAIL: should not be able to specify non-Function `setup` parameter.");
@@ -106,7 +106,7 @@ function test()
     try {
         badArgsSuite.addTestCase({
             name: "foo",
-            test: () => {},
+            test() {},
             teardown: "astd"
         });
         ProtocolTest.log("FAIL: should not be able to specify non-Function `teardown` parameter.");
@@ -116,7 +116,7 @@ function test()
     try {
         badArgsSuite.addTestCase({
             name: "foo",
-            test: () => {},
+            test() {},
             teardown: 123
         });
         ProtocolTest.log("FAIL: should not be able to specify non-Function `teardown` parameter.");
@@ -126,7 +126,7 @@ function test()
     try {
         badArgsSuite.addTestCase({
             name: "foo",
-            test: () => {},
+            test() {},
             teardown: {}
         });
         ProtocolTest.log("FAIL: should not be able to specify non-Function `teardown` parameter.");
@@ -146,7 +146,7 @@ function test()
     runTwiceSuite.addTestCase({
         name: "DummyTest0",
         description: "Check that a suite can't run more than once.",
-        test: (resolve, reject) => { resolve(); }
+        test(resolve, reject) { resolve(); }
     });
 
     let result = runTwiceSuite.runTestCases();
@@ -166,39 +166,39 @@ function test()
     sequentialExecutionSuite.addTestCase({
         name: "DummyTest1",
         description: "Check test case execution order.",
-        test: (resolve, reject) => { resolve(); }
+        test(resolve, reject) { resolve(); }
     });
     sequentialExecutionSuite.addTestCase({
         name: "DummyTest2",
         description: "Check test case execution order.",
-        test: (resolve, reject) => { resolve(); }
+        test(resolve, reject) { resolve(); }
     });
     sequentialExecutionSuite.addTestCase({
         name: "DummyTest3",
         description: "Check test case execution order.",
-        test: (resolve, reject) => { resolve(); }
+        test(resolve, reject) { resolve(); }
     });
     sequentialExecutionSuite.addTestCase({
         name: "FailingTest4",
         description: "Check that test fails by throwing an Error instance.",
-        test: (resolve, reject) => { throw thrownError; }
+        test(resolve, reject) { throw thrownError; }
     });
 
     let abortOnFailureSuite = ProtocolTest.createAsyncSuite("AsyncTestSuite.AbortOnFailure");
     abortOnFailureSuite.addTestCase({
         name: "PassingTest5",
         description: "This test is a dummy.",
-        test: (resolve, reject) => { resolve(); }
+        test(resolve, reject) { resolve(); }
     });
     abortOnFailureSuite.addTestCase({
         name: "FailingTest6",
         description: "This test should fail by explicitly calling the `reject` callback.",
-        test: (resolve, reject) => { reject(rejectToken); }
+        test(resolve, reject) { reject(rejectToken); }
     });
     abortOnFailureSuite.addTestCase({
         name: "PassingTest7",
         description: "This test should not executed when the preceding test fails.",
-        test: (resolve, reject) => { resolve(); }
+        test(resolve, reject) { resolve(); }
     });
 
     result = result.then(() => {
@@ -247,7 +247,7 @@ function test()
             window[setupAndTeardownSymbol] = 1;
             resolve();
         },
-        test: (resolve, reject) => {
+        test(resolve, reject) {
             ProtocolTest.expectThat(window[setupAndTeardownSymbol] === 1, "Test should see side effects of running setup() action.");
             window[setupAndTeardownSymbol] = 2;
             resolve();
@@ -261,7 +261,7 @@ function test()
     setupAndTeardownTestSuite.addTestCase({
         name: "TestRunningAfterTeardown",
         description: "Check execution order for test after a teardown action.",
-        test: (resolve, reject) => {
+        test(resolve, reject) {
             ProtocolTest.expectThat(window[setupAndTeardownSymbol] === 3, "Test should see side effects of previous test's teardown() action.");
             resolve();
         },
@@ -282,7 +282,7 @@ function test()
         name: "TestWithExceptionDuringSetup",
         description: "Check execution order for setup action that throws an exception.",
         setup: (resolve, reject) => { throw new Error() },
-        test: (resolve, reject) => {
+        test(resolve, reject) {
             ProtocolTest.assert(false, "Test should not execute if its setup action threw an exception.");
             reject();
         },
@@ -307,7 +307,7 @@ function test()
         name: "TestWithFailureDuringSetup",
         description: "Check execution order for setup action that has failed.",
         setup: (resolve, reject) => { reject(); },
-        test: (resolve, reject) => {
+        test(resolve, reject) {
             ProtocolTest.assert(false, "Test should not execute if its setup action failed.")
             reject();
         },
@@ -331,7 +331,7 @@ function test()
     teardownExceptionTestSuite.addTestCase({
         name: "TestWithExceptionDuringTeardown",
         description: "Check execution order for teardown action that throws an exception.",
-        test: (resolve, reject) => { resolve(); },
+        test(resolve, reject) { resolve(); },
         teardown: (resolve, reject) => { throw new Error() }
     });
     teardownExceptionTestSuite.addTestCase({
@@ -341,7 +341,7 @@ function test()
             ProtocolTest.assert(false, "Setup action should not execute if previous test's teardown action threw an exception.");
             reject();
         },
-        test: (resolve, reject) => {
+        test(resolve, reject) {
             ProtocolTest.assert(false, "Test should not execute if previous test's teardown action threw an exception.");
             reject();
         }
@@ -361,7 +361,7 @@ function test()
     teardownFailureTestSuite.addTestCase({
         name: "TestWithExceptionDuringTeardown",
         description: "Check execution order for teardown action that has failed.",
-        test: (resolve, reject) => { resolve(); },
+        test(resolve, reject) { resolve(); },
         teardown: (resolve, reject) => { reject(); },
     });
     teardownFailureTestSuite.addTestCase({
@@ -371,7 +371,7 @@ function test()
             ProtocolTest.assert(false, "Setup action should not execute if previous test's teardown action failed.");
             reject();
         },
-        test: (resolve, reject) => {
+        test(resolve, reject) {
             ProtocolTest.assert(false, "Test should not execute if previous test's teardown action failed.");
             reject();
         }
index bd02e59..6009805 100644 (file)
@@ -1,7 +1,7 @@
 Testing debounce proxy support.
 
 
-== Running test suite: Debouce
+== Running test suite: Debounce
 -- Running test case: Basic Debounce
 PASS: Debounced successfully.
 PASS: Call delayed at least 10ms.
index ae77594..a4f0071 100644 (file)
@@ -5,11 +5,11 @@
 <script>
 function test()
 {
-    let suite = InspectorTest.createAsyncSuite("Debouce");
+    let suite = InspectorTest.createAsyncSuite("Debounce");
 
     suite.addTestCase({
         name: "Basic Debounce",
-        test: function(resolve, reject) {
+        test(resolve, reject) {
             let callCount = 0;
             let startTime = performance.now();
 
@@ -47,7 +47,7 @@ function test()
 
     suite.addTestCase({
         name: "Multiple Debounce Delays",
-        test: function(resolve, reject) {
+        test(resolve, reject) {
             let callCount = 0;
             let startTime = performance.now();
 
@@ -84,7 +84,7 @@ function test()
 
     suite.addTestCase({
         name: "Soon Debounce",
-        test: function(resolve, reject) {
+        test(resolve, reject) {
             let callCount = 0;
 
             let object = {
@@ -118,7 +118,7 @@ function test()
 
     suite.addTestCase({
         name: "Cancel Debounce",
-        test: function(resolve, reject) {
+        test(resolve, reject) {
             let object = {
                 test() {
                     InspectorTest.fail("Debounced function call should have been canceled.");
index 81098f3..658f1aa 100644 (file)
@@ -20,6 +20,6 @@ Invoked callback for bar event.
 Invoked callback for baz event.
 Unregistering everything.
 Dispatching events.
-Unintalling and disconnecting listeners.
+Uninstalling and disconnecting listeners.
 Dispatching events.
 
index 451ebbd..95b44b9 100644 (file)
@@ -96,7 +96,7 @@ function test()
     emitter1.dispatchEventToListeners(BarEvent, data2); // Should not fire.
     emitter2.dispatchEventToListeners(BazEvent, data2); // Should not fire.
 
-    InspectorTest.log("Unintalling and disconnecting listeners.");
+    InspectorTest.log("Uninstalling and disconnecting listeners.");
     listenerSet.uninstall(true);
     InspectorTest.log("Dispatching events.");
     emitter1.dispatchEventToListeners(FooEvent, data1); // Should not fire.
index 298a4be..086d362 100644 (file)
@@ -21,7 +21,7 @@ function test()
 
     suite.addTestCase({
         name: "HeapSnapshot",
-        test: (resolve, reject) => {
+        test(resolve, reject) {
             InspectorTest.evaluateInPage("triggerCreateObjects()");
             HeapAgent.snapshot((error, timestamp, snapshotStringData) => {
                 InspectorTest.expectThat(!error, "Should not have an error creating a snapshot.");
@@ -37,7 +37,7 @@ function test()
 
     suite.addTestCase({
         name: "HeapSnapshotCollectionEvent",
-        test: (resolve, reject) => {
+        test(resolve, reject) {
             InspectorTest.evaluateInPage("triggerRemoveObjects()");
 
             let workerProxy = WebInspector.HeapSnapshotWorkerProxy.singleton();
@@ -59,7 +59,7 @@ function test()
 
     suite.addTestCase({
         name: "HeapSnapshot.prototype.update",
-        test: (resolve, reject) => {
+        test(resolve, reject) {
             let categoriesBefore = snapshot.categories;
             snapshot.update(() => {
                 let categoriesAfter = snapshot.categories;
index 3cffc98..470c579 100644 (file)
@@ -132,7 +132,7 @@ function test()
 
     suite.addTestCase({
         name: "HeapSnapshotProxy data",
-        test: (resolve, reject) => {
+        test(resolve, reject) {
             HeapAgent.snapshot((error, timestamp, snapshotStringData) => {
                 InspectorTest.expectThat(!error, "Should not have an error creating a snapshot.");
                 testSnapshot = WebInspector.TestHeapSnapshot.fromPayload(JSON.parse(snapshotStringData));
@@ -151,7 +151,7 @@ function test()
 
     suite.addTestCase({
         name: "HeapSnapshotProxy.prototype.instancesWithClassName",
-        test: (resolve, reject) => {
+        test(resolve, reject) {
             let windowObjects = testSnapshot.instancesWithClassName("Window")
             let windowObjectCount = windowObjects.length;
             let functionObjectCount = testSnapshot.instancesWithClassName("Function").length;
@@ -183,7 +183,7 @@ function test()
 
     suite.addTestCase({
         name: "HeapSnapshotProxy.prototype.nodeWithIdentifier and HeapSnapshotNodeProxy data",
-        test: (resolve, reject) => {
+        test(resolve, reject) {
             snapshot.nodeWithIdentifier(testSnapshotNodeForWindowObject.id, (heapSnapshotNode) => {
                 InspectorTest.expectThat(heapSnapshotNode.className === "Window", "Node className should be 'Window'.");
                 InspectorTest.expectThat(heapSnapshotNode.id === testSnapshotNodeForWindowObject.id, "Node identifier should match.")
@@ -198,7 +198,7 @@ function test()
 
     suite.addTestCase({
         name: "HeapSnapshotProxy.prototype.allocationBucketCounts",
-        test: (resolve, reject) => {
+        test(resolve, reject) {
             let testSmall = 0, testMedium = 0, testLarge = 0;
             const smallSize = 32, mediumSize = 128;
             // Skip <root>.
@@ -225,7 +225,7 @@ function test()
 
     suite.addTestCase({
         name: "HeapSnapshotNodeProxy.prototype.retainedNodes",
-        test: (resolve, reject) => {
+        test(resolve, reject) {
             let expectedNodes = testSnapshotNodeForWindowObject.outgoingEdges.map((edge) => edge.to);
             expectedNodes.sort((a, b) => a.id - b.id);
 
@@ -242,7 +242,7 @@ function test()
 
     suite.addTestCase({
         name: "HeapSnapshotNodeProxy.prototype.retainers",
-        test: (resolve, reject) => {
+        test(resolve, reject) {
             let expectedNodes = testSnapshotNodeForWindowObject.incomingEdges.map((edge) => edge.from);
             expectedNodes.sort((a, b) => a.id - b.id);
 
index b073226..c803257 100644 (file)
@@ -9,7 +9,7 @@ function test()
 
     suite.addTestCase({
         name: "Adding items",
-        test: function(resolve, reject) {
+        test(resolve, reject) {
             let list = new LinkedList;
 
             InspectorTest.log(list.length);
@@ -30,7 +30,7 @@ function test()
 
     suite.addTestCase({
         name: "Removing items",
-        test: function(resolve, reject) {
+        test(resolve, reject) {
             let list = new LinkedList;
 
             list.push("one");
@@ -49,7 +49,7 @@ function test()
 
     suite.addTestCase({
         name: "Removing all items",
-        test: function(resolve, reject) {
+        test(resolve, reject) {
             let list = new LinkedList;
 
             list.push("one");
@@ -65,7 +65,7 @@ function test()
 
     suite.addTestCase({
         name: "Iterating using forEach method",
-        test: function(resolve, reject) {
+        test(resolve, reject) {
             let list = new LinkedList;
 
             list.push("one");
index bdb1a40..4655422 100644 (file)
@@ -9,7 +9,7 @@ function test()
 
     suite.addTestCase({
         name: "Instantiating Multimap",
-        test: function(resolve, reject) {
+        test(resolve, reject) {
             let multimap = new ListMultimap;
 
             InspectorTest.log(multimap.size);
@@ -21,7 +21,7 @@ function test()
 
     suite.addTestCase({
         name: "Adding unique keys and values",
-        test: function(resolve, reject) {
+        test(resolve, reject) {
             let multimap = new ListMultimap;
 
             multimap.add("zero", "one");
@@ -36,7 +36,7 @@ function test()
 
     suite.addTestCase({
         name: "Adding repeating keys and unique values",
-        test: function(resolve, reject) {
+        test(resolve, reject) {
             let multimap = new ListMultimap;
 
             multimap.add("zero", "one");
@@ -51,7 +51,7 @@ function test()
 
     suite.addTestCase({
         name: "Adding unique keys and repeating values",
-        test: function(resolve, reject) {
+        test(resolve, reject) {
             let multimap = new ListMultimap;
 
             multimap.add("zero", "one");
@@ -67,7 +67,7 @@ function test()
 
     suite.addTestCase({
         name: "Adding repeating keys and values",
-        test: function(resolve, reject) {
+        test(resolve, reject) {
             let multimap = new ListMultimap;
 
             multimap.add("zero", "one");
@@ -84,7 +84,7 @@ function test()
 
     suite.addTestCase({
         name: "Deleting existing keys and values",
-        test: function(resolve, reject) {
+        test(resolve, reject) {
             let multimap = new ListMultimap;
 
             multimap.add(0, 1);
@@ -107,7 +107,7 @@ function test()
 
     suite.addTestCase({
         name: "Deleting non-existing keys and non-existing values",
-        test: function(resolve, reject) {
+        test(resolve, reject) {
             let multimap = new ListMultimap;
 
             multimap.add(0, 1);
@@ -129,7 +129,7 @@ function test()
 
     suite.addTestCase({
         name: "Deleting values for given key",
-        test: function(resolve, reject) {
+        test(resolve, reject) {
             let multimap = new ListMultimap;
 
             multimap.add("opossum", "badger");
@@ -152,7 +152,7 @@ function test()
 
     suite.addTestCase({
         name: "Deleting all keys and values",
-        test: function(resolve, reject) {
+        test(resolve, reject) {
             let multimap = new ListMultimap;
             multimap.add("badger", "raccoon");
             multimap.clear();
@@ -166,7 +166,7 @@ function test()
 
     suite.addTestCase({
         name: "Iterating using forEach method",
-        test: function(resolve, reject) {
+        test(resolve, reject) {
             let multimap = new ListMultimap;
             multimap.add("Platypus", "Sugar glider");
             multimap.add("Quoll", "Wallaby");
index f2392d6..c9830cc 100644 (file)
@@ -9,7 +9,7 @@ function test()
 
     suite.addTestCase({
         name: "Number.constrain",
-        test: () => {
+        test() {
             InspectorTest.expectThat(Number.constrain(0, -1, 1) === 0, "constrain of a value between min and max does not change");
             InspectorTest.expectThat(Number.constrain(0.5, -1, 1) === 0.5, "constrain of a value between min and max does not change");
             InspectorTest.expectThat(Number.constrain(1, -1, 1) === 1, "constrain of a value between min and max does not change");
@@ -30,7 +30,7 @@ function test()
 
     suite.addTestCase({
         name: "Number.secondsToString",
-        test: () => {
+        test() {
             // Normal resolution.
             InspectorTest.expectThat(Number.secondsToString(0, false) === "0ms", "normal resolution of 0ms should be ms with no decimals");
             InspectorTest.expectThat(Number.secondsToString(0.000123456, false) === "0.12ms", "normal resolution of sub 1ms should be ms");
@@ -62,7 +62,7 @@ function test()
 
     suite.addTestCase({
         name: "Number.bytesToString",
-        test: () => {
+        test() {
             const kb = 1024;
             const mb = kb * 1024;
 
@@ -86,7 +86,7 @@ function test()
 
     suite.addTestCase({
         name: "Number.percentageString",
-        test: () => {
+        test() {
             InspectorTest.expectThat(Number.percentageString(1 / 3) === "33.3%", "precision should default to 1 if unspecified");
 
             return true;
index 47a1d24..0e654f5 100644 (file)
@@ -7,8 +7,8 @@ PASS: shallowEqual of empty array and object literals should be false.
 PASS: shallowEqual of an object with itself should be true.
 PASS: shallowEqual of equal objects should be true.
 PASS: shallowEqual of equal objects should be true.
-PASS: shallowEqual of inequal objects should be false.
-PASS: shallowEqual of inequal objects should be false.
+PASS: shallowEqual of unequal objects should be false.
+PASS: shallowEqual of unequal objects should be false.
 PASS: shallowEqual of an object and null should be false.
 PASS: shallowEqual of an object and non-object should be false.
 PASS: shallowEqual of a non-object with itself should be false.
index e411203..a6ca147 100644 (file)
@@ -10,7 +10,7 @@ function test()
 
     suite.addTestCase({
         name: "Object.shallowEqual",
-        test: () => {
+        test() {
             InspectorTest.expectThat(Object.shallowEqual({}, {}), "shallowEqual of empty object literals should be true.");
             InspectorTest.expectThat(Object.shallowEqual([], []), "shallowEqual of empty array literals should be true.");
             InspectorTest.expectThat(!Object.shallowEqual([], {}), "shallowEqual of empty array and object literals should be false.");
@@ -23,8 +23,8 @@ function test()
             InspectorTest.expectThat(Object.shallowEqual(obj2, obj1), "shallowEqual of equal objects should be true.");
 
             let obj3 = {a: 1, b: 2, c: 3};
-            InspectorTest.expectThat(!Object.shallowEqual(obj1, obj3), "shallowEqual of inequal objects should be false.");
-            InspectorTest.expectThat(!Object.shallowEqual(obj3, obj1), "shallowEqual of inequal objects should be false.");
+            InspectorTest.expectThat(!Object.shallowEqual(obj1, obj3), "shallowEqual of unequal objects should be false.");
+            InspectorTest.expectThat(!Object.shallowEqual(obj3, obj1), "shallowEqual of unequal objects should be false.");
 
             InspectorTest.expectThat(!Object.shallowEqual({}, null), "shallowEqual of an object and null should be false.");
             InspectorTest.expectThat(!Object.shallowEqual({}, 1.23), "shallowEqual of an object and non-object should be false.");
index e89ef72..67e691c 100644 (file)
@@ -9,7 +9,7 @@ function test()
 
     suite.addTestCase({
         name: "String.format",
-        test: () => {
+        test() {
             InspectorTest.expectThat("%f".format(1.23456789) === "1.234568", "float format specifier with no sub-specifier should show 6 decimal digits");
             InspectorTest.expectThat("%.0f".format(1.23456789) === "1", "float format specifier with precision 0 should show 0 decimal digits");
             InspectorTest.expectThat("%.1f".format(1.23456789) === "1.2", "float format specifier with precision 1 should show 1 decimal digit");
index b5c3d95..6add405 100644 (file)
@@ -58,7 +58,7 @@ function test()
     try {
         badArgsSuite.addTestCase({
             name: "        ",
-            test: () => {},
+            test() {},
         });
         ProtocolTest.log("FAIL: test case should require non-whitespace name.");
     } catch (e) {
@@ -76,7 +76,7 @@ function test()
     try {
         badArgsSuite.addTestCase({
             name: "foo",
-            test: () => {},
+            test() {},
             setup: "astd"
         });
         ProtocolTest.log("FAIL: should not be able to specify non-Function `setup` parameter.");
@@ -86,7 +86,7 @@ function test()
     try {
         badArgsSuite.addTestCase({
             name: "foo",
-            test: () => {},
+            test() {},
             setup: 123
         });
         ProtocolTest.log("FAIL: should not be able to specify non-Function `setup` parameter.");
@@ -96,7 +96,7 @@ function test()
     try {
         badArgsSuite.addTestCase({
             name: "foo",
-            test: () => {},
+            test() {},
             setup: Promise.resolve()
         });
         ProtocolTest.log("FAIL: should not be able to specify non-Function `setup` parameter.");
@@ -106,7 +106,7 @@ function test()
     try {
         badArgsSuite.addTestCase({
             name: "foo",
-            test: () => {},
+            test() {},
             teardown: "astd"
         });
         ProtocolTest.log("FAIL: should not be able to specify non-Function `teardown` parameter.");
@@ -116,7 +116,7 @@ function test()
     try {
         badArgsSuite.addTestCase({
             name: "foo",
-            test: () => {},
+            test() {},
             teardown: 123
         });
         ProtocolTest.log("FAIL: should not be able to specify non-Function `teardown` parameter.");
@@ -126,7 +126,7 @@ function test()
     try {
         badArgsSuite.addTestCase({
             name: "foo",
-            test: () => {},
+            test() {},
             teardown: Promise.resolve()
         });
         ProtocolTest.log("FAIL: should not be able to specify non-Function `teardown` parameter.");
@@ -179,7 +179,7 @@ function test()
     sequentialExecutionSuite.addTestCase({
         name: "FailingTest4",
         description: "Check that test fails by throwing an Error instance.",
-        test: () => { throw thrownError; }
+        test() { throw thrownError; }
     });
 
     let result = sequentialExecutionSuite.runTestCases();
@@ -224,7 +224,7 @@ function test()
             window[setupAndTeardownSymbol] = 1;
             return true;
         },
-        test: () => {
+        test() {
             ProtocolTest.expectThat(window[setupAndTeardownSymbol] === 1, "Test should see side effects of running setup() action.");
             window[setupAndTeardownSymbol] = 2;
             return true;
@@ -238,7 +238,7 @@ function test()
     setupAndTeardownTestSuite.addTestCase({
         name: "TestRunningAfterTeardown",
         description: "Check execution order for test after a teardown action.",
-        test: () => {
+        test() {
             ProtocolTest.expectThat(window[setupAndTeardownSymbol] === 3, "Test should see side effects of previous test's teardown() action.");
             return true;
         },
@@ -251,7 +251,7 @@ function test()
         name: "TestWithExceptionDuringSetup",
         description: "Check execution order for setup action that throws an exception.",
         setup: () => { throw new Error() },
-        test: () => {
+        test() {
             ProtocolTest.assert(false, "Test should not execute if its setup action threw an exception.");
             return false;
         },
@@ -267,7 +267,7 @@ function test()
         name: "TestWithFailureDuringSetup",
         description: "Check execution order for setup action that has failed.",
         setup: () => false,
-        test: () => {
+        test() {
             ProtocolTest.assert(false, "Test should not execute if its setup action returned false.")
             return false;
         },
@@ -292,7 +292,7 @@ function test()
             ProtocolTest.assert(false, "Setup action should not execute if previous test's teardown action threw an exception.");
             return false;
         },
-        test: () => {
+        test() {
             ProtocolTest.assert(false, "Test should not execute if previous test's teardown action threw an exception.");
             return false;
         }
@@ -313,7 +313,7 @@ function test()
             ProtocolTest.assert(false, "Setup action should not execute if previous test's teardown action failed.");
             return false;
         },
-        test: () => {
+        test() {
             ProtocolTest.assert(false, "Test should not execute if previous test's teardown action failed.");
             return false;
         }
index b52a2c3..8e94689 100644 (file)
@@ -18,7 +18,7 @@ function test()
     suite.addTestCase({
         name: "ItemsAreProcessedInIterationOrder",
         description: "Check that yieldableTaskWillProcessItem delegate is called on input items in order.",
-        test: (resolve, reject) => {
+        test(resolve, reject) {
             let expectedItems = createTaskItems();
             let delegate = {
                 yieldableTaskWillProcessItem: (task, item) => {
@@ -40,7 +40,7 @@ function test()
     suite.addTestCase({
         name: "ProcessedItemsArgumentContainsExpectedItems",
         description: "The list of processed items after a yield should contain the items that were processed by yieldableTaskWillProcessItem.",
-        test: (resolve, reject) => {
+        test(resolve, reject) {
             const itemsToProcessBeforeForcingTaskToYield = 10;
             let actualProcessedItems = [];
 
@@ -73,7 +73,7 @@ function test()
     suite.addTestCase({
         name: "TaskFinishesWhenInterruptedByCancellation",
         description: "Check that a task can be cancelled.",
-        test: (resolve, reject) => {
+        test(resolve, reject) {
             let items = createTaskItems(40);
             let delegate = {
                 yieldableTaskWillProcessItem: (task, item) => {
@@ -95,7 +95,7 @@ function test()
     suite.addTestCase({
         name: "ShortTaskCompletesWithoutYielding",
         description: "A task with very few items should run to completion without yielding.",
-        test: (resolve, reject) => {
+        test(resolve, reject) {
             let processedItemsCount = 0;
 
             let delegate = {
@@ -120,7 +120,7 @@ function test()
         suite.addTestCase({
             name: `ProcessItemsFrom${name}`,
             description: "Check that any iterable object is supported.",
-            test: (resolve, reject) => {
+            test(resolve, reject) {
                 let delegate = {
                     yieldableTaskWillProcessItem: (task, item) => {
                         InspectorTest.log("Process item: " + JSON.stringify(item));