bindings: Throw a type error if converting an interface or callback function fails.

third_party/WebKit/Source/bindings/templates/callback_function.cpp.tmpl

 {% from 'utilities.cpp.tmpl' import v8_value_to_local_cpp_value %}
 {% filter format_blink_cpp_source_code %}
 
 {% include 'copyright_block.txt' %}
 
 #include "{{cpp_class}}.h"
 
 {% for filename in cpp_includes %}
 #include "{{filename}}"
 {% endfor %}
@@ skipping 63 matching lines @@
   }
 
   {% if return_value %}
   {{v8_value_to_local_cpp_value(return_value) | indent(2)}}
   returnValue = cppValue;
   {% endif %}
   return true;
 }
 
 {{cpp_class}}* NativeValueTraits<{{cpp_class}}>::nativeValue(v8::Isolate* isolate, v8::Local<v8::Value> value, ExceptionState& exceptionState) {
-  return {{cpp_class}}::create(ScriptState::current(isolate), value);
+  {{cpp_class}}* nativeValue = {{cpp_class}}::create(ScriptState::current(isolate), value);
+  if (!nativeValue)
+    exceptionState.throwTypeError("Unable to convert value to {{cpp_class}}.");

[Yuki, 2017/04/07 13:37:21]

nit: Can we use {{idl_type}} instead?
I'm not 100% sure if {{idl_type}} is appropriate, but my point is that we should
use an IDL name rather than an internal C++ class' name because this is a
user-visible message.

+  return nativeValue;
 }
 
 }  // namespace blink
 
 {% endfilter %}{# format_blink_cpp_source_code #}