Don't call LayoutPart::widgetPositionsUpdated unnecessarily.

third_party/WebKit/Source/core/layout/LayoutPart.cpp

 /*
  * Copyright (C) 1999 Lars Knoll (knoll@kde.org)
  *           (C) 2000 Simon Hausmann <hausmann@kde.org>
  *           (C) 2000 Stefan Schimanski (1Stein@gmx.de)
  * Copyright (C) 2004, 2005, 2006, 2009 Apple Inc. All rights reserved.
  * Copyright (C) Research In Motion Limited 2011. All rights reserved.
  *
  * This library is free software; you can redistribute it and/or
  * modify it under the terms of the GNU Library General Public
  * License as published by the Free Software Foundation; either
@@ skipping 249 matching lines @@
 void LayoutPart::updateOnWidgetChange()
 {
     Widget* widget = this->widget();
     if (!widget)
         return;
 
     if (!style())
         return;
 
     if (!needsLayout())
-        updateWidgetGeometry();
+        updateWidgetGeometryInternal();
 
     if (style()->visibility() != VISIBLE) {
         widget->hide();
     } else {
         widget->show();
         // FIXME: Why do we issue a full paint invalidation in this case, but not the other?
         setShouldDoFullPaintInvalidation();
     }
 }
 
-void LayoutPart::updateWidgetPosition()
+bool LayoutPart::updateWidgetGeometry()
 {
     Widget* widget = this->widget();
     if (!widget || !node()) // Check the node in case destroy() has been called.
-        return;
+        return false;
 
-    bool boundsChanged = updateWidgetGeometry();
+    bool boundsChanged = updateWidgetGeometryInternal();
 
     // If the frame bounds got changed, or if view needs layout (possibly indicating
     // content size is wrong) we have to do a layout to set the right widget size.
     if (widget && widget->isFrameView()) {
         FrameView* frameView = toFrameView(widget);
         // Check the frame's page to make sure that the frame isn't in the process of being destroyed.
         if ((boundsChanged || frameView->needsLayout()) && frameView->frame().page())
             frameView->layout();
     }
+    return boundsChanged;
 }
 
-void LayoutPart::widgetPositionsUpdated()
+void LayoutPart::widgetGeometriesUpdated()
 {
     Widget* widget = this->widget();
     if (!widget)
         return;
-    widget->widgetPositionsUpdated();
+    widget->widgetGeometriesUpdated();
 }
 
-bool LayoutPart::updateWidgetGeometry()
+bool LayoutPart::updateWidgetGeometryInternal()
 {
     Widget* widget = this->widget();
     ASSERT(widget);
 
     LayoutRect contentBox = contentBoxRect();
     LayoutRect absoluteContentBox(localToAbsoluteQuad(FloatQuad(FloatRect(contentBox))).boundingBox());
     if (widget->isFrameView()) {
         contentBox.setLocation(absoluteContentBox.location());
         return setWidgetGeometry(contentBox);
     }
@@ skipping 41 matching lines @@
 
 bool LayoutPart::isThrottledFrameView() const
 {
     if (!widget() || !widget()->isFrameView())
         return false;
     const FrameView* frameView = toFrameView(widget());
     return frameView->shouldThrottleRendering();
 }
 
 }