Finalizing parsing in a compile job should immediately report errors

Finalizing parsing in a compile job should immediately report errors

When preparing compilation, we can't delay error reporting either.

Also put handles during internalization into a deferred handle scope.

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

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

[jochen (gone - plz use gerrit), 2016-08-23 07:22:14]

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

[commit-bot: I haz the power, 2016-08-23 07:22:20]

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-08-23 07:44:56]

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

[commit-bot: I haz the power, 2016-08-23 07:44:57]

Dry run: This issue passed the CQ dry run.

[rmcilroy, 2016-08-23 11:34:28]

Description was changed from

==========
Finalizing parsing in a compile job should immediately report errors

When preparing compilation, we can't delay error reporting either.

Also put handles during internalization into a deferred handle scope.

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

to

==========
Finalizing parsing in a compile job should immediately report errors

When preparing compilation, we can't delay error reporting either.

Also put handles during internalization into a deferred handle scope.

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

[rmcilroy, 2016-08-23 11:34:28]

rmcilroy@chromium.org changed reviewers:
+ rmcilroy@chromium.org
- rmcilory@chromium.org

[marja, 2016-08-23 12:01:19]

lgtm

[jochen (gone - plz use gerrit), 2016-08-23 12:08:22]

The CQ bit was checked by jochen@chromium.org

[commit-bot: I haz the power, 2016-08-23 12:08:24]

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-08-23 12:10:48]

Description was changed from

==========
Finalizing parsing in a compile job should immediately report errors

When preparing compilation, we can't delay error reporting either.

Also put handles during internalization into a deferred handle scope.

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

to

==========
Finalizing parsing in a compile job should immediately report errors

When preparing compilation, we can't delay error reporting either.

Also put handles during internalization into a deferred handle scope.

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

[commit-bot: I haz the power, 2016-08-23 12:10:49]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-08-23 12:10:59]

Description was changed from

==========
Finalizing parsing in a compile job should immediately report errors

When preparing compilation, we can't delay error reporting either.

Also put handles during internalization into a deferred handle scope.

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

to

==========
Finalizing parsing in a compile job should immediately report errors

When preparing compilation, we can't delay error reporting either.

Also put handles during internalization into a deferred handle scope.

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

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

[commit-bot: I haz the power, 2016-08-23 12:11:00]

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