Remove SafePointScopes in HeapTest.cpp

Remove SafePointScopes in HeapTest.cpp

Currently SafePointScope is used for two purposes:

a) Declare that the thread is not operating on the oilpan heap
and let another thread start a GC.

b) Record the stack end and all registers before starting a GC.

Now that we have the per-thread heap, a) is not needed. b) is still needed though.
The SafePointScopes removed in this CL is a).

BUG=671856
Review-Url: https://codereview.chromium.org/2697383003
Cr-Commit-Position: refs/heads/master@{#451289}
Committed: https://chromium.googlesource.com/chromium/src/+/580f0df2c038eb1e575b169ed6491e9cbe92bc11

[haraken, 2017-02-17 11:20:46]

haraken@chromium.org changed reviewers:
+ sigbjornf@opera.com

[haraken, 2017-02-17 11:20:47]

PTAL

[haraken, 2017-02-17 11:50:01]

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

[commit-bot: I haz the power, 2017-02-17 11:50:22]

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

[sof, 2017-02-17 12:00:43]

lgtm

[haraken, 2017-02-17 12:08:13]

The CQ bit was unchecked by haraken@chromium.org

[haraken, 2017-02-17 12:08:16]

The CQ bit was checked by haraken@chromium.org

[commit-bot: I haz the power, 2017-02-17 12:08:30]

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-02-17 12:46:55]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1487333296730720, "parent_rev":
"0e60da02d2e136fa7cc20abf8858b00846cbd331", "commit_rev":
"580f0df2c038eb1e575b169ed6491e9cbe92bc11"}

[commit-bot: I haz the power, 2017-02-17 12:47:36]

Description was changed from

==========
Remove SafePointScopes in HeapTest.cpp

Currently SafePointScope is used for two purposes:

a) Declare that the thread is not operating on the oilpan heap
and let another thread start a GC.

b) Record the stack end and all registers before starting a GC.

Now that we have the per-thread heap, a) is not needed. b) is still needed
though.
The SafePointScopes removed in this CL is a).

BUG=671856
==========

to

==========
Remove SafePointScopes in HeapTest.cpp

Currently SafePointScope is used for two purposes:

a) Declare that the thread is not operating on the oilpan heap
and let another thread start a GC.

b) Record the stack end and all registers before starting a GC.

Now that we have the per-thread heap, a) is not needed. b) is still needed
though.
The SafePointScopes removed in this CL is a).

BUG=671856

Review-Url: https://codereview.chromium.org/2697383003
Cr-Commit-Position: refs/heads/master@{#451289}
Committed:
https://chromium.googlesource.com/chromium/src/+/580f0df2c038eb1e575b169ed649...
==========

[commit-bot: I haz the power, 2017-02-17 12:47:37]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/580f0df2c038eb1e575b169ed649...