[DevTools] Roll third_party/inspector_protocol to e23134c5aa6131e5a3a3afbefce255becce3a3bd.

[DevTools] Roll third_party/inspector_protocol to e23134c5aa6131e5a3a3afbefce255becce3a3bd.

This includes:
- [inspector_protocol] Allow custom json parser.
- [inspector_protocol] Allow overriding specific config values.
- [inspector_protocol] Fix NoneType error when parsing config_values.

BUG=635953
Committed: https://crrev.com/8bd45cc451e96d464522185ef1e946f094442f4b
Cr-Commit-Position: refs/heads/master@{#431123}

[dgozman, 2016-11-08 21:28:12]

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

[dgozman, 2016-11-08 21:28:13]

Could you please take a look?

[kozy, 2016-11-08 21:40:02]

lgtm

https://codereview.chromium.org/2490473004/diff/1/third_party/inspector_proto...
File third_party/inspector_protocol/CodeGenerator.py (right):

https://codereview.chromium.org/2490473004/diff/1/third_party/inspector_proto...
third_party/inspector_protocol/CodeGenerator.py:290: "create_type":
"protocol::Array<%s>::create()" % type["raw_type"],
Please just remove this.

[dgozman, 2016-11-08 22:04:29]

Description was changed from

==========
[DevTools] Allow custom json parser in inspector_protocol.

Drive-by: removed protocol platform, as it's not necessary anymore.

BUG=635953
==========

to

==========
[DevTools] Roll third_party/inspector_protocol to
b27ff7bff4770cf15b1187c10636222df1340ae2.

This includes:
- [inspector_protocol] Allow custom json parser.

BUG=635953
==========

[dgozman, 2016-11-08 22:05:33]

The CQ bit was checked by dgozman@chromium.org

[dgozman, 2016-11-08 22:05:34]

The patchset sent to the CQ was uploaded after l-g-t-m from
kozyatinskiy@chromium.org
Link to the patchset: https://codereview.chromium.org/2490473004/#ps20001
(title: "roll")

[commit-bot: I haz the power, 2016-11-08 22:06:05]

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-11-08 22:24:23]

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

[commit-bot: I haz the power, 2016-11-08 22:24:25]

Try jobs failed on following builders:
  chromeos_daisy_chromium_compile_only_ng on master.tryserver.chromium.linux
(JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromeos_daisy_...)

[dgozman, 2016-11-08 23:23:46]

Description was changed from

==========
[DevTools] Roll third_party/inspector_protocol to
b27ff7bff4770cf15b1187c10636222df1340ae2.

This includes:
- [inspector_protocol] Allow custom json parser.

BUG=635953
==========

to

==========
[DevTools] Roll third_party/inspector_protocol to
545a568e714053f8288543a97352f9c4f75e8417.

This includes:
- [inspector_protocol] Allow custom json parser.
- [inspector_protocol] Allow overriding specific config values.

BUG=635953
==========

[dgozman, 2016-11-08 23:30:25]

The CQ bit was checked by dgozman@chromium.org

[dgozman, 2016-11-08 23:30:26]

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

[commit-bot: I haz the power, 2016-11-08 23:31:08]

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-11-08 23:48:52]

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

[commit-bot: I haz the power, 2016-11-08 23:48:53]

Try jobs failed on following builders:
  chromeos_x86-generic_chromium_compile_only_ng on
master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromeos_x86-ge...)

[dgozman, 2016-11-09 01:12:11]

Description was changed from

==========
[DevTools] Roll third_party/inspector_protocol to
545a568e714053f8288543a97352f9c4f75e8417.

This includes:
- [inspector_protocol] Allow custom json parser.
- [inspector_protocol] Allow overriding specific config values.

BUG=635953
==========

to

==========
[DevTools] Roll third_party/inspector_protocol to
545a568e714053f8288543a97352f9c4f75e8417.

This includes:
- [inspector_protocol] Allow custom json parser.
- [inspector_protocol] Allow overriding specific config values.
- [inspector_protocol] Fix NoneType error when parsing config_values.

BUG=635953
==========

