Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(707)

Unified Diff: third_party/WebKit/Source/core/svg/SVGMPathElement.cpp

Issue 2633143002: SVG objects with same idrefs conflict when under different shadow root (Closed)
Patch Set: Tests Created 3 years, 11 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: third_party/WebKit/Source/core/svg/SVGMPathElement.cpp
diff --git a/third_party/WebKit/Source/core/svg/SVGMPathElement.cpp b/third_party/WebKit/Source/core/svg/SVGMPathElement.cpp
index 0e190034b95d5fb51151b652d20fb8fe61b6014d..4693d7d0d855f52ecc6a20f3115d3354cbad0121 100644
--- a/third_party/WebKit/Source/core/svg/SVGMPathElement.cpp
+++ b/third_party/WebKit/Source/core/svg/SVGMPathElement.cpp
@@ -21,8 +21,8 @@
#include "core/dom/Document.h"
#include "core/svg/SVGAnimateMotionElement.h"
-#include "core/svg/SVGDocumentExtensions.h"
#include "core/svg/SVGPathElement.h"
+#include "core/svg/SVGTreeScopeResources.h"
namespace blink {
@@ -50,12 +50,12 @@ void SVGMPathElement::buildPendingResource() {
hrefString(), treeScope(), &id);
if (!target) {
// Do not register as pending if we are already pending this resource.
- if (document().accessSVGExtensions().isElementPendingResource(this, id))
+ if (treeScope().accessSVGTreeScopedResources().isElementPendingResource(
+ this, id))
return;
-
if (!id.isEmpty()) {
- document().accessSVGExtensions().addPendingResource(id, this);
- ASSERT(hasPendingResources());
+ treeScope().accessSVGTreeScopedResources().addPendingResource(id, this);
+ DCHECK(hasPendingResources());
}
} else if (isSVGPathElement(target)) {
// Register us with the target in the dependencies map. Any change of

Powered by Google App Engine
This is Rietveld 408576698