Bug 543065, only scroll one ancestor of menus, back out due to scrolling regressions
authorNeil Deakin <neil@mozilla.com>
Wed, 24 Feb 2010 10:34:39 -0500
changeset 38462 ad972997528983d7b428bc984e8c9033027dde6b
parent 38461 ddfecbc93934b5dde2692fc20e13279090dd2991
child 38463 98a35ca2f5a1661a4eda7cd6622aa621b89528df
push idunknown
push userunknown
push dateunknown
bugs543065
milestone1.9.3a2pre
Bug 543065, only scroll one ancestor of menus, back out due to scrolling regressions
layout/xul/base/src/nsMenuPopupFrame.cpp
--- a/layout/xul/base/src/nsMenuPopupFrame.cpp
+++ b/layout/xul/base/src/nsMenuPopupFrame.cpp
@@ -1248,18 +1248,17 @@ nsIScrollableFrame* nsMenuPopupFrame::Ge
 void nsMenuPopupFrame::EnsureMenuItemIsVisible(nsMenuFrame* aMenuItem)
 {
   if (aMenuItem) {
     aMenuItem->PresContext()->PresShell()->
       ScrollFrameRectIntoView(aMenuItem,
                               nsRect(nsPoint(0,0), aMenuItem->GetRect().Size()),
                               NS_PRESSHELL_SCROLL_ANYWHERE,
                               NS_PRESSHELL_SCROLL_ANYWHERE,
-                              nsIPresShell::SCROLL_OVERFLOW_HIDDEN |
-                              nsIPresShell::SCROLL_FIRST_ANCESTOR_ONLY);
+                              nsIPresShell::SCROLL_OVERFLOW_HIDDEN);
   }
 }
 
 NS_IMETHODIMP nsMenuPopupFrame::SetCurrentMenuItem(nsMenuFrame* aMenuItem)
 {
   if (mCurrentMenu == aMenuItem)
     return NS_OK;