934316 - Fix some bad folderpane scrolling perf for feed folders, remove transition code

Status: RESOLVED FIXED

(MailNews Core :: Feed Reader, defect)

Comment 1

[alta88]

Attachment: fixFeedFolderPerf.patch

1. Folderpane properties were changed to indicate whether a feed folder had a subscription, running code that for legacy reasons had to check lots of places if a folder was empty.  This is now refactored.
2. Feed folder db sync with the feedUrl property (required to handle folder renames/moves) has been refactored.
3. Bug 271257.

Comment 2

[Magnus Melin [:mkmelin]]

Comment on attachment 826548 [details] [diff] [review]
fixFeedFolderPerf.patch

Review of attachment 826548 [details] [diff] [review]:
-----------------------------------------------------------------

Lots of changes. From what i can tell it seems ok. r=mkmelin

::: mailnews/extensions/newsblog/content/feed-subscriptions.js
@@ +1439,5 @@
>        {
>          win.updateStatusItem("progressMeter", 100);
>  
> +        // Add the feed to the databases.
> +        FeedUtils.addFeed(feed); 

trailing whitespace

::: mailnews/local/public/nsINewsBlogFeedDownloader.idl
@@ +17,2 @@
>  
> +  /* A convient method to subscribe to feeds without going through the Subscribe

please make this document comments javadoc/doxygen documentation comments while you're there
/**
 *
 */

::: mailnews/local/src/nsRssIncomingServer.cpp
@@ +192,5 @@
>  }
>  
>  NS_IMETHODIMP nsRssIncomingServer::FolderAdded(nsIMsgFolder *aFolder)
>  {
> +  // Not necessary for new folder adds.

"nothing to do", and maybe point to what it does?

Comment 3

[alta88]

Attachment: fixFeedPerf.patch

update for comments.

Comment 4

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/comm-central/rev/e3deddef504b