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

Unified Diff: third_party/WebKit/LayoutTests/fast/forms/time/time-valueasdate.html

Issue 2265443002: Fix an overflow in valueAsDate setter of temporal input types. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: accept null Created 4 years, 4 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/fast/forms/time/time-valueasdate.html
diff --git a/third_party/WebKit/LayoutTests/fast/forms/time/time-valueasdate.html b/third_party/WebKit/LayoutTests/fast/forms/time/time-valueasdate.html
index 3407e31efa8e7de161133890c0414a1871d24fc0..34b42f5f96773b17ba9531cf97018b041b510ef9 100644
--- a/third_party/WebKit/LayoutTests/fast/forms/time/time-valueasdate.html
+++ b/third_party/WebKit/LayoutTests/fast/forms/time/time-valueasdate.html
@@ -43,7 +43,7 @@ debug('Invalid Date:');
// Date of JavaScript can't represent 8.65E15.
shouldBe('var date = new Date(); date.setTime(8.65E15); input.valueAsDate = date; input.value', '""');
debug('Invalid objects:');
-shouldBe('input.value = "00:00"; input.valueAsDate = document; input.value', '""');
+shouldThrow('input.valueAsDate = document');
shouldBe('input.value = "00:00"; input.valueAsDate = null; input.value', '""');
debug('Step attribute value and string representation:');

Powered by Google App Engine
This is Rietveld 408576698