Recompute overflow after transform changes

Source/core/frame/FrameView.cpp

 /*
  * Copyright (C) 1998, 1999 Torben Weis <weis@kde.org>
  *                     1999 Lars Knoll <knoll@kde.org>
  *                     1999 Antti Koivisto <koivisto@kde.org>
  *                     2000 Dirk Mueller <mueller@kde.org>
  * Copyright (C) 2004, 2005, 2006, 2007, 2008 Apple Inc. All rights reserved.
  *           (C) 2006 Graham Dennis (graham.dennis@gmail.com)
  *           (C) 2006 Alexey Proskuryakov (ap@nypop.com)
  * Copyright (C) 2009 Google Inc. All rights reserved.
  *
@@ skipping 595 matching lines @@
         }
     }
 }
 
 void FrameView::updateAcceleratedCompositingSettings()
 {
     if (RenderView* renderView = this->renderView())
         renderView->compositor()->updateAcceleratedCompositingSettings();
 }
 
+void FrameView::recalcOverflowAfterStyleChange()
+{
+    RenderView* renderView = this->renderView();
+    if (!renderView || !renderView->needsOverflowRecalcAfterStyleChange())

[ojan, 2014/04/19 02:19:30]

Can renderView really ever be null here? We only ever call this as
view()->recalcOverflowAfterStyleChange(). So we should be able to
ASSERT(renderView) here.

[trchen, 2014/04/19 05:08:21]

Sounds good. Will do it.

+        return;
+
+    renderView->recalcOverflowAfterStyleChange();
+
+    // FIXME: We should adjust frame scrollbar here, but that will make many
+    // tests flake in debug build.
+}
+
 void FrameView::updateCompositingLayersAfterStyleChange()
 {
     TRACE_EVENT0("webkit", "FrameView::updateCompositingLayersAfterStyleChange");
     RenderView* renderView = this->renderView();
     if (!renderView)
         return;
 
     // If we expect to update compositing after an incipient layout, don't do so here.
     if (m_doingPreLayoutStyleUpdate || layoutPending() || renderView->needsLayout())
         return;
@@ skipping 2601 matching lines @@
 void FrameView::willRemoveScrollbar(Scrollbar* scrollbar, ScrollbarOrientation orientation)
 {
     ScrollableArea::willRemoveScrollbar(scrollbar, orientation);
     if (AXObjectCache* cache = axObjectCache()) {
         cache->remove(scrollbar);
         cache->handleScrollbarUpdate(this);
     }
 }
 
 } // namespace WebCore