Chromium Code Reviews| OLD | NEW |
|---|---|
| 1 /* | 1 /* |
| 2 * (C) 1999-2003 Lars Knoll (knoll@kde.org) | 2 * (C) 1999-2003 Lars Knoll (knoll@kde.org) |
| 3 * (C) 2002-2003 Dirk Mueller (mueller@kde.org) | 3 * (C) 2002-2003 Dirk Mueller (mueller@kde.org) |
| 4 * Copyright (C) 2002, 2005, 2006, 2008, 2009, 2010, 2012 Apple Inc. All rights reserved. | 4 * Copyright (C) 2002, 2005, 2006, 2008, 2009, 2010, 2012 Apple Inc. All rights reserved. |
| 5 * | 5 * |
| 6 * This library is free software; you can redistribute it and/or | 6 * This library is free software; you can redistribute it and/or |
| 7 * modify it under the terms of the GNU Library General Public | 7 * modify it under the terms of the GNU Library General Public |
| 8 * License as published by the Free Software Foundation; either | 8 * License as published by the Free Software Foundation; either |
| 9 * version 2 of the License, or (at your option) any later version. | 9 * version 2 of the License, or (at your option) any later version. |
| 10 * | 10 * |
| (...skipping 22 matching lines...) Expand all Loading... | |
| 33 CSSImportRule::CSSImportRule(StyleRuleImport* importRule, CSSStyleSheet* parent) | 33 CSSImportRule::CSSImportRule(StyleRuleImport* importRule, CSSStyleSheet* parent) |
| 34 : CSSRule(parent) | 34 : CSSRule(parent) |
| 35 , m_importRule(importRule) | 35 , m_importRule(importRule) |
| 36 { | 36 { |
| 37 } | 37 } |
| 38 | 38 |
| 39 CSSImportRule::~CSSImportRule() | 39 CSSImportRule::~CSSImportRule() |
| 40 { | 40 { |
| 41 if (m_styleSheetCSSOMWrapper) | 41 if (m_styleSheetCSSOMWrapper) |
| 42 m_styleSheetCSSOMWrapper->clearOwnerRule(); | 42 m_styleSheetCSSOMWrapper->clearOwnerRule(); |
| 43 #if !ENABLE(OILPAN) | |
| 44 // Oilpan automatically handles cycles and doesn't want destructors to touch potentially | |
|
Mads Ager (chromium)
2014/03/03 10:38:45
Maybe the comment should state that the MediaList
wibling-chromium
2014/03/03 10:45:08
Done.
| |
| 45 // dead objects to perform cleanup. | |
| 43 if (m_mediaCSSOMWrapper) | 46 if (m_mediaCSSOMWrapper) |
| 44 m_mediaCSSOMWrapper->clearParentRule(); | 47 m_mediaCSSOMWrapper->clearParentRule(); |
| 48 #endif // ENABLE(OILPAN) | |
| 45 } | 49 } |
| 46 | 50 |
| 47 String CSSImportRule::href() const | 51 String CSSImportRule::href() const |
| 48 { | 52 { |
| 49 return m_importRule->href(); | 53 return m_importRule->href(); |
| 50 } | 54 } |
| 51 | 55 |
| 52 MediaList* CSSImportRule::media() const | 56 MediaList* CSSImportRule::media() const |
| 53 { | 57 { |
| 54 if (!m_mediaCSSOMWrapper) | 58 if (!m_mediaCSSOMWrapper) |
| (...skipping 29 matching lines...) Expand all Loading... | |
| 84 m_styleSheetCSSOMWrapper = CSSStyleSheet::create(m_importRule->styleShee t(), const_cast<CSSImportRule*>(this)); | 88 m_styleSheetCSSOMWrapper = CSSStyleSheet::create(m_importRule->styleShee t(), const_cast<CSSImportRule*>(this)); |
| 85 return m_styleSheetCSSOMWrapper.get(); | 89 return m_styleSheetCSSOMWrapper.get(); |
| 86 } | 90 } |
| 87 | 91 |
| 88 void CSSImportRule::reattach(StyleRuleBase*) | 92 void CSSImportRule::reattach(StyleRuleBase*) |
| 89 { | 93 { |
| 90 // FIXME: Implement when enabling caching for stylesheets with import rules. | 94 // FIXME: Implement when enabling caching for stylesheets with import rules. |
| 91 ASSERT_NOT_REACHED(); | 95 ASSERT_NOT_REACHED(); |
| 92 } | 96 } |
| 93 | 97 |
| 98 void CSSImportRule::trace(Visitor* visitor) | |
| 99 { | |
| 100 visitor->trace(m_importRule); | |
| 101 visitor->trace(m_mediaCSSOMWrapper); | |
| 102 CSSRule::trace(visitor); | |
| 103 } | |
| 104 | |
| 94 } // namespace WebCore | 105 } // namespace WebCore |
| OLD | NEW |