Avoid auto-resize re-entrancy problems.

Source/core/frame/FrameView.cpp

Index: Source/core/frame/FrameView.cpp
diff --git a/Source/core/frame/FrameView.cpp b/Source/core/frame/FrameView.cpp
index 811fceaa7362576c08af53d67eee0bce25d4a619..5e0d2f8f1fcb6e68aa96c092a0d99905899093bc 100644
--- a/Source/core/frame/FrameView.cpp
+++ b/Source/core/frame/FrameView.cpp
@@ -802,6 +802,8 @@ void FrameView::layout(bool allowSubtree)
     TRACE_EVENT0("blink", "FrameView::layout");
     TRACE_EVENT_SCOPED_SAMPLING_STATE("blink", "Layout");
 
+    autoSizeIfEnabled();

[levin, 2014/08/22 02:11:00]

Why not after protector and changeInProgrammaticScroll?

[rune, 2014/08/22 08:52:26]

Because I thought those would be called when FrameView::layout is re-entered
from autoSizeIfEnabled(). There should be no harm having those twice on the
stack, so I'll move it down for safety.

+
     // Protect the view from being deleted during layout (in recalcStyle)
     RefPtr<FrameView> protector(this);
 
@@ -858,7 +860,6 @@ void FrameView::layout(bool allowSubtree)
             }
         }
         updateCounters();
-        autoSizeIfEnabled();
 
         ScrollbarMode hMode;
         ScrollbarMode vMode;