[runtime] Migrate Object.getOwnPropertyNames to C++.

[runtime] Migrate Object.getOwnPropertyNames to C++.

The Object.getOwnPropertyNames method always calls into C++ anyway,
so there's no point in having the JavaScript wrapper around at all.

Drive-by-fix: Inline GetOwnEnumerablePropertyNames into its single
call site.

CQ_INCLUDE_TRYBOTS=tryserver.chromium.win:win_chromium_rel_ng
R=yangguo@chromium.org

Committed: https://crrev.com/bf027fe756f62b4abcac8aa08134c8c5ed055620
Cr-Commit-Position: refs/heads/master@{#33380}

Committed: https://crrev.com/6fc3149939deb59d06f2d70e85306ece5272ff0c
Cr-Commit-Position: refs/heads/master@{#33417}

[Benedikt Meurer, 2016-01-19 06:54:45]

Hey Yang,

Another one from v8natives.
Please take a look.

Thanks,
Benedikt

[Benedikt Meurer, 2016-01-19 06:55:32]

The CQ bit was checked by bmeurer@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-01-19 06:55:42]

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

[commit-bot: I haz the power, 2016-01-19 06:56:13]

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

[commit-bot: I haz the power, 2016-01-19 06:56:13]

Dry run: Try jobs failed on following builders:
  v8_win_nosnap_shared_compile_rel on tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_win_nosnap_shared_compil...)

[Yang, 2016-01-19 07:32:44]

On 2016/01/19 06:56:13, commit-bot: I haz the power wrote:
> Dry run: Try jobs failed on following builders:
>   v8_win_nosnap_shared_compile_rel on tryserver.v8 (JOB_FAILED,
>
http://build.chromium.org/p/tryserver.v8/builders/v8_win_nosnap_shared_compil...)

lgtm.

[Benedikt Meurer, 2016-01-19 07:32:53]

The CQ bit was checked by bmeurer@chromium.org

[commit-bot: I haz the power, 2016-01-19 07:33:03]

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

[commit-bot: I haz the power, 2016-01-19 07:34:24]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-01-19 07:35:10]

Description was changed from

==========
[runtime] Migrate Object.getOwnPropertyNames to C++.

The Object.getOwnPropertyNames method always calls into C++ anyway,
so there's no point in having the JavaScript wrapper around at all.

Drive-by-fix: Inline GetOwnEnumerablePropertyNames into its single
call site.

R=yangguo@chromium.org
==========

to

==========
[runtime] Migrate Object.getOwnPropertyNames to C++.

The Object.getOwnPropertyNames method always calls into C++ anyway,
so there's no point in having the JavaScript wrapper around at all.

Drive-by-fix: Inline GetOwnEnumerablePropertyNames into its single
call site.

R=yangguo@chromium.org

