From 3546f54df95b9f6194d5e717ee646c6ee84cfaec Mon Sep 17 00:00:00 2001
From: "nick.kreeger%park.edu"
 <nick.kreeger%park.edu@18797224-902f-48f8-a5cc-f745e15eee43>
Date: Thu, 14 Sep 2006 17:02:44 +0000
Subject: [PATCH] Fixing bug 352078, RSS selection menu looks weird when
 there's no default app inserted. <Camino only>. r=stuart.morgan, sr=pinkerton

git-svn-id: svn://10.0.0.236/trunk@211696 18797224-902f-48f8-a5cc-f745e15eee43
---
 mozilla/camino/src/browser/AppListMenuFactory.mm | 12 ++++++++----
 1 file changed, 8 insertions(+), 4 deletions(-)

diff --git a/mozilla/camino/src/browser/AppListMenuFactory.mm b/mozilla/camino/src/browser/AppListMenuFactory.mm
index b23c4baeb0a..5b74fb9f4f6 100644
--- a/mozilla/camino/src/browser/AppListMenuFactory.mm
+++ b/mozilla/camino/src/browser/AppListMenuFactory.mm
@@ -114,6 +114,7 @@ static AppListMenuFactory* sAppListMenuFactoryInstance = nil;
   [menu addItem:[NSMenuItem separatorItem]];
   
   BOOL insertedDefaultApp = NO;
+  BOOL shouldInsertSeperatorAtEnd = NO;
   NSEnumerator* feedAppsEnum = [feedApps objectEnumerator];
   NSString* curBundleID = nil;
   while ((curBundleID = [feedAppsEnum nextObject])) {
@@ -135,8 +136,10 @@ static AppListMenuFactory* sAppListMenuFactoryInstance = nil;
       [menu insertItem:menuItem atIndex:0];
       insertedDefaultApp = YES;
     }
-    else
+    else {
       [menu addItem:menuItem];
+      shouldInsertSeperatorAtEnd = YES;
+    }
   }
   
   // The user selected an application that is not registered for "feed://".
@@ -147,19 +150,20 @@ static AppListMenuFactory* sAppListMenuFactoryInstance = nil;
                                         withBundleID:defaultFeedViewerID
                                            andAction:nil 
                                            andTarget:inTarget];
-      [[inPopupButton menu] insertItem:menuItem atIndex:0];
+      [menu insertItem:menuItem atIndex:0];
     }
     // Since we couldn't find a default application, add a blank menu item.
     else {
       NSMenuItem* dummyItem = [[NSMenuItem alloc] init];
       [dummyItem setTitle:@""];
-      [[inPopupButton menu] insertItem:dummyItem atIndex:0];
+      [menu insertItem:dummyItem atIndex:0];
       [dummyItem release];
     }
   }
   
   // allow the user to select a feed application
-  [menu addItem:[NSMenuItem separatorItem]];
+  if (shouldInsertSeperatorAtEnd)
+    [menu addItem:[NSMenuItem separatorItem]];
   NSMenuItem* selectItem = [[NSMenuItem alloc] initWithTitle:NSLocalizedString(@"Select...", nil)
                                                       action:inSelectAction 
                                                keyEquivalent:@""];