Widen definition of %HasComplexElements() to include non-enumerability

Widen definition of %HasComplexElements() to include non-enumerability

This avoids using the Sparse methods on objects with non-enumerable elements,
which can cause the 'enumerable: false' bit to get lost in the operation.

R=mstarzinger@chromium.org

Committed: https://code.google.com/p/v8/source/detail?r=24883

[adamk, 2014-10-23 23:56:23]

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

[adamk, 2014-10-23 23:56:23]

Quick followup to HasComplexElements().

[Michael Starzinger, 2014-10-24 17:04:08]

LGTM.

https://codereview.chromium.org/672323003/diff/1/test/mjsunit/regress/regress...
File test/mjsunit/regress/regress-enumerable-element.js (right):

https://codereview.chromium.org/672323003/diff/1/test/mjsunit/regress/regress...
test/mjsunit/regress/regress-enumerable-element.js:1: // Copyright 2014 the V8
project authors. All rights reserved.
suggestion: Maybe call this file "regress-shift-enumerable.js" instead?

[adamk, 2014-10-24 18:03:45]

https://codereview.chromium.org/672323003/diff/1/test/mjsunit/regress/regress...
File test/mjsunit/regress/regress-enumerable-element.js (right):

https://codereview.chromium.org/672323003/diff/1/test/mjsunit/regress/regress...
test/mjsunit/regress/regress-enumerable-element.js:1: // Copyright 2014 the V8
project authors. All rights reserved.
On 2014/10/24 at 17:04:08, Michael Starzinger wrote:
> suggestion: Maybe call this file "regress-shift-enumerable.js" instead?

Renamed

[adamk, 2014-10-24 18:04:14]

Committed patchset #2 (id:20001) manually as 24883 (presubmit successful).