Reland of move more dead code after Object.observe removal

Reland of Remove more dead code after Object.observe removal (patchset #1 id:1 of https://codereview.chromium.org/1929293002/ )

This moves __{define,lookup}{Getter,Setter}__ to builtins.cc.

Compared to the first attempt, this patch now preserves non-spec-compliant legacy behavior and does not throw on failure.

Original review: https://codereview.chromium.org/1904313004

Committed: https://crrev.com/1095835a61ae0b35d69f26aa59f76fa0a6ee96a9
Cr-Commit-Position: refs/heads/master@{#35899}

[Jakob Kummerow, 2016-04-29 07:35:45]

Created Reland of move more dead code after Object.observe removal

[Jakob Kummerow, 2016-04-29 09:40:02]

Patchset #2 (id:280001) has been deleted

[Jakob Kummerow, 2016-04-29 09:41:02]

Description was changed from

==========
Reland of move more dead code after Object.observe removal (patchset #1 id:1 of
https://codereview.chromium.org/1929293002/ )

Reason for revert:
Let's just preserve non-spec-compliant legacy behavior and not throw on failure.

Original issue's description:
> Revert of Remove more dead code after Object.observe removal (patchset #5
id:80001 of https://codereview.chromium.org/1904313004/ )
> 
> Reason for revert:
> [Sheriff] Looks like this breaks layout tests:
>
https://build.chromium.org/p/client.v8.fyi/builders/V8-Blink%20Linux%2064/bui...
> 
> Original issue's description:
> > Remove more dead code after Object.observe removal
> > 
> > This moves __{define,lookup}{Getter,Setter}__ to builtins.cc to free up the
JavaScript implementation of DefineOwnProperty for deletion.
> 
> TBR=verwaest@chromium.org,jkummerow@chromium.org
> # Skipping CQ checks because original CL landed less than 1 days ago.
> NOPRESUBMIT=true
> NOTREECHECKS=true
> NOTRY=true

TBR=verwaest@chromium.org,machenbach@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
==========

to

==========
Reland of Remove more dead code after Object.observe removal (patchset #1 id:1
of https://codereview.chromium.org/1929293002/ )

Reason for revert:
Let's just preserve non-spec-compliant legacy behavior and not throw on failure.

Original issue's description:
> Revert of Remove more dead code after Object.observe removal (patchset #5
id:80001 of https://codereview.chromium.org/1904313004/ )
> 
> Reason for revert:
> [Sheriff] Looks like this breaks layout tests:
>
https://build.chromium.org/p/client.v8.fyi/builders/V8-Blink%20Linux%2064/bui...
> 
> Original issue's description:
> > Remove more dead code after Object.observe removal
> > 
> > This moves __{define,lookup}{Getter,Setter}__ to builtins.cc to free up the
JavaScript implementation of DefineOwnProperty for deletion.
> 
> TBR=verwaest@chromium.org,jkummerow@chromium.org
> # Skipping CQ checks because original CL landed less than 1 days ago.
> NOPRESUBMIT=true
> NOTREECHECKS=true
> NOTRY=true

TBR=verwaest@chromium.org,machenbach@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
==========

[Jakob Kummerow, 2016-04-29 09:44:25]

Description was changed from

==========
Reland of Remove more dead code after Object.observe removal (patchset #1 id:1
of https://codereview.chromium.org/1929293002/ )

Reason for revert:
Let's just preserve non-spec-compliant legacy behavior and not throw on failure.

Original issue's description:
> Revert of Remove more dead code after Object.observe removal (patchset #5
id:80001 of https://codereview.chromium.org/1904313004/ )
> 
> Reason for revert:
> [Sheriff] Looks like this breaks layout tests:
>
https://build.chromium.org/p/client.v8.fyi/builders/V8-Blink%20Linux%2064/bui...
> 
> Original issue's description:
> > Remove more dead code after Object.observe removal
> > 
> > This moves __{define,lookup}{Getter,Setter}__ to builtins.cc to free up the
JavaScript implementation of DefineOwnProperty for deletion.
> 
> TBR=verwaest@chromium.org,jkummerow@chromium.org
> # Skipping CQ checks because original CL landed less than 1 days ago.
> NOPRESUBMIT=true
> NOTREECHECKS=true
> NOTRY=true

TBR=verwaest@chromium.org,machenbach@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
==========

to

==========
Reland of Remove more dead code after Object.observe removal (patchset #1 id:1
of https://codereview.chromium.org/1929293002/ )

This moves __{define,lookup}{Getter,Setter}__ to builtins.cc.

Compared to the first attempt, this patch now preserves non-spec-compliant
legacy behavior and does not throw on failure.

Original review: https://codereview.chromium.org/1904313004
==========

[Jakob Kummerow, 2016-04-29 09:44:25]

jkummerow@chromium.org changed reviewers:
- machenbach@chromium.org

[Jakob Kummerow, 2016-04-29 09:46:49]

As discussed, PTAL. The delta since the first attempt is in patch sets 2 -> 3.

[Toon Verwaest, 2016-04-29 09:48:20]

lgtm

[Jakob Kummerow, 2016-04-29 09:48:59]

The CQ bit was checked by jkummerow@chromium.org

[commit-bot: I haz the power, 2016-04-29 09:49:04]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1932163002/320001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1932163002/320001

[commit-bot: I haz the power, 2016-04-29 10:14:46]

Description was changed from

==========
Reland of Remove more dead code after Object.observe removal (patchset #1 id:1
of https://codereview.chromium.org/1929293002/ )

This moves __{define,lookup}{Getter,Setter}__ to builtins.cc.

Compared to the first attempt, this patch now preserves non-spec-compliant
legacy behavior and does not throw on failure.

Original review: https://codereview.chromium.org/1904313004
==========

to

==========
Reland of Remove more dead code after Object.observe removal (patchset #1 id:1
of https://codereview.chromium.org/1929293002/ )

This moves __{define,lookup}{Getter,Setter}__ to builtins.cc.

Compared to the first attempt, this patch now preserves non-spec-compliant
legacy behavior and does not throw on failure.

Original review: https://codereview.chromium.org/1904313004
==========

[commit-bot: I haz the power, 2016-04-29 10:14:47]

Committed patchset #3 (id:320001)

[commit-bot: I haz the power, 2016-05-01 22:03:35]

Description was changed from

==========
Reland of Remove more dead code after Object.observe removal (patchset #1 id:1
of https://codereview.chromium.org/1929293002/ )

This moves __{define,lookup}{Getter,Setter}__ to builtins.cc.

Compared to the first attempt, this patch now preserves non-spec-compliant
legacy behavior and does not throw on failure.

Original review: https://codereview.chromium.org/1904313004
==========

to

==========
Reland of Remove more dead code after Object.observe removal (patchset #1 id:1
of https://codereview.chromium.org/1929293002/ )

This moves __{define,lookup}{Getter,Setter}__ to builtins.cc.

Compared to the first attempt, this patch now preserves non-spec-compliant
legacy behavior and does not throw on failure.

Original review: https://codereview.chromium.org/1904313004

Committed: https://crrev.com/1095835a61ae0b35d69f26aa59f76fa0a6ee96a9
Cr-Commit-Position: refs/heads/master@{#35899}
==========

[commit-bot: I haz the power, 2016-05-01 22:03:36]

Patchset 3 (id:??) landed as
https://crrev.com/1095835a61ae0b35d69f26aa59f76fa0a6ee96a9
Cr-Commit-Position: refs/heads/master@{#35899}