Disable the CompilerDispatcher if we don't have idle time

Disable the CompilerDispatcher if we don't have idle time

Since we can't do all steps on background threads, we need idle time to
work

BUG=v8:5215
R=danno@chromium.org

Review-Url: https://codereview.chromium.org/2600743002
Cr-Commit-Position: refs/heads/master@{#41944}
Committed: https://chromium.googlesource.com/v8/v8/+/a0d9eb346bba90aa0b32a2d3184cbbfd6adb243e

[jochen (gone - plz use gerrit), 2016-12-23 14:41:31]

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

[commit-bot: I haz the power, 2016-12-23 14:41:39]

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

[danno, 2016-12-23 14:58:38]

lgtm

[jochen (gone - plz use gerrit), 2016-12-23 15:01:28]

The CQ bit was unchecked by jochen@chromium.org

[jochen (gone - plz use gerrit), 2016-12-23 15:01:30]

The CQ bit was checked by jochen@chromium.org

[commit-bot: I haz the power, 2016-12-23 15:01:39]

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-12-23 15:11:16]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1482505290000600, "parent_rev":
"b0a09d78095b49ce7d3551032a65f707456c1f27", "commit_rev":
"a0d9eb346bba90aa0b32a2d3184cbbfd6adb243e"}

[commit-bot: I haz the power, 2016-12-23 15:11:19]

Description was changed from

==========
Disable the CompilerDispatcher if we don't have idle time

Since we can't do all steps on background threads, we need idle time to
work

BUG=v8:5215
R=danno@chromium.org
==========

to

==========
Disable the CompilerDispatcher if we don't have idle time

Since we can't do all steps on background threads, we need idle time to
work

BUG=v8:5215
R=danno@chromium.org

Review-Url: https://codereview.chromium.org/2600743002
Cr-Commit-Position: refs/heads/master@{#41944}
Committed:
https://chromium.googlesource.com/v8/v8/+/a0d9eb346bba90aa0b32a2d3184cbbfd6ad...
==========

[commit-bot: I haz the power, 2016-12-23 15:11:20]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/v8/v8/+/a0d9eb346bba90aa0b32a2d3184cbbfd6ad...

[Michael Achenbach, 2016-12-23 19:03:30]

A revert of this CL (patchset #1 id:1) has been created in
https://codereview.chromium.org/2600773002/ by machenbach@chromium.org.

The reason for reverting is: [Sheriff] Speculative revert since we got
persistent timeouts on win32 debug:
https://build.chromium.org/p/client.v8/builders/V8%20Win32%20-%20debug/builds...

Please reland if it doesn't help..

[Michael Achenbach, 2016-12-23 21:29:58]

On 2016/12/23 19:03:30, Michael Achenbach (OOO) wrote:
> A revert of this CL (patchset #1 id:1) has been created in
> https://codereview.chromium.org/2600773002/ by mailto:machenbach@chromium.org.
> 
> The reason for reverting is: [Sheriff] Speculative revert since we got
> persistent timeouts on win32 debug:
>
https://build.chromium.org/p/client.v8/builders/V8%20Win32%20-%20debug/builds...
> 
> Please reland if it doesn't help..

Looks like the revert helped. In the output of the hanging shard the last
executed test is:
unittests/CompilerDispatcherTest.IdleTaskException: FAIL
But it also seems to hang on another test, but gets killed by the overall
timeout before we see which one.