[parser,ast] Simplify MarkExpressionAsAssigned.

[parser,ast] Simplify MarkExpressionAsAssigned.

There's no need to return anything.

Also add a DCHECK to make sure that we never mark a variable proxy as assigned
that is already resolved (to avoid potential inconsistency with the variable's
maybe_assigned status).

R=littledan@chromium.org
CC=mstarzinger@chromium.org
BUG=

Committed: https://crrev.com/48bbd1a7fefc601849ada069a09bbce4d57533eb
Cr-Commit-Position: refs/heads/master@{#41072}

[neis, 2016-11-15 15:28:52]

Description was changed from

==========
[parser/ast] Simplify MarkExpressionAsAssigned.

There's no need to return anything.

Also add a DCHECK to make sure that we never mark a variable proxy as assigned
that is already resolved (to avoid potential inconsistency with the variable's
maybe_assigned status).

R=littledan@chromium.org
CC=mstarzinger@chromium.org
BUG=
==========

to

==========
[parser,ast] Simplify MarkExpressionAsAssigned.

There's no need to return anything.

Also add a DCHECK to make sure that we never mark a variable proxy as assigned
that is already resolved (to avoid potential inconsistency with the variable's
maybe_assigned status).

R=littledan@chromium.org
CC=mstarzinger@chromium.org
BUG=
==========

[Dan Ehrenberg, 2016-11-17 11:39:58]

lgtm

Nice cleanup

[neis, 2016-11-17 11:42:58]

The CQ bit was checked by neis@chromium.org

[commit-bot: I haz the power, 2016-11-17 11:43:02]

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-17 12:09:31]

Description was changed from

==========
[parser,ast] Simplify MarkExpressionAsAssigned.

There's no need to return anything.

Also add a DCHECK to make sure that we never mark a variable proxy as assigned
that is already resolved (to avoid potential inconsistency with the variable's
maybe_assigned status).

R=littledan@chromium.org
CC=mstarzinger@chromium.org
BUG=
==========

to

==========
[parser,ast] Simplify MarkExpressionAsAssigned.

There's no need to return anything.

Also add a DCHECK to make sure that we never mark a variable proxy as assigned
that is already resolved (to avoid potential inconsistency with the variable's
maybe_assigned status).

R=littledan@chromium.org
CC=mstarzinger@chromium.org
BUG=
==========

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

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-11-17 22:38:05]

Description was changed from

==========
[parser,ast] Simplify MarkExpressionAsAssigned.

There's no need to return anything.

Also add a DCHECK to make sure that we never mark a variable proxy as assigned
that is already resolved (to avoid potential inconsistency with the variable's
maybe_assigned status).

R=littledan@chromium.org
CC=mstarzinger@chromium.org
BUG=
==========

to

==========
[parser,ast] Simplify MarkExpressionAsAssigned.

There's no need to return anything.

Also add a DCHECK to make sure that we never mark a variable proxy as assigned
that is already resolved (to avoid potential inconsistency with the variable's
maybe_assigned status).

R=littledan@chromium.org
CC=mstarzinger@chromium.org
BUG=

Committed: https://crrev.com/48bbd1a7fefc601849ada069a09bbce4d57533eb
Cr-Commit-Position: refs/heads/master@{#41072}
==========

[commit-bot: I haz the power, 2016-11-17 22:38:06]

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