Closed Bug 790034 Opened 9 years ago Closed 9 years ago

Remove nsIDOMWindowUtils.goOnline

Categories

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

Product:
Core
Component:
DOM: Core & HTML
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

()

Status:
RESOLVED FIXED
Milestone:
mozilla18

People

(Reporter: mccr8, Assigned: mccr8)

References

Details

Attachments

(1 file)

remove it
3.23 KB, patch
smaug
: review+
Details | Diff | Splinter Review 
This function is weird and should go away.
Assignee: nobody → continuation
Attached patch remove it — — Splinter Review 
Comment on attachment 659871 [details] [diff] [review]
remove it

I'm not really sure who is right to review this, so I'll just flag you, Olli. There were three uses of this function, one in each of our three frontends, but they have now been removed thanks to the fine work of Blair McBride.
Attachment #659871 - Flags: review?(bugs)
Attachment #659871 - Flags: review?(bugs) → review+
(In reply to Andrew McCreight from comment #2)
> There were three uses of this function, one in each of our three frontends
No, one of the uses was in the Gecko backend; two of your frontends had an override for that that file, while a third one didn't.
https://hg.mozilla.org/mozilla-central/rev/3403a77db505
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla18
Component: DOM → DOM: Core & HTML
You need to log in before you can comment on or make changes to this bug.