Make a dictionary that expresses a diff between two json files including AST.

bindings: Add generate_idl_diff.py

This script makes a dictionary that expresses a diff between two json files
including AST. The output includes annotations representing diff. See the
comment in the script for details.

BUG=530402
Committed: https://crrev.com/45d79a80177628745ec9a85736e0f5a39be7b18b
Cr-Commit-Position: refs/heads/master@{#350786}

[shimadaa, 2015-09-25 03:53:02]

shimadaa@google.com changed reviewers:
+ bashi@chromium.org, haraken@chromium.org, yukishiino@chromium.org

[bashi, 2015-09-25 03:53:24]

LGTM. We reviewed this CL on https://codereview.chromium.org/1329983002/

[shimadaa, 2015-09-25 03:54:42]

The CQ bit was checked by shimadaa@google.com

[commit-bot: I haz the power, 2015-09-25 03:55:14]

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

[Yuki, 2015-09-25 03:55:59]

lgtm

https://codereview.chromium.org/1363203003/diff/1/third_party/WebKit/Source/b...
File third_party/WebKit/Source/bindings/scripts/generate_idl_diff.py (right):

https://codereview.chromium.org/1363203003/diff/1/third_party/WebKit/Source/b...
third_party/WebKit/Source/bindings/scripts/generate_idl_diff.py:23:
{'InterfaceName': {
nit: Better to add 'Name' key in the example.

[haraken, 2015-09-25 03:58:29]

LGTM

Let's add more to the CL description so that others can follow what this CL is
doing.

[shimadaa, 2015-09-25 05:09:49]

The CQ bit was checked by shimadaa@google.com

[commit-bot: I haz the power, 2015-09-25 05:10:00]

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

[shimadaa, 2015-09-25 05:13:25]

Updated data structure's example. Thank you.

https://codereview.chromium.org/1363203003/diff/1/third_party/WebKit/Source/b...
File third_party/WebKit/Source/bindings/scripts/generate_idl_diff.py (right):

https://codereview.chromium.org/1363203003/diff/1/third_party/WebKit/Source/b...
third_party/WebKit/Source/bindings/scripts/generate_idl_diff.py:23:
{'InterfaceName': {
On 2015/09/25 03:55:59, Yuki wrote:
> nit: Better to add 'Name' key in the example.

Done.

[Yuki, 2015-09-25 05:16:41]

LGTM

As Haraken wrote, more detailed CL description is welcome.

https://codereview.chromium.org/1363203003/diff/20001/third_party/WebKit/Sour...
File third_party/WebKit/Source/bindings/scripts/generate_idl_diff.py (right):

https://codereview.chromium.org/1363203003/diff/20001/third_party/WebKit/Sour...
third_party/WebKit/Source/bindings/scripts/generate_idl_diff.py:172:
'<diff_file.jsson>\n')
s/jsson/json/

[bashi, 2015-09-25 06:08:44]

Could you update the description so that the first line only has one sentence?

---
bindings: Add generate_idl_diff.py

This script makes a dictionary that expresses a diff between two json files 
including AST. The output includes annotations representing diff. See the
comment in the script for details.

BUG=530402

[shimadaa, 2015-09-25 06:22:04]

The CQ bit was checked by shimadaa@google.com

[shimadaa, 2015-09-25 06:22:04]

The patchset sent to the CQ was uploaded after l-g-t-m from
haraken@chromium.org, bashi@chromium.org, yukishiino@chromium.org
Link to the patchset: https://codereview.chromium.org/1363203003/#ps40001
(title: "Modified a nit")

[commit-bot: I haz the power, 2015-09-25 06:22:07]

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

[commit-bot: I haz the power, 2015-09-25 08:18:11]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2015-09-25 08:19:02]

Patchset 3 (id:??) landed as
https://crrev.com/45d79a80177628745ec9a85736e0f5a39be7b18b
Cr-Commit-Position: refs/heads/master@{#350786}