Class methods should be non enumerable

Class methods should be non enumerable

Class methods and accessors (both prototype and static) should be
non enumerable.

BUG=v8:3330
LOG=Y
R=dslomov@chromium.org

Committed: https://crrev.com/176b68d11a2b485d9568d2e7fd03e6ed7b67dd54
Cr-Commit-Position: refs/heads/master@{#26437}

[arv (Not doing code reviews), 2015-02-03 17:36:59]

formatting

[arv (Not doing code reviews), 2015-02-03 17:37:34]

PTAL

[arv (Not doing code reviews), 2015-02-03 17:38:36]

https://codereview.chromium.org/896643003/diff/20001/src/runtime/runtime-obje...
File src/runtime/runtime-object.cc (right):

https://codereview.chromium.org/896643003/diff/20001/src/runtime/runtime-obje...
src/runtime/runtime-object.cc:1596: CONVERT_SMI_ARG_CHECKED(unchecked, 3);
I'm going to do a followup CL that introduces
CONVERT_PROPERTY_ATTRIBUTES_CHECKED since this code is really common.

[arv (Not doing code reviews), 2015-02-03 18:59:10]

arv@chromium.org changed reviewers:
+ adamk@chromium.org

[arv (Not doing code reviews), 2015-02-03 18:59:10]

Adding Adam

[adamk, 2015-02-03 19:07:04]

lgtm

[Dmitry Lomov (no reviews), 2015-02-04 10:36:09]

lgtm (I presume we have tests that check that object literal concise methods are
still enumerable?)

[arv (Not doing code reviews), 2015-02-04 16:01:36]

On 2015/02/04 10:36:09, Dmitry Lomov (chromium) wrote:
> lgtm (I presume we have tests that check that object literal concise methods
are
> still enumerable?)

Yes we have tests for property enumeration of concise methods in object
literals:

test/mjsunit/harmony/computed-property-names.js
test/mjsunit/harmony/object-literals-method.js

[arv (Not doing code reviews), 2015-02-04 16:01:43]

The CQ bit was checked by arv@chromium.org

[commit-bot: I haz the power, 2015-02-04 16:02:33]

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

[commit-bot: I haz the power, 2015-02-04 16:21:17]

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

[commit-bot: I haz the power, 2015-02-04 16:21:18]

Try jobs failed on following builders:
  v8_linux64_asan_rel on tryserver.v8
(http://build.chromium.org/p/tryserver.v8/builders/v8_linux64_asan_rel/builds/...)

[arv (Not doing code reviews), 2015-02-04 16:32:59]

New patchsets have been uploaded after l-g-t-m from dslomov@chromium.org

[arv (Not doing code reviews), 2015-02-04 16:33:04]

Use LanguageMode

[arv (Not doing code reviews), 2015-02-04 16:33:09]

The CQ bit was checked by arv@chromium.org

[commit-bot: I haz the power, 2015-02-04 16:34:02]

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

[commit-bot: I haz the power, 2015-02-04 17:23:16]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2015-02-04 17:23:23]

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