Revert of Remove more dead code after Object.observe removal

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/builds/6442

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

Committed: https://crrev.com/819371c6281c2dbe7646b7a691e4d7bb635c460a
Cr-Commit-Position: refs/heads/master@{#35886}

[Michael Achenbach, 2016-04-29 06:28:37]

Created Revert of Remove more dead code after Object.observe removal

[commit-bot: I haz the power, 2016-04-29 06:28:46]

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

[commit-bot: I haz the power, 2016-04-29 06:29:05]

Description was changed from

==========
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
==========

to

==========
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
==========

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

Committed patchset #1 (id:1)

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

A revert of this CL (patchset #1 id:1) has been created in
https://codereview.chromium.org/1932163002/ by jkummerow@chromium.org.

The reason for reverting is: Let's just preserve non-spec-compliant legacy
behavior and not throw on failure..

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

Description was changed from

==========
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
==========

to

==========
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

Committed: https://crrev.com/819371c6281c2dbe7646b7a691e4d7bb635c460a
Cr-Commit-Position: refs/heads/master@{#35886}
==========

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

Patchset 1 (id:??) landed as
https://crrev.com/819371c6281c2dbe7646b7a691e4d7bb635c460a
Cr-Commit-Position: refs/heads/master@{#35886}