Bommels05/Mozilla
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

bug 363112: nsDeckFrame::GetSelectedBox should use mIndex

Browse Source 
patch by Alex Vincent <ajvincent@gmail.com>, r=enn, sr=neil


git-svn-id: svn://10.0.0.236/trunk@216825 18797224-902f-48f8-a5cc-f745e15eee43
This commit is contained in:
db48x%yahoo.com
2006-12-10 18:13:48 +00:00
parent 61bd14e6ba
commit 4781091ef7
1 changed files with 6 additions and 10 deletions
Download Patch File Download Diff File Expand all files Collapse all files

16
mozilla/layout/xul/base/src/nsDeckFrame.cpp
View File

@@ -99,7 +99,7 @@ nsDeckFrame::AttributeChanged(PRInt32 aNameSpaceID,
aModType);
// if the index changed hide the old element and make the now element visible
// if the index changed hide the old element and make the new element visible
if (aAttribute == nsXULAtoms::selectedIndex) {
IndexChanged(GetPresContext());
}
@@ -157,16 +157,16 @@ nsDeckFrame::IndexChanged(nsPresContext* aPresContext)
Redraw(state);
// hide the currently showing box
nsIBox* currentBox = (mIndex >= 0) ? GetBoxAt(mIndex) : nsnull;
nsIBox* currentBox = GetSelectedBox();
if (currentBox) // only hide if it exists
HideBox(aPresContext, currentBox);
mIndex = index;
// show the new box
nsIBox* newBox = (index >= 0) ? GetBoxAt(index) : nsnull;
nsIBox* newBox = GetSelectedBox();
if (newBox) // only show if it exists
ShowBox(aPresContext, newBox);
mIndex = index;
}
PRInt32
@@ -191,11 +191,7 @@ nsDeckFrame::GetSelectedIndex()
nsIBox*
nsDeckFrame::GetSelectedBox()
{
// ok we want to paint only the child that as at the given index
PRInt32 index = GetSelectedIndex();
// get the child at that index.
return (index >= 0) ? GetBoxAt(index) : nsnull;
return (mIndex >= 0) ? GetBoxAt(mIndex) : nsnull;
}
NS_IMETHODIMP