Summary: | Web Inspector: add integration test for compiler source maps. | ||||||||
---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Pavel Podivilov <podivilov> | ||||||
Component: | Web Inspector (Deprecated) | Assignee: | Pavel Podivilov <podivilov> | ||||||
Status: | RESOLVED FIXED | ||||||||
Severity: | Normal | CC: | adamk, apavlov, bweinstein, dglazkov, joepeck, keishi, loislo, pfeldman, pmuellr, rik, timothy, webkit.review.bot, yurys | ||||||
Priority: | P2 | ||||||||
Version: | 528+ (Nightly build) | ||||||||
Hardware: | All | ||||||||
OS: | All | ||||||||
Bug Depends on: | 73032 | ||||||||
Bug Blocks: | 72824 | ||||||||
Attachments: |
|
Description
Pavel Podivilov
2011-11-18 07:34:47 PST
Created attachment 115806 [details]
Patch
Comment on attachment 115806 [details] Patch Attachment 115806 [details] did not pass chromium-ews (chromium-xvfb): Output: http://queues.webkit.org/results/10393163 New failing tests: http/tests/inspector/compiler-source-mapping-debug.html Created attachment 115817 [details]
Patch
Comment on attachment 115817 [details] Patch Attachment 115817 [details] did not pass chromium-ews (chromium-xvfb): Output: http://queues.webkit.org/results/10518133 New failing tests: http/tests/inspector/compiler-source-mapping-debug.html Comment on attachment 115817 [details]
Patch
Make sure it compiles under chromium prior to landing.
Committed r101069: <http://trac.webkit.org/changeset/101069> (In reply to comment #4) > (From update of attachment 115817 [details]) > Attachment 115817 [details] did not pass chromium-ews (chromium-xvfb): > Output: http://queues.webkit.org/results/10518133 > > New failing tests: > http/tests/inspector/compiler-source-mapping-debug.html As predicted by the EWS, this new test is always timing out on the bots: http://test-results.appspot.com/dashboards/flakiness_dashboard.html#tests=http%2Ftests%2Finspector%2Fcompiler-source-mapping-debug.html I intend to roll this out, since the only point of this patch was to add a test. Committed r101128: <http://trac.webkit.org/changeset/101128> (In reply to comment #7) > (In reply to comment #4) > > (From update of attachment 115817 [details] [details]) > > Attachment 115817 [details] [details] did not pass chromium-ews (chromium-xvfb): > > Output: http://queues.webkit.org/results/10518133 > > > > New failing tests: > > http/tests/inspector/compiler-source-mapping-debug.html > > As predicted by the EWS, this new test is always timing out on the bots: > > http://test-results.appspot.com/dashboards/flakiness_dashboard.html#tests=http%2Ftests%2Finspector%2Fcompiler-source-mapping-debug.html > > I intend to roll this out, since the only point of this patch was to add a test. Sorry and thanks for rolling it out! |