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

Unified Diff: LayoutTests/fast/innerHTML/javascript-url-expected.txt

Issue 1031783002: Escape '&' in javascript URLs for innerHTML/outerHTML (Closed) Base URL: https://chromium.googlesource.com/chromium/blink.git@master
Patch Set: Patch for landing Created 5 years, 9 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
« no previous file with comments | « LayoutTests/fast/innerHTML/javascript-url.html ('k') | Source/core/editing/MarkupAccumulator.cpp » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: LayoutTests/fast/innerHTML/javascript-url-expected.txt
diff --git a/LayoutTests/fast/innerHTML/javascript-url-expected.txt b/LayoutTests/fast/innerHTML/javascript-url-expected.txt
index 03769dcd511ffb3b2b8f91a3e470805778239be3..92225c0e8c3249d4191d387e24fabe1f29eb19e2 100644
--- a/LayoutTests/fast/innerHTML/javascript-url-expected.txt
+++ b/LayoutTests/fast/innerHTML/javascript-url-expected.txt
@@ -1,13 +1,12 @@
-Test that innerHTML does not mangle javascript: urls.
-r.innerHTML = r.innerHTML.replace(/&37;3C!--D--&37;3E/g, 123)
-PASS: r.innerHTML.indexOf('javascript:test(123)') > -1 should be true and is.
-r.firstChild.setAttribute('href', 'javascript:test("text<")')
-PASS: r.innerHTML.indexOf('javascript:test("text<")') > -1 should be true and is.
-r.firstChild.setAttribute("href", "javascript:test('text>')")
-PASS: r.innerHTML.indexOf("javascript:test('text>')") > -1 should be true and is.
-testString = javascript:test('text&',"test2&")
-r.firstChild.setAttribute("href", testString)
-PASS: r.innerHTML.indexOf("javascript:test('text&',&quot;test2&&quot;)") > 1 should be true and is.
-r.firstChild.setAttribute('href', 'http://www.google.fi/search?q=scarlett johansson&meta=&btnG=Google-haku')
-<a href="http://www.google.fi/search?q=scarlett johansson&amp;meta=&amp;btnG=Google-haku">link</a>
+Test that innerHTML/outerHTML does not mangle javascript: urls.
+PASS r.innerHTML is "<a href=\"javascript:test(123)\">link</a>"
+PASS r.innerHTML is "<a href='javascript:test(\"text<\")'>link</a>"
+PASS r.innerHTML is "<a href=\"javascript:test('text>')\">link</a>"
+PASS r.innerHTML is "<a href=\"javascript:test('text&amp;',&quot;test2&amp;&quot;)\">link</a>"
+PASS r.firstChild.outerHTML is "<a href=\"javascript:window.location='?x&amp;y'\">link</a>"
+PASS r.firstChild.outerHTML is "<a href=\"javascript:window.location='?x&amp;amp;y'\">link</a>"
+PASS r.firstChild.outerHTML is "<a href=\"javascript:window.location='?x&amp;y'\">link</a>"
+PASS successfullyParsed is true
+
+TEST COMPLETE
link
« no previous file with comments | « LayoutTests/fast/innerHTML/javascript-url.html ('k') | Source/core/editing/MarkupAccumulator.cpp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698