421781 - Remove fake-account hooks from front-end

Status: RESOLVED FIXED

(Thunderbird :: Mail Window Front End, defect)

Description

[Joey Minta]

Attachment: patch

As discussed previously, the fake-account stuff has become useless at this point.  This patch removes the front-end hooks from Thunderbird.

Comment 1

[Phil Ringnalda (:philor)]

Comment on attachment 308288 [details] [diff] [review]
patch

>+++ b/mail/base/content/msgMail3PaneWindow.js	Sun Mar 09 09:09:18 2008 -0400

There's a var k = 0; up here that can go, since you're making it unused.

>     for(var i = 0; i < rangeCount; i++)

Bleah. While you're here, a space after the for, por favor.

>-          if (folderResource.Value != "http://home.netscape.com/NC-rdf#PageTitleFakeAccount") {
>-            var msgFolder = folderResource.QueryInterface(Components.interfaces.nsIMsgFolder);
>-            if(msgFolder)
>-              folderArray[k++] = msgFolder;
>-          }
>+          var msgFolder = folderResource.QueryInterface(Components.interfaces.nsIMsgFolder);
>+          folderArray.push(msgFolder);


r=me, if you're absolutely certain the if (msgFolder) isn't needed, that we'll either blow up or just happily push null if something goes wrong.

Comment 2

[Joey Minta]

Changes made, patch checked in. (QI throws in js, it doesn't return null, so we'll blow up, and we would have already)

Checking in mail/base/content/mail3PaneWindowCommands.js;
/cvsroot/mozilla/mail/base/content/mail3PaneWindowCommands.js,v  <--  mail3PaneWindowCommands.js
new revision: 1.43; previous revision: 1.42
done
Checking in mail/base/content/mailContextMenus.js;
/cvsroot/mozilla/mail/base/content/mailContextMenus.js,v  <--  mailContextMenus.js
new revision: 1.30; previous revision: 1.29
done
Checking in mail/base/content/msgMail3PaneWindow.js;
/cvsroot/mozilla/mail/base/content/msgMail3PaneWindow.js,v  <--  msgMail3PaneWindow.js
new revision: 1.145; previous revision: 1.144
done
Removing mail/themes/qute/mail/fakeAccount.css;
/cvsroot/mozilla/mail/themes/qute/mail/fakeAccount.css,v  <--  fakeAccount.css
new revision: delete; previous revision: 1.1
done
Checking in mail/themes/qute/mail/jar.mn;
/cvsroot/mozilla/mail/themes/qute/mail/jar.mn,v  <--  jar.mn
new revision: 1.23; previous revision: 1.22
done

Comment 3

[Serge Gautherie (:sgautherie)]

(In reply to comment #2)

http://bonsai.mozilla.org/cvsquery.cgi?treeid=default&module=ThunderbirdTinderbox&sortby=Date&hours=2&date=explicit&mindate=2008-03-10+08%3A30&maxdate=2008-03-10+08%3A30