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

Unified Diff: Source/bindings/scripts/v8_attributes.py

Issue 313993002: Bindings: Add ScalarValueString support (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Created 6 years, 6 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: Source/bindings/scripts/v8_attributes.py
diff --git a/Source/bindings/scripts/v8_attributes.py b/Source/bindings/scripts/v8_attributes.py
index 46ea36fda60198eb405e250046fb420934db9bf8..fa872c0f8f16723458ceea9e7d1a6c039a68e9d3 100644
--- a/Source/bindings/scripts/v8_attributes.py
+++ b/Source/bindings/scripts/v8_attributes.py
@@ -108,7 +108,7 @@ def generate_attribute(interface, attribute):
'has_setter_exception_state':
is_setter_raises_exception or has_type_checking_interface or
has_type_checking_nullable or has_type_checking_unrestricted or
- idl_type.is_integer_type or idl_type.name == 'ByteString',
+ idl_type.is_integer_type or idl_type.name in ('ByteString', 'ScalarValueString'),
Nils Barth (inactive) 2014/06/11 03:54:25 Could you wrap this line?
jsbell 2014/06/12 17:45:55 Done.
'has_type_checking_interface': has_type_checking_interface,
'has_type_checking_nullable': has_type_checking_nullable,
'has_type_checking_unrestricted': has_type_checking_unrestricted,

Powered by Google App Engine
This is Rietveld 408576698