Improve gfx::Transform StructTraits.

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

 {%- set class_name = struct.name ~ "_Data" -%}
 
 class {{class_name}} {
  public:
   static {{class_name}}* New(mojo::internal::Buffer* buf) {
     return new (buf->Allocate(sizeof({{class_name}}))) {{class_name}}();
   }
 
   static bool Validate(const void* data,
                        mojo::internal::ValidationContext* validation_context);
@@ skipping 30 matching lines @@
 {%- if num_fields > 0 %}
 {%-   set last_field = struct.packed.packed_fields[num_fields - 1] %}
 {%-   set offset = last_field.offset + last_field.size %}
 {%-   set pad = offset|get_pad(8) %}
 {%-   if pad > 0 %}
   uint8_t padfinal_[{{pad}}];
 {%-   endif %}
 {%- endif %}
 
  private:
-  {{class_name}}();
+  {{class_name}}() {

[Fady Samuel, 2016/08/03 23:44:05]

Does inlining the constructor help here? Can you add these things to the
initializer list?

[yzshen1, 2016/08/03 23:58:14]

It doesn't help much actually. I saw some New() calls (line 5, which calls this
constructor) showed up in profiling. So I made this change.

I have changed to use initializer list. Thanks!

+    header_.num_bytes = sizeof(*this);
+    header_.version = {{struct.versions[-1].version}};
+  }
   ~{{class_name}}() = delete;
 };
 static_assert(sizeof({{class_name}}) == {{struct.versions[-1].num_bytes}},
               "Bad sizeof({{class_name}})");