Support default values for POD struct fields in mojom IDL.

Support default values for POD struct fields in mojom IDL.

TODO: default values for arrays and structs.

BUG=320082
Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=246424

[darin (slow to review), 2014-01-21 23:47:20]

LGTM

https://codereview.chromium.org/138143003/diff/30001/mojo/public/bindings/gen...
File
mojo/public/bindings/generators/cpp_templates/struct_builder_definition.tmpl
(right):

https://codereview.chromium.org/138143003/diff/30001/mojo/public/bindings/gen...
mojo/public/bindings/generators/cpp_templates/struct_builder_definition.tmpl:26:
{{set_default(pf.field)|indent(2)}}
Perhaps we should defer setting default values until Finish() is called? That
way we don't waste memory writing default values for fields that the user sets?

Hmm, ah... but we don't keep track of what methods are called, so we can't know
if, for example, a null string field was intentional or not.

OK, nevermind.

[commit-bot: I haz the power, 2014-01-22 00:58:06]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/mpcomplete@chromium.org/138143003/30001

[commit-bot: I haz the power, 2014-01-22 02:55:32]

Retried try job too often on win_rel for step(s) app_list_unittests,
ash_unittests, aura_unittests, browser_tests, cacheinvalidation_unittests,
cc_unittests, check_deps, check_deps2git, chrome_elf_unittests,
chromedriver_unittests, components_unittests, compositor_unittests,
content_browsertests, content_unittests, crypto_unittests, events_unittests,
google_apis_unittests, gpu_unittests, installer_util_unittests, ipc_tests,
jingle_unittests, media_unittests, mini_installer_test, nacl_integration,
ppapi_unittests, printing_unittests, remoting_unittests, sql_unittests,
sync_integration_tests, sync_unit_tests, telemetry_unittests, views_unittests
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=win_rel&nu...

[commit-bot: I haz the power, 2014-01-22 19:53:21]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/mpcomplete@chromium.org/138143003/30001

[commit-bot: I haz the power, 2014-01-22 21:53:14]

Change committed as 246424