Bug 90530 – REGRESSION(r121792): inspector/extensions/extensions-resources.html fails

• Status: RESOLVED
• Resolution: FIXED
• Component: Tools / Tests

[Zoltan Arvai]

Modified test fails after r121792 on Qt and Lion.

--- /ramdisk/qt-linux-64-release/build/layout-test-results/inspector/extensions/extensions-resources-expected.txt 
+++ /ramdisk/qt-linux-64-release/build/layout-test-results/inspector/extensions/extensions-resources-actual.txt 
@@ -1,4 +1,4 @@
-CONSOLE MESSAGE: line 11: don't panic!
+FAIL: Timed out waiting for notifyDone to be called
 Tests resource-related methods of WebInspector extension API
 
 
@@ -60,27 +60,4 @@
 content committed for resource .../audits-style1.css (type: stylesheet), new content: div.test { width: 220px; height: 42px; }
 Revision content: div.test { width: 220px; height: 42px; }
 RUNNING TEST: extension_testOnResourceAdded
-resource added:
-{
-    0 : {
-        url : data:application/javascript...
-        type : "script"
-        getContent : <function>
-        setContent : <function>
-    }
-}
-RUNNING TEST: extension_testOpenResourceHandler
-handleOpenResource() invoked [this should only appear once!]: 
-{
-    0 : {
-        url : .../extensions-resources.html
-        type : "document"
-        getContent : <function>
-        setContent : <function>
-    }
-    1 : 11
-}
-RUNNING TEST: extension_testSetResourceContent
-div.test width after stylesheet edited (should be 126): 126
-All tests done.

[Zoltan Arvai]

Skipped on Qt in http://trac.webkit.org/changeset/121839 .
Please unskip it with the proper fix.

[Vsevolod Vlasov]

Created attachment 150750 [details]
Patch

[Andrey Kosyakov]

Comment on attachment 150750 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=150750&action=review

> LayoutTests/http/tests/inspector/debugger-test.js:56
> +    InspectorTest.finishDebuggerTest(completeTest);
> +
> +    function completeTest()
> +    {
>          InspectorTest.completeTest();
>      }

Just InspectorTest.finishDebuggerTest(InspectorTest.completeTest.bind(this))?

> LayoutTests/inspector/extensions/extensions-resources-expected.txt:83
> -    1 : 11
> +    1 : 12

nit: you could as well have moved logMessage() to a separate <script> on the top of the file, so that we stop re-baselining the test upon adding new scripts.

> LayoutTests/inspector/extensions/extensions-resources.html:173
> +          return;

indent

[Vsevolod Vlasov]

Created attachment 150758 [details]
Patch

[Vsevolod Vlasov]

Committed r121842: <http://trac.webkit.org/changeset/121842>