[inspector] add test configuration for the parser test.

[inspector] add test configuration for the parser test.

BUG=chromium:580337
R=machenbach@chromium.org

Committed: https://crrev.com/5d49286c696d78b42227732347ff5f23ed2fc83b
Cr-Commit-Position: refs/heads/master@{#38198}

[Michael Achenbach, 2016-07-28 13:03:18]

https://codereview.chromium.org/2189023002/diff/1/tools/run-tests.py
File tools/run-tests.py (right):

https://codereview.chromium.org/2189023002/diff/1/tools/run-tests.py#newcode64
tools/run-tests.py:64: # This needs to stay in sync with
test/bot_default.isolate.
You also need to reference the isolate and gyp files from
test/{bot_}?default.{isolate|gyp}

https://codereview.chromium.org/2189023002/diff/1/tools/run-tests.py#newcode74
tools/run-tests.py:74: "inspector_protocol_parser_test",
Note that these are somewhat sorted slow->fast. Please add it only in the end if
it's lightning fast.

https://codereview.chromium.org/2189023002/diff/1/tools/run-tests.py#newcode103
tools/run-tests.py:103: "inspector_protocol_parser_test",
Hmm, if they should be executed when only executing unittests then they might
need to be isolated with unittests as well. Maybe remove it here? I think this
is only called on a few PPC and x390 bots.

[Yang, 2016-07-28 13:36:44]

https://codereview.chromium.org/2189023002/diff/1/tools/run-tests.py
File tools/run-tests.py (right):

https://codereview.chromium.org/2189023002/diff/1/tools/run-tests.py#newcode64
tools/run-tests.py:64: # This needs to stay in sync with
test/bot_default.isolate.
On 2016/07/28 13:03:18, Michael Achenbach (slow) wrote:
> You also need to reference the isolate and gyp files from
> test/{bot_}?default.{isolate|gyp}

Done.

https://codereview.chromium.org/2189023002/diff/1/tools/run-tests.py#newcode74
tools/run-tests.py:74: "inspector_protocol_parser_test",
On 2016/07/28 13:03:18, Michael Achenbach (slow) wrote:
> Note that these are somewhat sorted slow->fast. Please add it only in the end
if
> it's lightning fast.

I think this is the case. Debug mode running both tests cases takes 0.15s

[Michael Achenbach, 2016-07-28 14:03:09]

LGTM. Would be worth looking at the trybot test step output and checking if the
test is actually running.

[Yang, 2016-07-28 14:05:56]

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

[commit-bot: I haz the power, 2016-07-28 14:06:02]

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-07-28 14:07:31]

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

[commit-bot: I haz the power, 2016-07-28 14:07:32]

Dry run: Try jobs failed on following builders:
  v8_linux64_asan_rel_ng on master.tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_linux64_asan_rel_ng/buil...)
  v8_linux64_gyp_rel_ng on master.tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_linux64_gyp_rel_ng/build...)
  v8_linux_arm_rel_ng on master.tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_linux_arm_rel_ng/builds/...)
  v8_linux_mips64el_compile_rel on master.tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_linux_mips64el_compile_r...)
  v8_linux_mipsel_compile_rel on master.tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_linux_mipsel_compile_rel...)

[Yang, 2016-08-01 09:17:07]

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

[commit-bot: I haz the power, 2016-08-01 09:17:14]

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-01 09:38:43]

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

[commit-bot: I haz the power, 2016-08-01 09:38:44]

Dry run: This issue passed the CQ dry run.

[Yang, 2016-08-01 09:44:52]

The CQ bit was checked by yangguo@chromium.org

[Yang, 2016-08-01 09:44:52]

The patchset sent to the CQ was uploaded after l-g-t-m from
machenbach@chromium.org
Link to the patchset: https://codereview.chromium.org/2189023002/#ps120001
(title: "update DEPS")

[commit-bot: I haz the power, 2016-08-01 09:44:54]

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-01 09:48:55]

Committed patchset #7 (id:120001)

[commit-bot: I haz the power, 2016-08-01 09:50:21]

Description was changed from

==========
[inspector] add test configuration for the parser test.


BUG=chromium:580337

R=machenbach@chromium.org
==========

to

==========
[inspector] add test configuration for the parser test.

BUG=chromium:580337

R=machenbach@chromium.org

Committed: https://crrev.com/5d49286c696d78b42227732347ff5f23ed2fc83b
Cr-Commit-Position: refs/heads/master@{#38198}
==========

[commit-bot: I haz the power, 2016-08-01 09:50:22]

Patchset 7 (id:??) landed as
https://crrev.com/5d49286c696d78b42227732347ff5f23ed2fc83b
Cr-Commit-Position: refs/heads/master@{#38198}