[heap] Do not wait for marking deque uncommit task on tear down.

[heap] Do not wait for marking deque uncommit task on tear down.

Cancelable task already guarantees proper waiting.

BUG=

Committed: https://crrev.com/e12f9a113afbcc0ce6efcff0a92e224b55b47d30
Cr-Commit-Position: refs/heads/master@{#40647}

[ulan, 2016-10-28 09:42:22]

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

[commit-bot: I haz the power, 2016-10-28 09:42:25]

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

[ulan, 2016-10-28 09:49:59]

ulan@chromium.org changed reviewers:
+ mlippautz@chromium.org

[ulan, 2016-10-28 09:49:59]

ptal

[Michael Lippautz, 2016-10-28 09:53:29]

lgtm

[ulan, 2016-10-28 09:54:34]

The CQ bit was checked by ulan@chromium.org

[commit-bot: I haz the power, 2016-10-28 09:54:44]

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-28 10:14:52]

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

[commit-bot: I haz the power, 2016-10-28 10:14:56]

Try jobs failed on following builders:
  v8_linux64_rel_ng on master.tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_linux64_rel_ng/builds/15368)
  v8_linux64_rel_ng_triggered on master.tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_linux64_rel_ng_triggered...)

[ulan, 2016-10-28 12:24:37]

The CQ bit was checked by ulan@chromium.org

[ulan, 2016-10-28 12:24:37]

The patchset sent to the CQ was uploaded after l-g-t-m from
mlippautz@chromium.org
Link to the patchset: https://codereview.chromium.org/2462553002/#ps40001
(title: "fix test")

[commit-bot: I haz the power, 2016-10-28 12:24:52]

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-28 12:46:37]

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

[commit-bot: I haz the power, 2016-10-28 12:46:38]

Try jobs failed on following builders:
  v8_linux64_avx2_rel_ng on master.tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_linux64_avx2_rel_ng/buil...)
  v8_linux64_avx2_rel_ng_triggered on master.tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_linux64_avx2_rel_ng_trig...)

[ulan, 2016-10-28 13:25:56]

The CQ bit was checked by ulan@chromium.org

[ulan, 2016-10-28 13:25:56]

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

[commit-bot: I haz the power, 2016-10-28 13:26:06]

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-28 13:52:22]

Committed patchset #4 (id:60001)

[Michael Achenbach, 2016-10-28 17:44:56]

Note also these flaky races:
https://build.chromium.org/p/client.v8/builders/V8%20Linux64%20TSAN/builds/12...
https://build.chromium.org/p/client.v8/builders/V8%20Linux64%20TSAN/builds/12...

May have stared before this CL.

[Michael Achenbach, 2016-10-28 18:45:42]

Yea, the problem seems much older. Found this similar flake:
https://build.chromium.org/p/client.v8/builders/V8%20Linux64%20TSAN/builds/12...

[Michael Achenbach, 2016-10-28 18:50:27]

Filed https://bugs.chromium.org/p/v8/issues/detail?id=5583

[commit-bot: I haz the power, 2016-11-17 22:16:51]

Description was changed from

==========
[heap] Do not wait for marking deque uncommit task on tear down.

Cancelable task already guarantees proper waiting.

BUG=
==========

to

==========
[heap] Do not wait for marking deque uncommit task on tear down.

Cancelable task already guarantees proper waiting.

BUG=

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

[commit-bot: I haz the power, 2016-11-17 22:16:52]

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