Remove DCHECK for isolate->counters()

Remove DCHECK for isolate->counters()

This is a fix to https://codereview.chromium.org/2929853003 that got
reverted. The DCHECK checked to see that it was not in a background
thread. While this is a property we want for v8, it is also used
by blink, and blink violates this property.

Therefore, this CL removes the DCHECK for now.


BUG=v8:6361
Review-Url: https://codereview.chromium.org/2961443002
Cr-Commit-Position: refs/heads/master@{#46190}
Committed: https://chromium.googlesource.com/v8/v8/+/ee0e295d8e829b5c3fc1225b09c7c886e00305f6

[kschimpf, 2017-06-23 19:41:48]

Description was changed from

==========
Remove DCHECK for isolate->counters()

This is a fix to https://codereview.chromium.org/2929853003 that got
reverted. The DCHECK checked to see that it was not in a background
thread. While this is a property we want for v8, it is also used
by blink, and blink violates this property.

Therefore, this CL removes the DCHECK for now.


BUG=v8:6361
==========

to

==========
Remove DCHECK for isolate->counters()

This is a fix to https://codereview.chromium.org/2929853003 that got
reverted. The DCHECK checked to see that it was not in a background
thread. While this is a property we want for v8, it is also used
by blink, and blink violates this property.

Therefore, this CL removes the DCHECK for now.


BUG=v8:6361
==========

[kschimpf, 2017-06-23 19:41:49]

kschimpf@chromium.org changed reviewers:
+ mtrofin@google.com

[kschimpf, 2017-06-23 19:41:59]

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

[commit-bot: I haz the power, 2017-06-23 19:42:09]

Dry run: CQ is trying da patch.

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

[Mircea Trofin, 2017-06-23 20:07:38]

On 2017/06/23 19:42:09, commit-bot: I haz the power wrote:
> Dry run: CQ is trying da patch.
> 
> Follow status at:
>
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

lgtm

[commit-bot: I haz the power, 2017-06-23 20:11:12]

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

[commit-bot: I haz the power, 2017-06-23 20:11:13]

Dry run: This issue passed the CQ dry run.

[kschimpf, 2017-06-23 20:15:03]

The CQ bit was checked by kschimpf@chromium.org

[commit-bot: I haz the power, 2017-06-23 20:15:20]

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, 2017-06-23 20:17:04]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1498248903759830, "parent_rev":
"b6224c8fc0d2ba99ccb075d658df3ee99efc9c07", "commit_rev":
"ee0e295d8e829b5c3fc1225b09c7c886e00305f6"}

[commit-bot: I haz the power, 2017-06-23 20:17:13]

Description was changed from

==========
Remove DCHECK for isolate->counters()

This is a fix to https://codereview.chromium.org/2929853003 that got
reverted. The DCHECK checked to see that it was not in a background
thread. While this is a property we want for v8, it is also used
by blink, and blink violates this property.

Therefore, this CL removes the DCHECK for now.


BUG=v8:6361
==========

to

==========
Remove DCHECK for isolate->counters()

This is a fix to https://codereview.chromium.org/2929853003 that got
reverted. The DCHECK checked to see that it was not in a background
thread. While this is a property we want for v8, it is also used
by blink, and blink violates this property.

Therefore, this CL removes the DCHECK for now.


BUG=v8:6361

Review-Url: https://codereview.chromium.org/2961443002
Cr-Commit-Position: refs/heads/master@{#46190}
Committed:
https://chromium.googlesource.com/v8/v8/+/ee0e295d8e829b5c3fc1225b09c7c886e00...
==========

[commit-bot: I haz the power, 2017-06-23 20:17:14]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/v8/v8/+/ee0e295d8e829b5c3fc1225b09c7c886e00...