Move the setting of bindings code generator up the stack.

Move the setting of bindings code generator up the stack.

This opens the path for cleanly switching different types of
code generators. Also eliminates the need for IdlCompilerV8 and
IdlCompilerDictionaryImpl.

R=bashi,yukishiino
BUG=654129
Committed: https://crrev.com/28e4cd6cdb271055f445717508ebea7595fa4676
Cr-Commit-Position: refs/heads/master@{#424070}

[dglazkov, 2016-10-08 00:05:45]

The CQ bit was checked by dglazkov@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-10-08 00:06:05]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-10-08 01:58:08]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-10-08 01:58:09]

Dry run: This issue passed the CQ dry run.

[dglazkov, 2016-10-08 04:07:32]

PTAL.

[haraken, 2016-10-08 04:41:59]

LGTM

[dglazkov, 2016-10-08 04:45:34]

The CQ bit was checked by dglazkov@chromium.org

[commit-bot: I haz the power, 2016-10-08 04:45:43]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-10-08 04:51:19]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-10-08 04:52:58]

Description was changed from

==========
Move the setting of bindings code generator up the stack.

This opens the path for cleanly switching different types of
code generators. Also eliminates the need for IdlCompilerV8 and
IdlCompilerDictionaryImpl.

R=bashi,yukishiino
BUG=654129
==========

to

==========
Move the setting of bindings code generator up the stack.

This opens the path for cleanly switching different types of
code generators. Also eliminates the need for IdlCompilerV8 and
IdlCompilerDictionaryImpl.

R=bashi,yukishiino
BUG=654129

Committed: https://crrev.com/28e4cd6cdb271055f445717508ebea7595fa4676
Cr-Commit-Position: refs/heads/master@{#424070}
==========

[commit-bot: I haz the power, 2016-10-08 04:52:59]

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

[bashi, 2016-10-10 23:06:05]

bashi@chromium.org changed reviewers:
+ bashi@chromium.org

[bashi, 2016-10-10 23:06:06]

lgtm

[Yuki, 2016-11-01 13:09:48]

Description was changed from

==========
Move the setting of bindings code generator up the stack.

This opens the path for cleanly switching different types of
code generators. Also eliminates the need for IdlCompilerV8 and
IdlCompilerDictionaryImpl.

R=bashi,yukishiino
BUG=654129

Committed: https://crrev.com/28e4cd6cdb271055f445717508ebea7595fa4676
Cr-Commit-Position: refs/heads/master@{#424070}
==========

to

==========
Move the setting of bindings code generator up the stack.

This opens the path for cleanly switching different types of
code generators. Also eliminates the need for IdlCompilerV8 and
IdlCompilerDictionaryImpl.

R=bashi,yukishiino
BUG=654129

Committed: https://crrev.com/28e4cd6cdb271055f445717508ebea7595fa4676
Cr-Commit-Position: refs/heads/master@{#424070}
==========

[Yuki, 2016-11-01 13:09:48]

yukishiino@chromium.org changed reviewers:
+ yukishiino@chromium.org
- bashi@google.com, yukishiino@chromium.or