Bug 7343

Summary: REGRESSION: fast/js/toString-overrides.html fails when run multiple times
Product: WebKit Reporter: mitz
Component: JavaScriptCoreAssignee: Nobody <webkit-unassigned>
Status: VERIFIED FIXED    
Severity: Normal CC: ggaren, oliver
Priority: P1    
Version: 420+   
Hardware: Mac   
OS: OS X 10.4   
Attachments:
Description Flags
Remove object from visited elements set before returning an error mjs: review+

mitz
Reported 2006-02-18 10:41:21 PST
% run-webkit-tests fast/jstoString-overrides.html fast/js/toString-overrides.html fast/js/toString-overrides.html fast/js/toString-overrides.html fast/js/toString-overrides.html fast/js/toString-overrides.html fast/js/toString-overrides.html fast/js/toString-overrides.html [...] Testing 7 test cases. fast/js ..... fast/js/toString-overrides.html -> failed . fast/js/toString-overrides.html -> failed . fast/js/toString-overrides.html -> failed 4 test cases (57%) succeeded 3 test cases (42%) had incorrect layout A failure looks like this: This test checks for regression against 4147: Array.toString() and toLocaleString() improvements from KDE KJS. On success, you will see a series of "PASS" messages, followed by "TEST COMPLETE". PASS [1].toString() is '1' PASS [1].toLocaleString() is 'toLocaleString' PASS [1].toLocaleString() is '1' PASS [/r/].toString() is 'toString2' PASS [/r/].toLocaleString() is 'toLocaleString2' PASS [/r/].toLocaleString() is 'toString2' FAIL caught should be true. Was false PASS successfullyParsed is true TEST COMPLETE
Attachments
Remove object from visited elements set before returning an error (3.98 KB, patch)
2006-02-18 13:09 PST, mitz 		mjs: review+
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
mitz
Comment 1 2006-02-18 10:45:17 PST
I can also reproduce this by reloading the test in Safari repeatedly. Eventually it will display FAIL in that line.
mitz
Comment 2 2006-02-18 11:23:45 PST
Undoing the fix for bug 3539 fixes this bug.
mitz
Comment 3 2006-02-18 13:09:29 PST
Created attachment 6590 [details] Remove object from visited elements set before returning an error
Maciej Stachowiak
Comment 4 2006-02-18 13:30:35 PST
Comment on attachment 6590 [details] Remove object from visited elements set before returning an error r=me
Note You need to log in before you can comment on or make changes to this bug.