Bug 1569797 Comment 10 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 
Back in bug 254231 there was some hunk like this:

```
    if (uri.schemeIs("file"))
    {
      // Need to check for the file's existence explicitly due to Bug 875783.
      let file;
      try {
        file = Services.io.newURI(uri.spec, null, null).QueryInterface(Ci.nsIFileURL).file;
      }
      catch (ex) {}
      if (!file || !file.exists()) {
         // Simulate an invalid feed error.
        FeedUtils.log.info("Feed.download: file not found - " + uri.path);
        if (this.downloadCallback)
          this.downloadCallback.downloaded(this, FeedUtils.kNewsBlogRequestFailure);
        return;
      }
}
```

There referenced bug 875783 still exists. So is this still relevant?
Revision 1 by
on 
Back in bug 254231 there was some hunk like this:

```
if (uri.schemeIs("file"))
{
  // Need to check for the file's existence explicitly due to Bug 875783.
  let file;
  try {
    file = Services.io.newURI(uri.spec, null, null).QueryInterface(Ci.nsIFileURL).file;
  }
  catch (ex) {}
  if (!file || !file.exists()) {
     // Simulate an invalid feed error.
    FeedUtils.log.info("Feed.download: file not found - " + uri.path);
    if (this.downloadCallback)
      this.downloadCallback.downloaded(this, FeedUtils.kNewsBlogRequestFailure);
    return;
  }
}
```

There referenced bug 875783 still exists. So is this still relevant?

Back to Bug 1569797 Comment 10