I'm submitting a CL for review.

https://chromium-review.googlesource.com/c/chromium/src/+/925363

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

commit ca30a03eee533bc7c31dd160ec3a8b94ac896a51
Author: Claudio DeSouza <claudiomdsjr@gmail.com>
Date: Thu Feb 22 14:12:38 2018

JSON schema compiler fix for std::string

When adding support for static properties on the compiler, the wrong
field was assigned for the value of properties using std::string This
led to all string properties having the value "std::string".

We are not only fixing that, but also amending it to use const char[]
globals, rather than std::string.

Bug:  813604 
Change-Id: Ie386cb925ce3e55436e606fb5e512d7c1798788f
Reviewed-on: https://chromium-review.googlesource.com/925363
Commit-Queue: Elly Fong-Jones <ellyjones@chromium.org>
Reviewed-by: Devlin <rdevlin.cronin@chromium.org>
Cr-Commit-Position: refs/heads/master@{#538425}
[modify] https://crrev.com/ca30a03eee533bc7c31dd160ec3a8b94ac896a51/tools/json_schema_compiler/cpp_type_generator.py
[modify] https://crrev.com/ca30a03eee533bc7c31dd160ec3a8b94ac896a51/tools/json_schema_compiler/idl_schema_test.py
[modify] https://crrev.com/ca30a03eee533bc7c31dd160ec3a8b94ac896a51/tools/json_schema_compiler/model.py
[modify] https://crrev.com/ca30a03eee533bc7c31dd160ec3a8b94ac896a51/tools/json_schema_compiler/test/BUILD.gn
[modify] https://crrev.com/ca30a03eee533bc7c31dd160ec3a8b94ac896a51/tools/json_schema_compiler/test/idl_properties.idl
[modify] https://crrev.com/ca30a03eee533bc7c31dd160ec3a8b94ac896a51/tools/json_schema_compiler/test/idl_schemas_unittest.cc

@claudiomdsjr/@ellyjones can we close this one?

Yes. This ticket can be closed now!

Closing per #5.