Remove SafePointBarrier

Remove SafePointBarrier

We can simplify code by inlining what SafePointBarrier is doing in ThreadState.cpp.

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

[haraken, 2017-02-17 11:48:51]

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

[haraken, 2017-02-17 11:48:52]

PTAL

https://codereview.chromium.org/2703713003/diff/1/third_party/WebKit/Source/p...
File third_party/WebKit/Source/platform/heap/ThreadState.cpp (right):

https://codereview.chromium.org/2703713003/diff/1/third_party/WebKit/Source/p...
third_party/WebKit/Source/platform/heap/ThreadState.cpp:1128: // TODO(haraken):
The first void* pointer is unused. Remove it.

I need to update assembly files to fix this.

https://codereview.chromium.org/2703713003/diff/1/third_party/WebKit/Source/p...
third_party/WebKit/Source/platform/heap/ThreadState.cpp:1153: void
ThreadState::leaveSafePoint() {

I think I can remove leaveSafePoint() soon.

[haraken, 2017-02-17 11:49:36]

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:04]

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 11:56:34]

lgtm

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

The CQ bit was unchecked by haraken@chromium.org

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

The CQ bit was checked by haraken@chromium.org

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

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 14:01:34]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1487333310950900,
"parent_rev": "8e945543e2a82d11b0a2934919ad0ac7ad4e7633", "commit_rev":
"ea1c7044e3c4df9325089a41d320f4892d5bf041"}

[commit-bot: I haz the power, 2017-02-17 14:02:49]

Description was changed from

==========
Remove SafePointBarrier

We can simplify code by inlining what SafePointBarrier is doing in
ThreadState.cpp.

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

to

==========
Remove SafePointBarrier

We can simplify code by inlining what SafePointBarrier is doing in
ThreadState.cpp.

BUG=671856

Review-Url: https://codereview.chromium.org/2703713003
Cr-Commit-Position: refs/heads/master@{#451297}
Committed:
https://chromium.googlesource.com/chromium/src/+/ea1c7044e3c4df9325089a41d320...
==========

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

Committed patchset #2 (id:20001) as
https://chromium.googlesource.com/chromium/src/+/ea1c7044e3c4df9325089a41d320...