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

Unified Diff: LayoutTests/fast/frames/frames-with-frameborder-zero-can-be-resized.html

Issue 427563002: Remove HTMLFrameElement.width/height (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: fix tests Created 6 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/fast/frames/frames-with-frameborder-zero-can-be-resized.html
diff --git a/LayoutTests/fast/frames/frames-with-frameborder-zero-can-be-resized.html b/LayoutTests/fast/frames/frames-with-frameborder-zero-can-be-resized.html
index b92da519726eee1ab00d624f442b26e86097e0ac..ba49b77300efbd58d2cf1e41b02e2def266dac01 100644
--- a/LayoutTests/fast/frames/frames-with-frameborder-zero-can-be-resized.html
+++ b/LayoutTests/fast/frames/frames-with-frameborder-zero-can-be-resized.html
@@ -10,9 +10,10 @@
if (success)
frame.document.write('<p>Success!</p>');
else {
+ var rect = frame.frameElement.getBoundingClientRect();
frame.document.write('<p>Resize until ');
frame.document.write(isWidth ? 'width:' : 'height:');
- frame.document.write(isWidth ? frame.frameElement.width : frame.frameElement.height);
+ frame.document.write(isWidth ? rect.width : rect.height);
frame.document.write(' = ' + size + '</p>');
}
}
@@ -50,8 +51,8 @@
}
function checkSuccess() {
- log(One, One.frameElement.width == 100, true, 100)
- log(Two, Two.frameElement.height == 80, false, 80)
+ log(One, One.frameElement.getBoundingClientRect().width == 100, true, 100)
+ log(Two, Two.frameElement.getBoundingClientRect().height == 80, false, 80)
}
window.onload = run;

Powered by Google App Engine
This is Rietveld 408576698