From 5f12169f85c1a434d3b6d04019cd8d5001bfc2b0 Mon Sep 17 00:00:00 2001
From: "reed@reedloden.com" <reed@reedloden.com>
Date: Wed, 16 Jan 2008 23:06:43 -0800
Subject: [PATCH] Fix bustage from bug 405512 by removing unneeded |if| check.

---
 widget/src/cocoa/nsChildView.mm | 18 ++++++++----------
 1 file changed, 8 insertions(+), 10 deletions(-)

diff --git a/widget/src/cocoa/nsChildView.mm b/widget/src/cocoa/nsChildView.mm
index 35d54a759175..8ee437971e6f 100644
--- a/widget/src/cocoa/nsChildView.mm
+++ b/widget/src/cocoa/nsChildView.mm
@@ -2457,16 +2457,14 @@ NSEvent* gLastDragEvent = nil;
       if (menuRollup) {
         nsAutoTArray<nsIWidget*, 5> widgetChain;
         menuRollup->GetSubmenuWidgetChain(&widgetChain);
-        if (widgetChain) {
-          for (PRUint32 i = 0; i < widgetChain.Length(); i++) {
-            nsIWidget* widget = widgetChain[i];
-            NSWindow* currWindow = (NSWindow*)widget->GetNativeData(NS_NATIVE_WINDOW);
-            if (nsCocoaUtils::IsEventOverWindow(theEvent, currWindow)) {
-              rollup = PR_FALSE;
-              break;
-            }
-          } // foreach parent menu widget
-        }
+        for (PRUint32 i = 0; i < widgetChain.Length(); i++) {
+          nsIWidget* widget = widgetChain[i];
+          NSWindow* currWindow = (NSWindow*)widget->GetNativeData(NS_NATIVE_WINDOW);
+          if (nsCocoaUtils::IsEventOverWindow(theEvent, currWindow)) {
+            rollup = PR_FALSE;
+            break;
+          }
+        } // foreach parent menu widget
       } // if rollup listener knows about menus
 
       // if we've determined that we should still rollup, do it.