Bug 66357

Summary: make-script-test-wrappers should not look outside fast/js
Product: WebKit Reporter: Alexey Proskuryakov <ap>
Component: Tools / TestsAssignee: Alexey Proskuryakov <ap>
Status: RESOLVED FIXED    
Severity: Normal CC: webkit.review.bot
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Bug Depends on: 67747    
Bug Blocks:    
Attachments:
Description Flags
proposed patch
none
proposed patch none

Alexey Proskuryakov
Reported 2011-08-16 17:36:28 PDT
Splitting tests into HTML and JS parts is harmful: - it's more difficult to navigate to test source from results page to see what it's actually doing; - it's more difficult to navigate to it in Finder; - an extra file per test makes version control slower; - some people are led to believe that they should dynamically create a DOM from within JavaScript code, making tests even more difficult to follow; - other people manually modify wrappers, making it a pain to run the script for others. For fast/js, splitting the tests is likely worth it, as we want to eventually run them from within a pure JS interpreter without WebCore.
Attachments
proposed patch (4.00 KB, patch)
2011-08-16 17:38 PDT, Alexey Proskuryakov
no flags
proposed patch (1.61 KB, patch)
2011-08-16 17:43 PDT, Alexey Proskuryakov
no flags
Alexey Proskuryakov
Comment 1 2011-08-16 17:38:15 PDT
Created attachment 104131 [details] proposed patch
Alexey Proskuryakov
Comment 2 2011-08-16 17:41:01 PDT
Comment on attachment 104131 [details] proposed patch That's too much, should also look in fast/js subdirectories.
Alexey Proskuryakov
Comment 3 2011-08-16 17:43:43 PDT
Created attachment 104132 [details] proposed patch
Maciej Stachowiak
Comment 4 2011-10-25 12:59:49 PDT
Comment on attachment 104132 [details] proposed patch r=me
WebKit Review Bot
Comment 5 2011-10-25 14:15:47 PDT
Comment on attachment 104132 [details] proposed patch Clearing flags on attachment: 104132 Committed r98395: <http://trac.webkit.org/changeset/98395>
WebKit Review Bot
Comment 6 2011-10-25 14:15:52 PDT
All reviewed patches have been landed. Closing bug.
Note You need to log in before you can comment on or make changes to this bug.