Slightly tidy up the usage of SweepForbiddenScope and ScriptForbiddenScope

Slightly tidy up the usage of SweepForbiddenScope and ScriptForbiddenScope

SweepForbiddenScope and ScriptForbiddenScope are used together.
This CL just makes it clearer in the code base.

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

[haraken, 2017-02-15 09:37:14]

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

[haraken, 2017-02-15 09:37:15]

PTAL

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

https://codereview.chromium.org/2692043008/diff/1/third_party/WebKit/Source/p...
third_party/WebKit/Source/platform/heap/ThreadState.cpp:1279:
SweepForbiddenScope sweepForbidden(this);

Actually this is not needed though.

[sof, 2017-02-15 09:42:55]

lgtm, having ScriptForbiddenIfMainThreadScope now also apply across postGC()
isn't problematic.

[haraken, 2017-02-15 09:46:11]

The CQ bit was checked by haraken@chromium.org

[commit-bot: I haz the power, 2017-02-15 09:46:25]

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-15 11:17:53]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1487151971905480, "parent_rev":
"53ddb6092f87b0b6eea70c0c099237bf3b8f8fcd", "commit_rev":
"df877d86769f52421f838b8b947569aabe80db6e"}

[commit-bot: I haz the power, 2017-02-15 11:18:27]

Description was changed from

==========
Slightly tidy up the usage of SweepForbiddenScope and ScriptForbiddenScope

SweepForbiddenScope and ScriptForbiddenScope are used together.
This CL just makes it clearer in the code base.

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

to

==========
Slightly tidy up the usage of SweepForbiddenScope and ScriptForbiddenScope

SweepForbiddenScope and ScriptForbiddenScope are used together.
This CL just makes it clearer in the code base.

BUG=671856

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

[commit-bot: I haz the power, 2017-02-15 11:18:28]

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