Revert of Fix async/await memory leak

Revert of Fix async/await memory leak (patchset #5 id:80001 of https://codereview.chromium.org/2334323006/ )

Reason for revert:
newly introduced test async-await-loop times out: https://build.chromium.org/p/client.v8/builders/V8%20Linux%20-%20arm64%20-%20sim%20-%20MSAN/builds/10894/steps/Ignition%20-%20turbofan%20%28flakes%29/logs/async-await-loop

Original issue's description:
> Fix async/await memory leak
>
> This patch closes a memory leak in async/await where the desugaring
> was creating a situation analagous to that described in v8:5002.
> Intermediate Promises were being kept alive, so a long-running loop
> would cause linear memory usage on the heap. This patch returns
> undefined to the 'then' callback passed into PerformPromiseThen
> in order to avoid this hazard. Test expectations are fixed to remove
> expecting extraneous events which occurred on Promises that are
> now not given unnecessarily complex resolution paths before being
> thrown away.
>
> BUG=v8:5390
>
> Committed: https://crrev.com/a0ba18e9634c5e2d439033ab61a77cff54f9af35
> Cr-Commit-Position: refs/heads/master@{#39479}

TBR=adamk@chromium.org,caitp@igalia.com,littledan@chromium.org
NOTRY=true
BUG=v8:5390
Committed: https://crrev.com/196db1999da130019bbf8e3bd65977f840e8afaf
Cr-Commit-Position: refs/heads/master@{#39493}

[Michael Hablich, 2016-09-19 07:38:20]

The CQ bit was checked by hablich@chromium.org

[Michael Hablich, 2016-09-19 07:38:20]

Created Revert of Fix async/await memory leak

[commit-bot: I haz the power, 2016-09-19 07:38:26]

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-09-19 07:45:36]

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

[commit-bot: I haz the power, 2016-09-19 07:45:38]

Try jobs failed on following builders:
  v8_linux_chromium_gn_rel on master.tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_linux_chromium_gn_rel/bu...)

[Michael Hablich, 2016-09-19 07:48:50]

The CQ bit was checked by hablich@chromium.org

[commit-bot: I haz the power, 2016-09-19 07:48:56]

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-09-19 07:55:46]

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

[commit-bot: I haz the power, 2016-09-19 07:55:48]

Try jobs failed on following builders:
  v8_linux_chromium_gn_rel on master.tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_linux_chromium_gn_rel/bu...)

[Michael Hablich, 2016-09-19 08:09:10]

Description was changed from

