Move StatsTable into the Counters class.

Move StatsTable into the Counters class.

By moving StatsTable from class Isolate to class Counters, it make the
class StatsTable thead safe. This is needed because these two classes
call each other, and for background compilation, instances of the
Counters class can persist longer that the corresponding Isolate it
came from.

It also removes unnecessary hops to the the Isolate, and checks if the
StatsTable has been created, for these communications.

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

[kschimpf, 2017-05-26 16:10:05]

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

[commit-bot: I haz the power, 2017-05-26 16:10:07]

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, 2017-05-26 16:30:17]

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

[commit-bot: I haz the power, 2017-05-26 16:30:18]

Dry run: Try jobs failed on following builders:
  v8_linux64_gyp_rel_ng on master.tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_linux64_gyp_rel_ng/build...)
  v8_linux64_gyp_rel_ng_triggered on master.tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_linux64_gyp_rel_ng_trigg...)

[kschimpf, 2017-05-26 17:23:15]

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

[commit-bot: I haz the power, 2017-05-26 17:23:16]

Dry run: CQ is trying da patch.

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

[kschimpf, 2017-05-26 17:43:16]

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

[commit-bot: I haz the power, 2017-05-26 17:43:25]

Dry run: CQ is trying da patch.

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

[kschimpf, 2017-05-26 17:44:03]

Description was changed from

==========
Move StatsTable into the Counters class.

By moving StatsTable from class Isolate to class Counters, it make the
class StatsTable thead safe. This is needed because these two classes
call each other, and for background compilation, instances of the
Counters class can persist longer that the corresponding Isolate it
came from.

It also removes unnecessary hops to the the Isolate, and checks if the
StatsTable has been created, for these communications.

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

to

==========
Move StatsTable into the Counters class.

By moving StatsTable from class Isolate to class Counters, it make the
class StatsTable thead safe. This is needed because these two classes
call each other, and for background compilation, instances of the
Counters class can persist longer that the corresponding Isolate it
came from.

It also removes unnecessary hops to the the Isolate, and checks if the
StatsTable has been created, for these communications.

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

[kschimpf, 2017-05-26 17:44:03]

kschimpf@chromium.org changed reviewers:
+ jochen@chromium.org, mtrofin@chromium.org

[kschimpf, 2017-05-26 17:44:15]

Please review. Thanks.

[commit-bot: I haz the power, 2017-05-26 18:15:58]

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

[commit-bot: I haz the power, 2017-05-26 18:15:59]

Dry run: This issue passed the CQ dry run.

[jochen (gone - plz use gerrit), 2017-05-29 14:11:46]

lgtm

[kschimpf, 2017-05-29 17:03:57]

The CQ bit was checked by kschimpf@chromium.org

[commit-bot: I haz the power, 2017-05-29 17:04:09]

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-05-29 18:18:26]

CQ is committing da patch.

Bot data: {"patchset_id": 40001, "attempt_start_ts": 1496077437436200,
"parent_rev": "68a723996bb39bacdf5e9fc7e11c329deab68cfa", "commit_rev":
"2a9965bd0ecf90393d785425b97c5045fc0b5316"}

[commit-bot: I haz the power, 2017-05-29 18:18:31]

Description was changed from

==========
Move StatsTable into the Counters class.

By moving StatsTable from class Isolate to class Counters, it make the
class StatsTable thead safe. This is needed because these two classes
call each other, and for background compilation, instances of the
Counters class can persist longer that the corresponding Isolate it
came from.

It also removes unnecessary hops to the the Isolate, and checks if the
StatsTable has been created, for these communications.

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

to

==========
Move StatsTable into the Counters class.

By moving StatsTable from class Isolate to class Counters, it make the
class StatsTable thead safe. This is needed because these two classes
call each other, and for background compilation, instances of the
Counters class can persist longer that the corresponding Isolate it
came from.

It also removes unnecessary hops to the the Isolate, and checks if the
StatsTable has been created, for these communications.

BUG=v8:6361

Review-Url: https://codereview.chromium.org/2906063002
Cr-Commit-Position: refs/heads/master@{#45576}
Committed:
https://chromium.googlesource.com/v8/v8/+/2a9965bd0ecf90393d785425b97c5045fc0...
==========

[commit-bot: I haz the power, 2017-05-29 18:18:33]

Committed patchset #3 (id:40001) as
https://chromium.googlesource.com/v8/v8/+/2a9965bd0ecf90393d785425b97c5045fc0...