third_party/WebKit/LayoutTests/svg/custom/duplicate-ids-removal.html - Issue 2722543002: Improve handling of duplicate id's for SVG resources

Unified Diff: third_party/WebKit/LayoutTests/svg/custom/duplicate-ids-removal.html

Issue 2722543002: Improve handling of duplicate id's for SVG resources (Closed)

Patch Set: Created 3 years, 10 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

« third_party/WebKit/LayoutTests/svg/custom/duplicate-ids-addition.html ('K') | « third_party/WebKit/LayoutTests/svg/custom/duplicate-ids-id-mutation-expected.html ('k') | third_party/WebKit/LayoutTests/svg/custom/duplicate-ids-removal-expected.html » ('j') | third_party/WebKit/Source/core/svg/SVGTreeScopeResources.cpp » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Hide Comments ('s')

Index: third_party/WebKit/LayoutTests/svg/custom/duplicate-ids-removal.html

diff --git a/third_party/WebKit/LayoutTests/svg/custom/duplicate-ids-removal.html b/third_party/WebKit/LayoutTests/svg/custom/duplicate-ids-removal.html

new file mode 100644

index 0000000000000000000000000000000000000000..a766a3c8bc3499f2918ee923945eec944c0b8896

--- /dev/null

+++ b/third_party/WebKit/LayoutTests/svg/custom/duplicate-ids-removal.html

@@ -0,0 +1,16 @@

+<!DOCTYPE html>

+<script src="../../resources/run-after-layout-and-paint.js"></script>

+<svg>

+ <linearGradient id="f">

+ <stop stop-color="green"/>

+ </linearGradient>

+ <linearGradient id="f">

+ <stop stop-color="red"/>

+ </linearGradient>

+ <rect width="100" height="100" fill="url(#f) orange"/>

+</svg>

+<script>

+runAfterLayoutAndPaint(function() {

+ document.querySelectorAll('lineargradient')[1].remove();

Stephen Chennney 2017/02/27 16:21:22 Another test comes to mind. If you remove the firs

fs 2017/02/27 17:47:30 I wrote this test, but it's currently failing (so

fs 2017/02/27 19:11:25 I realize that that test likely worked "by acciden

+}, true);

+</script>