search

petea / sage

A feed reader for Firefox.
http://sagerss.com
27 stars 8 forks source link

Not loading content with 42.0b9 #139

Closed coder66 closed 8 years ago

coder66 commented 8 years ago

Clicking on a feed in 42.0b9 does nothing, there is no content loaded by sage.

Worked fine with 42.0b8.

moyenager commented 8 years ago

Sage 1.5.4 not working for me either in FF 42.0b9 (Windows 10): no content loaded in main window.

gialloporpora commented 8 years ago

I have the same issue. In my Error Console I find these messages:

JS frame :: chrome://sage/content/sidebar.js :: openURI :: line 612
JS frame :: chrome://sage/content/sidebar.js :: sidebarController.loadFeedFromNode/onURIResolved< :: line 279
JS frame :: chrome://sage/content/sidebar.js :: sidebarController.loadFeedFromNode/< :: line 287
petea commented 8 years ago

Here's the full trace. For some reason loadURI() isn't working as expected at this point:

https://github.com/petea/sage/blob/master/src/chrome/content/sidebar.js#L612

Full Message: Component returned failure code: 0x8000ffff (NS_ERROR_UNEXPECTED) [nsIWebNavigation.loadURIWithOptions]
Full Stack: JS frame :: chrome://browser/content/browser.js :: _loadURIWithFlags :: line 11704
JS frame :: chrome://browser/content/tabbrowser.xml :: loadURIWithFlags :: line 6354
JS frame :: chrome://global/content/bindings/browser.xml :: loadURI :: line 119
JS frame :: chrome://browser/content/tabbrowser.xml :: loadURI :: line 3690
JS frame :: chrome://sage/content/sidebar.js :: openURI :: line 612
JS frame :: chrome://sage/content/sidebar.js :: sidebarController.loadFeedFromNode/onURIResolved< :: line 279
JS frame :: chrome://sage/content/sidebar.js :: sidebarController.loadFeedFromNode/< :: line 287
JS frame :: resource://gre/modules/Promise.jsm -> resource://gre/modules/Promise-backend.js :: Handler.prototype.process :: line 937
JS frame :: resource://gre/modules/Promise.jsm -> resource://gre/modules/Promise-backend.js :: this.PromiseWalker.walkerLoop :: line 813
JS frame :: resource://gre/modules/Promise.jsm -> resource://gre/modules/Promise-backend.js :: this.PromiseWalker.scheduleWalkerLoop/< :: line 747
native frame :: <unknown filename> :: <TOP_LEVEL> :: line 0
coder66 commented 8 years ago

42.0 is now available - has anyone tested with the final build? Peter, when to expect a fix?

petea commented 8 years ago

Looks like this has been fixed in 42 final.

coder66 commented 8 years ago

Confirmed with 42.0 final 64-bit

gialloporpora commented 8 years ago

I have tested with:

Mozilla/5.0 (Windows NT 5.1; rv:42.0) Gecko/20100101 Firefox/42.0

and I works fine, thanks.

Unfortunately, there is another issue in next version, I suppose, because FDE said that the sage protocol is not available.

petea commented 8 years ago

@gialloporpora If you can confirm an issue in FF 43, please create a new bug.