From 3a1bebfb87d994b810574fdfeba09683e80cd2c9 Mon Sep 17 00:00:00 2001
From: win7-7 <win7-7@users.noreply.github.com>
Date: Sun, 10 May 2020 15:00:08 +0300
Subject: Issue #1355 - Fix the nit, remove nsPoint offset in nsFrame.cpp and 
 change PresShellState member order

---
 layout/base/nsDisplayList.cpp | 3 +--
 layout/base/nsDisplayList.h   | 2 +-
 layout/generic/nsFrame.cpp    | 3 +--
 3 files changed, 3 insertions(+), 5 deletions(-)

diff --git a/layout/base/nsDisplayList.cpp b/layout/base/nsDisplayList.cpp
index 55461f7c0..73e991246 100644
--- a/layout/base/nsDisplayList.cpp
+++ b/layout/base/nsDisplayList.cpp
@@ -882,8 +882,7 @@ void nsDisplayListBuilder::MarkOutOfFlowFrameForDisplay(nsIFrame* aDirtyFrame,
     }
   }
 
-  nsPoint offset = aFrame->GetOffsetTo(aDirtyFrame);
-  nsRect dirty = dirtyRectRelativeToDirtyFrame - offset;
+  nsRect dirty = dirtyRectRelativeToDirtyFrame - aFrame->GetOffsetTo(aDirtyFrame);
   nsRect overflowRect = aFrame->GetVisualOverflowRect();
 
   if (aFrame->IsTransformed() &&
diff --git a/layout/base/nsDisplayList.h b/layout/base/nsDisplayList.h
index ff2a14dd7..9eac132fa 100644
--- a/layout/base/nsDisplayList.h
+++ b/layout/base/nsDisplayList.h
@@ -1195,11 +1195,11 @@ private:
     nsRect        mCaretRect;
     uint32_t      mFirstFrameMarkedForDisplay;
     bool          mIsBackgroundOnly;
-    nsIFrame*     mPresShellIgnoreScrollFrame;
     // This is a per-document flag turning off event handling for all content
     // in the document, and is set when we enter a subdocument for a pointer-
     // events:none frame.
     bool          mInsidePointerEventsNoneDoc;
+    nsIFrame*     mPresShellIgnoreScrollFrame;
   };
 
   PresShellState* CurrentPresShellState() {
diff --git a/layout/generic/nsFrame.cpp b/layout/generic/nsFrame.cpp
index 95d79bcc4..95420d0d5 100644
--- a/layout/generic/nsFrame.cpp
+++ b/layout/generic/nsFrame.cpp
@@ -2709,8 +2709,7 @@ nsIFrame::BuildDisplayListForChild(nsDisplayListBuilder*   aBuilder,
 
   // dirty rect in child-relative coordinates
   NS_ASSERTION(aBuilder->GetCurrentFrame() == this, "Wrong coord space!");
-  nsPoint offset = child->GetOffsetTo(this);
-  nsRect dirty = aBuilder->GetDirtyRect() - offset;
+  nsRect dirty = aBuilder->GetDirtyRect() - child->GetOffsetTo(this);
 
   if (childType == nsGkAtoms::placeholderFrame) {
     isPlaceholder = true;
-- 
cgit v1.2.3