bindings: Stop using Nullable<T> in dictionary impl

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

 {% filter format_blink_cpp_source_code %}
 
 {% include 'copyright_block.txt' %}
 #include "{{cpp_class}}.h"
 
 {% for filename in cpp_includes %}
 #include "{{filename}}"
 {% endfor %}
 
 namespace blink {
 
 {# Constructor #}
 {{cpp_class}}::{{cpp_class}}() {
   {% for member in members if member.cpp_default_value %}
   {{member.setter_name}}({{member.cpp_default_value}});
   {% endfor %}
 }
 
 {{cpp_class}}::~{{cpp_class}}() {}
 
+{% for member in members %}
+bool {{cpp_class}}::{{member.has_method_name}}() const {
+  return {{member.has_method_expression}};
+}
+{{member.rvalue_cpp_type}} {{cpp_class}}::{{member.cpp_name}}() const {
+  return {{member.getter_expression}};
+}
+void {{cpp_class}}::{{member.setter_name}}({{member.rvalue_cpp_type}} value) {
+  m_{{member.cpp_name}} = value;
+  {% if member.has_flag_variable %}
+  {{member.has_flag_variable}} = true;
+  {% endif %}
+}
+{% if member.null_setter_name %}
+void {{cpp_class}}::{{member.null_setter_name}}() {
+  {% if member.has_flag_variable %}
+  {{member.has_flag_variable}} = false;
+  {% else %}

[peria, 2016/11/11 07:45:16]

{%endif%} here.

otherwise, it keeps unused m_{{member.cpp_name}}.

[bashi, 2016/11/11 07:52:57]

IIUC the purpose of this method is make hasXXX() return false and in that case
accessing m_xxx is illegal anyway. Added DCHECK().

+  m_{{member.cpp_name}} = {{member.member_cpp_type}}();
+  {% endif %}
+}
+{% endif %}
+{% endfor %}
+
 DEFINE_TRACE({{cpp_class}}) {
   {% for member in members if member.is_traceable %}
   visitor->trace(m_{{member.cpp_name}});
   {% endfor %}
   {% if parent_cpp_class %}
   {{parent_cpp_class}}::trace(visitor);
   {% endif %}
 }
 
 }  // namespace blink
 
 {% endfilter %}{# format_blink_cpp_source_code #}