Committed: https://crrev.com/bf027fe756f62b4abcac8aa08134c8c5ed055620
Cr-Commit-Position: refs/heads/master@{#33380}
==========

[commit-bot: I haz the power, 2016-01-19 07:35:11]

Patchset 2 (id:??) landed as
https://crrev.com/bf027fe756f62b4abcac8aa08134c8c5ed055620
Cr-Commit-Position: refs/heads/master@{#33380}

[Michael Hablich, 2016-01-20 08:19:27]

A revert of this CL (patchset #2 id:20001) has been created in
https://codereview.chromium.org/1609173002/ by hablich@chromium.org.

The reason for reverting is: Breaks roll:
https://codereview.chromium.org/1603953002/.

[Benedikt Meurer, 2016-01-20 18:13:46]

Description was changed from

==========
[runtime] Migrate Object.getOwnPropertyNames to C++.

The Object.getOwnPropertyNames method always calls into C++ anyway,
so there's no point in having the JavaScript wrapper around at all.

Drive-by-fix: Inline GetOwnEnumerablePropertyNames into its single
call site.

R=yangguo@chromium.org

Committed: https://crrev.com/bf027fe756f62b4abcac8aa08134c8c5ed055620
Cr-Commit-Position: refs/heads/master@{#33380}
==========

to

==========
[runtime] Migrate Object.getOwnPropertyNames to C++.

The Object.getOwnPropertyNames method always calls into C++ anyway,
so there's no point in having the JavaScript wrapper around at all.

Drive-by-fix: Inline GetOwnEnumerablePropertyNames into its single
call site.

R=yangguo@chromium.org

Committed: https://crrev.com/bf027fe756f62b4abcac8aa08134c8c5ed055620
Cr-Commit-Position: refs/heads/master@{#33380}
==========

[Benedikt Meurer, 2016-01-20 18:14:01]

Description was changed from

==========
[runtime] Migrate Object.getOwnPropertyNames to C++.

The Object.getOwnPropertyNames method always calls into C++ anyway,
so there's no point in having the JavaScript wrapper around at all.

Drive-by-fix: Inline GetOwnEnumerablePropertyNames into its single
call site.

R=yangguo@chromium.org

Committed: https://crrev.com/bf027fe756f62b4abcac8aa08134c8c5ed055620
Cr-Commit-Position: refs/heads/master@{#33380}
==========

to

==========
[runtime] Migrate Object.getOwnPropertyNames to C++.

The Object.getOwnPropertyNames method always calls into C++ anyway,
so there's no point in having the JavaScript wrapper around at all.

Drive-by-fix: Inline GetOwnEnumerablePropertyNames into its single
call site.

CQ_INCLUDE_TRYBOTS=tryserver.chromium.win:win_chromium_rel_ng
R=yangguo@chromium.org

Committed: https://crrev.com/bf027fe756f62b4abcac8aa08134c8c5ed055620
Cr-Commit-Position: refs/heads/master@{#33380}
==========

[Benedikt Meurer, 2016-01-20 18:15:22]

The CQ bit was checked by bmeurer@chromium.org

[Benedikt Meurer, 2016-01-20 18:15:22]

The patchset sent to the CQ was uploaded after l-g-t-m from yangguo@chromium.org
Link to the patchset: https://codereview.chromium.org/1605803002/#ps40001
(title: "REBASE")

[commit-bot: I haz the power, 2016-01-20 18:15:31]

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

[commit-bot: I haz the power, 2016-01-20 19:30:16]

Description was changed from

==========
[runtime] Migrate Object.getOwnPropertyNames to C++.

The Object.getOwnPropertyNames method always calls into C++ anyway,
so there's no point in having the JavaScript wrapper around at all.

Drive-by-fix: Inline GetOwnEnumerablePropertyNames into its single
call site.

CQ_INCLUDE_TRYBOTS=tryserver.chromium.win:win_chromium_rel_ng
R=yangguo@chromium.org

Committed: https://crrev.com/bf027fe756f62b4abcac8aa08134c8c5ed055620
Cr-Commit-Position: refs/heads/master@{#33380}
==========

to

==========
[runtime] Migrate Object.getOwnPropertyNames to C++.

The Object.getOwnPropertyNames method always calls into C++ anyway,
so there's no point in having the JavaScript wrapper around at all.

Drive-by-fix: Inline GetOwnEnumerablePropertyNames into its single
call site.

CQ_INCLUDE_TRYBOTS=tryserver.chromium.win:win_chromium_rel_ng
R=yangguo@chromium.org

Committed: https://crrev.com/bf027fe756f62b4abcac8aa08134c8c5ed055620
Cr-Commit-Position: refs/heads/master@{#33380}
==========

[commit-bot: I haz the power, 2016-01-20 19:30:16]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2016-01-20 19:30:57]

Description was changed from

==========
[runtime] Migrate Object.getOwnPropertyNames to C++.

The Object.getOwnPropertyNames method always calls into C++ anyway,
so there's no point in having the JavaScript wrapper around at all.

Drive-by-fix: Inline GetOwnEnumerablePropertyNames into its single
call site.

CQ_INCLUDE_TRYBOTS=tryserver.chromium.win:win_chromium_rel_ng
R=yangguo@chromium.org

Committed: https://crrev.com/bf027fe756f62b4abcac8aa08134c8c5ed055620
Cr-Commit-Position: refs/heads/master@{#33380}
==========

to

==========
[runtime] Migrate Object.getOwnPropertyNames to C++.

The Object.getOwnPropertyNames method always calls into C++ anyway,
so there's no point in having the JavaScript wrapper around at all.

Drive-by-fix: Inline GetOwnEnumerablePropertyNames into its single
call site.

CQ_INCLUDE_TRYBOTS=tryserver.chromium.win:win_chromium_rel_ng
R=yangguo@chromium.org

Committed: https://crrev.com/bf027fe756f62b4abcac8aa08134c8c5ed055620
Cr-Commit-Position: refs/heads/master@{#33380}

Committed: https://crrev.com/6fc3149939deb59d06f2d70e85306ece5272ff0c
Cr-Commit-Position: refs/heads/master@{#33417}
==========

[commit-bot: I haz the power, 2016-01-20 19:30:57]

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