Remove CSSOM mutability in StyleRule.

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 17 matching lines @@
 #include "sky/engine/core/css/parser/BisonCSSParser.h"
 #include "sky/engine/core/dom/Document.h"
 #include "sky/engine/core/dom/Node.h"
 #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 {
 
-// Rough size estimate for the memory cache.
-unsigned StyleSheetContents::estimatedSizeInBytes() const
-{
-    // Note that this does not take into account size of the strings hanging from various objects.
-    // The assumption is that nearly all of of them are atomic and would exist anyway.
-    unsigned size = sizeof(*this);
-
-    // FIXME: This ignores the children of media rules.
-    // Most rules are StyleRules.
-    size += ruleCount() * StyleRule::averageSizeInBytes();
-    return size;
-}
-
 StyleSheetContents::StyleSheetContents(const String& originalURL, const CSSParserContext& context)
-    : m_originalURL(originalURL)
-    , m_hasSyntacticallyValidCSSHeader(true)
-    , m_didLoadErrorOccur(false)
+    : m_hasSyntacticallyValidCSSHeader(true)
     , m_usesRemUnits(false)
-    , m_isMutable(false)
-    , m_isInMemoryCache(false)
-    , m_hasFontFaceRule(false)
     , m_hasMediaQueries(false)
     , m_hasSingleOwnerDocument(true)
+    , m_originalURL(originalURL)
     , m_parserContext(context)
 {
 }
 
