Hoist layout update out of SVGGraphicsElement::GetBBox and overrides

third_party/WebKit/Source/core/svg/SVGUseElement.cpp

 /*
  * Copyright (C) 2004, 2005, 2006, 2007, 2008 Nikolas Zimmermann
  * <zimmermann@kde.org>
  * Copyright (C) 2004, 2005, 2006, 2007 Rob Buis <buis@kde.org>
  * Copyright (C) Research In Motion Limited 2009-2010. All rights reserved.
  * Copyright (C) 2011 Torch Mobile (Beijing) Co. Ltd. All rights reserved.
  * Copyright (C) 2012 University of Szeged
  * Copyright (C) 2012 Renata Hodovan <reni@webkit.org>
  *
  * This library is free software; you can redistribute it and/or
@@ skipping 658 matching lines @@
       height_->CurrentValue()->IsRelative())
     return true;
 
   if (!target_element_instance_)
     return false;
 
   return target_element_instance_->HasRelativeLengths();
 }
 
 FloatRect SVGUseElement::GetBBox() {
-  GetDocument().UpdateStyleAndLayoutIgnorePendingStylesheets();
-
-  if (!GetLayoutObject())
-    return FloatRect();
-
+  DCHECK(GetLayoutObject());
   LayoutSVGTransformableContainer& transformable_container =
       ToLayoutSVGTransformableContainer(*GetLayoutObject());
   // Don't apply the additional translation if the oBB is invalid.
   if (!transformable_container.IsObjectBoundingBoxValid())
     return FloatRect();
 
   // TODO(fs): Preferably this would just use objectBoundingBox() (and hence
   // don't need to override SVGGraphicsElement::getBBox at all) and be
   // correct without additional work. That will not work out ATM without
   // additional quirks. The problem stems from including the additional
@@ skipping 56 matching lines @@
 
   if (resource_)
     resource_->RemoveClient(this);
 
   resource_ = resource;
   if (resource_)
     resource_->AddClient(this);
 }
 
 }  // namespace blink