Abstract out MozIntl's add-these-functions-to-Intl behavior into a single function usable by all mozIntl functions

RESOLVED FIXED in Firefox 53

Status

()

RESOLVED FIXED
2 years ago
2 years ago

People

(Reporter: Waldo, Assigned: Waldo)

Tracking

unspecified
mozilla53
Points:
---

Firefox Tracking Flags

(firefox53 fixed)

Details

Attachments

(1 attachment)

Comment hidden (empty)
(Assignee)

Comment 1

2 years ago
Created attachment 8821599 [details] [diff] [review]
Patch

Haven't actually tested/compiled this yet, but it passes an eyeball test at least, for now.
Attachment #8821599 - Flags: review?(andrebargull)
Comment on attachment 8821599 [details] [diff] [review]
Patch

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

> Haven't actually tested/compiled this yet, but it passes an eyeball test at least, for now.

Confirmed it compiles and works. :-)

https://treeherder.mozilla.org/#/jobs?repo=try&revision=cc3918041bb59a435f3a5231ecec622eae5f8746
Attachment #8821599 - Flags: review?(andrebargull) → review+

Comment 3

2 years ago
Pushed by jwalden@mit.edu:
https://hg.mozilla.org/integration/mozilla-inbound/rev/b93b58cfd16f
Abstract out MozIntl's add-these-functions-to-Intl behavior into a single function usable by all mozIntl functions.  r=anba

Comment 4

2 years ago
Pushed by jwalden@mit.edu:
https://hg.mozilla.org/integration/mozilla-inbound/rev/1a867dd80624
Followup bustage fix for a newly-landed test that needs an update along the same lines as the others this bug just touched.  r=sparky

Comment 5

2 years ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/b93b58cfd16f
https://hg.mozilla.org/mozilla-central/rev/1a867dd80624
Status: ASSIGNED → RESOLVED
Last Resolved: 2 years ago
status-firefox53: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla53
You need to log in before you can comment on or make changes to this bug.