Revert of Parser: Make skipping HTML comments optional.

Revert of Parser: Make skipping HTML comments optional. (patchset #6 id:140001 of https://codereview.chromium.org/1801203002/ )

Reason for revert:
Violates ES6 spec (crbug.com/4850), and implementation was over-eager. Will revert for now.

Original issue's description:
> Parser: Make skipping HTML comments optional.
>
> API change: This adds a new flag skip_html_comments to v8::ScriptOriginOptions. This flag controls whether V8 will attempt to honour HTML-style comments in JS sources.
>
> (That is: Gracefully ignore <!-- ... ---> in JS sources, which was a popular technique in the early days of JavaScript, to prevent non-JS-enabled browsers from displaying script sources to uses.)
>
> The flag defaults to 'true' when using v8::ScriptOrigin constructor, which preserves the existing behaviour. Embedders which are happy with the existing behaviour will thus not need any changes.
>
> BUG=chromium:573887
> LOG=Y
>
> Committed: https://crrev.com/91d344288aa51ed03eaaa1cb3e368ac1e82f0173
> Cr-Commit-Position: refs/heads/master@{#34904}

TBR=jochen@chromium.org,rossberg@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=chromium:573887, v8:4850
LOG=Y

Committed: https://crrev.com/09ac4f295c111811fffc91dcc9fd468c2c3207be
Cr-Commit-Position: refs/heads/master@{#34958}

[vogelheim, 2016-03-21 16:59:05]

Created Revert of Parser: Make skipping HTML comments optional.

[commit-bot: I haz the power, 2016-03-21 16:59:15]

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

[commit-bot: I haz the power, 2016-03-21 17:02:24]

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

[commit-bot: I haz the power, 2016-03-21 17:04:32]

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

[commit-bot: I haz the power, 2016-03-21 17:04:33]

Try jobs failed on following builders:
  v8_presubmit on tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_presubmit/builds/12689)

[vogelheim, 2016-03-21 17:15:40]

Description was changed from

==========
Revert of Parser: Make skipping HTML comments optional. (patchset #6 id:140001
of https://codereview.chromium.org/1801203002/ )

Reason for revert:
Violates ES6 spec (crbug.com/4850), and implementation was over-eager. Will
revert for now.

Original issue's description:
> Parser: Make skipping HTML comments optional.
> 
> API change: This adds a new flag skip_html_comments to
v8::ScriptOriginOptions. This flag controls whether V8 will attempt to honour
HTML-style comments in JS sources.
> 
> (That is: Gracefully ignore <!-- ... ---> in JS sources, which was a popular
technique in the early days of JavaScript, to prevent non-JS-enabled browsers
from displaying script sources to uses.)
> 
> The flag defaults to 'true' when using v8::ScriptOrigin constructor, which
preserves the existing behaviour. Embedders which are happy with the existing
behaviour will thus not need any changes.
> 
> BUG=chromium:573887
> LOG=Y
> 
> Committed: https://crrev.com/91d344288aa51ed03eaaa1cb3e368ac1e82f0173
> Cr-Commit-Position: refs/heads/master@{#34904}

TBR=jochen@chromium.org,rossberg@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=chromium:573887
==========

to

==========
Revert of Parser: Make skipping HTML comments optional. (patchset #6 id:140001
of https://codereview.chromium.org/1801203002/ )

Reason for revert:
Violates ES6 spec (crbug.com/4850), and implementation was over-eager. Will
revert for now.

Original issue's description:
> Parser: Make skipping HTML comments optional.
> 
> API change: This adds a new flag skip_html_comments to
v8::ScriptOriginOptions. This flag controls whether V8 will attempt to honour
HTML-style comments in JS sources.
> 
> (That is: Gracefully ignore <!-- ... ---> in JS sources, which was a popular
technique in the early days of JavaScript, to prevent non-JS-enabled browsers
from displaying script sources to uses.)
> 
> The flag defaults to 'true' when using v8::ScriptOrigin constructor, which
preserves the existing behaviour. Embedders which are happy with the existing
behaviour will thus not need any changes.
> 
> BUG=chromium:573887
> LOG=Y
> 
> Committed: https://crrev.com/91d344288aa51ed03eaaa1cb3e368ac1e82f0173
> Cr-Commit-Position: refs/heads/master@{#34904}

TBR=jochen@chromium.org,rossberg@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=chromium:573887, v8:4850
LOG=Y
==========

[vogelheim, 2016-03-21 17:15:45]

The CQ bit was checked by vogelheim@chromium.org

[commit-bot: I haz the power, 2016-03-21 17:15:53]

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

[commit-bot: I haz the power, 2016-03-21 17:48:33]

Description was changed from

==========
Revert of Parser: Make skipping HTML comments optional. (patchset #6 id:140001
of https://codereview.chromium.org/1801203002/ )

Reason for revert:
Violates ES6 spec (crbug.com/4850), and implementation was over-eager. Will
revert for now.

Original issue's description:
> Parser: Make skipping HTML comments optional.
> 
> API change: This adds a new flag skip_html_comments to
v8::ScriptOriginOptions. This flag controls whether V8 will attempt to honour
HTML-style comments in JS sources.
> 
> (That is: Gracefully ignore <!-- ... ---> in JS sources, which was a popular
technique in the early days of JavaScript, to prevent non-JS-enabled browsers
from displaying script sources to uses.)
> 
> The flag defaults to 'true' when using v8::ScriptOrigin constructor, which
preserves the existing behaviour. Embedders which are happy with the existing
behaviour will thus not need any changes.
> 
> BUG=chromium:573887
> LOG=Y
> 
> Committed: https://crrev.com/91d344288aa51ed03eaaa1cb3e368ac1e82f0173
> Cr-Commit-Position: refs/heads/master@{#34904}

TBR=jochen@chromium.org,rossberg@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=chromium:573887, v8:4850
LOG=Y
==========

to

==========
Revert of Parser: Make skipping HTML comments optional. (patchset #6 id:140001
of https://codereview.chromium.org/1801203002/ )

Reason for revert:
Violates ES6 spec (crbug.com/4850), and implementation was over-eager. Will
revert for now.

Original issue's description:
> Parser: Make skipping HTML comments optional.
> 
> API change: This adds a new flag skip_html_comments to
v8::ScriptOriginOptions. This flag controls whether V8 will attempt to honour
HTML-style comments in JS sources.
> 
> (That is: Gracefully ignore <!-- ... ---> in JS sources, which was a popular
technique in the early days of JavaScript, to prevent non-JS-enabled browsers
from displaying script sources to uses.)
> 
> The flag defaults to 'true' when using v8::ScriptOrigin constructor, which
preserves the existing behaviour. Embedders which are happy with the existing
behaviour will thus not need any changes.
> 
> BUG=chromium:573887
> LOG=Y
> 
> Committed: https://crrev.com/91d344288aa51ed03eaaa1cb3e368ac1e82f0173
> Cr-Commit-Position: refs/heads/master@{#34904}

TBR=jochen@chromium.org,rossberg@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=chromium:573887, v8:4850
LOG=Y
==========

[commit-bot: I haz the power, 2016-03-21 17:48:34]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-03-21 17:50:26]

Description was changed from

==========
Revert of Parser: Make skipping HTML comments optional. (patchset #6 id:140001
of https://codereview.chromium.org/1801203002/ )

Reason for revert:
Violates ES6 spec (crbug.com/4850), and implementation was over-eager. Will
revert for now.

Original issue's description:
> Parser: Make skipping HTML comments optional.
> 
> API change: This adds a new flag skip_html_comments to
v8::ScriptOriginOptions. This flag controls whether V8 will attempt to honour
HTML-style comments in JS sources.
> 
> (That is: Gracefully ignore <!-- ... ---> in JS sources, which was a popular
technique in the early days of JavaScript, to prevent non-JS-enabled browsers
from displaying script sources to uses.)
> 
> The flag defaults to 'true' when using v8::ScriptOrigin constructor, which
preserves the existing behaviour. Embedders which are happy with the existing
behaviour will thus not need any changes.
> 
> BUG=chromium:573887
> LOG=Y
> 
> Committed: https://crrev.com/91d344288aa51ed03eaaa1cb3e368ac1e82f0173
> Cr-Commit-Position: refs/heads/master@{#34904}

TBR=jochen@chromium.org,rossberg@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=chromium:573887, v8:4850
LOG=Y
==========

to

==========
Revert of Parser: Make skipping HTML comments optional. (patchset #6 id:140001
of https://codereview.chromium.org/1801203002/ )

Reason for revert:
Violates ES6 spec (crbug.com/4850), and implementation was over-eager. Will
revert for now.

Original issue's description:
> Parser: Make skipping HTML comments optional.
>
> API change: This adds a new flag skip_html_comments to
v8::ScriptOriginOptions. This flag controls whether V8 will attempt to honour
HTML-style comments in JS sources.
>
> (That is: Gracefully ignore <!-- ... ---> in JS sources, which was a popular
technique in the early days of JavaScript, to prevent non-JS-enabled browsers
from displaying script sources to uses.)
>
> The flag defaults to 'true' when using v8::ScriptOrigin constructor, which
preserves the existing behaviour. Embedders which are happy with the existing
behaviour will thus not need any changes.
>
> BUG=chromium:573887
> LOG=Y
>
> Committed: https://crrev.com/91d344288aa51ed03eaaa1cb3e368ac1e82f0173
> Cr-Commit-Position: refs/heads/master@{#34904}

TBR=jochen@chromium.org,rossberg@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=chromium:573887, v8:4850
LOG=Y

Committed: https://crrev.com/09ac4f295c111811fffc91dcc9fd468c2c3207be
Cr-Commit-Position: refs/heads/master@{#34958}
==========

[commit-bot: I haz the power, 2016-03-21 17:50:27]

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