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

Unified Diff: LayoutTests/fast/images/exif-orientation-image-document.html

Issue 196523007: Revert of id of iframe incorrectly sets window name (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Created 6 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
Index: LayoutTests/fast/images/exif-orientation-image-document.html
diff --git a/LayoutTests/fast/images/exif-orientation-image-document.html b/LayoutTests/fast/images/exif-orientation-image-document.html
index 942869163302dd719a173cfce41dd5d1bc25a347..fbf896efac8512f5233a063ecc97883b4b39c376 100644
--- a/LayoutTests/fast/images/exif-orientation-image-document.html
+++ b/LayoutTests/fast/images/exif-orientation-image-document.html
@@ -13,8 +13,8 @@
console.appendChild(li);
}
-function imageSize(name) {
- var img = window.frames[name].document.querySelector('img');
+function imageSize(id) {
+ var img = window.frames[id].document.querySelector('img');
return [img.offsetWidth, img.offsetHeight];
}
@@ -41,17 +41,17 @@
</head>
<body onload="load()">
<b>The images should be rotated respecting their EXIF orientation. In image documents, this happens independent of WebKitShouldRespectImageOrientation.</b><br><br>
-<iframe name="img1" src="resources/exif-orientation-1-ul.jpg" frameborder=0></iframe>
-<iframe name="img2" src="resources/exif-orientation-2-ur.jpg" frameborder=0></iframe>
-<iframe name="img3" src="resources/exif-orientation-3-lr.jpg" frameborder=0></iframe>
-<iframe name="img4" src="resources/exif-orientation-4-lol.jpg" frameborder=0></iframe>
+<iframe id="img1" src="resources/exif-orientation-1-ul.jpg" frameborder=0></iframe>
+<iframe id="img2" src="resources/exif-orientation-2-ur.jpg" frameborder=0></iframe>
+<iframe id="img3" src="resources/exif-orientation-3-lr.jpg" frameborder=0></iframe>
+<iframe id="img4" src="resources/exif-orientation-4-lol.jpg" frameborder=0></iframe>
<br>
-<iframe name="img5" src="resources/exif-orientation-5-lu.jpg" frameborder=0></iframe>
-<iframe name="img6" src="resources/exif-orientation-6-ru.jpg" frameborder=0></iframe>
-<iframe name="img7" src="resources/exif-orientation-7-rl.jpg" frameborder=0></iframe>
-<iframe name="img8" src="resources/exif-orientation-8-llo.jpg" frameborder=0></iframe>
+<iframe id="img5" src="resources/exif-orientation-5-lu.jpg" frameborder=0></iframe>
+<iframe id="img6" src="resources/exif-orientation-6-ru.jpg" frameborder=0></iframe>
+<iframe id="img7" src="resources/exif-orientation-7-rl.jpg" frameborder=0></iframe>
+<iframe id="img8" src="resources/exif-orientation-8-llo.jpg" frameborder=0></iframe>
<br>
-<iframe name="img9" src="resources/exif-orientation-9-u.jpg" frameborder=0></iframe>
+<iframe id="img9" src="resources/exif-orientation-9-u.jpg" frameborder=0></iframe>
<br>
<ul id="console"></ul>
</body>

Powered by Google App Engine
This is Rietveld 408576698