Closed Bug 980586 Opened 6 years ago Closed 6 years ago

Remove old e10s prompt stuff

Categories

(Core :: IPC, defect)

x86_64
Linux
defect
Not set

Tracking

()

RESOLVED FIXED
mozilla30

People

(Reporter: mrbkap, Assigned: mrbkap)

Details

Attachments

(1 file, 1 obsolete file)

Olli wrote a bunch of code back in the day to implement prompts and alerts for e10s. Unfortunately, since that time, the entire system got overhauled and his code got outdated. Now, with bug 899648, we're reimplementing it entirely (in JS this time), meaning that we can get rid of the old code.
Attached patch Patch (obsolete) — Splinter Review
smaug, I'm sorry to do this to you, but would you mind stamping the removal of this code?
Attachment #8387142 - Flags: review?(bugs)
no worry. I finally managed to clear my review queue. I'll review this tomorrow.
Comment on attachment 8387142 [details] [diff] [review]
Patch

I think you could also remove nsIDialogCreator.idl. That was added for this stuff.
Attachment #8387142 - Flags: review?(bugs) → review+
Attached patch For checkinSplinter Review
Attachment #8387142 - Attachment is obsolete: true
Attachment #8387871 - Flags: review+
https://hg.mozilla.org/mozilla-central/rev/6e5d9cc36cd4
Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla30
You need to log in before you can comment on or make changes to this bug.