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

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

Issue 184633004: Revert 168553 "id of iframe incorrectly sets window name" (Closed) Base URL: svn://svn.chromium.org/blink/
Patch Set: Created 6 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
Download patch
Index: trunk/LayoutTests/fast/images/exif-orientation-image-document.html
===================================================================
--- trunk/LayoutTests/fast/images/exif-orientation-image-document.html (revision 168578)
+++ trunk/LayoutTests/fast/images/exif-orientation-image-document.html (working copy)
@@ -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