Migrate Object.definePropert{ies,y} from v8natives to builtins.

Migrate Object.definePropert{ies,y} from v8natives to builtins.

R=bmeurer@chromium.org
BUG=

Committed: https://crrev.com/0ff89ea75b3807d751c8cc7bb955a4325776de71
Cr-Commit-Position: refs/heads/master@{#35761}

[Benedikt Meurer, 2016-04-25 11:14:52]

lgtm

[caitp (gmail), 2016-04-25 11:25:23]

caitpotter88@gmail.com changed reviewers:
+ caitpotter88@gmail.com

[caitp (gmail), 2016-04-25 11:25:24]

Just a thought: if the pattern of moving things to builtins.cc continues, maybe
it's worth re-organizing these so that different groups of builtin functions can
have their own source file.

[neis, 2016-04-25 11:44:40]

The CQ bit was checked by neis@chromium.org

[commit-bot: I haz the power, 2016-04-25 11:44:52]

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

[commit-bot: I haz the power, 2016-04-25 11:46:20]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-04-25 11:46:21]

Try jobs failed on following builders:
  v8_android_arm_compile_rel on tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_android_arm_compile_rel/...)
  v8_linux64_asan_rel_ng on tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_linux64_asan_rel_ng/buil...)
  v8_linux64_avx2_rel_ng on tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_linux64_avx2_rel_ng/buil...)
  v8_linux64_rel_ng on tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_linux64_rel_ng/builds/4785)
  v8_linux_arm64_rel_ng on tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_linux_arm64_rel_ng/build...)
  v8_linux_dbg_ng on tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_linux_dbg_ng/builds/4807)
  v8_linux_gcc_compile_rel on tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_linux_gcc_compile_rel/bu...)
  v8_linux_mips64el_compile_rel on tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_linux_mips64el_compile_r...)
  v8_linux_mipsel_compile_rel on tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_linux_mipsel_compile_rel...)
  v8_linux_nodcheck_rel_ng on tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_linux_nodcheck_rel_ng/bu...)
  v8_linux_rel_ng on tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_linux_rel_ng/builds/4789)
  v8_presubmit on tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_presubmit/builds/14095)
  v8_win_nosnap_shared_rel_ng on tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_win_nosnap_shared_rel_ng...)

[neis, 2016-04-25 11:59:14]

The CQ bit was checked by neis@chromium.org

[commit-bot: I haz the power, 2016-04-25 11:59:21]

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

[commit-bot: I haz the power, 2016-04-25 12:24:07]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-04-25 12:24:57]

Description was changed from

==========
Migrate Object.definePropert{ies,y} from v8natives to builtins.

R=bmeurer@chromium.org
BUG=
==========

to

==========
Migrate Object.definePropert{ies,y} from v8natives to builtins.

R=bmeurer@chromium.org
BUG=

Committed: https://crrev.com/0ff89ea75b3807d751c8cc7bb955a4325776de71
Cr-Commit-Position: refs/heads/master@{#35761}
==========

[commit-bot: I haz the power, 2016-04-25 12:24:58]

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