Bug 120696

Summary: Unified test infrastructure via the jsc shell
Product: WebKit Reporter: Filip Pizlo <fpizlo>
Component: Tools / TestsAssignee: Filip Pizlo <fpizlo>
Status: RESOLVED FIXED    
Severity: Normal CC: ap, barraclough, ggaren, mark.lam, mhahnenberg, msaboff, oliver, sam
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: All   
OS: All   
Bug Depends on: 120697, 120722, 120723, 120725, 120759, 120765, 120766, 120808, 120809, 120918, 120974, 120994, 120996, 121011, 121043, 121578, 122376, 122385, 122397    
Bug Blocks: 112840, 122418    
Attachments:
Description Flags
the ptach
none
the patch oliver: review+

Filip Pizlo
Reported 2013-09-04 13:48:30 PDT
- Old mozilla tests - fast/js tests - fast/js/regress tests - Benchmarks as regression tests - Tests that instrument jsc with additional command-line options
Attachments
the ptach (150.89 KB, patch)
2013-10-06 12:01 PDT, Filip Pizlo 		no flags
DetailsFormatted DiffDiff
the patch (151.15 KB, patch)
2013-10-06 12:04 PDT, Filip Pizlo 		oliver: review+
DetailsFormatted DiffDiff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.
Filip Pizlo
Comment 1 2013-09-04 13:49:20 PDT
This blocks https://bugs.webkit.org/show_bug.cgi?id=112840 because I don't want to proceed with more FTL stuff without a way to test it.
Filip Pizlo
Comment 2 2013-10-06 12:01:45 PDT
Created attachment 213519 [details] the ptach
Filip Pizlo
Comment 3 2013-10-06 12:04:12 PDT
Created attachment 213520 [details] the patch
Geoffrey Garen
Comment 4 2013-10-06 12:10:35 PDT
Should we throw sputnik into the mix?
Filip Pizlo
Comment 5 2013-10-06 12:34:31 PDT
(In reply to comment #4) > Should we throw sputnik into the mix? We could but it's hard since Sputnik is in the form of .html files.
Filip Pizlo
Comment 6 2013-10-06 19:14:47 PDT
Landed in http://trac.webkit.org/changeset/157014
Alexey Proskuryakov
Comment 7 2013-10-06 21:01:40 PDT
> Should we throw sputnik into the mix? Or delete it for good?
Filip Pizlo
Comment 8 2013-10-06 21:06:02 PDT
(In reply to comment #7) > > Should we throw sputnik into the mix? > > Or delete it for good? It actually catches things that the other suites don't catch. So if we did delete it, it would probably only be if we replaced it with test262. While that might be a good thing to do at some point, it's not critical since test262 is mostly just sputnik + other tests (which we also already have).
Note You need to log in before you can comment on or make changes to this bug.