Remove virtual accessor GetURIExternal from Link

RESOLVED WONTFIX

Status

()

defect
RESOLVED WONTFIX
8 years ago
8 years ago

People

(Reporter: khuey, Assigned: khuey)

Tracking

(Blocks 1 bug)

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

This was added in Bug 510202 and can go away now that we live in a libxul only world.
Assignee: nobody → khuey
Comment on attachment 527959 [details] [diff] [review]
Remove virtual accessor used as a non-libxul workaround now that disable-libxul is dead.

r=sdwilsh
Attachment #527959 - Flags: review?(sdwilsh) → review+
Is this not meant for use by non-core code, then?  _That_'s not linked into libxul....
(In reply to comment #3)
> Is this not meant for use by non-core code, then?  _That_'s not linked into
> libxul....
That's a good point.  Maybe we don't want this change?
Well, I got the impression that we did this just for that one use case ... if we don't want this, WONTFIX the bug.
So, who gets to make that call? Bz?
Unless there's obvious harm from having this (which I don't see yet), I'd just keep it.
Status: NEW → RESOLVED
Last Resolved: 8 years ago
Resolution: --- → WONTFIX
You need to log in before you can comment on or make changes to this bug.