>+ mLastModified: null,
Not sure how this change relates.
>- win.updateStatusItem("statusText", message, aErrorCode);
>+ let code = feed.url.startsWith("http") ? aErrorCode : null;
>+ win.updateStatusItem("statusText", message, code);
Not sure how this change relates.
>- onunload="return FeedSubscriptions.onUnload();"
>+ onclose="return FeedSubscriptions.onClose();"
Not sure how this change relates.
>+ HIGH_COUNT_LIMIT_SUBSCRIBE: 200,
>+ HIGH_COUNT_LIMIT_UPDATE: 1000,
>+ CANCEL_REQUESTED: false,
What happens when there are too many items? (By comparison, for newsgroups, you can choose to either download the latest N (ignoring the rest) or the next N, although I don't know whether the RSS code would be able to let you do this. Also, N is pref-controlled, although we can leave that to a separate bug.)
Also, wouldn't it be more likely for there to be many items when you first subscribe? (I know a feed that only used to update every few days or so but the RSS feed contained the entire archive should you so want it.)
>+ confirmHighItemCount: function(aFeed, aCount) {
Where does this actually get called?
Comment on attachment 755964
updated for comments
>+ mLastModified: null,
Not sure how this change relates.
>- win.updateStatu sItem(" statusText" , message, aErrorCode); startsWith( "http") ? aErrorCode : null; sItem(" statusText" , message, code);
>+ let code = feed.url.
>+ win.updateStatu
Not sure how this change relates.
>- onunload="return FeedSubscriptio ns.onUnload( );" ns.onClose( );"
>+ onclose="return FeedSubscriptio
Not sure how this change relates.
>+ HIGH_COUNT_ LIMIT_SUBSCRIBE : 200, LIMIT_UPDATE: 1000,
>+ HIGH_COUNT_
>+ CANCEL_REQUESTED: false,
What happens when there are too many items? (By comparison, for newsgroups, you can choose to either download the latest N (ignoring the rest) or the next N, although I don't know whether the RSS code would be able to let you do this. Also, N is pref-controlled, although we can leave that to a separate bug.)
Also, wouldn't it be more likely for there to be many items when you first subscribe? (I know a feed that only used to update every few days or so but the RSS feed contained the entire archive should you so want it.)
>+ confirmHighItem Count: function(aFeed, aCount) {
Where does this actually get called?