Move CSSRuleList to the garbage collected heap

Source/core/css/CSSImportRule.cpp

 /*
  * (C) 1999-2003 Lars Knoll (knoll@kde.org)
  * (C) 2002-2003 Dirk Mueller (mueller@kde.org)
  * Copyright (C) 2002, 2005, 2006, 2008, 2009, 2010, 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 20 matching lines @@
 namespace WebCore {
 
 CSSImportRule::CSSImportRule(StyleRuleImport* importRule, CSSStyleSheet* parent)
     : CSSRule(parent)
     , m_importRule(importRule)
 {
 }
 
 CSSImportRule::~CSSImportRule()
 {
-    if (m_styleSheetCSSOMWrapper)

[Mads Ager (chromium), 2014/03/12 12:24:42]

I think you still need this when !ENABLE(OILPAN).

[Erik Corry, 2014/03/12 12:31:27]

Done.

-        m_styleSheetCSSOMWrapper->clearOwnerRule();
 #if !ENABLE(OILPAN)
     // MediaList and the parent CSSImportRule are both on the oilpan heap and die together.
     // Therefor clearing is not needed nor allowed since it could be touching already
     // finalized memory.
     if (m_mediaCSSOMWrapper)
         m_mediaCSSOMWrapper->clearParentRule();
 #endif // ENABLE(OILPAN)
 }
 
 String CSSImportRule::href() const
@@ skipping 40 matching lines @@
 void CSSImportRule::reattach(StyleRuleBase*)
 {
     // FIXME: Implement when enabling caching for stylesheets with import rules.
     ASSERT_NOT_REACHED();
 }
 
 void CSSImportRule::trace(Visitor* visitor)
 {
     visitor->trace(m_importRule);
     visitor->trace(m_mediaCSSOMWrapper);
+    visitor->trace(m_styleSheetCSSOMWrapper);
     CSSRule::trace(visitor);
 }
 
 } // namespace WebCore