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

Unified Diff: third_party/WebKit/Source/bindings/scripts/v8_types.py

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/Source/bindings/scripts/v8_types.py
diff --git a/third_party/WebKit/Source/bindings/scripts/v8_types.py b/third_party/WebKit/Source/bindings/scripts/v8_types.py
index 87d6ef20a9fa11c86a90c7259505d9f72300f803..1fe8b5aced561215d284a302b4bec9f7b16c107f 100644
--- a/third_party/WebKit/Source/bindings/scripts/v8_types.py
+++ b/third_party/WebKit/Source/bindings/scripts/v8_types.py
@@ -472,7 +472,7 @@ def set_component_dirs(new_component_dirs):
V8_VALUE_TO_CPP_VALUE = {
# Basic
- 'Date': 'toCoreDate({isolate}, {v8_value})',
+ 'Date': 'toCoreDate({isolate}, {v8_value}, exceptionState)',
'DOMString': '{v8_value}',
'ByteString': 'toByteString({isolate}, {arguments})',
'USVString': 'toUSVString({isolate}, {arguments})',
@@ -506,7 +506,7 @@ def v8_conversion_needs_exception_state(idl_type):
return (idl_type.is_numeric_type or
idl_type.is_enum or
idl_type.is_dictionary or
- idl_type.name in ('Boolean', 'ByteString', 'Dictionary', 'USVString', 'SerializedScriptValue'))
+ idl_type.name in ('Boolean', 'ByteString', 'Date', 'Dictionary', 'USVString', 'SerializedScriptValue'))
IdlType.v8_conversion_needs_exception_state = property(v8_conversion_needs_exception_state)
IdlArrayOrSequenceType.v8_conversion_needs_exception_state = True

Powered by Google App Engine
This is Rietveld 408576698