Closed
Bug 591885
Opened 14 years ago
Closed 14 years ago
Test cases should be upgraded to use USES_BUGCOMPAT
Categories
(Tamarin Graveyard :: Virtual Machine, defect)
Tamarin Graveyard
Virtual Machine
Tracking
(Not tracked)
VERIFIED
FIXED
People
(Reporter: lhansen, Assigned: cpeyer)
References
Details
Attachments
(1 file)
21.28 KB,
patch
|
jsudduth
:
review+
|
Details | Diff | Splinter Review |
A few test cases still have individual -bugcompat flags in the avm_args file, they should be rewritten to use USES_BUGCOMPAT.
Bug #590879 will change those flags to be -swfversion, this bug should wait on that bug to land.
The tests I know about are:
$ grep swfversion $(find . -name '*.as.avm_args')
test/acceptance/as3/Vector/bug504525/bug504525a.as.avm_args:-swfversion 9
test/acceptance/as3/Vector/bug504525/bug504525b.as.avm_args:-swfversion 10
test/acceptance/as3/Vector/bug504525/bug504525c.as.avm_args:-swfversion 11
test/acceptance/ecma3/GlobalObject/parseFloat_513018a.as.avm_args:-swfversion 9
test/acceptance/ecma3/GlobalObject/parseFloat_513018b.as.avm_args:-swfversion 11
test/acceptance/ecma3/String/localeCompare_585791a.as.avm_args:-swfversion 9
test/acceptance/ecma3/String/localeCompare_585791b.as.avm_args:-swfversion 11
Assignee | ||
Comment 1•14 years ago
|
||
1. replace all uses of USES_BUGCOMPAT with USES_SWFVERSION (see Bug 590879)
2. replace tests that use individual -swfversion avm_args files to use the future compatible USES_SWFVERSION
Attachment #470904 -
Flags: review?(jsudduth)
Comment 2•14 years ago
|
||
Comment on attachment 470904 [details] [diff] [review]
Update any tests that use versioning to use USES_SWFVERSION
Looks good - just pull out the unrelated metricinfo.py code.
Attachment #470904 -
Flags: review?(jsudduth) → review+
Assignee | ||
Comment 3•14 years ago
|
||
Pushed to redux -metricinfo stuffs:
http://hg.mozilla.org/tamarin-redux/rev/41ee7ef53bab
Status: ASSIGNED → RESOLVED
Closed: 14 years ago
Resolution: --- → FIXED
Assignee | ||
Updated•14 years ago
|
Status: RESOLVED → VERIFIED
You need to log in
before you can comment on or make changes to this bug.
Description
•