[Extensions] Remove the "use_movable_types" entry from idl/json files

[Extensions] Remove the "use_movable_types" entry from idl/json files

All APIs are converted to using movable types, and it is the default in code
generation since r384649. We can remove the entry in the json/idl files.

Also print an error with the filename if a file fails to parse as json.

BUG=595949
Committed: https://crrev.com/e7b32419fe1b86cea29cad5dc1d54787f3d0ee88
Cr-Commit-Position: refs/heads/master@{#384773}

[Devlin, 2016-04-01 23:33:50]

Description was changed from

==========
[Extensions] Remove the "use_movable_types" entry from idl/json files

All APIs are converted to using movable types, and it is now the default in
code generation. We can remove the entry in the json/idl files.

BUG=595949
==========

to

==========
[Extensions] Remove the "use_movable_types" entry from idl/json files

All APIs are converted to using movable types, and it is now the default in
code generation. We can remove the entry in the json/idl files.

Also print an error with the filename if a file fails to parse as json.

BUG=595949
==========

[Devlin, 2016-04-01 23:34:27]

rdevlin.cronin@chromium.org changed reviewers:
+ lazyboy@chromium.org

[Devlin, 2016-04-01 23:34:29]

Istiaque, mind taking a look?  Sorry for the size - with the exception of the
json_schema.py change, this just removing all the use_movable_types entries.

https://codereview.chromium.org/1848103005/diff/1/tools/json_schema_compiler/...
File tools/json_schema_compiler/json_schema.py (right):

https://codereview.chromium.org/1848103005/diff/1/tools/json_schema_compiler/...
tools/json_schema_compiler/json_schema.py:45: print('FAILED: Exception
encountered while loading "%s"' % filename)
drive-by: compile failures where I forgot to remove a trailing comma are much
easier to read when we know the filename. :)

[lazyboy, 2016-04-02 00:04:34]

lgtm.
nit: In CL description: "it is now the default" -> "it is default since r384649"

https://codereview.chromium.org/1848103005/diff/1/tools/json_schema_compiler/...
File tools/json_schema_compiler/json_schema.py (right):

https://codereview.chromium.org/1848103005/diff/1/tools/json_schema_compiler/...
tools/json_schema_compiler/json_schema.py:45: print('FAILED: Exception
encountered while loading "%s"' % filename)
On 2016/04/01 23:34:29, Devlin wrote:
> drive-by: compile failures where I forgot to remove a trailing comma are much
> easier to read when we know the filename. :)

Acknowledged.

[Devlin, 2016-04-02 00:11:43]

Description was changed from

==========
[Extensions] Remove the "use_movable_types" entry from idl/json files

All APIs are converted to using movable types, and it is now the default in
code generation. We can remove the entry in the json/idl files.

Also print an error with the filename if a file fails to parse as json.

BUG=595949
==========

to

==========
[Extensions] Remove the "use_movable_types" entry from idl/json files

All APIs are converted to using movable types, and it is the default in code
generation since r384649. We can remove the entry in the json/idl files.

Also print an error with the filename if a file fails to parse as json.

BUG=595949
==========

[Devlin, 2016-04-02 01:14:30]

The CQ bit was checked by rdevlin.cronin@chromium.org

[Devlin, 2016-04-02 01:14:44]

On 2016/04/02 00:04:34, lazyboy wrote:
> lgtm.
> nit: In CL description: "it is now the default" -> "it is default since
r384649"

Done.

[commit-bot: I haz the power, 2016-04-02 01:14:54]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1848103005/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1848103005/1

[commit-bot: I haz the power, 2016-04-02 01:31:14]

Description was changed from

==========
[Extensions] Remove the "use_movable_types" entry from idl/json files

All APIs are converted to using movable types, and it is the default in code
generation since r384649. We can remove the entry in the json/idl files.

Also print an error with the filename if a file fails to parse as json.

BUG=595949
==========

to

==========
[Extensions] Remove the "use_movable_types" entry from idl/json files

All APIs are converted to using movable types, and it is the default in code
generation since r384649. We can remove the entry in the json/idl files.

Also print an error with the filename if a file fails to parse as json.

BUG=595949
==========

[commit-bot: I haz the power, 2016-04-02 01:31:16]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-04-02 01:32:17]

Description was changed from

==========
[Extensions] Remove the "use_movable_types" entry from idl/json files

All APIs are converted to using movable types, and it is the default in code
generation since r384649. We can remove the entry in the json/idl files.

Also print an error with the filename if a file fails to parse as json.

BUG=595949
==========

to

==========
[Extensions] Remove the "use_movable_types" entry from idl/json files

All APIs are converted to using movable types, and it is the default in code
generation since r384649. We can remove the entry in the json/idl files.

Also print an error with the filename if a file fails to parse as json.

BUG=595949

Committed: https://crrev.com/e7b32419fe1b86cea29cad5dc1d54787f3d0ee88
Cr-Commit-Position: refs/heads/master@{#384773}
==========

[commit-bot: I haz the power, 2016-04-02 01:32:19]

Patchset 1 (id:??) landed as
https://crrev.com/e7b32419fe1b86cea29cad5dc1d54787f3d0ee88
Cr-Commit-Position: refs/heads/master@{#384773}