Corrected assert for cacheable stylesheets.

third_party/WebKit/Source/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 69 matching lines @@
     , m_defaultNamespace(o.m_defaultNamespace)
     , m_hasSyntacticallyValidCSSHeader(o.m_hasSyntacticallyValidCSSHeader)
     , m_didLoadErrorOccur(false)
     , m_isMutable(false)
     , m_isReferencedFromResource(false)
     , m_hasFontFaceRule(o.m_hasFontFaceRule)
     , m_hasMediaQueries(o.m_hasMediaQueries)
     , m_hasSingleOwnerDocument(true)
     , m_parserContext(o.m_parserContext)
 {
-    ASSERT(o.isCacheable());
-
     // FIXME: Copy import rules.
     ASSERT(o.m_importRules.isEmpty());
 
     for (unsigned i = 0; i < m_childRules.size(); ++i)
         m_childRules[i] = o.m_childRules[i]->copy();
 }
 
 StyleSheetContents::~StyleSheetContents()
 {
 }
 
 void StyleSheetContents::setHasSyntacticallyValidCSSHeader(bool isValidCss)
 {
     if (!isValidCss) {
         if (Document* document = clientSingleOwnerDocument())
             removeSheetFromCache(document);
     }
     m_hasSyntacticallyValidCSSHeader = isValidCss;
 }
 
-bool StyleSheetContents::isCacheable() const
+bool StyleSheetContents::isCacheableForResource() const
 {
     // This would require dealing with multiple clients for load callbacks.
     if (!loadCompleted())
         return false;
     // 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;
     // FIXME: Support copying import rules.
     if (!m_importRules.isEmpty())
         return false;
     // FIXME: Support cached stylesheets in import rules.
     if (m_ownerRule)
         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;
 }
 
+bool StyleSheetContents::isCacheableForStyleElement() const
+{
+    // FIXME: Support copying import rules.
+    if (!importRules().isEmpty())
+        return false;
+    // Until import rules are supported in cached sheets it's not possible for loading to fail.
+    DCHECK(!didLoadErrorOccur());
+    // It is not the original sheet anymore.
+    if (isMutable())
+        return false;
+    if (!hasSyntacticallyValidCSSHeader())
+        return false;
+    return true;
+}
+
+
 void StyleSheetContents::parserAppendRule(StyleRuleBase* rule)
 {
     if (rule->isImportRule()) {
         // Parser enforces that @import rules come before anything else
         ASSERT(m_childRules.isEmpty());
         StyleRuleImport* importRule = toStyleRuleImport(rule);
         if (importRule->mediaQueries())
             setHasMediaQueries();
         m_importRules.append(importRule);
         m_importRules.last()->setParentStyleSheet(this);
@@ skipping 341 matching lines @@
         case StyleRuleBase::Supports:
         case StyleRuleBase::Viewport:
             break;
         }
     }
     return false;
 }
 
 bool StyleSheetContents::hasFailedOrCanceledSubresources() const
 {
-    ASSERT(isCacheable());
+    ASSERT(isCacheableForResource());
     return childRulesHaveFailedOrCanceledSubresources(m_childRules);
 }
 
 Document* StyleSheetContents::clientSingleOwnerDocument() const
 {
     if (!m_hasSingleOwnerDocument || clientSize() <= 0)
         return nullptr;
 
     if (m_loadingClients.size())
         return (*m_loadingClients.begin())->ownerDocument();
@@ skipping 54 matching lines @@
 
 void StyleSheetContents::removeSheetFromCache(Document* document)
 {
     ASSERT(document);
     document->styleEngine().removeSheet(this);
 }
 
 void StyleSheetContents::setReferencedFromResource(bool referenced)
 {
     ASSERT(referenced != m_isReferencedFromResource);
-    ASSERT(isCacheable());
+    ASSERT(isCacheableForResource());
     m_isReferencedFromResource = referenced;
 }
 
 RuleSet& StyleSheetContents::ensureRuleSet(const MediaQueryEvaluator& medium, AddRuleFlags addRuleFlags)
 {
     if (!m_ruleSet) {
         m_ruleSet = RuleSet::create();
         m_ruleSet->addRulesFromSheet(this, medium, addRuleFlags);
     }
     return *m_ruleSet.get();
@@ skipping 72 matching lines @@
     visitor->trace(m_ownerRule);
     visitor->trace(m_importRules);
     visitor->trace(m_namespaceRules);
     visitor->trace(m_childRules);
     visitor->trace(m_loadingClients);
     visitor->trace(m_completedClients);
     visitor->trace(m_ruleSet);
 }
 
 } // namespace blink