Remove single client special cases for StyleSheetContents.

sky/engine/core/css/StyleSheetContents.cpp

 /*
  * (C) 1999-2003 Lars Knoll (knoll@kde.org)
  * Copyright (C) 2004, 2006, 2007, 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.
  *
  * This library is distributed in the hope that it will be useful,
@@ skipping 20 matching lines @@
 #include "sky/engine/core/dom/StyleEngine.h"
 #include "sky/engine/core/frame/UseCounter.h"
 #include "sky/engine/platform/TraceEvent.h"
 #include "sky/engine/wtf/Deque.h"
 
 namespace blink {
 
 StyleSheetContents::StyleSheetContents(const CSSParserContext& context)
     : m_usesRemUnits(false)
     , m_hasMediaQueries(false)
-    , m_hasSingleOwnerDocument(true)
     , m_parserContext(context)
 {
 }
 
 StyleSheetContents::~StyleSheetContents()
 {
-#if !ENABLE(OILPAN)
-    clearRules();
-#endif
+    // TODO(esprehn): Why is this here? The rules will be cleared immediately
+    // after this destructor runs anyway.
+    m_childRules.clear();
 }
 
 void StyleSheetContents::parserAppendRule(PassRefPtr<StyleRuleBase> rule)
 {
-    // Add warning message to inspector if dpi/dpcm values are used for screen media.
-    if (rule->isMediaRule()) {
+    if (rule->isMediaRule())
         setHasMediaQueries();
-        reportMediaQueryWarningIfNeeded(singleOwnerDocument(), toStyleRuleMedia(rule.get())->mediaQueries());
-    }
-
     m_childRules.append(rule);
 }
 
 void StyleSheetContents::setHasMediaQueries()
 {
     m_hasMediaQueries = true;
 }
 
 StyleRuleBase* StyleSheetContents::ruleAt(unsigned index) const
 {
     ASSERT_WITH_SECURITY_IMPLICATION(index < ruleCount());
     return m_childRules[index].get();
 }
 
 unsigned StyleSheetContents::ruleCount() const
 {
     return m_childRules.size();
 }
 
-void StyleSheetContents::clearRules()
-{
-    m_childRules.clear();
-}
-
 bool StyleSheetContents::parseString(const String& sheetText)
 {
     CSSParserContext context(parserContext(), UseCounter::getFrom(this));
     BisonCSSParser p(context);
     p.parseSheet(this, sheetText);
     return true;
 }
 
-bool StyleSheetContents::hasSingleOwnerNode() const
-{
-    return hasOneClient();
-}
-
-Node* StyleSheetContents::singleOwnerNode() const
-{
-    if (!hasOneClient())
-        return 0;
-    if (m_loadingClients.size())
-        return (*m_loadingClients.begin())->ownerNode();
-    return (*m_completedClients.begin())->ownerNode();
-}
-
-Document* StyleSheetContents::singleOwnerDocument() const
-{
-    return clientSingleOwnerDocument();
-}
-
-Document* StyleSheetContents::clientSingleOwnerDocument() const
-{
-    if (!m_hasSingleOwnerDocument || clientSize() <= 0)
-        return 0;
-
-    if (m_loadingClients.size())
-        return (*m_loadingClients.begin())->ownerDocument();
-    return (*m_completedClients.begin())->ownerDocument();
-}
-
 void StyleSheetContents::registerClient(CSSStyleSheet* sheet)
 {
     ASSERT(!m_loadingClients.contains(sheet) && !m_completedClients.contains(sheet));
 
     // InspectorCSSAgent::buildObjectForRule creates CSSStyleSheet without any owner node.
     if (!sheet->ownerDocument())
         return;
-
-    if (Document* document = clientSingleOwnerDocument()) {
-        if (sheet->ownerDocument() != document)
-            m_hasSingleOwnerDocument = false;
-    }
     m_loadingClients.add(sheet);
 }
 
 void StyleSheetContents::unregisterClient(CSSStyleSheet* sheet)
 {
     m_loadingClients.remove(sheet);
     m_completedClients.remove(sheet);
 
     if (!sheet->ownerDocument() || !m_loadingClients.isEmpty() || !m_completedClients.isEmpty())
         return;
-
-    if (m_hasSingleOwnerDocument)
-        removeSheetFromCache(sheet->ownerDocument());
-    m_hasSingleOwnerDocument = true;
-}
-
-void StyleSheetContents::removeSheetFromCache(Document* document)
-{
-    ASSERT(document);
-    document->styleEngine()->removeSheet(this);
+    sheet->ownerDocument()->styleEngine()->removeSheet(this);
 }
 
 void StyleSheetContents::shrinkToFit()
 {
     m_childRules.shrinkToFit();
 }
 
 RuleSet& StyleSheetContents::ensureRuleSet(const MediaQueryEvaluator& medium, AddRuleFlags addRuleFlags)
 {
     if (!m_ruleSet) {
@@ skipping 34 matching lines @@
     }
 }
 
 void StyleSheetContents::notifyRemoveFontFaceRule(const StyleRuleFontFace* fontFaceRule)
 {
     removeFontFaceRules(m_loadingClients, fontFaceRule);
     removeFontFaceRules(m_completedClients, fontFaceRule);
 }
 
 }