Support for CSSOM CSSNamespaceRule interface

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 138 matching lines @@
         StyleRuleImport* importRule = toStyleRuleImport(rule.get());
         if (importRule->mediaQueries())
             setHasMediaQueries();
         m_importRules.append(importRule);
         m_importRules.last()->setParentStyleSheet(this);
         m_importRules.last()->requestStyleSheet();
         return;
     }
 
     if (rule->isNamespaceRule()) {
-        // Parser enforces that @namespace rules come before anything else
+        // Parser enforces that @namespace rules come before all rules except for import
+        // or namespace rules

[Timothy Loh, 2015/09/02 04:45:14]

namespace -> charset?

[ramya.v, 2015/09/02 09:20:54]

Done.

         ASSERT(m_childRules.isEmpty());
         StyleRuleNamespace& namespaceRule = toStyleRuleNamespace(*rule);
         parserAddNamespace(namespaceRule.prefix(), namespaceRule.uri());
+        m_namespaceRules.append(&namespaceRule);
         return;
     }
 
     if (rule->isMediaRule())
         setHasMediaQueries();
 
     m_childRules.append(rule);
 }
 
 void StyleSheetContents::setHasMediaQueries()
 {
     m_hasMediaQueries = true;
     if (parentStyleSheet())
         parentStyleSheet()->setHasMediaQueries();
 }
 
 StyleRuleBase* StyleSheetContents::ruleAt(unsigned index) const
 {
     ASSERT_WITH_SECURITY_IMPLICATION(index < ruleCount());
 
     if (index < m_importRules.size())
         return m_importRules[index].get();
 
     index -= m_importRules.size();
+
+    if (index < m_namespaceRules.size())
+        return m_namespaceRules[index].get();
+
+    index -= m_namespaceRules.size();
+
     return m_childRules[index].get();
 }
 
 unsigned StyleSheetContents::ruleCount() const
 {
-    return m_importRules.size() + m_childRules.size();
+    return m_importRules.size() + m_namespaceRules.size() + m_childRules.size();
 }
 
 void StyleSheetContents::clearRules()
 {
     for (unsigned i = 0; i < m_importRules.size(); ++i) {
         ASSERT(m_importRules.at(i)->parentStyleSheet() == this);
         m_importRules[i]->clearParentStyleSheet();
     }
     m_importRules.clear();
+    m_namespaceRules.clear();
     m_childRules.clear();
 }
 
 bool StyleSheetContents::wrapperInsertRule(PassRefPtrWillBeRawPtr<StyleRuleBase> rule, unsigned index)
 {
     ASSERT(m_isMutable);
     ASSERT_WITH_SECURITY_IMPLICATION(index <= ruleCount());
 
     if (index < m_importRules.size() || (index == m_importRules.size() && rule->isImportRule())) {
         // Inserting non-import rule before @import is not allowed.
@@ skipping 12 matching lines @@
     }
     // Inserting @import rule after a non-import rule is not allowed.
     if (rule->isImportRule())
         return false;
 
     if (rule->isMediaRule())
         setHasMediaQueries();
 
     index -= m_importRules.size();
 
+    if (index < m_namespaceRules.size() || (index == m_namespaceRules.size() && rule->isNamespaceRule())) {
+        // Inserting non-namespace rules other than import rule before @namespace is not allowed.
+        if (!rule->isNamespaceRule())
+            return false;
+
+        StyleRuleNamespace* namespaceRule = toStyleRuleNamespace(rule.get());
+        m_namespaceRules.insert(index, namespaceRule);

[Timothy Loh, 2015/09/02 04:45:14]

..don't we need to also update m_namespaces? Currently (during stylesheet
parsing) we'll get into parserAddNamespace, although that currently assumes
we're only adding to the end.

[ramya.v, 2015/09/02 09:20:54]

Done.

+        return true;
+    }
+
+    if (rule->isNamespaceRule())
+        return false;
+
+    index -= m_namespaceRules.size();
+
     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 (index < m_importRules.size()) {
         m_importRules[index]->clearParentStyleSheet();
         if (m_importRules[index]->isFontFaceRule())
             notifyRemoveFontFaceRule(toStyleRuleFontFace(m_importRules[index].get()));
         m_importRules.remove(index);
         return;
     }
     index -= m_importRules.size();
 
+    if (index < m_namespaceRules.size()) {

[Timothy Loh, 2015/09/02 04:45:14]

From the spec:
If old rule is an @namespace at-rule, and list contains anything other than
@import at-rules, and @namespace at-rules, throw an InvalidStateError exception.

[ramya.v, 2015/09/02 09:20:54]

Done.

+        m_namespaceRules.remove(index);
+        return;
+    }
+    index -= m_namespaceRules.size();
+
     if (m_childRules[index]->isFontFaceRule())
         notifyRemoveFontFaceRule(toStyleRuleFontFace(m_childRules[index].get()));
     m_childRules.remove(index);
 }
 
 void StyleSheetContents::parserAddNamespace(const AtomicString& prefix, const AtomicString& uri)
 {
     if (uri.isNull() || prefix.isNull())
         return;
     PrefixNamespaceURIMap::AddResult result = m_namespaces.add(prefix, uri);
@@ skipping 353 matching lines @@
     }
 
     findFontFaceRulesFromRules(childRules(), fontFaceRules);
 }
 
 DEFINE_TRACE(StyleSheetContents)
 {
 #if ENABLE(OILPAN)
     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);
 #endif
 }
 
 }