Refactor ParseObjectLiteral

Refactor ParseObjectLiteral

This extracts the parsing of the ObjectLiteralProperty into its own
function. This is in preparation for adding support for parsing classes.

BUG=None
LOG=Y
R=dslomov@chromium.org, marja@chromium.org

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

[arv (Not doing code reviews), 2014-08-11 22:28:47]

This depends on https://codereview.chromium.org/459463002

[marja, 2014-08-12 13:56:37]

lgtm. I was temporarily confused because the boilerplate property checking,
function name infering and consuming the comma were done in two different
places, depending on whether we just parsed a getter / setter or a normal
property. But this unified version is correct, afaics.

[arv (Not doing code reviews), 2014-08-12 14:46:26]

On 2014/08/12 at 13:56:37, marja wrote:
> lgtm. I was temporarily confused because the boilerplate property checking,
function name infering and consuming the comma were done in two different
places, depending on whether we just parsed a getter / setter or a normal
property. But this unified version is correct, afaics.

I want to do more cleanup when I add static, methods and property shorthand.
Then there is computed properties... At, least moving this out of ObjectLiteral
allows sharing the mess instead of repeating it.

[Dmitry Lomov (no reviews), 2014-08-18 09:03:11]

lgtm

[arv (Not doing code reviews), 2014-08-20 15:51:13]

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