SVG object with same idrefs get conflicted even they are under different shadow root

third_party/WebKit/Source/core/dom/TreeScope.cpp

Index: third_party/WebKit/Source/core/dom/TreeScope.cpp
diff --git a/third_party/WebKit/Source/core/dom/TreeScope.cpp b/third_party/WebKit/Source/core/dom/TreeScope.cpp
index fdea83b380fc9ff57a3dc8783132d9474952136d..027352bec310e6860799ff6617f21f8a20a89540 100644
--- a/third_party/WebKit/Source/core/dom/TreeScope.cpp
+++ b/third_party/WebKit/Source/core/dom/TreeScope.cpp
@@ -493,6 +493,18 @@ void TreeScope::setNeedsStyleRecalcForViewportUnits()
             element->setNeedsStyleRecalc(LocalStyleChange, StyleChangeReasonForTracing::create(StyleChangeReason::ViewportUnits));
     }
 }
+const SVGTreeScopeResources* TreeScope::svgTreeScopedResources()

[hayato, 2016/07/04 04:27:29]

Could we rename this to TreeScope::svgExtensions()?

"TreeScope" in a function name looks redundant to me.

[taijin, 2016/07/05 06:37:14]

Sure, thanks, will do it:)

+    {
+        return m_svgTreeScopedResources.get();
+    }
+
+SVGTreeScopeResources& TreeScope::accessSVGTreeScopedResources()
+{
+    if (!m_svgTreeScopedResources)
+        m_svgTreeScopedResources = new SVGTreeScopeResources(this);
+    return *m_svgTreeScopedResources;
+}
+
 
 DEFINE_TRACE(TreeScope)
 {
@@ -505,6 +517,7 @@ DEFINE_TRACE(TreeScope)
     visitor->trace(m_imageMapsByName);
     visitor->trace(m_scopedStyleResolver);
     visitor->trace(m_radioButtonGroupScope);
+    visitor->trace(m_svgTreeScopedResources);
 }
 
 } // namespace blink