Worker: Stop running worker tests with BlinkGC::MainThreadHeapMode

Worker: Stop running worker tests with BlinkGC::MainThreadHeapMode

Oilpan's per-thread heap mode is already shipped for workers and we no longer
have to run the tests on both BlinkGC::MainThreadHeapMode and
BlinkGC::PerThreadHeapMode.

BUG=674400
Committed: https://crrev.com/d82e162920b55095efa40a8052e9539d2cf861bb
Cr-Commit-Position: refs/heads/master@{#438790}

[nhiroki, 2016-12-15 07:44:00]

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

[commit-bot: I haz the power, 2016-12-15 07:44:07]

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

[nhiroki, 2016-12-15 07:46:11]

nhiroki@chromium.org changed reviewers:
+ haraken@chromium.org, keishi@chromium.org

[nhiroki, 2016-12-15 07:46:12]

PTAL, thanks.

[haraken, 2016-12-15 08:00:15]

keishi@: Are we ready to remove code for non-per-thread heaps?

[keishi, 2016-12-15 08:49:09]

On 2016/12/15 08:00:15, haraken wrote:
> keishi@: Are we ready to remove code for non-per-thread heaps?

Yes. Thanks for doing this, I should be cleaning up after myself.
Removing everything like safepoints and threadAttachMutex and everything turned
out to be quite complex, so after this I'll land a CL to remote
PerThreadHeapMode and work incrementaly from there.

[keishi, 2016-12-15 08:50:04]

On 2016/12/15 08:49:09, keishi wrote:
> On 2016/12/15 08:00:15, haraken wrote:
> > keishi@: Are we ready to remove code for non-per-thread heaps?
> 
> Yes. Thanks for doing this, I should be cleaning up after myself.
> Removing everything like safepoints and threadAttachMutex and everything
turned
> out to be quite complex, so after this I'll land a CL to remote
> PerThreadHeapMode and work incrementaly from there.

LGTM

[haraken, 2016-12-15 08:50:23]

LGTM

[nhiroki, 2016-12-15 09:10:15]

Thank you for reviewing :)

On 2016/12/15 08:49:09, keishi wrote:
> On 2016/12/15 08:00:15, haraken wrote:
> > keishi@: Are we ready to remove code for non-per-thread heaps?
> 
> Yes. Thanks for doing this, I should be cleaning up after myself.
> Removing everything like safepoints and threadAttachMutex and everything
turned
> out to be quite complex, so after this I'll land a CL to remote
> PerThreadHeapMode and work incrementaly from there.

Acknowledged.

[nhiroki, 2016-12-15 09:10:25]

The CQ bit was unchecked by nhiroki@chromium.org

[nhiroki, 2016-12-15 09:10:29]

The CQ bit was checked by nhiroki@chromium.org

[commit-bot: I haz the power, 2016-12-15 09:10:44]

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-12-15 09:14:29]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1481793029107870, "parent_rev":
"c97e52dd1ee38e042fe9878eb01fc3e57ae40fc7", "commit_rev":
"49148010afc2b4a85f7c103fde6738218e5ae602"}

[commit-bot: I haz the power, 2016-12-15 09:14:54]

Description was changed from

==========
Worker: Stop running worker tests with BlinkGC::MainThreadHeapMode

Oilpan's per-thread heap mode is already shipped for workers and we no longer
have to run the tests on both BlinkGC::MainThreadHeapMode and
BlinkGC::PerThreadHeapMode.

BUG=674400
==========

to

==========
Worker: Stop running worker tests with BlinkGC::MainThreadHeapMode

Oilpan's per-thread heap mode is already shipped for workers and we no longer
have to run the tests on both BlinkGC::MainThreadHeapMode and
BlinkGC::PerThreadHeapMode.

BUG=674400

Review-Url: https://codereview.chromium.org/2578053002
==========

[commit-bot: I haz the power, 2016-12-15 09:14:55]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-12-15 09:19:33]

Description was changed from

==========
Worker: Stop running worker tests with BlinkGC::MainThreadHeapMode

Oilpan's per-thread heap mode is already shipped for workers and we no longer
have to run the tests on both BlinkGC::MainThreadHeapMode and
BlinkGC::PerThreadHeapMode.

BUG=674400

Review-Url: https://codereview.chromium.org/2578053002
==========

to

==========
Worker: Stop running worker tests with BlinkGC::MainThreadHeapMode

Oilpan's per-thread heap mode is already shipped for workers and we no longer
have to run the tests on both BlinkGC::MainThreadHeapMode and
BlinkGC::PerThreadHeapMode.

BUG=674400

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

[commit-bot: I haz the power, 2016-12-15 09:19:34]

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