Oilpan: Remove Persistent handles from heap allocated CSSRule objects and fix finalization of CSSKe…

Source/core/css/CSSPageRule.cpp

 /*
  * (C) 1999-2003 Lars Knoll (knoll@kde.org)
  * (C) 2002-2003 Dirk Mueller (mueller@kde.org)
  * Copyright (C) 2002, 2005, 2006, 2008, 2012 Apple Inc. All rights reserved.
  *
  * This library is free software; you can redistribute it and/or
  * modify it under the terms of the GNU Library General Public
  * License as published by the Free Software Foundation; either
  * version 2 of the License, or (at your option) any later version.
  *
@@ skipping 83 matching lines @@
 }
 
 void CSSPageRule::reattach(StyleRuleBase* rule)
 {
     ASSERT(rule);
     m_pageRule = toStyleRulePage(rule);
     if (m_propertiesCSSOMWrapper)
         m_propertiesCSSOMWrapper->reattach(m_pageRule->mutableProperties());
 }
 
+void CSSPageRule::trace(Visitor* visitor)
+{
+    visitor->trace(m_pageRule);
+    CSSRule::trace(visitor);

[Erik Corry, 2014/03/03 09:44:29]

I think mostly we have done the superclass call first.

+}
+
 } // namespace WebCore