Make AstConsString::length constant-time instead of O(N)

Make AstConsString::length constant-time instead of O(N)

This makes it consistent in behavior with its heap-resident equivalent.

Also some minor cleanup in the AstString class hierarchy.

BUG=v8:4595
LOG=n

Committed: https://crrev.com/8a7e6fc34dbb8a5102e9f256fa343ce49de5dc28
Cr-Commit-Position: refs/heads/master@{#32733}

[adamk, 2015-12-10 00:15:44]

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

[adamk, 2015-12-10 00:15:44]

Relatively minor cleanup/reasonable-ness fix found while investigating FNI bug.

[Dan Ehrenberg, 2015-12-10 01:02:26]

lgtm

[adamk, 2015-12-10 01:11:47]

The CQ bit was checked by adamk@chromium.org

[commit-bot: I haz the power, 2015-12-10 01:11:59]

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

[commit-bot: I haz the power, 2015-12-10 04:42:00]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2015-12-10 04:42:48]

Description was changed from

==========
Make AstConsString::length constant-time instead of O(N)

This makes it consistent in behavior with its heap-resident equivalent.

Also some minor cleanup in the AstString class hierarchy.

BUG=v8:4595
LOG=n
==========

to

==========
Make AstConsString::length constant-time instead of O(N)

This makes it consistent in behavior with its heap-resident equivalent.

Also some minor cleanup in the AstString class hierarchy.

BUG=v8:4595
LOG=n

Committed: https://crrev.com/8a7e6fc34dbb8a5102e9f256fa343ce49de5dc28
Cr-Commit-Position: refs/heads/master@{#32733}
==========

[commit-bot: I haz the power, 2015-12-10 04:42:49]

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