Avoid calling updateAllLifecyclePhasesExceptPaint in SVGImage if not needed.

third_party/WebKit/Source/platform/Widget.cpp

 /*
  * Copyright (C) 2004, 2005, 2006, 2010 Apple Inc. All rights reserved.
  * Copyright (C) 2013 Google Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions
  * are met:
  * 1. Redistributions of source code must retain the above copyright
  *    notice, this list of conditions and the following disclaimer.
  * 2. Redistributions in binary form must reproduce the above copyright
  *    notice, this list of conditions and the following disclaimer in the
  *    documentation and/or other materials provided with the distribution.
  *
  * THIS SOFTWARE IS PROVIDED BY APPLE COMPUTER, INC. ``AS IS'' AND ANY
  * EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO, THE
  * IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR
  * PURPOSE ARE DISCLAIMED.  IN NO EVENT SHALL APPLE COMPUTER, INC. OR
  * CONTRIBUTORS BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL,
  * EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT LIMITED TO,
  * PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF USE, DATA, OR
  * PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY THEORY
  * OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT
  * (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE
  * OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
  */
 
 #include "platform/Widget.h"
 
 #include "wtf/Assertions.h"
+#include "wtf/text/WTFString.h"

[fs, 2016/05/23 21:25:56]

Hmm?

 
 namespace blink {
 
 Widget::Widget()
     : m_parent(nullptr)
     , m_selfVisible(false)
     , m_parentVisible(false)
 {
 }
 
@@ skipping 19 matching lines @@
 Widget* Widget::root() const
 {
     const Widget* top = this;
     while (top->parent())
         top = top->parent();
     if (top->isFrameView())
         return const_cast<Widget*>(static_cast<const Widget*>(top));
     return 0;
 }
 
+bool Widget::resize(int w, int h)
+{
+    return resize(IntSize(w, h));
+}
+
+bool Widget::resize(const IntSize& s)
+{
+    IntRect newFrameRect(IntRect(location(), s));

[fs, 2016/05/23 21:25:56]

newFrameRect(location(), s);

Could also just compare the size first since the location is trivially
invariant.

+    if (newFrameRect == frameRect())
+        return false;
+    setFrameRect(newFrameRect);
+    return true;
+}
+
 IntRect Widget::convertFromRootFrame(const IntRect& rectInRootFrame) const
 {
     if (const Widget* parentWidget = parent()) {
         IntRect parentRect = parentWidget->convertFromRootFrame(rectInRootFrame);
         return convertFromContainingWidget(parentRect);
     }
     return rectInRootFrame;
 }
 
 IntRect Widget::convertToRootFrame(const IntRect& localRect) const
@@ skipping 83 matching lines @@
 {
     return point;
 }
 
 IntPoint Widget::convertSelfToChild(const Widget*, const IntPoint& point) const
 {
     return point;
 }
 
 } // namespace blink