Simplify accesses to Script::line_ends

Simplify accesses to Script::line_ends

line_ends is either undefined (if uninitialized) or a fixed array (after
Script::InitLineEnds). All accesses from JS have been removed, therefore
we can skip assigning a COW map and remove the accessor.

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

[jgruber, 2016-11-09 09:02:39]

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

[commit-bot: I haz the power, 2016-11-09 09:02:57]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[jgruber, 2016-11-09 09:23:24]

jgruber@chromium.org changed reviewers:
+ kozyatinskiy@chromium.org, yangguo@chromium.org

[Yang, 2016-11-09 09:29:18]

lgtm. awesome.

https://codereview.chromium.org/2490903002/diff/1/src/api.cc
File src/api.cc (right):

https://codereview.chromium.org/2490903002/diff/1/src/api.cc#newcode8892
src/api.cc:8892: result.push_back(line_end->value());
since we already know the resulting size of the vector, wouldn't it make sense
to reserve the capacity upfront?

[commit-bot: I haz the power, 2016-11-09 09:29:40]

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

[commit-bot: I haz the power, 2016-11-09 09:29:41]

Dry run: This issue passed the CQ dry run.

[jgruber, 2016-11-09 11:54:02]

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

[commit-bot: I haz the power, 2016-11-09 11:54:04]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-11-09 12:19:50]

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

[commit-bot: I haz the power, 2016-11-09 12:19:51]

Dry run: This issue passed the CQ dry run.

[kozy, 2016-11-09 17:40:06]

lgtm

[jgruber, 2016-11-10 07:57:20]

The CQ bit was checked by jgruber@chromium.org

[jgruber, 2016-11-10 07:57:21]

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/2490903002/#ps20001
(title: "Address comments")

[commit-bot: I haz the power, 2016-11-10 07:57:25]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-11-10 07:59:48]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-11-17 22:28:15]

Description was changed from

==========
Simplify accesses to Script::line_ends

line_ends is either undefined (if uninitialized) or a fixed array (after
Script::InitLineEnds). All accesses from JS have been removed, therefore
we can skip assigning a COW map and remove the accessor.
==========

to

==========
Simplify accesses to Script::line_ends

line_ends is either undefined (if uninitialized) or a fixed array (after
Script::InitLineEnds). All accesses from JS have been removed, therefore
we can skip assigning a COW map and remove the accessor.

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

[commit-bot: I haz the power, 2016-11-17 22:28:17]

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