From c48759a9d0bfe4aa54c5496dfb5c4a79efff5bc8 Mon Sep 17 00:00:00 2001 From: Robert O'Callahan Date: Tue, 21 Jul 2015 16:22:02 +1200 Subject: [PATCH] Bug 1184842. Remove frameset mVisibilityOverride since it's always false now. r=mats --- layout/generic/nsFrameSetFrame.cpp | 10 +++------- 1 file changed, 3 insertions(+), 7 deletions(-) diff --git a/layout/generic/nsFrameSetFrame.cpp b/layout/generic/nsFrameSetFrame.cpp index 64a371110a47..bd4c1839135d 100644 --- a/layout/generic/nsFrameSetFrame.cpp +++ b/layout/generic/nsFrameSetFrame.cpp @@ -108,7 +108,7 @@ public: const nsHTMLReflowState& aReflowState, nsReflowStatus& aStatus) override; - bool GetVisibility() { return mVisibility || mVisibilityOverride; } + bool GetVisibility() { return mVisibility; } void SetVisibility(bool aVisibility); void SetColor(nscolor aColor); @@ -128,7 +128,6 @@ protected: int32_t mWidth; bool mVertical; bool mVisibility; - bool mVisibilityOverride; bool mCanResize; friend class nsHTMLFramesetFrame; }; @@ -1176,7 +1175,6 @@ nsHTMLFramesetFrame::RecalculateBorderResize() for (verX = 0; verX < mNumCols-1; verX++) { if (mVerBorders[verX]) { mVerBorders[verX]->mCanResize = true; - mVerBorders[verX]->mVisibilityOverride = false; SetBorderResize(mVerBorders[verX]); } } @@ -1184,7 +1182,6 @@ nsHTMLFramesetFrame::RecalculateBorderResize() for (horX = 0; horX < mNumRows-1; horX++) { if (mHorBorders[horX]) { mHorBorders[horX]->mCanResize = true; - mHorBorders[horX]->mVisibilityOverride = false; SetBorderResize(mHorBorders[horX]); } } @@ -1346,7 +1343,6 @@ nsHTMLFramesetBorderFrame::nsHTMLFramesetBorderFrame(nsStyleContext* aContext, bool aVisibility) : nsLeafFrame(aContext), mWidth(aWidth), mVertical(aVertical), mVisibility(aVisibility) { - mVisibilityOverride = false; mCanResize = true; mColor = NO_COLOR; mPrevNeighbor = 0; @@ -1467,7 +1463,7 @@ void nsHTMLFramesetBorderFrame::PaintBorder(DrawTarget* aDrawTarget, NS_RGB(128, 128, 128)))); ColorPattern color(ToDeviceColor(NS_RGB(255, 255, 255))); // default to white - if (mVisibility || mVisibilityOverride) { + if (mVisibility) { color = (NO_COLOR == mColor) ? bgColor : ColorPattern(ToDeviceColor(mColor)); } @@ -1496,7 +1492,7 @@ void nsHTMLFramesetBorderFrame::PaintBorder(DrawTarget* aDrawTarget, } } - if (!mVisibility && !mVisibilityOverride) + if (!mVisibility) return; if (widthInPixels >= 5) {