Bug 121021

Summary: Enable JSC stress tests for the rest of non-Windows platforms
Product: WebKit Reporter: Zan Dobersek <zan>
Component: New BugsAssignee: Zan Dobersek <zan>
Status: RESOLVED FIXED    
Severity: Normal CC: fpizlo, lforschler, ossy
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Bug Depends on:    
Bug Blocks: 120809    
Attachments:
Description Flags
Patch
none
Patch fpizlo: review+

Zan Dobersek
Reported 2013-09-09 00:47:27 PDT
Enable JSC stress tests for the rest of non-Windows platforms
Attachments
Patch (2.05 KB, patch)
2013-09-09 00:53 PDT, Zan Dobersek 		no flags
DetailsFormatted DiffDiff
Patch (6.63 KB, patch)
2013-09-10 11:48 PDT, Zan Dobersek 		fpizlo: review+
DetailsFormatted DiffDiff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.
Zan Dobersek
Comment 1 2013-09-09 00:53:13 PDT
Created attachment 211015 [details] Patch
Csaba Osztrogonác
Comment 2 2013-09-09 03:15:33 PDT
I think the master.cfg should be adjusted to the new run-javascriptcore-tests output before it lands, similar to https://trac.webkit.org/changeset/155124 Or the output of the run-javascriptcore-tests should be simpler and/or easily parsable by master.cfg. Results for Mozilla tests: 0 regressions found. -----> catched by "regression" 0 tests fixed. OK. Results for fast/js tests: 0 failures found. -----> catched by "failure" 0 crashes found. OK. Results for JSC stress tests: 0 failures found. -----> one more "failure", master.cfg can be confused OK.
Zan Dobersek
Comment 3 2013-09-10 11:48:41 PDT
Created attachment 211216 [details] Patch
Filip Pizlo
Comment 4 2013-09-10 11:57:02 PDT
Comment on attachment 211216 [details] Patch Looks good, but you might need to rebase.
Zan Dobersek
Comment 5 2013-09-10 12:38:37 PDT
Committed r155467: <http://trac.webkit.org/changeset/155467>
Csaba Osztrogonác
Comment 6 2013-09-10 14:19:05 PDT
(In reply to comment #5) > Committed r155467: <http://trac.webkit.org/changeset/155467> Lucas, could you restart the build.webkit.org master after this change please?
Note You need to log in before you can comment on or make changes to this bug.