Bug 1326409 Part 1 - Annotate EllipseShapeInfo as 'final'. r=dholbert
authorTing-Yu Lin <tlin@mozilla.com>
Mon, 13 Mar 2017 12:03:39 +0800
changeset 497498 c41191ce3b13f0f179036234fdfe69c0f906a5be
parent 497274 f923de5a11109e677c993d7d95d1a9aba1a89e9a
child 497499 23c0f2c6ee17e78ab6eec6de29a365ff40442898
push id48933
push userbmo:james@hoppipolla.co.uk
push dateMon, 13 Mar 2017 13:53:04 +0000
reviewersdholbert
bugs1326409
milestone55.0a1
Bug 1326409 Part 1 - Annotate EllipseShapeInfo as 'final'. r=dholbert MozReview-Commit-ID: IReCWO8JRSQ
layout/generic/nsFloatManager.h
--- a/layout/generic/nsFloatManager.h
+++ b/layout/generic/nsFloatManager.h
@@ -457,17 +457,17 @@ private:
     nsRect mRect;
     // The half corner radii of the reference box. It's an nscoord[8] array
     // in the float manager's coordinate space. If there are no radii, it's
     // nullptr.
     mozilla::UniquePtr<nscoord[]> mRadii;
   };
 
   // Implements shape-outside: circle() and shape-outside: ellipse().
-  class EllipseShapeInfo : public ShapeInfo
+  class EllipseShapeInfo final : public ShapeInfo
   {
   public:
     EllipseShapeInfo(const nsPoint& aCenter,
                      const nsSize& aRadii)
       : mCenter(aCenter)
       , mRadii(aRadii)
     {}
 
@@ -481,17 +481,17 @@ private:
     nscoord BEnd() const override { return mCenter.y + mRadii.height; }
     bool IsEmpty() const override { return mRadii.IsEmpty(); };
 
     void Translate(nscoord aLineLeft, nscoord aBlockStart) override
     {
       mCenter.MoveBy(aLineLeft, aBlockStart);
     }
 
-  protected:
+  private:
     // The position of the center of the ellipse. The coordinate space is the
     // same as FloatInfo::mRect.
     nsPoint mCenter;
     // The radii of the ellipse in app units. The width and height represent
     // the line-axis and block-axis radii of the ellipse.
     nsSize mRadii;
   };