From 653b2cba7f2cd7e4b61eb7d9cc148725dd159162 Mon Sep 17 00:00:00 2001
From: "bzbarsky%mit.edu"
 <bzbarsky%mit.edu@18797224-902f-48f8-a5cc-f745e15eee43>
Date: Fri, 9 Nov 2007 05:58:47 +0000
Subject: [PATCH] Handle GetPositionAndSize flushing reflow.  Bug 399410,
 r+sr=roc, a=beltzner

git-svn-id: svn://10.0.0.236/trunk@239016 18797224-902f-48f8-a5cc-f745e15eee43
---
 mozilla/layout/generic/nsFrameFrame.cpp | 31 ++++++++++++++++++-------
 1 file changed, 22 insertions(+), 9 deletions(-)

diff --git a/mozilla/layout/generic/nsFrameFrame.cpp b/mozilla/layout/generic/nsFrameFrame.cpp
index aec39931fc4..955dcbd8188 100644
--- a/mozilla/layout/generic/nsFrameFrame.cpp
+++ b/mozilla/layout/generic/nsFrameFrame.cpp
@@ -437,15 +437,6 @@ nsSubDocumentFrame::Reflow(nsPresContext*          aPresContext,
 PRBool
 nsSubDocumentFrame::ReflowFinished()
 {
-  mPostedReflowCallback = PR_FALSE;
-  
-  nsSize innerSize(GetSize());
-  if (IsInline()) {
-    nsMargin usedBorderPadding = GetUsedBorderAndPadding();
-    innerSize.width  -= usedBorderPadding.LeftRight();
-    innerSize.height -= usedBorderPadding.TopBottom();
-  }
-  
   nsCOMPtr<nsIDocShell> docShell;
   GetDocShell(getter_AddRefs(docShell));
 
@@ -456,11 +447,33 @@ nsSubDocumentFrame::ReflowFinished()
     PRInt32 x = 0;
     PRInt32 y = 0;
 
+    nsWeakFrame weakFrame(this);
+    
     nsPresContext* presContext = PresContext();
     baseWindow->GetPositionAndSize(&x, &y, nsnull, nsnull);
+
+    if (!weakFrame.IsAlive()) {
+      // GetPositionAndSize() killed us
+      return;
+    }
+
+    // GetPositionAndSize might have resized us.  So now is the time to
+    // get our size.
+    mPostedReflowCallback = PR_FALSE;
+  
+    nsSize innerSize(GetSize());
+    if (IsInline()) {
+      nsMargin usedBorderPadding = GetUsedBorderAndPadding();
+      innerSize.width  -= usedBorderPadding.LeftRight();
+      innerSize.height -= usedBorderPadding.TopBottom();
+    }  
+
     PRInt32 cx = presContext->AppUnitsToDevPixels(innerSize.width);
     PRInt32 cy = presContext->AppUnitsToDevPixels(innerSize.height);
     baseWindow->SetPositionAndSize(x, y, cx, cy, PR_FALSE);
+  } else {
+    // Make sure that we can post a reflow callback in the future.
+    mPostedReflowCallback = PR_FALSE;
   }
 
   return PR_FALSE;