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

Unified Diff: LayoutTests/svg/W3C-SVG-1.1-SE/types-dom-06-f.svg

Issue 20738002: Make DOMImplementation::hasFeature() behave according to specification (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Created 7 years, 5 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: LayoutTests/svg/W3C-SVG-1.1-SE/types-dom-06-f.svg
diff --git a/LayoutTests/svg/W3C-SVG-1.1-SE/types-dom-06-f.svg b/LayoutTests/svg/W3C-SVG-1.1-SE/types-dom-06-f.svg
index e88e2475aa9317d69d7bd577a44bcb10b199dce8..15b6eb225821d13e6545dad6c36f95a921e2bfb6 100644
--- a/LayoutTests/svg/W3C-SVG-1.1-SE/types-dom-06-f.svg
+++ b/LayoutTests/svg/W3C-SVG-1.1-SE/types-dom-06-f.svg
@@ -61,7 +61,7 @@
r1.removeAttribute("requiredFeatures");
return;
}
- if(i1 != "this.is.a.bogus.feature.string")
abarth-chromium 2013/07/26 17:24:25 There are SVG conformance tests. Should we be cha
do-not-use 2013/07/26 17:49:29 Sorry, I updated many SVG tests and I missed that
+ if(i1 != "http://www.w3.org/TR/SVG11/feature#BogusFeature")
{
r1.removeAttribute("requiredFeatures");
return;
@@ -113,7 +113,7 @@
</g>
<!-- tests -->
- <rect id="r1" requiredFeatures="http://www.w3.org/TR/SVG11/feature#Shape this.is.a.bogus.feature.string" width="100" height="100" fill="red" transform="translate(100 100)"/>
+ <rect id="r1" requiredFeatures="http://www.w3.org/TR/SVG11/feature#Shape http://www.w3.org/TR/SVG11/feature#BogusFeature" width="100" height="100" fill="red" transform="translate(100 100)"/>
<rect id="r2" width="100" height="100" fill="red" transform="translate(190 100)"/>
<rect id="r3" width="100" height="100" fill="lime" requiredFeatures="http://www.w3.org/TR/SVG11/feature#Shape" transform="translate(300 100)"/>

Powered by Google App Engine
This is Rietveld 408576698