Remove unneeded destructors and add finalization support to various classes.

Source/core/css/CSSBasicShapes.h

Index: Source/core/css/CSSBasicShapes.h
diff --git a/Source/core/css/CSSBasicShapes.h b/Source/core/css/CSSBasicShapes.h
index 78b84df13d793817ce10a33bcdce42719be87afb..cab20cad15112185db076f4f570e2255d5932d1b 100644
--- a/Source/core/css/CSSBasicShapes.h
+++ b/Source/core/css/CSSBasicShapes.h
@@ -60,9 +60,6 @@ public:
 
     virtual void trace(Visitor* visitor) { visitor->trace(m_layoutBox); }
 
-public:
-    virtual ~CSSBasicShape() { }

[kouhei (in TOK), 2014/03/05 07:05:01]

Isn't this needed for RefCounted<CSSBasicType>, built without enable_oilpan=1?

[zerny-chromium, 2014/03/05 07:07:04]

It is a no-op, so the default generated destructor should be sufficient. Is the
default not generated here for some reason?

[haraken, 2014/03/05 07:09:15]

Probably the default destructor is not virtual?

[kouhei (in TOK), 2014/03/05 07:11:55]

RefCountedBase's d-tor isn't declared virtual, and there are
RefPtr<CSSBasicShape>, so I thought derived classes' implicitly declared d-tor
like ~CSSBasicShapeRectangle() wouldn't be called without this explicit virtual
decl.

-
 protected:
     CSSBasicShape() { }
     RefPtrWillBeMember<CSSPrimitiveValue> m_layoutBox;