Bug 1379334 - Convert mis-indented code to 2-space indent, plus bracing and logical operator position fixes when reindenting.
MozReview-Commit-ID: ElsSNF40LZQ
--- a/layout/generic/nsGfxScrollFrame.cpp
+++ b/layout/generic/nsGfxScrollFrame.cpp
@@ -5095,103 +5095,105 @@ nsXULScrollFrame::XULLayout(nsBoxLayoutS
scrollbars are visible), and reflow any fixed position views
(if we're the viewport and we added or removed a scrollbar).
**************/
ScrollbarStyles styles = GetScrollbarStyles();
// Look at our style do we always have vertical or horizontal scrollbars?
if (styles.mHorizontal == NS_STYLE_OVERFLOW_SCROLL)
- mHelper.mHasHorizontalScrollbar = true;
+ mHelper.mHasHorizontalScrollbar = true;
if (styles.mVertical == NS_STYLE_OVERFLOW_SCROLL)
- mHelper.mHasVerticalScrollbar = true;
+ mHelper.mHasVerticalScrollbar = true;
if (mHelper.mHasHorizontalScrollbar)
- AddHorizontalScrollbar(aState, scrollbarBottom);
+ AddHorizontalScrollbar(aState, scrollbarBottom);
if (mHelper.mHasVerticalScrollbar)
- AddVerticalScrollbar(aState, scrollbarRight);
+ AddVerticalScrollbar(aState, scrollbarRight);
// layout our the scroll area
LayoutScrollArea(aState, oldScrollPosition);
// now look at the content area and see if we need scrollbars or not
bool needsLayout = false;
// if we have 'auto' scrollbars look at the vertical case
if (styles.mVertical != NS_STYLE_OVERFLOW_SCROLL) {
// These are only good until the call to LayoutScrollArea.
nsRect scrolledRect = mHelper.GetScrolledRect();
// There are two cases to consider
- if (scrolledRect.height <= mHelper.mScrollPort.height
- || styles.mVertical != NS_STYLE_OVERFLOW_AUTO) {
- if (mHelper.mHasVerticalScrollbar) {
- // We left room for the vertical scrollbar, but it's not needed;
- // remove it.
- RemoveVerticalScrollbar(aState, scrollbarRight);
+ if (scrolledRect.height <= mHelper.mScrollPort.height ||
+ styles.mVertical != NS_STYLE_OVERFLOW_AUTO) {
+ if (mHelper.mHasVerticalScrollbar) {
+ // We left room for the vertical scrollbar, but it's not needed;
+ // remove it.
+ RemoveVerticalScrollbar(aState, scrollbarRight);
+ needsLayout = true;
+ }
+ } else {
+ if (!mHelper.mHasVerticalScrollbar) {
+ // We didn't leave room for the vertical scrollbar, but it turns
+ // out we needed it
+ if (AddVerticalScrollbar(aState, scrollbarRight)) {
needsLayout = true;
}
- } else {
- if (!mHelper.mHasVerticalScrollbar) {
- // We didn't leave room for the vertical scrollbar, but it turns
- // out we needed it
- if (AddVerticalScrollbar(aState, scrollbarRight))
- needsLayout = true;
- }
+ }
}
// ok layout at the right size
if (needsLayout) {
- nsBoxLayoutState resizeState(aState);
- LayoutScrollArea(resizeState, oldScrollPosition);
- needsLayout = false;
+ nsBoxLayoutState resizeState(aState);
+ LayoutScrollArea(resizeState, oldScrollPosition);
+ needsLayout = false;
}
}
// if scrollbars are auto look at the horizontal case
if (styles.mHorizontal != NS_STYLE_OVERFLOW_SCROLL)
{
// These are only good until the call to LayoutScrollArea.
nsRect scrolledRect = mHelper.GetScrolledRect();
// if the child is wider that the scroll area
// and we don't have a scrollbar add one.
if ((scrolledRect.width > mHelper.mScrollPort.width)
&& styles.mHorizontal == NS_STYLE_OVERFLOW_AUTO) {
if (!mHelper.mHasHorizontalScrollbar) {
- // no scrollbar?
- if (AddHorizontalScrollbar(aState, scrollbarBottom))
- needsLayout = true;
-
- // if we added a horizontal scrollbar and we did not have a vertical
- // there is a chance that by adding the horizontal scrollbar we will
- // suddenly need a vertical scrollbar. Is a special case but its
- // important.
- //if (!mHasVerticalScrollbar && scrolledRect.height > scrollAreaRect.height - sbSize.height)
- // printf("****Gfx Scrollbar Special case hit!!*****\n");
+ // no scrollbar?
+ if (AddHorizontalScrollbar(aState, scrollbarBottom)) {
+ needsLayout = true;
+ }
+
+ // if we added a horizontal scrollbar and we did not have a vertical
+ // there is a chance that by adding the horizontal scrollbar we will
+ // suddenly need a vertical scrollbar. Is a special case but its
+ // important.
+ //if (!mHasVerticalScrollbar && scrolledRect.height > scrollAreaRect.height - sbSize.height)
+ // printf("****Gfx Scrollbar Special case hit!!*****\n");
}
} else {
// if the area is smaller or equal to and we have a scrollbar then
// remove it.
if (mHelper.mHasHorizontalScrollbar) {
RemoveHorizontalScrollbar(aState, scrollbarBottom);
needsLayout = true;
}
}
}
// we only need to set the rect. The inner child stays the same size.
if (needsLayout) {
- nsBoxLayoutState resizeState(aState);
- LayoutScrollArea(resizeState, oldScrollPosition);
- needsLayout = false;
+ nsBoxLayoutState resizeState(aState);
+ LayoutScrollArea(resizeState, oldScrollPosition);
+ needsLayout = false;
}
// get the preferred size of the scrollbars
nsSize hMinSize(0, 0);
if (mHelper.mHScrollbarBox && mHelper.mHasHorizontalScrollbar) {
GetScrollbarMetrics(aState, mHelper.mHScrollbarBox, &hMinSize, nullptr, false);
}
nsSize vMinSize(0, 0);