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

Unified Diff: content/test/data/click-noreferrer-links.html

Issue 1378403002: Move RFHM browser tests to the embedded test server. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 5 years, 3 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 | « content/browser/frame_host/render_frame_host_manager_browsertest.cc ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: content/test/data/click-noreferrer-links.html
diff --git a/content/test/data/click-noreferrer-links.html b/content/test/data/click-noreferrer-links.html
index ee011b7008e6ba491fcdffb09feb02a8ce458001..663ba259f11710ab5ffb3e4481312c9783853027 100644
--- a/content/test/data/click-noreferrer-links.html
+++ b/content/test/data/click-noreferrer-links.html
@@ -2,6 +2,13 @@
<head><title>Click noreferrer links</title>
<script>
+ function setBaseURL(baseUrl) {
Charlie Reis 2015/10/01 17:43:46 I'm torn on this name. It sounds like it's about
nasko 2015/10/01 18:10:18 Done.
+ var links = ['noref_and_tblank_link', 'tblank_link', 'noref_link'];
+ links.forEach(function(linkId) {
+ link = document.getElementById(linkId);
+ link.href = baseUrl + link.pathname.substr(1);
+ });
+ }
function simulateClick(target) {
var evt = document.createEvent("MouseEvents");
evt.initMouseEvent("click", true, true, window,
@@ -96,7 +103,7 @@
</script>
</head>
-<a href="http://REPLACE_WITH_HOST_AND_PORT/files/title2.html"
+<a href="http://REPLACE_WITH_HOST_AND_PORT/title2.html"
id="noref_and_tblank_link" rel="noreferrer" target="_blank">
rel=noreferrer and target=_blank</a><br>
<a href="title2.html" id="samesite_noref_and_targeted_link"
@@ -106,9 +113,9 @@
same-site target=foo</a><br>
<a href="title2.html" id="samesite_tblank_link" target="_blank">
same-site target=_blank</a><br>
-<a href="http://REPLACE_WITH_HOST_AND_PORT/files/title2.html" id="tblank_link"
+<a href="http://REPLACE_WITH_HOST_AND_PORT/title2.html" id="tblank_link"
target="_blank">target=_blank</a><br>
-<a href="http://REPLACE_WITH_HOST_AND_PORT/files/title2.html" id="noref_link"
+<a href="http://REPLACE_WITH_HOST_AND_PORT/title2.html" id="noref_link"
Charlie Reis 2015/10/01 17:43:46 Why keep the REPLACE_WITH_HOST_AND_PORT part if we
nasko 2015/10/01 18:10:18 It helps visually see which links are expected to
rel="noreferrer">rel=noreferrer</a><br>
<a href="about:blank" id="blank_targeted_link" target="foo">
blank_targeted_link=foo</a><br>
« no previous file with comments | « content/browser/frame_host/render_frame_host_manager_browsertest.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698