Bug 90385 - inspector/debugger/script-snippet-model.html fails
Summary: inspector/debugger/script-snippet-model.html fails
Alias: None
Product: WebKit
Classification: Unclassified
Component: Tools / Tests (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Vsevolod Vlasov
Keywords: Qt
Depends on:
Blocks: 87008 88707
  Show dependency treegraph
Reported: 2012-07-02 07:52 PDT by Zoltan Arvai
Modified: 2012-07-03 08:00 PDT (History)
3 users (show)

See Also:

Patch (4.17 KB, patch)
2012-07-03 06:57 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 Zoltan Arvai 2012-07-02 07:52:45 PDT
New test inspector/debugger/script-snippet-model.html added in r121673 fails.
Comment 1 Csaba Osztrogonác 2012-07-02 07:55:59 PDT
No, it isn't Qt specific, it fails on Apple Lion too.

--- /ramdisk/qt-linux-64-release/build/layout-test-results/inspector/debugger/script-snippet-model-expected.txt 
+++ /ramdisk/qt-linux-64-release/build/layout-test-results/inspector/debugger/script-snippet-model-actual.txt 
@@ -4,16 +4,12 @@
 Running: testEvaluate
 Last evaluation source url for snippet: snippets:///1_1
-Snippet script added, sourceURL = snippets:///1_1
 Snippet execution result: undefined
 Last evaluation source url for snippet: snippets:///2_2
-Snippet script added, sourceURL = snippets:///2_2
-Snippet execution result: function doesNothing()
+Snippet execution result: function doesNothing() {
     var  i = 2+2;
 Last evaluation source url for snippet: snippets:///1_3
-Snippet script added, sourceURL = snippets:///1_3
 Snippet execution result: undefined
 Debugger was disabled.
Comment 2 Csaba Osztrogonác 2012-07-02 08:32:52 PDT
Skipped on Qt by r121680. Please unskip it with the proper fix.
Comment 3 Vsevolod Vlasov 2012-07-03 06:57:28 PDT
Created attachment 150600 [details]
Comment 4 Vsevolod Vlasov 2012-07-03 08:00:27 PDT
Committed r121765: <http://trac.webkit.org/changeset/121765>