Summary: | REGRESSION (r72351): fast/profiler/throw-exception-from-eval.html fails | ||
---|---|---|---|
Product: | WebKit | Reporter: | Csaba Osztrogonác <ossy> |
Component: | New Bugs | Assignee: | Nobody <webkit-unassigned> |
Status: | RESOLVED INVALID | ||
Severity: | Normal | CC: | abarth, aestes, alex, aroben, barraclough, eric, jberlin, joepeck, oliver, webkit.review.bot |
Priority: | P2 | Keywords: | InRadar, LayoutTestFailure, Regression |
Version: | 528+ (Nightly build) | ||
Hardware: | All | ||
OS: | All | ||
Bug Depends on: | 49635 | ||
Bug Blocks: |
Description
Csaba Osztrogonác
2010-11-19 06:14:53 PST
I added the failing test to the Skipped list: http://trac.webkit.org/changeset/72397 These tests also fail on the Leopard bots. Skipped on Leopard bots in http://trac.webkit.org/changeset/72406. I'm not sure what these tests are supposed to output, but my guess here is that this is a progression, and that the new Qt/Leopard results are correct. Looking at the results, they are showing 'line 19' at the top and bottom of the call stack. Looking at the .html source, this corresponds to a script tag containing an eval. It is unlikely that there is a reason for this to be at both ends of the chain, so I think this indicates an error previously in failing to remove layers from the call stack upon exception unwind, where they should have been being removed. I'll have to investigate more to confirm this. It also fails in gtk+, I'm skipping it until we find the issue. (In reply to comment #6) > It also fails in gtk+, I'm skipping it until we find the issue. It also fails on Win-XP. I will check in the failing results soon. Hi Alejandro, This problem relates to: https://bugs.webkit.org/show_bug.cgi?id=26276 If GTK is testing on a platform that is not yet correctly measuring its stack size, then fixing this will fix the problem. However we may have to expect different results here. Is it also possible that GTK is testing on a platform where the stack extent is being measured correctly, but the machine stack is smaller. If this is the case, the test will not be able to recurse as many times, and will be expected to produce different results. cheers, G. (In reply to comment #7) > (In reply to comment #6) > > It also fails in gtk+, I'm skipping it until we find the issue. > > It also fails on Win-XP. I will check in the failing results soon. Jessie checked in failing results in r74560. The Windows XP bots are now not matching those failing results anymore; they're matching the cross-platform results: <http://build.webkit.org/results/Windows%20XP%20Debug%20(Tests)/r75711%20(24147)/fast/profiler/throw-exception-from-eval-pretty-diff.html>. It looks like the test produces the Windows-XP-specific results when run on its own, and produces the cross-platform results when run after other tests in fast/profiler. I haven't been able to determine a single other test that makes this one start producing cross-platform results. (In reply to comment #9) > It looks like the test produces the Windows-XP-specific results when run on its own, and produces the cross-platform results when run after other tests in fast/profiler. I haven't been able to determine a single other test that makes this one start producing cross-platform results. Even worse, it doesn't seem to be deterministic. This command produced different results when run two times in a row: run-webkit-tests fast/profiler/anonymous-event-handler.html fast/profiler/throw-exception-from-eval.html (In reply to comment #10) > Even worse, it doesn't seem to be deterministic. This command produced different results when run two times in a row: > > run-webkit-tests fast/profiler/anonymous-event-handler.html fast/profiler/throw-exception-from-eval.html Given the non-deterministic failure, I think it makes sense to disable this test on all platforms. (In reply to comment #11) > Given the non-deterministic failure, I think it makes sense to disable this test on all platforms. Disabled the test in r75723. http://trac.webkit.org/changeset/75723 might have broken SnowLeopard Intel Release (Tests) The following tests are not passing: fast/css/pseudo-in-range-invalid-value.html fast/css/pseudo-in-range.html fast/forms/form-collection-elements.html fast/forms/range-keyoperation.html Legacy Profiler has been removed. This test no longer exists. |