Add back the duplicate property checker

Add back the duplicate property checker

We're not quite ready to make this change.

BUG=v8:3498
LOG=Y
R=rossberg@chromium.org

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

[arv (Not doing code reviews), 2014-08-20 17:49:10]

This is mostly a revert from the old CL. The only difference is that the
ObjectLiteralChecker is passed to the ParsePropertyDefinition function.

[rossberg, 2014-08-21 11:19:43]

On 2014/08/20 17:49:10, arv wrote:
> This is mostly a revert from the old CL. The only difference is that the
> ObjectLiteralChecker is passed to the ParsePropertyDefinition function.

FWIW, I'm perfectly fine with implementing the new behaviour, it's just
preferable to do it behind a flag first. I suggest using
--harmony-object-literals, because it naturally goes with the other extensions
there.

[arv (Not doing code reviews), 2014-08-21 15:16:46]

On 2014/08/21 at 11:19:43, rossberg wrote:
> On 2014/08/20 17:49:10, arv wrote:
> > This is mostly a revert from the old CL. The only difference is that the
> > ObjectLiteralChecker is passed to the ParsePropertyDefinition function.
> 
> FWIW, I'm perfectly fine with implementing the new behaviour, it's just
preferable to do it behind a flag first. I suggest using
--harmony-object-literals, because it naturally goes with the other extensions
there.

Agreed. Personally I would prefer doing that in another CL.

[rossberg, 2014-08-21 15:20:03]

On 2014/08/21 15:16:46, arv wrote:
> On 2014/08/21 at 11:19:43, rossberg wrote:
> > On 2014/08/20 17:49:10, arv wrote:
> > > This is mostly a revert from the old CL. The only difference is that the
> > > ObjectLiteralChecker is passed to the ParsePropertyDefinition function.
> > 
> > FWIW, I'm perfectly fine with implementing the new behaviour, it's just
> preferable to do it behind a flag first. I suggest using
> --harmony-object-literals, because it naturally goes with the other extensions
> there.
> 
> Agreed. Personally I would prefer doing that in another CL.

Sure, fine by me.

LGTM

[arv (Not doing code reviews), 2014-08-21 15:32:29]

Committed patchset #2 manually as 23284 (presubmit successful).