[dgozman, 2016-11-09 01:12:29]

Description was changed from

==========
[DevTools] Roll third_party/inspector_protocol to
545a568e714053f8288543a97352f9c4f75e8417.

This includes:
- [inspector_protocol] Allow custom json parser.
- [inspector_protocol] Allow overriding specific config values.
- [inspector_protocol] Fix NoneType error when parsing config_values.

BUG=635953
==========

to

==========
[DevTools] Roll third_party/inspector_protocol to
e23134c5aa6131e5a3a3afbefce255becce3a3bd.

This includes:
- [inspector_protocol] Allow custom json parser.
- [inspector_protocol] Allow overriding specific config values.
- [inspector_protocol] Fix NoneType error when parsing config_values.

BUG=635953
==========

[dgozman, 2016-11-09 01:12:48]

The CQ bit was checked by dgozman@chromium.org

[dgozman, 2016-11-09 01:12:48]

The patchset sent to the CQ was uploaded after l-g-t-m from
kozyatinskiy@chromium.org
Link to the patchset: https://codereview.chromium.org/2490473004/#ps80001
(title: "roll further")

[commit-bot: I haz the power, 2016-11-09 01:13:25]

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-11-09 01:29:42]

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

[commit-bot: I haz the power, 2016-11-09 01:29:43]

Try jobs failed on following builders:
  chromeos_amd64-generic_chromium_compile_only_ng on
master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromeos_amd64-...)
  chromeos_daisy_chromium_compile_only_ng on master.tryserver.chromium.linux
(JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromeos_daisy_...)
  linux_chromium_chromeos_rel_ng on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[dgozman, 2016-11-09 22:59:21]

The CQ bit was checked by dgozman@chromium.org

[dgozman, 2016-11-09 22:59:22]

The patchset sent to the CQ was uploaded after l-g-t-m from
kozyatinskiy@chromium.org
Link to the patchset: https://codereview.chromium.org/2490473004/#ps100001
(title: "ui_devtools")

[commit-bot: I haz the power, 2016-11-09 22:59:55]

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-11-10 01:06:40]

Description was changed from

==========
[DevTools] Roll third_party/inspector_protocol to
e23134c5aa6131e5a3a3afbefce255becce3a3bd.

This includes:
- [inspector_protocol] Allow custom json parser.
- [inspector_protocol] Allow overriding specific config values.
- [inspector_protocol] Fix NoneType error when parsing config_values.

BUG=635953
==========

to

==========
[DevTools] Roll third_party/inspector_protocol to
e23134c5aa6131e5a3a3afbefce255becce3a3bd.

This includes:
- [inspector_protocol] Allow custom json parser.
- [inspector_protocol] Allow overriding specific config values.
- [inspector_protocol] Fix NoneType error when parsing config_values.

BUG=635953
==========

[commit-bot: I haz the power, 2016-11-10 01:06:41]

Committed patchset #6 (id:100001)

[commit-bot: I haz the power, 2016-11-10 01:22:19]

Description was changed from

==========
[DevTools] Roll third_party/inspector_protocol to
e23134c5aa6131e5a3a3afbefce255becce3a3bd.

This includes:
- [inspector_protocol] Allow custom json parser.
- [inspector_protocol] Allow overriding specific config values.
- [inspector_protocol] Fix NoneType error when parsing config_values.

BUG=635953
==========

to

==========
[DevTools] Roll third_party/inspector_protocol to
e23134c5aa6131e5a3a3afbefce255becce3a3bd.

This includes:
- [inspector_protocol] Allow custom json parser.
- [inspector_protocol] Allow overriding specific config values.
- [inspector_protocol] Fix NoneType error when parsing config_values.

BUG=635953

Committed: https://crrev.com/8bd45cc451e96d464522185ef1e946f094442f4b
Cr-Commit-Position: refs/heads/master@{#431123}
==========

[commit-bot: I haz the power, 2016-11-10 01:22:19]

Patchset 6 (id:??) landed as
https://crrev.com/8bd45cc451e96d464522185ef1e946f094442f4b
Cr-Commit-Position: refs/heads/master@{#431123}