Bug 33382 - Bring the Mozilla JavaScript tests up-to-date
Summary: Bring the Mozilla JavaScript tests up-to-date
Status: ASSIGNED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Tools / Tests (show other bugs)
Version: 528+ (Nightly build)
Hardware: PC OS X 10.5
: P2 Normal
Assignee: Kent Hansen
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2010-01-08 06:11 PST by Kent Hansen
Modified: 2011-04-12 22:05 PDT (History)
3 users (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Kent Hansen 2010-01-08 06:11:15 PST
The tests under JavaScriptCore/tests/mozilla were imported into trunk in October 2003. It would be nice to try to update them to the most recent tests from Mozilla trunk.
1) This would increase test coverage (adding the ES5 tests, for example), and 2) Mozilla have made behavioral changes since the tests were imported, and consequently updated their tests (see e.g. https://bugs.webkit.org/show_bug.cgi?id=33319); in such cases we should consider whether JavaScriptCore should adopt the same behavior.
Comment 1 Kent Hansen 2010-01-08 06:57:11 PST
Some potential issues I noticed with doing a "batch-update":
1) Mozilla's tests use tabs. These were not converted to spaces when imported into WebKit trunk. Should they be when importing the updated tests?
2) The indentation is different.
3) Some Mozilla tests have been patched in WebKit trunk. Care must be taken so that those are preserved (i.e. check whether the updated tests cover those changes, or if they must be kept).
4) Different license headers ("Netscape Public License" vs "Mozilla Public License").
Comment 2 Alexey Proskuryakov 2010-01-08 08:07:58 PST
> 1) Mozilla's tests use tabs. These were not converted to spaces when imported
> into WebKit trunk. Should they be when importing the updated tests?
> 2) The indentation is different.

I think that these aspects should be preserved for easier merging of changes in the future.