JavaScript has to be enabled to parse feeds

RESOLVED FIXED

Status

SeaMonkey
Feed Discovery and Preview
RESOLVED FIXED
5 years ago
5 years ago

People

(Reporter: neil@parkwaycc.co.uk, Assigned: neil@parkwaycc.co.uk)

Tracking

({regression})

Trunk
regression
Dependency tree / graph

SeaMonkey Tracking Flags

(seamonkey2.13 unaffected, seamonkey2.14 fixed, seamonkey2.15 fixed, seamonkey2.16 fixed)

Details

Attachments

(1 attachment)

(Assignee)

Description

5 years ago
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

5 years ago
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)
(Assignee)

Comment 2

5 years ago
(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.
(Assignee)

Comment 3

5 years ago
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
(Assignee)

Comment 4

5 years ago
5. Add the custom principal in FeedConverter.js
(Assignee)

Comment 5

5 years ago
Created attachment 674441 [details] [diff] [review]
Proposed patch
Assignee: nobody → neil
Status: NEW → ASSIGNED
Attachment #674441 - Flags: review?(iann_bugzilla)

Updated

5 years ago
Attachment #674441 - Flags: review?(iann_bugzilla) → review+
(Assignee)

Comment 6

5 years ago
Pushed comm-central changeset 0e457d4ab692.
(Assignee)

Comment 7

5 years ago
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
Attachment #674441 - Flags: approval-comm-beta?
Attachment #674441 - Flags: approval-comm-aurora?

Updated

5 years ago
Attachment #674441 - Flags: approval-comm-beta?
Attachment #674441 - Flags: approval-comm-beta+
Attachment #674441 - Flags: approval-comm-aurora?
Attachment #674441 - Flags: approval-comm-aurora+
(Assignee)

Comment 8

5 years ago
http://hg.mozilla.org/releases/comm-aurora/rev/1c1dd649ea89
http://hg.mozilla.org/releases/comm-beta/rev/2ccb24c2d876
Status: ASSIGNED → RESOLVED
Last Resolved: 5 years ago
status-seamonkey2.14: affected → fixed
status-seamonkey2.15: affected → fixed
status-seamonkey2.16: affected → fixed
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.