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 57 matching lines @@
     , m_hasMediaQueries(false)
     , m_hasSingleOwnerDocument(true)
     , m_parserContext(context)
 {
 }
 
 StyleSheetContents::StyleSheetContents(const StyleSheetContents& o)
     : m_ownerRule(nullptr)
     , m_originalURL(o.m_originalURL)
     , m_importRules(o.m_importRules.size())
+    , m_namespaceRules(o.m_namespaceRules.size())
     , m_childRules(o.m_childRules.size())
     , m_namespaces(o.m_namespaces)
     , m_hasSyntacticallyValidCSSHeader(o.m_hasSyntacticallyValidCSSHeader)
     , m_didLoadErrorOccur(false)
     , m_isMutable(false)
     , m_isInMemoryCache(false)
     , m_hasFontFaceRule(o.m_hasFontFaceRule)
     , m_hasMediaQueries(o.m_hasMediaQueries)
     , m_hasSingleOwnerDocument(true)
     , m_parserContext(o.m_parserContext)
@@ skipping 61 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 other than
+        // import/namespace/charset rules

[Timothy Loh, 2015/09/04 01:35:50]

not sure you want to write namespace here

[ramya.v, 2015/09/07 09:06:28]

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;
+        // Inserting @namespace rule when rules other than import/namespace/charset are present is not allowed.
+        if (m_childRules.size() > 0)
+            return false;
+
+        StyleRuleNamespace* namespaceRule = toStyleRuleNamespace(rule.get());
+        m_namespaceRules.insert(index, namespaceRule);
+        parserAddNamespace(namespaceRule->prefix(), namespaceRule->uri());

[Timothy Loh, 2015/09/04 01:35:50]

parserAddNamespace won't do what you want here. If a stylesheet specifies a
namespace prefix multiple times, the url is given by the last valid declaration.
If a namespace rule is inserted in the middle of a bunch of namespace rules, we
might end up using that rule instead of a later one with the same prefix.

[ramya.v, 2015/09/07 05:28:23]

Hi Tim just one doubt.
As per spec it was given
If a namespace prefix or default namespace is declared more than once only the
last declaration shall be used. Declaring a namespace prefix or default
namespace more than once is nonconforming.


As per your comments what I understood is if there are bunch of namespace rules
with same prefix, the rule which comes last in css rule list should be used.

However firefox and IE are using the rule which is declared later in time frame
even if it is added an index previous to existing namespace rules.

Ex: In the below example as per our understanding div3 with mango should have
background color red, however in Firefox and IE div2 with apple is having
background color red.

<html xmlns="http://www.w3.org/1999/xhtml">
    
    <style>
    @namespace abc url(banana);
     </style>


     <div id="div1" xmlns="banana"> div1</div>
	 <div id="div3" xmlns="apple">div3</div>
	 <div id="div2" xmlns="mango">div2</div>
    <script>
	document.styleSheets[0].insertRule('@namespace abc url(mango);',
document.styleSheets[0].cssRules.length);
	document.styleSheets[0].insertRule('@namespace abc url(apple);', 0);
	document.styleSheets[0].insertRule('div { color: red;}',
document.styleSheets[0].cssRules.length);
	document.styleSheets[0].insertRule('abc|div { color: green;}',
document.styleSheets[0].cssRules.length);
	</script>
</html>


Can you please let me know if this compatibility difference is fine?
Do I have to confirm the behavior with spec editors?

+        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)
+bool 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;
+        return true;
     }
     index -= m_importRules.size();
 
+    if (index < m_namespaceRules.size()) {
+        if (m_childRules.size() > 0)

[Timothy Loh, 2015/09/04 01:35:50]

!m_childRules.isEmpty() probably reads better

[ramya.v, 2015/09/07 09:06:28]

Done.

+            return false;
+        m_namespaceRules.remove(index);
+        return true;
+    }
+    index -= m_namespaceRules.size();
+
     if (m_childRules[index]->isFontFaceRule())
         notifyRemoveFontFaceRule(toStyleRuleFontFace(m_childRules[index].get()));
     m_childRules.remove(index);
+    return true;
 }
 
 void StyleSheetContents::parserAddNamespace(const AtomicString& prefix, const AtomicString& uri)
 {
     if (uri.isNull() || prefix.isNull())
         return;
     PrefixNamespaceURIMap::AddResult result = m_namespaces.add(prefix, uri);
     if (result.isNewEntry)
         return;
     result.storedValue->value = uri;
@@ skipping 350 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
 }
 
 }