Create a character stream and hook it up to the parse info

Create a character stream and hook it up to the parse info

Also make it possible to use the background parser from a character
stream. The External{One,Two}ByteStringUtf16CharacterStreams work both
on foreground and background threads.

BUG=v8:5215
R=marja@chromium.org,vogelheim@chromium.org

Committed: https://crrev.com/84b07ec24d0cb9588dc22ca5f9b0f709afbf91fe
Cr-Commit-Position: refs/heads/master@{#38162}

[jochen (gone - plz use gerrit), 2016-07-29 07:08:03]

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

[commit-bot: I haz the power, 2016-07-29 07:08:10]

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

[jochen (gone - plz use gerrit), 2016-07-29 07:19:25]

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

[commit-bot: I haz the power, 2016-07-29 07:19:29]

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-07-29 07:41:10]

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

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

Dry run: This issue passed the CQ dry run.

[marja, 2016-07-29 08:43:52]

lgtm w/ comment

https://codereview.chromium.org/2195603002/diff/20001/src/parsing/parser.cc
File src/parsing/parser.cc (right):

https://codereview.chromium.org/2195603002/diff/20001/src/parsing/parser.cc#n...
src/parsing/parser.cc:5491: // When streaming, we don't know the length of the
source until we have
These changes seem bogus, it was under 80 chars before, plus the new version is
weird (lines with only one word).

[jochen (gone - plz use gerrit), 2016-07-29 08:54:35]

https://codereview.chromium.org/2195603002/diff/20001/src/parsing/parser.cc
File src/parsing/parser.cc (right):

https://codereview.chromium.org/2195603002/diff/20001/src/parsing/parser.cc#n...
src/parsing/parser.cc:5491: // When streaming, we don't know the length of the
source until we have
On 2016/07/29 at 08:43:52, marja wrote:
> These changes seem bogus, it was under 80 chars before, plus the new version
is weird (lines with only one word).

reverted

[jochen (gone - plz use gerrit), 2016-07-29 08:54:48]

The CQ bit was checked by jochen@chromium.org

[jochen (gone - plz use gerrit), 2016-07-29 08:54:49]

The patchset sent to the CQ was uploaded after l-g-t-m from marja@chromium.org
Link to the patchset: https://codereview.chromium.org/2195603002/#ps30001
(title: "updates")

[commit-bot: I haz the power, 2016-07-29 08:54:52]

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-07-29 09:17:16]

Committed patchset #3 (id:30001)

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

Description was changed from

==========
Create a character stream and hook it up to the parse info

Also make it possible to use the background parser from a character
stream. The External{One,Two}ByteStringUtf16CharacterStreams work both
on foreground and background threads.

BUG=v8:5215
R=marja@chromium.org,vogelheim@chromium.org
==========

to

==========
Create a character stream and hook it up to the parse info

Also make it possible to use the background parser from a character
stream. The External{One,Two}ByteStringUtf16CharacterStreams work both
on foreground and background threads.

BUG=v8:5215
R=marja@chromium.org,vogelheim@chromium.org

Committed: https://crrev.com/84b07ec24d0cb9588dc22ca5f9b0f709afbf91fe
Cr-Commit-Position: refs/heads/master@{#38162}
==========

[commit-bot: I haz the power, 2016-07-29 09:20:12]

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