-StyleSheetContents::StyleSheetContents(const StyleSheetContents& o)
-    : m_originalURL(o.m_originalURL)
-    , m_childRules(o.m_childRules.size())
-    , m_namespaces(o.m_namespaces)
-    , m_hasSyntacticallyValidCSSHeader(o.m_hasSyntacticallyValidCSSHeader)
-    , m_didLoadErrorOccur(false)
-    , m_usesRemUnits(o.m_usesRemUnits)
-    , m_isMutable(false)
-    , m_isInMemoryCache(false)
-    , m_hasFontFaceRule(o.m_hasFontFaceRule)
-    , m_hasMediaQueries(o.m_hasMediaQueries)
-    , m_hasSingleOwnerDocument(true)
-    , m_parserContext(o.m_parserContext)
-{
-    ASSERT(o.isCacheable());
-
-    for (unsigned i = 0; i < m_childRules.size(); ++i)
-        m_childRules[i] = o.m_childRules[i]->copy();
-}
-
 StyleSheetContents::~StyleSheetContents()
 {
 #if !ENABLE(OILPAN)
     clearRules();
 #endif
 }
 
 void StyleSheetContents::setHasSyntacticallyValidCSSHeader(bool isValidCss)
 {
     if (!isValidCss) {
         if (Document* document = clientSingleOwnerDocument())
             removeSheetFromCache(document);
     }
     m_hasSyntacticallyValidCSSHeader = isValidCss;
 }
 
 bool StyleSheetContents::isCacheable() const
 {
     // FIXME: StyleSheets with media queries can't be cached because their RuleSet
     // is processed differently based off the media queries, which might resolve
     // differently depending on the context of the parent CSSStyleSheet (e.g.
     // if they are in differently sized iframes). Once RuleSets are media query
     // agnostic, we can restore sharing of StyleSheetContents with medea queries.
     if (m_hasMediaQueries)
         return false;
-    if (m_didLoadErrorOccur)
-        return false;
-    // It is not the original sheet anymore.
-    if (m_isMutable)
-        return false;
     // If the header is valid we are not going to need to check the SecurityOrigin.
     // FIXME: Valid mime type avoids the check too.
     if (!m_hasSyntacticallyValidCSSHeader)
         return false;
     return true;
 }
 
 void StyleSheetContents::parserAppendRule(PassRefPtr<StyleRuleBase> rule)
 {
     // Add warning message to inspector if dpi/dpcm values are used for screen media.
@@ skipping 19 matching lines @@
 unsigned StyleSheetContents::ruleCount() const
 {
     return m_childRules.size();
 }
 
 void StyleSheetContents::clearRules()
 {
     m_childRules.clear();
 }
 
-bool StyleSheetContents::wrapperInsertRule(PassRefPtr<StyleRuleBase> rule, unsigned index)
-{
-    ASSERT(m_isMutable);
-    ASSERT_WITH_SECURITY_IMPLICATION(index <= ruleCount());
-
-    if (rule->isMediaRule())
-        setHasMediaQueries();
-
-    if (rule->isFontFaceRule())
-        setHasFontFaceRule(true);
-    m_childRules.insert(index, rule);
-    return true;
-}
-
-void StyleSheetContents::wrapperDeleteRule(unsigned index)
-{
-    ASSERT(m_isMutable);
-    ASSERT_WITH_SECURITY_IMPLICATION(index < ruleCount());
-
-    if (m_childRules[index]->isFontFaceRule())
-        notifyRemoveFontFaceRule(toStyleRuleFontFace(m_childRules[index].get()));
-    m_childRules.remove(index);
-}
-
 bool StyleSheetContents::parseString(const String& sheetText)
 {
     return parseStringAtPosition(sheetText, TextPosition::minimumPosition(), false);
 }
 
 bool StyleSheetContents::parseStringAtPosition(const String& sheetText, const TextPosition& startPosition, bool createdByParser)
 {
     CSSParserContext context(parserContext(), UseCounter::getFrom(this));
     BisonCSSParser p(context);
     p.parseSheet(this, sheetText, startPosition, 0, createdByParser);
@@ skipping 97 matching lines @@
         removeSheetFromCache(sheet->ownerDocument());
     m_hasSingleOwnerDocument = true;
 }
 
 void StyleSheetContents::removeSheetFromCache(Document* document)
 {
     ASSERT(document);
     document->styleEngine()->removeSheet(this);
 }
 
-void StyleSheetContents::addedToMemoryCache()
-{
-    ASSERT(!m_isInMemoryCache);
-    ASSERT(isCacheable());
-    m_isInMemoryCache = true;
-}
-
-void StyleSheetContents::removedFromMemoryCache()
-{
-    ASSERT(m_isInMemoryCache);
-    ASSERT(isCacheable());
-    m_isInMemoryCache = false;
-}
-
 void StyleSheetContents::shrinkToFit()
 {
     m_childRules.shrinkToFit();
 }
 
 RuleSet& StyleSheetContents::ensureRuleSet(const MediaQueryEvaluator& medium, AddRuleFlags addRuleFlags)
 {
     if (!m_ruleSet) {
         m_ruleSet = RuleSet::create();
         m_ruleSet->addRulesFromSheet(this, medium, addRuleFlags);
@@ skipping 31 matching lines @@
             ownerNode->document().styleEngine()->removeFontFaceRules(Vector<RawPtr<const StyleRuleFontFace> >(1, fontFaceRule));
     }
 }
 
 void StyleSheetContents::notifyRemoveFontFaceRule(const StyleRuleFontFace* fontFaceRule)
 {
     removeFontFaceRules(m_loadingClients, fontFaceRule);
     removeFontFaceRules(m_completedClients, fontFaceRule);
 }
 
-static void findFontFaceRulesFromRules(const Vector<RefPtr<StyleRuleBase> >& rules, Vector<RawPtr<const StyleRuleFontFace> >& fontFaceRules)
-{
-    for (unsigned i = 0; i < rules.size(); ++i) {
-        StyleRuleBase* rule = rules[i].get();
-
-        if (rule->isFontFaceRule()) {
-            fontFaceRules.append(toStyleRuleFontFace(rule));
-        } else if (rule->isMediaRule()) {
-            StyleRuleMedia* mediaRule = toStyleRuleMedia(rule);
-            // We cannot know whether the media rule matches or not, but
-            // for safety, remove @font-face in the media rule (if exists).
-            findFontFaceRulesFromRules(mediaRule->childRules(), fontFaceRules);
-        }
-    }
 }
-
-void StyleSheetContents::findFontFaceRules(Vector<RawPtr<const StyleRuleFontFace> >& fontFaceRules)
-{
-    findFontFaceRulesFromRules(childRules(), fontFaceRules);
-}
-
-}