Add field-initializing constructors to generated mojo structs.

mojo/public/tools/bindings/generators/cpp_templates/wrapper_class_template_definition.tmpl

 template <typename StructPtrType>
 {{struct.name}}Ptr {{struct.name}}::Clone() const {
   // Use StructPtrType to prevent the compiler from trying to compile this
   // without being asked.
-  StructPtrType rv(New());
+  return StructPtrType::Struct::New(

[yzshen1, 2017/02/14 17:21:53]

This could be
  "return New("

Right?

[Sam McNally, 2017/02/15 06:35:49]

Done.

 {%- for field in struct.fields %}
-  rv->{{field.name}} = mojo::internal::Clone({{field.name}});
+      mojo::internal::Clone({{field.name}})
+{%-   if not loop.last -%},{%- endif %}
 {%- endfor %}
-  return rv;
+  );
 }
 
 template <typename T,
           typename std::enable_if<std::is_same<
               T, {{struct.name}}>::value>::type*>
 bool {{struct.name}}::Equals(const T& other) const {
 {%- for field in struct.fields %}
   if (!mojo::internal::Equals(this->{{field.name}}, other.{{field.name}}))
     return false;
 {%- endfor %}
   return true;
 }