Parser: Refactor strict mode checks for functions

Parser: Refactor strict mode checks for functions

Moves the strict mode checks and error reporting for the function and
parameter names into a separate CheckStrictFunctionNameAndParameters()
function in ParserBase. Parsing of arrow functions will then use this
new function instead of duplicating the error code.

BUG=
R=marja@chromium.org

Committed: https://code.google.com/p/v8/source/detail?r=21896

[aperez, 2014-06-14 20:44:22]

It would be nice to have this change applied so arrow function
parsing could use the factored out code, as suggested in
https://codereview.chromium.org/160073006/

[marja, 2014-06-16 07:04:14]

lgtm. I can also land this, but I'd like my other Parser changes to stabilize a
bit before landing more... so maybe tomorrow-ish, k?

[aperez, 2014-06-16 09:50:08]

On 2014/06/16 07:04:14, marja wrote:
> lgtm. I can also land this, but I'd like my other Parser changes to stabilize
a
> bit before landing more... so maybe tomorrow-ish, k?

Sure, sounds good to me. Thanks!

[marja, 2014-06-20 09:45:21]

Committed patchset #1 manually as r21896 (presubmit successful).