Closed Bug 884279 Opened 11 years ago Closed 11 years ago

renaming future to promise

Categories

(Core :: DOM: Core & HTML, defect)

x86_64
Linux
defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla25

People

(Reporter: baku, Assigned: baku)

References

(Blocks 1 open bug)

Details

Attachments

(1 file, 5 obsolete files)

Splitting bug 875289
Attached patch patch (obsolete) — Splinter Review
Attachment #764103 - Flags: review?(mounir)
Comment on attachment 764103 [details] [diff] [review] patch Review of attachment 764103 [details] [diff] [review]: ----------------------------------------------------------------- Please use `hg mv` for Future.webidl -> Promise.webidl. ::: modules/libpref/src/init/all.js @@ +1821,5 @@ > > // If true, ArchiveReader will be enabled > pref("dom.archivereader.enabled", false); > > // If true, Future will be enabled s/Future/Promise/
Attachment #764103 - Flags: review?(mounir) → review+
Attached patch patch (obsolete) — Splinter Review
Attachment #764103 - Attachment is obsolete: true
Keywords: checkin-needed
This doesn't apply cleanly on inbound since Promise.h does not exist there.
Keywords: checkin-needed
?!? Promise.h is a renaming of Future.h it's included in this patch.
Attached patch patch for m-i (obsolete) — Splinter Review
Keywords: checkin-needed
Attached patch patch (obsolete) — Splinter Review
Rebased on top of m-c and m-i
Attachment #764622 - Attachment is obsolete: true
Attachment #765025 - Attachment is obsolete: true
Keywords: checkin-needed
Blocks: 885333
BTW, this patch has bug 875289 in the commit message. I fixed it when I pushed it the first time, but you need to fix it in your local queue before you push it again.
Attached patch patch (obsolete) — Splinter Review
rebased
Attachment #765243 - Attachment is obsolete: true
Attached patch patchSplinter Review
Attachment #767707 - Attachment is obsolete: true
Keywords: checkin-needed
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla25
Component: DOM → DOM: Core & HTML
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: