Reduce how often LayoutSVGShape::updateShapeFromElement is called

Source/core/layout/svg/LayoutSVGEllipse.h

 /*
  * Copyright (C) 2012 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
@@ skipping 23 matching lines @@
 class LayoutSVGEllipse final : public LayoutSVGShape {
 public:
     explicit LayoutSVGEllipse(SVGGeometryElement*);
     virtual ~LayoutSVGEllipse();
 
     virtual ShapeGeometryCodePath geometryCodePath() const override { return m_usePathFallback ? PathGeometry : EllipseGeometryFastPath; }
 
     virtual const char* name() const override { return "LayoutSVGEllipse"; }
 
 private:
-    virtual void updateShapeFromElement() override;
+    void updateShapeFromElement() override final;

[fs, 2015/06/16 10:54:02]

Nit: This class is final

[pdr., 2015/06/17 03:50:49]

Done

+    void updateStrokeAndFillBoundingBoxes() override final;
     virtual bool isShapeEmpty() const override { return m_usePathFallback ? LayoutSVGShape::isShapeEmpty() : m_fillBoundingBox.isEmpty(); }
     virtual bool shapeDependentStrokeContains(const FloatPoint&) override;
     virtual bool shapeDependentFillContains(const FloatPoint&, const WindRule) const override;
     void calculateRadiiAndCenter();
     bool hasContinuousStroke() const;
 
 private:
     FloatPoint m_center;
     FloatSize m_radii;
     bool m_usePathFallback;
 };
 
 }
 
 #endif