Bug 1298209 - Use mFrame instead of Frame() because it's less confusing. r?mattwoodrow draft
authorMarkus Stange <mstange@themasta.com>
Mon, 22 Aug 2016 13:32:57 -0400
changeset 405729 b6687474459221af6e1988a1e472e31397ae3694
parent 405728 95b4de9b034372b5f2aff463ba5defb1805e529e
child 405730 ea73ec564583bb5e13052626f046383c95204525
push id27567
push usermstange@themasta.com
push dateThu, 25 Aug 2016 23:19:20 +0000
reviewersmattwoodrow
bugs1298209
milestone51.0a1
Bug 1298209 - Use mFrame instead of Frame() because it's less confusing. r?mattwoodrow MozReview-Commit-ID: 3UotUmhpEXR
layout/base/nsDisplayList.cpp
--- a/layout/base/nsDisplayList.cpp
+++ b/layout/base/nsDisplayList.cpp
@@ -5083,18 +5083,18 @@ already_AddRefed<Layer>
 nsDisplayFixedPosition::BuildLayer(nsDisplayListBuilder* aBuilder,
                                    LayerManager* aManager,
                                    const ContainerLayerParameters& aContainerParameters) {
   RefPtr<Layer> layer =
     nsDisplayOwnLayer::BuildLayer(aBuilder, aManager, aContainerParameters);
 
   layer->SetIsFixedPosition(true);
 
-  nsPresContext* presContext = Frame()->PresContext();
-  nsIFrame* fixedFrame = mIsFixedBackground ? presContext->PresShell()->GetRootFrame() : Frame();
+  nsPresContext* presContext = mFrame->PresContext();
+  nsIFrame* fixedFrame = mIsFixedBackground ? presContext->PresShell()->GetRootFrame() : mFrame;
 
   const nsIFrame* viewportFrame = fixedFrame->GetParent();
   // anchorRect will be in the container's coordinate system (aLayer's parent layer).
   // This is the same as the display items' reference frame.
   nsRect anchorRect;
   if (viewportFrame) {
     // Fixed position frames are reflowed into the scroll-port size if one has
     // been set.