Allow comments in extension config files.

Allow comments in extension config files.

Added a script to remove comments from the extension api JSON files before
processing for the extension docs. We will now be able to comment in the JSON
files, and they should be much easier to read. Also added the license header
to all the JSON files.

BUG=114233
TEST=Put comments in one of the JSON files and remake the docs. They will make
with no problems.


Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=124660

Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=124878

[cduvall, 2012-02-29 02:16:20]

CL for change to allow comments in JSON files. Thanks for the review.

[Aaron Boodman, 2012-02-29 06:58:11]

Sweet!

I think you will also need to change some files in tools/json_schema_compiler/
(just look for "loads").

[cduvall, 2012-03-02 01:19:24]

On 2012/02/29 06:58:11, Aaron Boodman wrote:
> Sweet!
> 
> I think you will also need to change some files in tools/json_schema_compiler/
> (just look for "loads").

Done. I'm pretty sure that's all the files that need to be changed, but if you
know of any more just let me know.

[Aaron Boodman, 2012-03-02 01:24:49]

LGTM, but I'd like kalman to double-check the changes to the schema compiler.

[not at google - send to devlin, 2012-03-02 02:15:08]

https://chromiumcodereview.appspot.com/9447090/diff/7001/tools/json_schema_co...
File tools/json_schema_compiler/compiler.py (right):

https://chromiumcodereview.appspot.com/9447090/diff/7001/tools/json_schema_co...
tools/json_schema_compiler/compiler.py:31: '/../../third_party/'
Won't this cause problems on Windows? os.path.join.

also you may need to use os.pardir instead of '..'

https://chromiumcodereview.appspot.com/9447090/diff/7001/tools/json_schema_co...
File tools/json_schema_compiler/cpp_type_generator_test.py (right):

https://chromiumcodereview.appspot.com/9447090/diff/7001/tools/json_schema_co...
tools/json_schema_compiler/cpp_type_generator_test.py:15: '/../../third_party/'
ditto

it would probably be nicer to pull all of this logic into a module ('json' or
'json_schema'?) that exposes a LoadJSON(filename) method and manages the
inclusion of the library, the minification, the reading, etc itself.

https://chromiumcodereview.appspot.com/9447090/diff/7001/tools/json_schema_co...
File tools/json_schema_compiler/model_test.py (right):

https://chromiumcodereview.appspot.com/9447090/diff/7001/tools/json_schema_co...
tools/json_schema_compiler/model_test.py:17: import json_minify as minify
yeah, definitely nicer to have this all packaged away in a module.

[not at google - send to devlin, 2012-03-02 02:18:35]

By the way, I've frequently wanted comments in those schema files, so this is
awesome.

[cduvall, 2012-03-02 02:53:25]

Ok, I put all the logic in json_schema.py so we can just call
LoadJSON(filename), and I fixed up the third_party path inclusion.

[not at google - send to devlin, 2012-03-02 03:01:28]

lgtm

[commit-bot: I haz the power, 2012-03-02 03:15:33]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/cduvall@chromium.org/9447090/12002

[commit-bot: I haz the power, 2012-03-02 16:09:18]

Change committed as 124660

[commit-bot: I haz the power, 2012-03-03 19:38:07]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/cduvall@chromium.org/9447090/20001

[cduvall, 2012-03-03 19:38:49]

Had to recommit due to small license header problem.

[commit-bot: I haz the power, 2012-03-03 21:20:17]

Change committed as 124878