All graphs for this bug:
  https://chromeperf.appspot.com/group_report?bug_id=737372

Original alerts at time of bug-filing:
  https://chromeperf.appspot.com/group_report?keys=agxzfmNocm9tZXBlcmZyFAsSB0Fub21hbHkYgIDg7pzpvgoM,agxzfmNocm9tZXBlcmZyFAsSB0Fub21hbHkYgIDgrr_hqwoM,agxzfmNocm9tZXBlcmZyFAsSB0Fub21hbHkYgIDg7rb1qgkM


Bot(s) for this bug's original alert(s):

linux
mac

rockot: looks like https://chromium-review.googlesource.com/546957 caused a ~200kib binary size regression. expected?

Expected yes, but I do have some follow-up work to bring it back down
significantly.

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/7ccd5748af65f98edbb5fbd44e60b7e61f8cec3f

commit 7ccd5748af65f98edbb5fbd44e60b7e61f8cec3f
Author: Ken Rockot <rockot@chromium.org>
Date: Wed Jun 28 12:49:01 2017

Mojo C++ Bindings: Move bits of InterfacePtrState out-of-line

BUG= 737372 

Change-Id: I84b735900c0f065d80b927868696688b46ca7caa
Reviewed-on: https://chromium-review.googlesource.com/551086
Commit-Queue: Ken Rockot <rockot@chromium.org>
Reviewed-by: Yuzhu Shen <yzshen@chromium.org>
Cr-Commit-Position: refs/heads/master@{#482960}
[modify] https://crrev.com/7ccd5748af65f98edbb5fbd44e60b7e61f8cec3f/mojo/public/cpp/bindings/BUILD.gn
[add] https://crrev.com/7ccd5748af65f98edbb5fbd44e60b7e61f8cec3f/mojo/public/cpp/bindings/lib/interface_ptr_state.cc
[modify] https://crrev.com/7ccd5748af65f98edbb5fbd44e60b7e61f8cec3f/mojo/public/cpp/bindings/lib/interface_ptr_state.h

 Issue 737983  has been merged into this issue.

 Issue 738201  has been merged into this issue.

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/916a4ce665a069b167196957b1d38af1251f59a8

commit 916a4ce665a069b167196957b1d38af1251f59a8
Author: Ken Rockot <rockot@chromium.org>
Date: Thu Jun 29 23:29:09 2017

Mojo C++ Bindings: Shave off some more code size

Makes all SerializationContext state private and adds out-of-line
helper methods to manipulate it.

Also adds a base AssociatedInterfacePtrState class to force shared
code across template instantiations for a minor enhancement.

Overall decrease of Android binary size is ~60 kB.

BUG= 737372 

Change-Id: I17f26bef075faad70357cf90239e0960935553b0
Reviewed-on: https://chromium-review.googlesource.com/553429
Commit-Queue: Ken Rockot <rockot@chromium.org>
Reviewed-by: Yuzhu Shen <yzshen@chromium.org>
Cr-Commit-Position: refs/heads/master@{#483544}
[modify] https://crrev.com/916a4ce665a069b167196957b1d38af1251f59a8/mojo/public/cpp/bindings/BUILD.gn
[modify] https://crrev.com/916a4ce665a069b167196957b1d38af1251f59a8/mojo/public/cpp/bindings/lib/array_serialization.h
[add] https://crrev.com/916a4ce665a069b167196957b1d38af1251f59a8/mojo/public/cpp/bindings/lib/associated_interface_ptr_state.cc
[modify] https://crrev.com/916a4ce665a069b167196957b1d38af1251f59a8/mojo/public/cpp/bindings/lib/associated_interface_ptr_state.h
[modify] https://crrev.com/916a4ce665a069b167196957b1d38af1251f59a8/mojo/public/cpp/bindings/lib/handle_interface_serialization.h
[modify] https://crrev.com/916a4ce665a069b167196957b1d38af1251f59a8/mojo/public/cpp/bindings/lib/map_serialization.h
[modify] https://crrev.com/916a4ce665a069b167196957b1d38af1251f59a8/mojo/public/cpp/bindings/lib/serialization_context.cc
[modify] https://crrev.com/916a4ce665a069b167196957b1d38af1251f59a8/mojo/public/cpp/bindings/lib/serialization_context.h
[modify] https://crrev.com/916a4ce665a069b167196957b1d38af1251f59a8/mojo/public/cpp/bindings/lib/serialization_util.h
[modify] https://crrev.com/916a4ce665a069b167196957b1d38af1251f59a8/mojo/public/cpp/bindings/lib/string_serialization.h
[modify] https://crrev.com/916a4ce665a069b167196957b1d38af1251f59a8/mojo/public/cpp/bindings/tests/union_unittest.cc
[modify] https://crrev.com/916a4ce665a069b167196957b1d38af1251f59a8/mojo/public/tools/bindings/generators/cpp_templates/interface_definition.tmpl
[modify] https://crrev.com/916a4ce665a069b167196957b1d38af1251f59a8/mojo/public/tools/bindings/generators/cpp_templates/struct_macros.tmpl
[modify] https://crrev.com/916a4ce665a069b167196957b1d38af1251f59a8/mojo/public/tools/bindings/generators/cpp_templates/struct_serialization_declaration.tmpl
[modify] https://crrev.com/916a4ce665a069b167196957b1d38af1251f59a8/mojo/public/tools/bindings/generators/cpp_templates/union_serialization_declaration.tmpl

Between r482960 and r483544 (totaling a reduction of ~68k for Android binary)  the cost of the expected 80 kB increase is mostly paid down.

For other data, the Mac binary size increase was around 430k and these changes have reduced it by ~367k.

Calling this Fixed as I haven't found any other practical ways to cut down the code size further.