Fix function name inference corruption for async functions

Fix function name inference corruption for async functions

The code which pushes and pops to the function name inference stack
generally checks if the stack is active with the IsOpen method. One
piece of code pertaining to async functions was missing that check.
This patch adds it.

BUG=chromium:658267
R=gsathya,caitp

Committed: https://crrev.com/06f8e877267c9acae10e53e3054bea209dc92b78
Cr-Commit-Position: refs/heads/master@{#41120}

[Dan Ehrenberg, 2016-11-18 16:59:56]

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

[commit-bot: I haz the power, 2016-11-18 16:59:58]

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

[caitp, 2016-11-18 17:18:33]

https://codereview.chromium.org/2514893002/diff/1/src/parsing/func-name-infer...
File src/parsing/func-name-inferrer.cc (right):

https://codereview.chromium.org/2514893002/diff/1/src/parsing/func-name-infer...
src/parsing/func-name-inferrer.cc:48: if (IsOpen()) {
I like the straight forward fix, thanks. lgtm

[Dan Ehrenberg, 2016-11-18 17:19:43]

The CQ bit was unchecked by littledan@chromium.org

[Dan Ehrenberg, 2016-11-18 17:19:50]

The CQ bit was checked by littledan@chromium.org

[commit-bot: I haz the power, 2016-11-18 17:19:56]

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

[Dan Ehrenberg, 2016-11-18 17:25:24]

The CQ bit was unchecked by littledan@chromium.org

[marja, 2016-11-18 18:28:43]

marja@chromium.org changed reviewers:
+ marja@chromium.org

[marja, 2016-11-18 18:28:43]

lgtm (looks like i don't have access to the bug - weird!)

[marja, 2016-11-18 18:28:47]

lgtm (looks like i don't have access to the bug - weird!)

[marja, 2016-11-18 18:28:49]

lgtm (looks like i don't have access to the bug - weird!)

[Dan Ehrenberg, 2016-11-18 18:29:29]

The CQ bit was checked by littledan@chromium.org

[commit-bot: I haz the power, 2016-11-18 18:29:34]

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-18 18:31:38]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-11-18 18:32:11]

Description was changed from

==========
Fix function name inference corruption for async functions

The code which pushes and pops to the function name inference stack
generally checks if the stack is active with the IsOpen method. One
piece of code pertaining to async functions was missing that check.
This patch adds it.

BUG=chromium:658267
R=gsathya,caitp
==========

to

==========
Fix function name inference corruption for async functions

The code which pushes and pops to the function name inference stack
generally checks if the stack is active with the IsOpen method. One
piece of code pertaining to async functions was missing that check.
This patch adds it.

BUG=chromium:658267
R=gsathya,caitp

Committed: https://crrev.com/06f8e877267c9acae10e53e3054bea209dc92b78
Cr-Commit-Position: refs/heads/master@{#41120}
==========

[commit-bot: I haz the power, 2016-11-18 18:32:12]

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