Make sure to flush reflows before trying to scroll so that frame coordinates

will be up-to-date.  Bug 186149, r=peterv, sr=jst, a=asa

content/html/content/src/nsGenericHTMLElement.cpp

@@ -1283,6 +1283,9 @@ nsGenericHTMLElement::ScrollIntoView(PRBool aTop)
     return NS_OK;
   }
 
+  // Now flush to make sure things are up to date
+  presShell->FlushPendingNotifications(PR_FALSE);
+  
   // Get the primary frame for this element
   nsIFrame *frame = nsnull;
   presShell->GetPrimaryFrameFor(this, &frame);

layout/base/nsPresShell.cpp

@@ -4118,6 +4118,9 @@ PresShell::GoToAnchor(const nsAString& aAnchorName, PRBool aScroll)
   }
 
   if (content) {
+    // Flush notifications so we scroll to the right place
+    FlushPendingNotifications(PR_FALSE);
+    
     // Get the primary frame
     nsIFrame* frame = nsnull;
     if (aScroll &&

layout/html/base/src/nsPresShell.cpp

@@ -4118,6 +4118,9 @@ PresShell::GoToAnchor(const nsAString& aAnchorName, PRBool aScroll)
   }
 
   if (content) {
+    // Flush notifications so we scroll to the right place
+    FlushPendingNotifications(PR_FALSE);
+    
     // Get the primary frame
     nsIFrame* frame = nsnull;
     if (aScroll &&