Land tests for various bugs

VERIFIED FIXED in mozilla22

Status

()

Core
JavaScript Engine
VERIFIED FIXED
5 years ago
5 years ago

People

(Reporter: gkw, Assigned: gkw)

Tracking

Trunk
mozilla22
x86_64
Mac OS X
Points:
---
Bug Flags:
in-testsuite +

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment, 1 obsolete attachment)

(Assignee)

Description

5 years ago
I'll be attaching a patch to land testcases for bug 831424 (2 testcases), bug 836601 and bug 839758.
(Assignee)

Comment 1

5 years ago
Created attachment 718707 [details] [diff] [review]
patch v1

$ python -u jit_test.py --no-slow --tbpl -t 1200 <js shell> bug836601.js bug839758.js bug831424-1.js bug831424-2.js
[40| 0| 0| 0] 100% ==================================================>|  20.4s
PASSED ALL

bug831424-2.js seemed slightly slower than the rest but it does not seem to be much of a problem, I guess?
Attachment #718707 - Flags: review?(terrence)
(Assignee)

Updated

5 years ago
Assignee: general → gary
Status: NEW → ASSIGNED
Comment on attachment 718707 [details] [diff] [review]
patch v1

Review of attachment 718707 [details] [diff] [review]:
-----------------------------------------------------------------

Seems reasonable.
Attachment #718707 - Flags: review?(terrence) → review+
(Assignee)

Comment 5

5 years ago
Created attachment 719094 [details] [diff] [review]
patch v2

I've removed the testcase for bug 839758, reopened it, brought forward r+ in this patch, and will reland this.
Attachment #718707 - Attachment is obsolete: true
Attachment #719094 - Flags: review+
https://hg.mozilla.org/mozilla-central/rev/e85c2b64d0ed
Status: ASSIGNED → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla22
(Assignee)

Comment 8

5 years ago
VERIFIED because this is in-testsuite+.
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.