Land self-hosted assertion that previously caused CGC test failures

RESOLVED FIXED in Firefox 61

Status

()

P2
normal
RESOLVED FIXED
7 months ago
7 months ago

People

(Reporter: jonco, Assigned: jonco)

Tracking

unspecified
mozilla61
Points:
---

Firefox Tracking Flags

(firefox61 fixed)

Details

Attachments

(1 attachment)

(Assignee)

Description

7 months ago
In bug 1440309 an assertion in self-hosted code was left out of the final patch because it caused CGC tests to fail.  I don't know why this happened but it doesn't happen any more.
(Assignee)

Comment 2

7 months ago
Created attachment 8972949 [details] [diff] [review]
bug1458961-land-string-assertion
Attachment #8972949 - Flags: review?(andrebargull)
Comment on attachment 8972949 [details] [diff] [review]
bug1458961-land-string-assertion

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

Thanks!
Attachment #8972949 - Flags: review?(andrebargull) → review+
Priority: -- → P2

Comment 4

7 months ago
Pushed by jcoppeard@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/6262a4f457d1
Re-add removed assertion to String_pad r=anba

Comment 5

7 months ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/6262a4f457d1
Status: NEW → RESOLVED
Last Resolved: 7 months ago
status-firefox61: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla61
You need to log in before you can comment on or make changes to this bug.