From 97fcb884ad0027b8960d3c92420bbe212df774f8 Mon Sep 17 00:00:00 2001 From: "nick.kreeger@park.edu" Date: Fri, 2 May 2008 12:30:03 -0700 Subject: [PATCH] Fixing bug 431606, content drop-down menus should appear with the parent menu. r=enndeakin, sr=roc, approval1.9=beltzner --- layout/xul/base/src/nsMenuPopupFrame.cpp | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/layout/xul/base/src/nsMenuPopupFrame.cpp b/layout/xul/base/src/nsMenuPopupFrame.cpp index ca53238c2755..1738bf64a879 100644 --- a/layout/xul/base/src/nsMenuPopupFrame.cpp +++ b/layout/xul/base/src/nsMenuPopupFrame.cpp @@ -1129,8 +1129,7 @@ nsMenuPopupFrame::SetPopupPosition(nsIFrame* aAnchorFrame) */ if ( (screenViewLocY + mRect.height) > screenBottomTwips ) { - // XXX Bug 84121 comment 48 says the next line has to use screenHeightTwips, why not screenBottomTwips? - PRInt32 moveDistY = (screenViewLocY + mRect.height) - screenHeightTwips; + PRInt32 moveDistY = (screenViewLocY + mRect.height) - screenBottomTwips; if ( screenViewLocY - moveDistY < screenTopTwips ) moveDistY = screenViewLocY - screenTopTwips; screenViewLocY -= moveDistY;