804411 - JavaScript has to be enabled to parse feeds

Status: RESOLVED FIXED

(SeaMonkey :: Feed Discovery and Preview, defect)

Description

[[e-mail]]

Steps to reproduce problem:
1) disable JavaScript in browser
2) open: http://www.irregularwebcomic.net/rss3.xml

Actual result: feed doesn't get parsed.

Expected result: feed should be displayed.

Additional information: This bug was created as a clone of bug 786009.

Comment 1

[therube]

Confirmed.

2.13.1 works properly.

Mozilla/5.0 (Windows NT 6.1; WOW64; rv:16.0) Gecko/20121011 Firefox/16.0 SeaMonkey/2.13.1
Build identifier: 20121011080919

By the time we reach here we're broken:

Mozilla/5.0 (Windows NT 6.1; WOW64; rv:17.0) Gecko/17.0 Firefox/17.0 SeaMonkey/2.14a1
Build identifier: 20120726003003

In that case, no page actually loads.  It shows "done", but the page is entirely gray, & Page Info shows it to be about:blank.

Error Console shows:

Error: Services.scriptSecurityManager.getCodebasePrincipal is not a function
Source File: resource:///components/nsAbout.js
Line: 43

I did see this, once:

Error: Search service falling back to synchronous initialization at SRCH_SVC__ensureInitialized@resource:///components/nsSearchService.js:2498
@resource:///components/nsSearchService.js:3468
@chrome://communicator/content/nsContextMenu.js:1284
@chrome://communicator/content/nsContextMenu.js:45
nsContextMenu@chrome://communicator/content/nsContextMenu.js:26
onpopupshowing@chrome://navigator/content/navigator.xul:1

Source File: resource:///components/nsSearchService.js
Line: 2499

A later build is different again, actually loading the page, though not displaying its data, & no Error Console message:

Mozilla/5.0 (Windows NT 6.1; WOW64; rv:17.0) Gecko/17.0 Firefox/17.0 SeaMonkey/2.14a1
Build identifier: 20120814003006

(these are builds I had on hand, so can't necessarily give a precise regression range)

Comment 2

[[e-mail]]

(In reply to therube from comment #1)
> Mozilla/5.0 (Windows NT 6.1; WOW64; rv:17.0) Gecko/17.0 Firefox/17.0
> SeaMonkey/2.14a1
> Build identifier: 20120726003003
> 
> In that case, no page actually loads.  It shows "done", but the page is
> entirely gray, & Page Info shows it to be about:blank.
> 
> Error Console shows:
> 
> Error: Services.scriptSecurityManager.getCodebasePrincipal is not a function
> Source File: resource:///components/nsAbout.js
> Line: 43
This was fixed later on in the 2.14 cycle, so 2.14b doesn't throw this error.

Comment 3

[[e-mail]]

Well, there appear to be a number of options:

1. Restore nsAboutFeeds.js and add the custom principal there
2. Always add the custom principal in nsAbout.js
   (I tried this and I couldn't get anything to break)
3. Add a special case for about:feeds in nsAbout.js
4. Add a special case for about:feeds in nsAbout.js BUT
   hide it under its own flag

Comment 4

[[e-mail]]

5. Add the custom principal in FeedConverter.js

Comment 5

[[e-mail]]

Attachment: Proposed patch

Comment 6

[[e-mail]]

Pushed comm-central changeset 0e457d4ab692.

Comment 7

[[e-mail]]

Comment on attachment 674441 [details] [diff] [review]
Proposed patch

[Approval Request Comment]
Regression caused by (bug #): 774585
User impact if declined: Unable to view feeds when JavaScript is disabled
Testing completed (on m-c, etc.): 
Risk to taking this patch (and alternatives if risky): Low
String changes made by this patch: None

Comment 8

[[e-mail]]

http://hg.mozilla.org/releases/comm-aurora/rev/1c1dd649ea89
http://hg.mozilla.org/releases/comm-beta/rev/2ccb24c2d876