Closed Bug 1325675 Opened 8 years ago Closed 8 years ago

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

Categories

(Core :: JavaScript: Internationalization API, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla53
Tracking Status
firefox53 --- fixed

People

(Reporter: Waldo, Assigned: Waldo)

Details

Attachments

(1 file)

No description provided.
Attached patch PatchSplinter Review
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+
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
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
Status: ASSIGNED → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla53
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: