[strong] Deprecate arguments

[strong] Deprecate arguments

R=marja@chromium.org
BUG=

Committed: https://crrev.com/0ebd870f720c8f3175b89257dd42386f98b056f8
Cr-Commit-Position: refs/heads/master@{#26753}

[marja, 2015-02-19 13:37:36]

lgtm *except* pls remove the IsArguments from ast_value_factory.h, it's now dead
code.

[marja, 2015-02-19 13:38:11]

... actually, do all test_parsing tests run now? Aren't there any tests there
which test eval and arguments?

[rossberg, 2015-02-19 14:17:59]

New patchsets have been uploaded after l-g-t-m from marja@chromium.org

[rossberg, 2015-02-19 14:19:40]

On 2015/02/19 13:37:36, marja wrote:
> pls remove the IsArguments from ast_value_factory.h, it's now dead
> code.

Done.

> ... actually, do all test_parsing tests run now? Aren't there any tests there
> which test eval and arguments?

I didn't add any sync tests for this, since it's all in base. All tests succeed.
Shouldn't they?

[marja, 2015-02-19 14:33:14]

> > ... actually, do all test_parsing tests run now? Aren't there any tests
there
> > which test eval and arguments?
> 
> I didn't add any sync tests for this, since it's all in base. All tests
succeed.
> Shouldn't they?

The interesting test case would've been NoErrorsEvalAndArgumentsStrict where the
difference between strict and strong shows up, but that test didn't contain the
strong directive, so, everything should succeed.

[rossberg, 2015-02-19 14:35:53]

The CQ bit was checked by rossberg@chromium.org

[commit-bot: I haz the power, 2015-02-19 14:37:37]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/932333004/20001

[commit-bot: I haz the power, 2015-02-19 14:58:38]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2015-02-19 14:58:52]

Patchset 2 (id:??) landed as
https://crrev.com/0ebd870f720c8f3175b89257dd42386f98b056f8
Cr-Commit-Position: refs/heads/master@{#26753}