Add a test for subclass maps.

Add a test for subclass maps.

R=arv@chromium.org

Committed: https://crrev.com/3c5218f6fa3b44a95c9bd0b36bf4b9e344d9939e
Cr-Commit-Position: refs/heads/master@{#27812}

[Dmitry Lomov (no reviews), 2015-04-02 09:45:07]

ptal, just realized this was lying around on my machine, never committed.

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

https://codereview.chromium.org/1052963002/diff/1/test/mjsunit/harmony/classe...
File test/mjsunit/harmony/classes-maps.js (right):

https://codereview.chromium.org/1052963002/diff/1/test/mjsunit/harmony/classe...
test/mjsunit/harmony/classes-maps.js:45: assertThrows(function() {
this.byteLength; }, TypeError);
wrong `this` so you get a TypeError because this === undefined in strict mode

use eval or unroll the assertThrows instead

[Dmitry Lomov (no reviews), 2015-04-13 09:50:40]

ptal

https://codereview.chromium.org/1052963002/diff/1/test/mjsunit/harmony/classe...
File test/mjsunit/harmony/classes-maps.js (right):

https://codereview.chromium.org/1052963002/diff/1/test/mjsunit/harmony/classe...
test/mjsunit/harmony/classes-maps.js:45: assertThrows(function() {
this.byteLength; }, TypeError);
On 2015/04/02 13:02:32, arv wrote:
> wrong `this` so you get a TypeError because this === undefined in strict mode
> 
> use eval or unroll the assertThrows instead

Oh, embarassing!
Done.

[arv (Not doing code reviews), 2015-04-13 15:59:39]

LGTM

https://codereview.chromium.org/1052963002/diff/20001/test/mjsunit/harmony/cl...
File test/mjsunit/harmony/classes-maps.js (right):

https://codereview.chromium.org/1052963002/diff/20001/test/mjsunit/harmony/cl...
test/mjsunit/harmony/classes-maps.js:48: }
nit: 

} catch (e) {

[Dmitry Lomov (no reviews), 2015-04-14 09:34:16]

The CQ bit was checked by dslomov@chromium.org

[Dmitry Lomov (no reviews), 2015-04-14 09:34:17]

The patchset sent to the CQ was uploaded after l-g-t-m from arv@chromium.org
Link to the patchset: https://codereview.chromium.org/1052963002/#ps40001
(title: "Patch for landing")

[Dmitry Lomov (no reviews), 2015-04-14 09:34:27]

https://codereview.chromium.org/1052963002/diff/20001/test/mjsunit/harmony/cl...
File test/mjsunit/harmony/classes-maps.js (right):

https://codereview.chromium.org/1052963002/diff/20001/test/mjsunit/harmony/cl...
test/mjsunit/harmony/classes-maps.js:48: }
On 2015/04/13 15:59:39, arv wrote:
> nit: 
> 
> } catch (e) {

Done.

[commit-bot: I haz the power, 2015-04-14 09:34:33]

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

[commit-bot: I haz the power, 2015-04-14 09:59:12]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2015-04-14 09:59:22]

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