ES6: Duplicate properties are no longer an error

ES6: Duplicate properties are no longer an error

This removes the duplicate property checker and updates the tests.

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

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

[arv (Not doing code reviews), 2014-08-08 19:30:41]

PTAL

[arv (Not doing code reviews), 2014-08-08 22:09:18]

Fix more tests

[arv (Not doing code reviews), 2014-08-11 17:51:55]

Update webkit test too

[marja, 2014-08-12 07:47:44]

lgtm modulo 2 comments - great that this code is no longer needed! :)

1) Will this affect tests ran by WebKit? (So that you'd need to update test
expectations on the WebKit side before V8 can roll into Chromium.)

2) below

https://codereview.chromium.org/459463002/diff/40001/test/preparser/duplicate...
File test/preparser/duplicate-property.pyt (left):

https://codereview.chromium.org/459463002/diff/40001/test/preparser/duplicate...
test/preparser/duplicate-property.pyt:134: 
It's kinda surprising that this test apparently tests number to string
conversions too! Do we have enough confidence that these cases are covered by
other tests, or should you add the number-to-string conversion test cases
somewhere separately?

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

Thanks for the review

I'll run the blink layout tests and disable/update the tests on the blink side
if needed.

https://codereview.chromium.org/459463002/diff/40001/test/preparser/duplicate...
File test/preparser/duplicate-property.pyt (left):

https://codereview.chromium.org/459463002/diff/40001/test/preparser/duplicate...
test/preparser/duplicate-property.pyt:134: 
On 2014/08/12 07:47:44, marja wrote:
> It's kinda surprising that this test apparently tests number to string
> conversions too! Do we have enough confidence that these cases are covered by
> other tests, or should you add the number-to-string conversion test cases
> somewhere separately?

I can't find anything that covers this. I'll move these into an mjsunit test in
the form of.

assertSame(0x20, 32, "hex-int")

[arv (Not doing code reviews), 2014-08-12 18:09:51]

Add test for number literals that was previously part of duplicate property test

[arv (Not doing code reviews), 2014-08-13 16:16:37]

Blink TestExpectation suppression: https://codereview.chromium.org/467903003

[arv (Not doing code reviews), 2014-08-20 14:25:56]

Committed patchset #4 manually as 23239 (presubmit successful).