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
Rebased

https://tbpl.mozilla.org/?tree=Try&rev=4e48e1903f13
Attachment #767707 - Attachment is obsolete: true
Keywords: checkin-needed
https://hg.mozilla.org/mozilla-central/rev/cc1d6a5a4840
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: