Remove dependence on .json files for IDL-specified extensions/apps APIs

Remove dependence on .json files for IDL-specified extensions/apps APIs

This is an incremental step towards fully IDL-specified API bindings. This
patch adds code which generates JSON based on IDL for the purpose of exposing
bindings to javascript.

BUG=119137
TEST=Code should compile and existing tests should pass.


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

[asargent_no_longer_on_chrome, 2012-03-20 21:42:28]

Ok, ready for review now. Note that I tried to prune what seemed like some
unneeded includes from json_schema_bundle_compile.gypi.

[miket_OOO, 2012-03-20 21:52:16]

lgtm

http://codereview.chromium.org/9774001/diff/4001/build/json_schema_bundle_com...
File build/json_schema_bundle_compile.gypi (right):

http://codereview.chromium.org/9774001/diff/4001/build/json_schema_bundle_com...
build/json_schema_bundle_compile.gypi:35:
'<(SHARED_INTERMEDIATE_DIR)/<(cc_dir)/generated_schemas.cc',
Nice!

http://codereview.chromium.org/9774001/diff/4001/chrome/chrome_common.gypi
File chrome/chrome_common.gypi (left):

http://codereview.chromium.org/9774001/diff/4001/chrome/chrome_common.gypi#ol...
chrome/chrome_common.gypi:263:
'<(SHARED_INTERMEDIATE_DIR)/chrome/common/extensions/api/generated_api.h'
Thank you.

http://codereview.chromium.org/9774001/diff/4001/chrome/common/extensions/api...
File chrome/common/extensions/api/extension_api.cc (left):

http://codereview.chromium.org/9774001/diff/4001/chrome/common/extensions/api...
chrome/common/extensions/api/extension_api.cc:113:
IDR_EXTENSION_API_JSON_EXPERIMENTAL_DNS,
Thanks (etc.)

http://codereview.chromium.org/9774001/diff/4001/tools/json_schema_compiler/s...
File tools/json_schema_compiler/schema_bundle_generator.py (right):

http://codereview.chromium.org/9774001/diff/4001/tools/json_schema_compiler/s...
tools/json_schema_compiler/schema_bundle_generator.py:11: SOURCE_BASE_PATH =
'chrome/common/extensions/api'
I'd like to parameterize this eventually... would you mind adding a TODO for me?
(If that doesn't make sense, ask me.)

http://codereview.chromium.org/9774001/diff/4001/tools/json_schema_compiler/s...
tools/json_schema_compiler/schema_bundle_generator.py:13: class
SchemaBundleGenerator(object):
Style: doc class purpose, please.

http://codereview.chromium.org/9774001/diff/4001/tools/json_schema_compiler/s...
tools/json_schema_compiler/schema_bundle_generator.py:86: 
Stray CR

[asargent_no_longer_on_chrome, 2012-03-20 23:34:07]

New version up that addresses comments. I also realized I had forgotten to
remove the h_bundle_generator.py file since I moved its code to
schema_bundle_generator.py, so I did that as well. 

CQ, here I come.

http://codereview.chromium.org/9774001/diff/4001/tools/json_schema_compiler/s...
File tools/json_schema_compiler/schema_bundle_generator.py (right):

http://codereview.chromium.org/9774001/diff/4001/tools/json_schema_compiler/s...
tools/json_schema_compiler/schema_bundle_generator.py:11: SOURCE_BASE_PATH =
'chrome/common/extensions/api'
On 2012/03/20 21:52:16, miket wrote:
> I'd like to parameterize this eventually... would you mind adding a TODO for
me?
> (If that doesn't make sense, ask me.)

Done.

http://codereview.chromium.org/9774001/diff/4001/tools/json_schema_compiler/s...
tools/json_schema_compiler/schema_bundle_generator.py:13: class
SchemaBundleGenerator(object):
On 2012/03/20 21:52:16, miket wrote:
> Style: doc class purpose, please.

Done.

http://codereview.chromium.org/9774001/diff/4001/tools/json_schema_compiler/s...
tools/json_schema_compiler/schema_bundle_generator.py:86: 
On 2012/03/20 21:52:16, miket wrote:
> Stray CR

Done.

[commit-bot: I haz the power, 2012-03-20 23:34:50]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/asargent@chromium.org/9774001/12001

[commit-bot: I haz the power, 2012-03-20 23:35:01]

Can't apply patch for file chrome/common/extensions/docs/samples.json.
While running patch -p1 --forward --force;
patching file chrome/common/extensions/docs/samples.json
Hunk #1 FAILED at 18.
Hunk #2 FAILED at 142.
2 out of 2 hunks FAILED -- saving rejects to file
chrome/common/extensions/docs/samples.json.rej

[commit-bot: I haz the power, 2012-03-20 23:43:03]

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

[commit-bot: I haz the power, 2012-03-21 00:44:02]

Try job failure for 9774001-12002 (retry) on mac_rel for step "check_deps"
(clobber build).
It's a second try, previously, step "compile" failed.
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=mac_rel&nu...

[commit-bot: I haz the power, 2012-03-21 04:00:15]

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

[commit-bot: I haz the power, 2012-03-21 05:40:19]

Change committed as 127908