Bug 90979 - REGRESSION(r122327): It made http/tests/inspector/network/network-request-revision-content.html fail on Lion, Qt
Summary: REGRESSION(r122327): It made http/tests/inspector/network/network-request-rev...
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Tools / Tests (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Vsevolod Vlasov
URL:
Keywords:
Depends on:
Blocks: 79666 90888
  Show dependency treegraph
 
Reported: 2012-07-11 06:42 PDT by Ádám Kallai
Modified: 2012-07-12 03:03 PDT (History)
5 users (show)

See Also:


Attachments
Patch (4.02 KB, patch)
2012-07-12 02:45 PDT, Vsevolod Vlasov
pfeldman: review+
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Ádám Kallai 2012-07-11 06:42:47 PDT
* http/tests/inspector/network/network-request-revision-content.html

Diff:

--- /buildbot/lion-qt-release/lion-qt-intel-release/build/layout-test-results/http/tests/inspector/network/network-request-revision-content-expected.txt
+++ /buildbot/lion-qt-release/lion-qt-intel-release/build/layout-test-results/http/tests/inspector/network/network-request-revision-content-actual.txt
@@ -2,6 +2,21 @@
 Tests how revision requests content if original content was not loaded yet.
 
 Bug 63631
-http://127.0.0.1:8000/inspector/network/resources/script.js
+Exception while running: function step2(msg) {
+        var request = WebInspector.panels.network.requests[WebInspector.panels.network.requests.length - 1];
+        uiSourceCode = WebInspector.workspace.uiSourceCodeForURL(request.url);
+        resource = WebInspector.resourceForURL(request.url);
+        uiSourceCode.addRevision("");
+        uiSourceCode.history[0].requestContent(step3);
+    }
+step2
+result
 
+_consoleMessageAdded@qrc:/webkit/inspector/ConsoleView.js:371
+dispatchEventToListeners@qrc:/webkit/inspector/Object.js:101
+addMessage@qrc:/webkit/inspector/ConsoleModel.js:67
+messageAdded@qrc:/webkit/inspector/ConsoleModel.js:258
+dispatch@qrc:/webkit/inspector/InspectorBackend.js:228
+qrc:/webkit/inspector/inspector.js:608
+[native code]
Comment 1 Ádám Kallai 2012-07-11 07:08:53 PDT
I skipped this test on Qt.

http://trac.webkit.org/changeset/122330/trunk/LayoutTests/platform/qt/Skipped
Comment 2 Vsevolod Vlasov 2012-07-12 02:45:42 PDT
Created attachment 151896 [details]
Patch
Comment 3 Vsevolod Vlasov 2012-07-12 03:03:13 PDT
Committed r122436: <http://trac.webkit.org/changeset/122436>