RESOLVED FIXED 81334
Web Inspector: Implement snippets evaluation. 
https://bugs.webkit.org/show_bug.cgi?id=81334
Summary Web Inspector: Implement snippets evaluation.
Vsevolod Vlasov
Reported 2012-03-16 05:09:04 PDT
Implement snippets evaluation.
Attachments
Patch (6.52 KB, patch)
2012-03-16 05:12 PDT, Vsevolod Vlasov 		pfeldman: review+
webkit.review.bot: commit-queue-
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Vsevolod Vlasov
Comment 1 2012-03-16 05:12:11 PDT
Created attachment 132252 [details] Patch
Pavel Podivilov
Comment 2 2012-03-16 05:17:15 PDT
Comment on attachment 132252 [details] Patch looks good!
Pavel Feldman
Comment 3 2012-03-16 05:37:29 PDT
Comment on attachment 132252 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=132252&action=review > Source/WebCore/inspector/front-end/ConsoleView.js:665 > + evaluateAndAppendResult: function(text) Should evaluateUsingTextPrompt accept optional parameter that allows not showing the expression in the text prompt?
WebKit Review Bot
Comment 4 2012-03-16 05:47:38 PDT
Comment on attachment 132252 [details] Patch Attachment 132252 [details] did not pass chromium-ews (chromium-xvfb): Output: http://queues.webkit.org/results/11960375 New failing tests: inspector/debugger/snippets-model.html
Vsevolod Vlasov
Comment 5 2012-03-19 12:02:46 PDT
Committed r111212: <http://trac.webkit.org/changeset/111212>
Szilard Ledan
Comment 6 2012-03-20 02:56:18 PDT
The inspector/debugger/snippets-model.html test fails on Qt. --- /ramdisk/qt-linux-64-release/build/layout-test-results/inspector/debugger/snippets-model-expected.txt +++ /ramdisk/qt-linux-64-release/build/layout-test-results/inspector/debugger/snippets-model-actual.txt @@ -1,3 +1,4 @@ +FAIL: Timed out waiting for notifyDone to be called Tests snippets model. @@ -37,9 +38,4 @@ Running: testEvaluate Last evaluation source url for snippet: snippets://1_1 -Snippet script added, sourceURL = snippets://1_1 -Last evaluation source url for snippet: snippets://2_2 -Snippet script added, sourceURL = snippets://2_2 -Last evaluation source url for snippet: snippets://1_3 -Snippet script added, sourceURL = snippets://1_3 I will skip this until it is fixed.
Csaba Osztrogonác
Comment 7 2012-03-20 03:44:42 PDT
Reopen to fix it. Skipped on Qt - http://trac.webkit.org/changeset/111378/trunk/LayoutTests/platform/qt/Skipped Please unskip with the proper fix.
Vsevolod Vlasov
Comment 8 2012-03-20 10:47:10 PDT
(In reply to comment #7) > Reopen to fix it. Skipped on Qt - http://trac.webkit.org/changeset/111378/trunk/LayoutTests/platform/qt/Skipped > > Please unskip with the proper fix. Committed r111409: <http://trac.webkit.org/changeset/111409>
Note You need to log in before you can comment on or make changes to this bug.