Bug 1569797 Comment 13 Edit History

Note: The actual edited comment in the bug view page will always show the original commenter’s name and original timestamp.

Original comment by
on 
(In reply to Jorg K (GMT+2) from comment #12)
> Can you give me a line number where to insert this block? Here, right:
> https://searchfox.org/comm-central/rev/61a2e430379fa81b1afdc4b88cfd9f97b55f955a/mailnews/extensions/newsblog/content/Feed.js#125
> 
yes.

> So the block above with s/uri/this.url/ ? What's the `this.downloadCallback`?
this.url is a string, so the test would have to be startsWith("file:"). downloadCallback() cleans up and sets messages, and is different depending on whether the download is initiated from Subscribe dialog, which has its own handler.
Revision 1 by
on 
(In reply to Jorg K (GMT+2) from comment #12)
> Can you give me a line number where to insert this block? Here, right:
> https://searchfox.org/comm-central/rev/61a2e430379fa81b1afdc4b88cfd9f97b55f955a/mailnews/extensions/newsblog/content/Feed.js#125
> 
yes.

> So the block above with s/uri/this.url/ ? What's the `this.downloadCallback`?

this.url is a string, so the test would have to be startsWith("file:"). downloadCallback() cleans up and sets messages, and is different depending on whether the download is initiated from Subscribe dialog, which has its own handler.

Back to Bug 1569797 Comment 13