Only record one in n line endings to save space.

Only record one in n line endings to save space.

R=yangguo@chromium.org
BUG=

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

[Yang, 2015-05-11 06:39:41]

I wonder whether it would make sense to move all the implementation to C++.
Getting line ends from a script object is implemented as executable accessor,
which means we have to jump into runtime anyways. If we have one single
implementation to calculate line ends in C++, we could also save the trouble of
special casing for exotic line ends.

[Erik Corry, 2015-05-11 11:44:15]

erik.corry@gmail.com changed reviewers:
+ erik.corry@gmail.com

[Erik Corry, 2015-05-11 11:44:15]

Moving everything to C++ would be one option, but not something I am planning to
work on.

[Yang, 2015-05-19 14:10:28]

lgtm. Though we should do a clean up at some point, moving all of the code to
C++.

[Erik Corry Chromium.org, 2015-05-19 14:11:09]

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

[commit-bot: I haz the power, 2015-05-19 14:11:23]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1137683003/1

[commit-bot: I haz the power, 2015-05-19 14:18:01]

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

[commit-bot: I haz the power, 2015-05-19 14:18:02]

Dry run: Try jobs failed on following builders:
  v8_linux64_avx2_rel on tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_linux64_avx2_rel/builds/491)

[Erik Corry Chromium.org, 2015-05-20 09:34:24]

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

[Erik Corry Chromium.org, 2015-05-20 09:34:24]

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/1137683003/#ps20001
(title: "Try to make compilers happy")

[commit-bot: I haz the power, 2015-05-20 09:34:40]

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

[commit-bot: I haz the power, 2015-05-20 10:01:36]

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

[commit-bot: I haz the power, 2015-05-20 10:01:36]

Dry run: This issue passed the CQ dry run.

[Erik Corry Chromium.org, 2015-06-08 11:26:18]

The CQ bit was checked by erikcorry@chromium.org

[Erik Corry Chromium.org, 2015-06-08 11:26:18]

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/1137683003/#ps60001
(title: "Fix false detection of exotic newlines")

[commit-bot: I haz the power, 2015-06-08 11:26:28]

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

[commit-bot: I haz the power, 2015-06-08 12:00:52]

Committed patchset #4 (id:60001)

[commit-bot: I haz the power, 2015-06-08 12:01:10]

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

[Erik Corry Chromium.org, 2015-06-08 14:24:03]

A revert of this CL (patchset #4 id:60001) has been created in
https://codereview.chromium.org/1155683010/ by erikcorry@chromium.org.

The reason for reverting is: Test failures
https://storage.googleapis.com/chromium-layout-test-archives/V8-Blink_Win/345....