[DevTools] Prepare inspector_protocol build to move.

[DevTools] Prepare inspector_protocol build to move.

- Do not list inspector_protocol files anywhere outside;
- Separate concatenation from compatibility check;
- Improve files layout;
- Pass path to jinja module to CodeGenerator.

BUG=637032
Committed: https://crrev.com/a73483cd5dd6d569d079e7ea43210f8482d12552
Cr-Commit-Position: refs/heads/master@{#415137}

[dgozman, 2016-08-27 01:23:54]

Description was changed from

==========
[DevTools] Prepare inspector_protocol build to move.

- Do not list inspector_protocol files anywhere outside;
- Separate concatenation from compatibility check;
- Improve files layout.

BUG=637032
==========

to

==========
[DevTools] Prepare inspector_protocol build to move.

- Do not list inspector_protocol files anywhere outside;
- Separate concatenation from compatibility check;
- Improve files layout;
- Pass path to jinja module to CodeGenerator.

BUG=637032
==========

[dgozman, 2016-08-27 01:24:07]

dgozman@chromium.org changed reviewers:
+ pfeldman@chromium.org

[dgozman, 2016-08-27 01:24:07]

Take a look please.

[pfeldman, 2016-08-27 01:45:42]

lgtm 2

[dgozman, 2016-08-28 18:57:36]

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

[commit-bot: I haz the power, 2016-08-28 18:57:41]

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-08-28 19:41:22]

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

[commit-bot: I haz the power, 2016-08-28 19:41:23]

Dry run: Try jobs failed on following builders:
  linux_android_rel_ng on master.tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/linux_androi...)

[dgozman, 2016-08-29 02:47:07]

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

[commit-bot: I haz the power, 2016-08-29 02:47:18]

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-08-29 04:11:56]

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

[commit-bot: I haz the power, 2016-08-29 04:11:57]

Dry run: Try jobs failed on following builders:
  linux_android_rel_ng on master.tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/linux_androi...)

[dgozman, 2016-08-29 05:30:42]

The CQ bit was checked by dgozman@chromium.org

[dgozman, 2016-08-29 05:30:43]

The patchset sent to the CQ was uploaded after l-g-t-m from
pfeldman@chromium.org
Link to the patchset: https://codereview.chromium.org/2282283002/#ps40001
(title: "win compile")

[commit-bot: I haz the power, 2016-08-29 05:31:14]

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-08-29 07:12:35]

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

[commit-bot: I haz the power, 2016-08-29 07:12:37]

Try jobs failed on following builders:
  linux_android_rel_ng on master.tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/linux_androi...)

[dgozman, 2016-08-29 13:55:52]

The CQ bit was checked by dgozman@chromium.org

[commit-bot: I haz the power, 2016-08-29 13:56:22]

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-08-29 14:33:11]

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

[commit-bot: I haz the power, 2016-08-29 14:33:13]

Try jobs failed on following builders:
  linux_android_rel_ng on master.tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/linux_androi...)

[dgozman, 2016-08-29 16:39:55]

The CQ bit was checked by dgozman@chromium.org

[commit-bot: I haz the power, 2016-08-29 16:45:11]

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

[dgozman, 2016-08-29 20:16:50]

The CQ bit was unchecked by dgozman@chromium.org

[dgozman, 2016-08-29 20:16:54]

The CQ bit was checked by dgozman@chromium.org

[commit-bot: I haz the power, 2016-08-29 20:49:13]

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

[dgozman, 2016-08-29 20:55:31]

The CQ bit was unchecked by dgozman@chromium.org

[dgozman, 2016-08-29 20:55:42]

Description was changed from

==========
[DevTools] Prepare inspector_protocol build to move.

- Do not list inspector_protocol files anywhere outside;
- Separate concatenation from compatibility check;
- Improve files layout;
- Pass path to jinja module to CodeGenerator.

BUG=637032
==========

to

==========
[DevTools] Prepare inspector_protocol build to move.

- Do not list inspector_protocol files anywhere outside;
- Separate concatenation from compatibility check;
- Improve files layout;
- Pass path to jinja module to CodeGenerator.

BUG=637032
NOTRY=true
==========

[dgozman, 2016-08-29 20:55:48]

The CQ bit was checked by dgozman@chromium.org

[commit-bot: I haz the power, 2016-08-29 21:21:21]

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

[dgozman, 2016-08-29 22:57:26]

The CQ bit was unchecked by dgozman@chromium.org

[dgozman, 2016-08-29 22:57:36]

Description was changed from

==========
[DevTools] Prepare inspector_protocol build to move.

- Do not list inspector_protocol files anywhere outside;
- Separate concatenation from compatibility check;
- Improve files layout;
- Pass path to jinja module to CodeGenerator.

BUG=637032
NOTRY=true
==========

to

==========
[DevTools] Prepare inspector_protocol build to move.

- Do not list inspector_protocol files anywhere outside;
- Separate concatenation from compatibility check;
- Improve files layout;
- Pass path to jinja module to CodeGenerator.

BUG=637032
==========

[dgozman, 2016-08-29 23:05:30]

The CQ bit was checked by dgozman@chromium.org

[dgozman, 2016-08-29 23:05:31]

The patchset sent to the CQ was uploaded after l-g-t-m from
pfeldman@chromium.org
Link to the patchset: https://codereview.chromium.org/2282283002/#ps60001
(title: "NOTREACHED")

[commit-bot: I haz the power, 2016-08-29 23:55:12]

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-08-30 05:09:24]

Description was changed from

==========
[DevTools] Prepare inspector_protocol build to move.

- Do not list inspector_protocol files anywhere outside;
- Separate concatenation from compatibility check;
- Improve files layout;
- Pass path to jinja module to CodeGenerator.

BUG=637032
==========

to

==========
[DevTools] Prepare inspector_protocol build to move.

- Do not list inspector_protocol files anywhere outside;
- Separate concatenation from compatibility check;
- Improve files layout;
- Pass path to jinja module to CodeGenerator.

BUG=637032
==========

[commit-bot: I haz the power, 2016-08-30 05:09:26]

Committed patchset #4 (id:60001)

[commit-bot: I haz the power, 2016-08-30 05:12:00]

Description was changed from

==========
[DevTools] Prepare inspector_protocol build to move.

- Do not list inspector_protocol files anywhere outside;
- Separate concatenation from compatibility check;
- Improve files layout;
- Pass path to jinja module to CodeGenerator.

BUG=637032
==========

to

==========
[DevTools] Prepare inspector_protocol build to move.

- Do not list inspector_protocol files anywhere outside;
- Separate concatenation from compatibility check;
- Improve files layout;
- Pass path to jinja module to CodeGenerator.

BUG=637032

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

[commit-bot: I haz the power, 2016-08-30 05:12:01]

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