==========
Revert of Fix async/await memory leak (patchset #5 id:80001 of
https://codereview.chromium.org/2334323006/ )

Reason for revert:
newly introduced test async-await-loop times out:
https://build.chromium.org/p/client.v8/builders/V8%20Linux%20-%20arm64%20-%20...

Original issue's description:
> Fix async/await memory leak
> 
> This patch closes a memory leak in async/await where the desugaring
> was creating a situation analagous to that described in v8:5002.
> Intermediate Promises were being kept alive, so a long-running loop
> would cause linear memory usage on the heap. This patch returns
> undefined to the 'then' callback passed into PerformPromiseThen
> in order to avoid this hazard. Test expectations are fixed to remove
> expecting extraneous events which occurred on Promises that are
> now not given unnecessarily complex resolution paths before being
> thrown away.
> 
> BUG=v8:5390
> 
> Committed: https://crrev.com/a0ba18e9634c5e2d439033ab61a77cff54f9af35
> Cr-Commit-Position: refs/heads/master@{#39479}

TBR=adamk@chromium.org,caitp@igalia.com,littledan@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=v8:5390
==========

to

==========
Revert of Fix async/await memory leak (patchset #5 id:80001 of
https://codereview.chromium.org/2334323006/ )

Reason for revert:
newly introduced test async-await-loop times out:
https://build.chromium.org/p/client.v8/builders/V8%20Linux%20-%20arm64%20-%20...

Original issue's description:
> Fix async/await memory leak
> 
> This patch closes a memory leak in async/await where the desugaring
> was creating a situation analagous to that described in v8:5002.
> Intermediate Promises were being kept alive, so a long-running loop
> would cause linear memory usage on the heap. This patch returns
> undefined to the 'then' callback passed into PerformPromiseThen
> in order to avoid this hazard. Test expectations are fixed to remove
> expecting extraneous events which occurred on Promises that are
> now not given unnecessarily complex resolution paths before being
> thrown away.
> 
> BUG=v8:5390
> 
> Committed: https://crrev.com/a0ba18e9634c5e2d439033ab61a77cff54f9af35
> Cr-Commit-Position: refs/heads/master@{#39479}

TBR=adamk@chromium.org,caitp@igalia.com,littledan@chromium.org
NOTRY=true
BUG=v8:5390
==========

[Michael Hablich, 2016-09-19 08:09:17]

The CQ bit was checked by hablich@chromium.org

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

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-09-19 08:12:30]

Description was changed from

==========
Revert of Fix async/await memory leak (patchset #5 id:80001 of
https://codereview.chromium.org/2334323006/ )

Reason for revert:
newly introduced test async-await-loop times out:
https://build.chromium.org/p/client.v8/builders/V8%20Linux%20-%20arm64%20-%20...

Original issue's description:
> Fix async/await memory leak
> 
> This patch closes a memory leak in async/await where the desugaring
> was creating a situation analagous to that described in v8:5002.
> Intermediate Promises were being kept alive, so a long-running loop
> would cause linear memory usage on the heap. This patch returns
> undefined to the 'then' callback passed into PerformPromiseThen
> in order to avoid this hazard. Test expectations are fixed to remove
> expecting extraneous events which occurred on Promises that are
> now not given unnecessarily complex resolution paths before being
> thrown away.
> 
> BUG=v8:5390
> 
> Committed: https://crrev.com/a0ba18e9634c5e2d439033ab61a77cff54f9af35
> Cr-Commit-Position: refs/heads/master@{#39479}

TBR=adamk@chromium.org,caitp@igalia.com,littledan@chromium.org
NOTRY=true
BUG=v8:5390
==========

to

==========
Revert of Fix async/await memory leak (patchset #5 id:80001 of
https://codereview.chromium.org/2334323006/ )

Reason for revert:
newly introduced test async-await-loop times out:
https://build.chromium.org/p/client.v8/builders/V8%20Linux%20-%20arm64%20-%20...

Original issue's description:
> Fix async/await memory leak
> 
> This patch closes a memory leak in async/await where the desugaring
> was creating a situation analagous to that described in v8:5002.
> Intermediate Promises were being kept alive, so a long-running loop
> would cause linear memory usage on the heap. This patch returns
> undefined to the 'then' callback passed into PerformPromiseThen
> in order to avoid this hazard. Test expectations are fixed to remove
> expecting extraneous events which occurred on Promises that are
> now not given unnecessarily complex resolution paths before being
> thrown away.
> 
> BUG=v8:5390
> 
> Committed: https://crrev.com/a0ba18e9634c5e2d439033ab61a77cff54f9af35
> Cr-Commit-Position: refs/heads/master@{#39479}

TBR=adamk@chromium.org,caitp@igalia.com,littledan@chromium.org
NOTRY=true
BUG=v8:5390
==========

[commit-bot: I haz the power, 2016-09-19 08:12:31]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-09-19 08:13:00]

Description was changed from

==========
Revert of Fix async/await memory leak (patchset #5 id:80001 of
https://codereview.chromium.org/2334323006/ )

Reason for revert:
newly introduced test async-await-loop times out:
https://build.chromium.org/p/client.v8/builders/V8%20Linux%20-%20arm64%20-%20...

Original issue's description:
> Fix async/await memory leak
> 
> This patch closes a memory leak in async/await where the desugaring
> was creating a situation analagous to that described in v8:5002.
> Intermediate Promises were being kept alive, so a long-running loop
> would cause linear memory usage on the heap. This patch returns
> undefined to the 'then' callback passed into PerformPromiseThen
> in order to avoid this hazard. Test expectations are fixed to remove
> expecting extraneous events which occurred on Promises that are
> now not given unnecessarily complex resolution paths before being
> thrown away.
> 
> BUG=v8:5390
> 
> Committed: https://crrev.com/a0ba18e9634c5e2d439033ab61a77cff54f9af35
> Cr-Commit-Position: refs/heads/master@{#39479}

TBR=adamk@chromium.org,caitp@igalia.com,littledan@chromium.org
NOTRY=true
BUG=v8:5390
==========

to

==========
Revert of Fix async/await memory leak (patchset #5 id:80001 of
https://codereview.chromium.org/2334323006/ )

Reason for revert:
newly introduced test async-await-loop times out:
https://build.chromium.org/p/client.v8/builders/V8%20Linux%20-%20arm64%20-%20...

Original issue's description:
> Fix async/await memory leak
>
> This patch closes a memory leak in async/await where the desugaring
> was creating a situation analagous to that described in v8:5002.
> Intermediate Promises were being kept alive, so a long-running loop
> would cause linear memory usage on the heap. This patch returns
> undefined to the 'then' callback passed into PerformPromiseThen
> in order to avoid this hazard. Test expectations are fixed to remove
> expecting extraneous events which occurred on Promises that are
> now not given unnecessarily complex resolution paths before being
> thrown away.
>
> BUG=v8:5390
>
> Committed: https://crrev.com/a0ba18e9634c5e2d439033ab61a77cff54f9af35
> Cr-Commit-Position: refs/heads/master@{#39479}

TBR=adamk@chromium.org,caitp@igalia.com,littledan@chromium.org
NOTRY=true
BUG=v8:5390

Committed: https://crrev.com/196db1999da130019bbf8e3bd65977f840e8afaf
Cr-Commit-Position: refs/heads/master@{#39493}
==========

[commit-bot: I haz the power, 2016-09-19 08:13:01]

Patchset 1 (id:??) landed as
https://crrev.com/196db1999da130019bbf8e3bd65977f840e8afaf
Cr-Commit-Position: refs/heads/master@{#39493}

[Dan Ehrenberg, 2016-09-19 17:33:15]

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

The reason for reverting is: Relanding with faster-running test.