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

Unified Diff: third_party/WebKit/LayoutTests/imported/wpt/html/semantics/tabular-data/the-table-element/remove-row.html

Issue 2446483002: Import wpt@c5a14f553cba5f197743b9af605a84eddd8692a2 (Closed)
Patch Set: Created 4 years, 2 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: third_party/WebKit/LayoutTests/imported/wpt/html/semantics/tabular-data/the-table-element/remove-row.html
diff --git a/third_party/WebKit/LayoutTests/imported/wpt/html/semantics/tabular-data/the-table-element/remove-row.html b/third_party/WebKit/LayoutTests/imported/wpt/html/semantics/tabular-data/the-table-element/remove-row.html
index b0e529f91ec8b2d810bcbefbaaa89409d566e980..dc152292e2748f60edd953c82e83f61bc106339b 100644
--- a/third_party/WebKit/LayoutTests/imported/wpt/html/semantics/tabular-data/the-table-element/remove-row.html
+++ b/third_party/WebKit/LayoutTests/imported/wpt/html/semantics/tabular-data/the-table-element/remove-row.html
@@ -42,9 +42,23 @@ test(function() {
assert_equals(old_length, el.rows.length);
}, "check normal deleteRow");
test(function() {
- while (el.rows.length > 1) {
+ assert_equals(el.rows.length, 3);
+ do {
+ var old_length = el.rows.length;
el.deleteRow(-1);
- }
- assert_equals(1, el.rows.length);
+ assert_equals(el.rows.length, old_length - 1);
+ } while (el.rows.length);
}, "check normal deleteRow bis");
+
+test(function() {
+ assert_equals(el.rows.length, 0);
+ el.deleteRow(-1);
+}, 'deleteRow(-1) with no rows');
+
+test(function() {
+ assert_equals(el.rows.length, 0);
+ assert_throws("IndexSizeError", function() {
+ el.deleteRow(0);
+ });
+}, 'deleteRow(0) with no rows');
</script>

Powered by Google App Engine
This is Rietveld 408576698