From 46c2b20d442b5ca1f4bcf671b4a0eb1c5e3dc73f Mon Sep 17 00:00:00 2001 From: Robert O'Callahan Date: Mon, 23 Jun 2014 16:24:51 +1200 Subject: [PATCH] Bug 1022612. Part 37: Remove aAllowVisibleRegionExpansion. r=mattwoodrow This is no longer needed thanks to the readback simplification. --HG-- extra : rebase_source : 6dad4c34681c478f9d4bf4829723ff9542b9d083 --- layout/base/nsDisplayList.cpp | 85 ++++++++++---------------------- layout/base/nsDisplayList.h | 39 +++++---------- layout/generic/nsCanvasFrame.h | 3 +- layout/generic/nsObjectFrame.cpp | 6 +-- layout/generic/nsObjectFrame.h | 3 +- 5 files changed, 44 insertions(+), 92 deletions(-) diff --git a/layout/base/nsDisplayList.cpp b/layout/base/nsDisplayList.cpp index 06a33f89aa4..acd85e4b5b2 100644 --- a/layout/base/nsDisplayList.cpp +++ b/layout/base/nsDisplayList.cpp @@ -1046,8 +1046,7 @@ nsDisplayList::ComputeVisibilityForRoot(nsDisplayListBuilder* aBuilder, nsRegion r; r.And(*aVisibleRegion, GetBounds(aBuilder)); return ComputeVisibilityForSublist(aBuilder, aVisibleRegion, - r.GetBounds(), r.GetBounds(), - aDisplayPortFrame); + r.GetBounds(), aDisplayPortFrame); } static nsRegion @@ -1086,7 +1085,6 @@ bool nsDisplayList::ComputeVisibilityForSublist(nsDisplayListBuilder* aBuilder, nsRegion* aVisibleRegion, const nsRect& aListVisibleBounds, - const nsRect& aAllowVisibleRegionExpansion, nsIFrame* aDisplayPortFrame) { #ifdef DEBUG nsRegion r; @@ -1108,8 +1106,7 @@ nsDisplayList::ComputeVisibilityForSublist(nsDisplayListBuilder* aBuilder, itemVisible.And(*aVisibleRegion, bounds); item->mVisibleRect = itemVisible.GetBounds(); - if (item->ComputeVisibility(aBuilder, aVisibleRegion, - aAllowVisibleRegionExpansion.Intersect(bounds))) { + if (item->ComputeVisibility(aBuilder, aVisibleRegion)) { anyVisible = true; nsRegion opaque = TreatAsOpaque(item, aBuilder); @@ -1611,8 +1608,7 @@ nsDisplayItem::ZIndex() const bool nsDisplayItem::ComputeVisibility(nsDisplayListBuilder* aBuilder, - nsRegion* aVisibleRegion, - const nsRect& aAllowVisibleRegionExpansion) + nsRegion* aVisibleRegion) { return !mVisibleRect.IsEmpty() && !IsInvisibleInRect(aVisibleRegion->GetBounds()); @@ -1630,7 +1626,7 @@ nsDisplayItem::RecomputeVisibility(nsDisplayListBuilder* aBuilder, // When we recompute visibility within layers we don't need to // expand the visible region for content behind plugins (the plugin // is not in the layer). - if (!ComputeVisibility(aBuilder, aVisibleRegion, nsRect())) { + if (!ComputeVisibility(aBuilder, aVisibleRegion)) { mVisibleRect = nsRect(); return false; } @@ -2043,11 +2039,9 @@ nsDisplayBackgroundImage::HitTest(nsDisplayListBuilder* aBuilder, bool nsDisplayBackgroundImage::ComputeVisibility(nsDisplayListBuilder* aBuilder, - nsRegion* aVisibleRegion, - const nsRect& aAllowVisibleRegionExpansion) + nsRegion* aVisibleRegion) { - if (!nsDisplayItem::ComputeVisibility(aBuilder, aVisibleRegion, - aAllowVisibleRegionExpansion)) { + if (!nsDisplayItem::ComputeVisibility(aBuilder, aVisibleRegion)) { return false; } @@ -2792,10 +2786,8 @@ nsDisplayBoxShadowOuter::IsInvisibleInRect(const nsRect& aRect) bool nsDisplayBoxShadowOuter::ComputeVisibility(nsDisplayListBuilder* aBuilder, - nsRegion* aVisibleRegion, - const nsRect& aAllowVisibleRegionExpansion) { - if (!nsDisplayItem::ComputeVisibility(aBuilder, aVisibleRegion, - aAllowVisibleRegionExpansion)) { + nsRegion* aVisibleRegion) { + if (!nsDisplayItem::ComputeVisibility(aBuilder, aVisibleRegion)) { return false; } @@ -2854,10 +2846,8 @@ nsDisplayBoxShadowInner::Paint(nsDisplayListBuilder* aBuilder, bool nsDisplayBoxShadowInner::ComputeVisibility(nsDisplayListBuilder* aBuilder, - nsRegion* aVisibleRegion, - const nsRect& aAllowVisibleRegionExpansion) { - if (!nsDisplayItem::ComputeVisibility(aBuilder, aVisibleRegion, - aAllowVisibleRegionExpansion)) { + nsRegion* aVisibleRegion) { + if (!nsDisplayItem::ComputeVisibility(aBuilder, aVisibleRegion)) { return false; } @@ -2960,8 +2950,7 @@ nsDisplayWrapList::GetBounds(nsDisplayListBuilder* aBuilder, bool* aSnap) { bool nsDisplayWrapList::ComputeVisibility(nsDisplayListBuilder* aBuilder, - nsRegion* aVisibleRegion, - const nsRect& aAllowVisibleRegionExpansion) { + nsRegion* aVisibleRegion) { // Convert the passed in visible region to our appunits. nsRegion visibleRegion; // mVisibleRect has been clipped to GetClippedBounds @@ -2970,8 +2959,7 @@ nsDisplayWrapList::ComputeVisibility(nsDisplayListBuilder* aBuilder, bool retval = mList.ComputeVisibilityForSublist(aBuilder, &visibleRegion, - mVisibleRect, - aAllowVisibleRegionExpansion); + mVisibleRect); nsRegion removed; // removed = originalVisibleRegion - visibleRegion @@ -3244,8 +3232,7 @@ nsDisplayOpacity::GetLayerState(nsDisplayListBuilder* aBuilder, bool nsDisplayOpacity::ComputeVisibility(nsDisplayListBuilder* aBuilder, - nsRegion* aVisibleRegion, - const nsRect& aAllowVisibleRegionExpansion) { + nsRegion* aVisibleRegion) { // Our children are translucent so we should not allow them to subtract // area from aVisibleRegion. We do need to find out what is visible under // our children in the temporary compositing buffer, because if our children @@ -3254,10 +3241,8 @@ nsDisplayOpacity::ComputeVisibility(nsDisplayListBuilder* aBuilder, nsRect bounds = GetClippedBounds(aBuilder); nsRegion visibleUnderChildren; visibleUnderChildren.And(*aVisibleRegion, bounds); - nsRect allowExpansion = bounds.Intersect(aAllowVisibleRegionExpansion); return - nsDisplayWrapList::ComputeVisibility(aBuilder, &visibleUnderChildren, - allowExpansion); + nsDisplayWrapList::ComputeVisibility(aBuilder, &visibleUnderChildren); } bool nsDisplayOpacity::TryMerge(nsDisplayListBuilder* aBuilder, nsDisplayItem* aItem) { @@ -3335,8 +3320,7 @@ nsDisplayMixBlendMode::BuildLayer(nsDisplayListBuilder* aBuilder, } bool nsDisplayMixBlendMode::ComputeVisibility(nsDisplayListBuilder* aBuilder, - nsRegion* aVisibleRegion, - const nsRect& aAllowVisibleRegionExpansion) { + nsRegion* aVisibleRegion) { // Our children are need their backdrop so we should not allow them to subtract // area from aVisibleRegion. We do need to find out what is visible under // our children in the temporary compositing buffer, because if our children @@ -3345,10 +3329,7 @@ bool nsDisplayMixBlendMode::ComputeVisibility(nsDisplayListBuilder* aBuilder, nsRect bounds = GetClippedBounds(aBuilder); nsRegion visibleUnderChildren; visibleUnderChildren.And(*aVisibleRegion, bounds); - nsRect allowExpansion = bounds.Intersect(aAllowVisibleRegionExpansion); - return - nsDisplayWrapList::ComputeVisibility(aBuilder, &visibleUnderChildren, - allowExpansion); + return nsDisplayWrapList::ComputeVisibility(aBuilder, &visibleUnderChildren); } bool nsDisplayMixBlendMode::TryMerge(nsDisplayListBuilder* aBuilder, nsDisplayItem* aItem) { @@ -3521,16 +3502,14 @@ nsDisplaySubDocument::GetBounds(nsDisplayListBuilder* aBuilder, bool* aSnap) bool nsDisplaySubDocument::ComputeVisibility(nsDisplayListBuilder* aBuilder, - nsRegion* aVisibleRegion, - const nsRect& aAllowVisibleRegionExpansion) + nsRegion* aVisibleRegion) { nsRect displayport; bool usingDisplayPort = nsLayoutUtils::ViewportHasDisplayPort(mFrame->PresContext(), &displayport); if (!(mFlags & GENERATE_SCROLLABLE_LAYER) || !usingDisplayPort) { - return nsDisplayWrapList::ComputeVisibility(aBuilder, aVisibleRegion, - aAllowVisibleRegionExpansion); + return nsDisplayWrapList::ComputeVisibility(aBuilder, aVisibleRegion); } nsRegion childVisibleRegion; @@ -3541,9 +3520,8 @@ nsDisplaySubDocument::ComputeVisibility(nsDisplayListBuilder* aBuilder, nsRect boundedRect = childVisibleRegion.GetBounds().Intersect(mList.GetBounds(aBuilder)); - nsRect allowExpansion = boundedRect.Intersect(aAllowVisibleRegionExpansion); bool visible = mList.ComputeVisibilityForSublist( - aBuilder, &childVisibleRegion, boundedRect, allowExpansion, + aBuilder, &childVisibleRegion, boundedRect, usingDisplayPort ? mFrame : nullptr); // We don't allow this computation to influence aVisibleRegion, on the // assumption that the layer can be asynchronously scrolled so we'll @@ -3830,8 +3808,7 @@ nsDisplayScrollLayer::ShouldBuildLayerEvenIfInvisible(nsDisplayListBuilder* aBui bool nsDisplayScrollLayer::ComputeVisibility(nsDisplayListBuilder* aBuilder, - nsRegion* aVisibleRegion, - const nsRect& aAllowVisibleRegionExpansion) + nsRegion* aVisibleRegion) { nsRect displayport; bool usingDisplayPort = @@ -3840,10 +3817,9 @@ nsDisplayScrollLayer::ComputeVisibility(nsDisplayListBuilder* aBuilder, usingDisplayPort ? &displayport : nullptr); nsRect boundedRect = scrolledContentRect.Intersect(mList.GetBounds(aBuilder)); - nsRect allowExpansion = boundedRect.Intersect(aAllowVisibleRegionExpansion); nsRegion childVisibleRegion = scrolledContentRect; bool visible = mList.ComputeVisibilityForSublist( - aBuilder, &childVisibleRegion, boundedRect, allowExpansion, + aBuilder, &childVisibleRegion, boundedRect, usingDisplayPort ? mScrollFrame : nullptr); // We don't allow this computation to influence aVisibleRegion, on the // assumption that the layer can be asynchronously scrolled so we'll @@ -4101,8 +4077,7 @@ void nsDisplayZoom::Paint(nsDisplayListBuilder* aBuilder, } bool nsDisplayZoom::ComputeVisibility(nsDisplayListBuilder *aBuilder, - nsRegion *aVisibleRegion, - const nsRect& aAllowVisibleRegionExpansion) + nsRegion *aVisibleRegion) { // Convert the passed in visible region to our appunits. nsRegion visibleRegion; @@ -4113,8 +4088,6 @@ bool nsDisplayZoom::ComputeVisibility(nsDisplayListBuilder *aBuilder, nsRect transformedVisibleRect = mVisibleRect.ConvertAppUnitsRoundOut(mParentAPD, mAPD); - nsRect allowExpansion = - aAllowVisibleRegionExpansion.ConvertAppUnitsRoundIn(mParentAPD, mAPD); bool retval; // If we are to generate a scrollable layer we call // nsDisplaySubDocument::ComputeVisibility to make the necessary adjustments @@ -4124,12 +4097,10 @@ bool nsDisplayZoom::ComputeVisibility(nsDisplayListBuilder *aBuilder, if (!(mFlags & GENERATE_SCROLLABLE_LAYER) || !usingDisplayPort) { retval = mList.ComputeVisibilityForSublist(aBuilder, &visibleRegion, - transformedVisibleRect, - allowExpansion); + transformedVisibleRect); } else { retval = - nsDisplaySubDocument::ComputeVisibility(aBuilder, &visibleRegion, - allowExpansion); + nsDisplaySubDocument::ComputeVisibility(aBuilder, &visibleRegion); } nsRegion removed; @@ -4764,8 +4735,7 @@ nsDisplayTransform::GetLayerState(nsDisplayListBuilder* aBuilder, } bool nsDisplayTransform::ComputeVisibility(nsDisplayListBuilder *aBuilder, - nsRegion *aVisibleRegion, - const nsRect& aAllowVisibleRegionExpansion) + nsRegion *aVisibleRegion) { /* As we do this, we need to be sure to * untransform the visible rect, since we want everything that's painting to @@ -5231,8 +5201,7 @@ nsDisplaySVGEffects::BuildLayer(nsDisplayListBuilder* aBuilder, } bool nsDisplaySVGEffects::ComputeVisibility(nsDisplayListBuilder* aBuilder, - nsRegion* aVisibleRegion, - const nsRect& aAllowVisibleRegionExpansion) { + nsRegion* aVisibleRegion) { nsPoint offset = ToReferenceFrame(); nsRect dirtyRect = nsSVGIntegrationUtils::GetRequiredSourceForInvalidArea(mFrame, @@ -5243,7 +5212,7 @@ bool nsDisplaySVGEffects::ComputeVisibility(nsDisplayListBuilder* aBuilder, // not allow them to subtract area from aVisibleRegion. nsRegion childrenVisible(dirtyRect); nsRect r = dirtyRect.Intersect(mList.GetBounds(aBuilder)); - mList.ComputeVisibilityForSublist(aBuilder, &childrenVisible, r, nsRect()); + mList.ComputeVisibilityForSublist(aBuilder, &childrenVisible, r); return true; } diff --git a/layout/base/nsDisplayList.h b/layout/base/nsDisplayList.h index 043019cd74c..4066e6a9f5c 100644 --- a/layout/base/nsDisplayList.h +++ b/layout/base/nsDisplayList.h @@ -1184,8 +1184,7 @@ public: * is visible. */ virtual bool ComputeVisibility(nsDisplayListBuilder* aBuilder, - nsRegion* aVisibleRegion, - const nsRect& aAllowVisibleRegionExpansion); + nsRegion* aVisibleRegion); /** * Try to merge with the other item (which is below us in the display @@ -1266,7 +1265,7 @@ public: * -- Subtracts bounds from aVisibleRegion if the item is opaque */ bool RecomputeVisibility(nsDisplayListBuilder* aBuilder, - nsRegion* aVisibleRegion); + nsRegion* aVisibleRegion); /** * Returns the result of aBuilder->ToReferenceFrame(GetUnderlyingFrame()) @@ -1550,7 +1549,6 @@ public: bool ComputeVisibilityForSublist(nsDisplayListBuilder* aBuilder, nsRegion* aVisibleRegion, const nsRect& aListVisibleBounds, - const nsRect& aAllowVisibleRegionExpansion, nsIFrame* aDisplayPortFrame = nullptr); /** @@ -2122,8 +2120,7 @@ public: virtual void HitTest(nsDisplayListBuilder* aBuilder, const nsRect& aRect, HitTestState* aState, nsTArray *aOutFrames) MOZ_OVERRIDE; virtual bool ComputeVisibility(nsDisplayListBuilder* aBuilder, - nsRegion* aVisibleRegion, - const nsRect& aAllowVisibleRegionExpansion) MOZ_OVERRIDE; + nsRegion* aVisibleRegion) MOZ_OVERRIDE; virtual nsRegion GetOpaqueRegion(nsDisplayListBuilder* aBuilder, bool* aSnap) MOZ_OVERRIDE; virtual bool IsUniform(nsDisplayListBuilder* aBuilder, nscolor* aColor) MOZ_OVERRIDE; @@ -2327,8 +2324,7 @@ public: virtual nsRect GetBounds(nsDisplayListBuilder* aBuilder, bool* aSnap) MOZ_OVERRIDE; virtual bool IsInvisibleInRect(const nsRect& aRect) MOZ_OVERRIDE; virtual bool ComputeVisibility(nsDisplayListBuilder* aBuilder, - nsRegion* aVisibleRegion, - const nsRect& aAllowVisibleRegionExpansion) MOZ_OVERRIDE; + nsRegion* aVisibleRegion) MOZ_OVERRIDE; NS_DISPLAY_DECL_NAME("BoxShadowOuter", TYPE_BOX_SHADOW_OUTER) virtual void ComputeInvalidationRegion(nsDisplayListBuilder* aBuilder, @@ -2371,8 +2367,7 @@ public: virtual void Paint(nsDisplayListBuilder* aBuilder, nsRenderingContext* aCtx) MOZ_OVERRIDE; virtual bool ComputeVisibility(nsDisplayListBuilder* aBuilder, - nsRegion* aVisibleRegion, - const nsRect& aAllowVisibleRegionExpansion) MOZ_OVERRIDE; + nsRegion* aVisibleRegion) MOZ_OVERRIDE; NS_DISPLAY_DECL_NAME("BoxShadowInner", TYPE_BOX_SHADOW_INNER) virtual nsDisplayItemGeometry* AllocateGeometry(nsDisplayListBuilder* aBuilder) MOZ_OVERRIDE @@ -2542,8 +2537,7 @@ public: virtual bool IsUniform(nsDisplayListBuilder* aBuilder, nscolor* aColor) MOZ_OVERRIDE; virtual void Paint(nsDisplayListBuilder* aBuilder, nsRenderingContext* aCtx) MOZ_OVERRIDE; virtual bool ComputeVisibility(nsDisplayListBuilder* aBuilder, - nsRegion* aVisibleRegion, - const nsRect& aAllowVisibleRegionExpansion) MOZ_OVERRIDE; + nsRegion* aVisibleRegion) MOZ_OVERRIDE; virtual bool TryMerge(nsDisplayListBuilder* aBuilder, nsDisplayItem* aItem) MOZ_OVERRIDE { return false; } @@ -2674,8 +2668,7 @@ public: LayerManager* aManager, const ContainerLayerParameters& aParameters) MOZ_OVERRIDE; virtual bool ComputeVisibility(nsDisplayListBuilder* aBuilder, - nsRegion* aVisibleRegion, - const nsRect& aAllowVisibleRegionExpansion) MOZ_OVERRIDE; + nsRegion* aVisibleRegion) MOZ_OVERRIDE; virtual bool TryMerge(nsDisplayListBuilder* aBuilder, nsDisplayItem* aItem) MOZ_OVERRIDE; virtual void ComputeInvalidationRegion(nsDisplayListBuilder* aBuilder, const nsDisplayItemGeometry* aGeometry, @@ -2723,8 +2716,7 @@ public: LayerManager* aManager, const ContainerLayerParameters& aParameters) MOZ_OVERRIDE; virtual bool ComputeVisibility(nsDisplayListBuilder* aBuilder, - nsRegion* aVisibleRegion, - const nsRect& aAllowVisibleRegionExpansion) MOZ_OVERRIDE; + nsRegion* aVisibleRegion) MOZ_OVERRIDE; virtual bool TryMerge(nsDisplayListBuilder* aBuilder, nsDisplayItem* aItem) MOZ_OVERRIDE; virtual bool ShouldFlattenAway(nsDisplayListBuilder* aBuilder) MOZ_OVERRIDE { return false; @@ -2848,8 +2840,7 @@ public: virtual nsRect GetBounds(nsDisplayListBuilder* aBuilder, bool* aSnap) MOZ_OVERRIDE; virtual bool ComputeVisibility(nsDisplayListBuilder* aBuilder, - nsRegion* aVisibleRegion, - const nsRect& aAllowVisibleRegionExpansion) MOZ_OVERRIDE; + nsRegion* aVisibleRegion) MOZ_OVERRIDE; virtual bool ShouldBuildLayerEvenIfInvisible(nsDisplayListBuilder* aBuilder) MOZ_OVERRIDE; @@ -2966,8 +2957,7 @@ public: } virtual bool ComputeVisibility(nsDisplayListBuilder* aBuilder, - nsRegion* aVisibleRegion, - const nsRect& aAllowVisibleRegionExpansion) MOZ_OVERRIDE; + nsRegion* aVisibleRegion) MOZ_OVERRIDE; virtual LayerState GetLayerState(nsDisplayListBuilder* aBuilder, LayerManager* aManager, @@ -3066,8 +3056,7 @@ public: virtual void HitTest(nsDisplayListBuilder* aBuilder, const nsRect& aRect, HitTestState* aState, nsTArray *aOutFrames) MOZ_OVERRIDE; virtual bool ComputeVisibility(nsDisplayListBuilder* aBuilder, - nsRegion* aVisibleRegion, - const nsRect& aAllowVisibleRegionExpansion) MOZ_OVERRIDE; + nsRegion* aVisibleRegion) MOZ_OVERRIDE; virtual LayerState GetLayerState(nsDisplayListBuilder* aBuilder, LayerManager* aManager, const ContainerLayerParameters& aParameters) MOZ_OVERRIDE @@ -3108,8 +3097,7 @@ public: return mEffectsBounds + ToReferenceFrame(); } virtual bool ComputeVisibility(nsDisplayListBuilder* aBuilder, - nsRegion* aVisibleRegion, - const nsRect& aAllowVisibleRegionExpansion) MOZ_OVERRIDE; + nsRegion* aVisibleRegion) MOZ_OVERRIDE; virtual bool TryMerge(nsDisplayListBuilder* aBuilder, nsDisplayItem* aItem) MOZ_OVERRIDE; virtual bool ShouldFlattenAway(nsDisplayListBuilder* aBuilder) MOZ_OVERRIDE { @@ -3213,8 +3201,7 @@ public: const ContainerLayerParameters& aContainerParameters) MOZ_OVERRIDE; virtual bool ShouldBuildLayerEvenIfInvisible(nsDisplayListBuilder* aBuilder) MOZ_OVERRIDE; virtual bool ComputeVisibility(nsDisplayListBuilder *aBuilder, - nsRegion *aVisibleRegion, - const nsRect& aAllowVisibleRegionExpansion) MOZ_OVERRIDE; + nsRegion *aVisibleRegion) MOZ_OVERRIDE; virtual bool TryMerge(nsDisplayListBuilder *aBuilder, nsDisplayItem *aItem) MOZ_OVERRIDE; virtual uint32_t GetPerFrameKey() MOZ_OVERRIDE { return (mIndex << nsDisplayItem::TYPE_BITS) | nsDisplayItem::GetPerFrameKey(); } diff --git a/layout/generic/nsCanvasFrame.h b/layout/generic/nsCanvasFrame.h index 66d73d9e5a5..b7f17d1231e 100644 --- a/layout/generic/nsCanvasFrame.h +++ b/layout/generic/nsCanvasFrame.h @@ -155,8 +155,7 @@ public: } virtual bool ComputeVisibility(nsDisplayListBuilder* aBuilder, - nsRegion* aVisibleRegion, - const nsRect& aAllowVisibleRegionExpansion) MOZ_OVERRIDE + nsRegion* aVisibleRegion) MOZ_OVERRIDE { return NS_GET_A(mColor) > 0; } diff --git a/layout/generic/nsObjectFrame.cpp b/layout/generic/nsObjectFrame.cpp index 95537d3bf24..c5ef72c2e7b 100644 --- a/layout/generic/nsObjectFrame.cpp +++ b/layout/generic/nsObjectFrame.cpp @@ -959,8 +959,7 @@ nsDisplayPlugin::Paint(nsDisplayListBuilder* aBuilder, bool nsDisplayPlugin::ComputeVisibility(nsDisplayListBuilder* aBuilder, - nsRegion* aVisibleRegion, - const nsRect& aAllowVisibleRegionExpansion) + nsRegion* aVisibleRegion) { if (aBuilder->IsForPluginGeometry()) { nsObjectFrame* f = static_cast(mFrame); @@ -999,8 +998,7 @@ nsDisplayPlugin::ComputeVisibility(nsDisplayListBuilder* aBuilder, } } - return nsDisplayItem::ComputeVisibility(aBuilder, aVisibleRegion, - aAllowVisibleRegionExpansion); + return nsDisplayItem::ComputeVisibility(aBuilder, aVisibleRegion); } nsRegion diff --git a/layout/generic/nsObjectFrame.h b/layout/generic/nsObjectFrame.h index 8e7b3d27dce..ce7d056745b 100644 --- a/layout/generic/nsObjectFrame.h +++ b/layout/generic/nsObjectFrame.h @@ -319,8 +319,7 @@ public: virtual void Paint(nsDisplayListBuilder* aBuilder, nsRenderingContext* aCtx) MOZ_OVERRIDE; virtual bool ComputeVisibility(nsDisplayListBuilder* aBuilder, - nsRegion* aVisibleRegion, - const nsRect& aAllowVisibleRegionExpansion) MOZ_OVERRIDE; + nsRegion* aVisibleRegion) MOZ_OVERRIDE; NS_DISPLAY_DECL_NAME("Plugin", TYPE_PLUGIN)