Ensure proper finalization of garbage-collected types.

Source/core/css/CSSCalculationValue.h

Index: Source/core/css/CSSCalculationValue.h
diff --git a/Source/core/css/CSSCalculationValue.h b/Source/core/css/CSSCalculationValue.h
index ca4c5bfd24818561f43f06a899103a523a8bfd8d..b4c581ad0a6dda7899b51e4dc6d97386cf60823d 100644
--- a/Source/core/css/CSSCalculationValue.h
+++ b/Source/core/css/CSSCalculationValue.h
@@ -56,7 +56,7 @@ enum CalculationCategory {
     CalcOther
 };
 
-class CSSCalcExpressionNode : public RefCountedWillBeGarbageCollected<CSSCalcExpressionNode> {
+class CSSCalcExpressionNode : public RefCountedWillBeGarbageCollectedFinalized<CSSCalcExpressionNode> {

[haraken, 2014/03/11 10:26:37]

Why does this class need to be GarbageCollectedFinalized?

[zerny-chromium, 2014/03/11 11:05:25]

Because it has a virtual destructor.

[Mads Ager (chromium), 2014/03/11 13:34:41]

The virtual destructor is definitely needed when !ENABLE(OILPAN), however, maybe
we should ifdef it out for the ENABLE(OILPAN) build and leave this 
GarbageCollected? As far as I can tell no destructor call is needed here with
Oilpan.

[zerny-chromium, 2014/03/11 13:54:12]

Will do.

 public:
     enum Type {
         CssCalcPrimitiveValue = 1,