Closed
Bug 506119
Opened 15 years ago
Closed 9 years ago
deCOMtaminate FeedWriter
Categories
(Firefox Graveyard :: RSS Discovery and Preview, defect, P3)
Firefox Graveyard
RSS Discovery and Preview
Tracking
(Not tracked)
RESOLVED
WORKSFORME
People
(Reporter: dietrich, Unassigned)
References
(Blocks 2 open bugs)
Details
(Keywords: perf, Whiteboard: [ts])
nsIFeedWriter and impl have only a single caller and use-case: displaying the feed preview. we should investigate if we can either bundle it directly with the content page, or convert it to a js module. for background, see bug 336903.
Comment 1•15 years ago
|
||
Bug 499123 seems to be doing something very much along these lines. I think that will help.
Depends on: 499123
Reporter | ||
Updated•15 years ago
|
Priority: -- → P3
Comment 2•13 years ago
|
||
Can I just remove BrowserFeedWriter and include the FeedWriter.js into subscribe.js? This will allow us to remove the nsIFeedWriter interface.
Comment 3•13 years ago
|
||
(In reply to Dietrich Ayala from comment #0) > nsIFeedWriter and impl have only a single caller and use-case: displaying > the feed preview. > > we should investigate if we can either bundle it directly with the content > page, or convert it to a js module. > > for background, see bug 336903. Surely the point of bug 336903 was that the component was created to make the feed preview page safe, by not having it run as chrome? Currently only an XPCOM component can do this.
Comment 4•13 years ago
|
||
So can't we deCOMtaminate it?
Reporter | ||
Comment 5•13 years ago
|
||
(In reply to neil@parkwaycc.co.uk from comment #3) > Surely the point of bug 336903 was that the component was created to make > the feed preview page safe, by not having it run as chrome? Currently only > an XPCOM component can do this. Yeah, the idea is to explore other ways of exposing the data to content. We now have framescripts, as well as trivial means of exposing JS APIs to web content. I haven't dug around the feed preview page enough to know what exactly the danger points are, or whether these things are feasible.
Comment 6•9 years ago
|
||
nsIFeedWriter seems to not exist any more.
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → WORKSFORME
Updated•5 years ago
|
Product: Firefox → Firefox Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•