Chromium Code Reviews| Index: third_party/WebKit/LayoutTests/fast/dom/HTMLTableElement/exceptions.html |
| diff --git a/third_party/WebKit/LayoutTests/fast/dom/HTMLTableElement/exceptions.html b/third_party/WebKit/LayoutTests/fast/dom/HTMLTableElement/exceptions.html |
| index 38a24130ef91f5536802bfabb2129e965b6f04e8..93f6d4d9b5391e1bbdaf39dbffeabe8517914606 100644 |
| --- a/third_party/WebKit/LayoutTests/fast/dom/HTMLTableElement/exceptions.html |
| +++ b/third_party/WebKit/LayoutTests/fast/dom/HTMLTableElement/exceptions.html |
| @@ -9,15 +9,13 @@ |
| var t = document.createElement('table'); |
| - // TODO(foolip): Setting caption/tHead/tFoot to null should not throw |
|
tkent
2016/09/14 22:55:41
The comment is still valid for caption and tHead.
rwlbuis
2016/09/15 14:32:33
Done.
|
| - // an exception, it should just remove the old caption/thead/tfoot. |
| shouldThrow("t.caption = null"); |
| shouldThrow("t.caption = document.body"); |
| shouldThrow("t.tHead = null"); |
| shouldThrow("t.tHead = document.body"); |
| - shouldThrow("t.tFoot = null"); |
| + shouldNotThrow("t.tFoot = null"); |
|
tkent
2016/09/14 22:55:41
We should confirm that t.tFoot is removed from |t|
rwlbuis
2016/09/15 14:32:33
Done.
|
| shouldThrow("t.tFoot = document.body"); |
| shouldThrow("t.insertRow(-2)"); |