Move collectGarbage* methods to ThreadState

Move collectGarbage* methods to ThreadState

Move collectGarbage* methods to ThreadState to indicate that these run GC for the current thread heap.

BUG=

Committed: https://crrev.com/89d83a49a6c126d91e25e2d3520cb398e8e1f7d5
Committed: https://crrev.com/f7a7635e3c139b0230d4ac74bdb517ded2e26385
Cr-Original-Commit-Position: refs/heads/master@{#417269}
Cr-Commit-Position: refs/heads/master@{#417538}

[keishi, 2016-09-02 12:36:37]

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

[commit-bot: I haz the power, 2016-09-02 12:36:48]

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, 2016-09-02 14:14:57]

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

[commit-bot: I haz the power, 2016-09-02 14:14:58]

Dry run: This issue passed the CQ dry run.

[keishi, 2016-09-07 10:05:40]

Description was changed from

==========
Move collectGarbage* methods to ThreadState

BUG=
==========

to

==========
Move collectGarbage* methods to ThreadState

Move collectGarbage* methods to ThreadState to indicate that these run GC for
the current thread heap.

BUG=
==========

[keishi, 2016-09-07 10:05:53]

keishi@chromium.org changed reviewers:
+ haraken@chromium.org, oilpan-reviews@chromium.org

[haraken, 2016-09-07 14:38:55]

LGTM

https://codereview.chromium.org/2307003002/diff/20001/third_party/WebKit/Sour...
File third_party/WebKit/Source/bindings/core/v8/V8GCController.cpp (right):

https://codereview.chromium.org/2307003002/diff/20001/third_party/WebKit/Sour...
third_party/WebKit/Source/bindings/core/v8/V8GCController.cpp:355: if
(currentThreadState && !ThreadState::current()->isGCForbidden()) {


ThreadState::current() => currentThread

[keishi, 2016-09-08 01:48:18]

https://codereview.chromium.org/2307003002/diff/20001/third_party/WebKit/Sour...
File third_party/WebKit/Source/bindings/core/v8/V8GCController.cpp (right):

https://codereview.chromium.org/2307003002/diff/20001/third_party/WebKit/Sour...
third_party/WebKit/Source/bindings/core/v8/V8GCController.cpp:355: if
(currentThreadState && !ThreadState::current()->isGCForbidden()) {
On 2016/09/07 14:38:55, haraken wrote:
> 
> 
> ThreadState::current() => currentThread

Done.

[keishi, 2016-09-08 01:48:23]

The CQ bit was checked by keishi@chromium.org

[keishi, 2016-09-08 01:48:23]

The patchset sent to the CQ was uploaded after l-g-t-m from haraken@chromium.org
Link to the patchset: https://codereview.chromium.org/2307003002/#ps40001
(title: "fix")

[commit-bot: I haz the power, 2016-09-08 01:48:37]

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-09-08 06:18:57]

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

[commit-bot: I haz the power, 2016-09-08 06:18:59]

Try jobs failed on following builders:
  win_chromium_rel_ng on master.tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_rel_...)

[keishi, 2016-09-08 08:16:47]

The CQ bit was checked by keishi@chromium.org

[commit-bot: I haz the power, 2016-09-08 08:17:04]

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-09-08 13:21:02]

Description was changed from

==========
Move collectGarbage* methods to ThreadState

Move collectGarbage* methods to ThreadState to indicate that these run GC for
the current thread heap.

BUG=
==========

to

==========
Move collectGarbage* methods to ThreadState

Move collectGarbage* methods to ThreadState to indicate that these run GC for
the current thread heap.

BUG=
==========

[commit-bot: I haz the power, 2016-09-08 13:21:03]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2016-09-08 13:23:42]

Description was changed from

==========
Move collectGarbage* methods to ThreadState

Move collectGarbage* methods to ThreadState to indicate that these run GC for
the current thread heap.

BUG=
==========

to

==========
Move collectGarbage* methods to ThreadState

Move collectGarbage* methods to ThreadState to indicate that these run GC for
the current thread heap.

BUG=

Committed: https://crrev.com/89d83a49a6c126d91e25e2d3520cb398e8e1f7d5
Cr-Commit-Position: refs/heads/master@{#417269}
==========

[commit-bot: I haz the power, 2016-09-08 13:23:44]

Patchset 3 (id:??) landed as
https://crrev.com/89d83a49a6c126d91e25e2d3520cb398e8e1f7d5
Cr-Commit-Position: refs/heads/master@{#417269}

[dmurph, 2016-09-08 22:05:09]

A revert of this CL (patchset #3 id:40001) has been created in
https://codereview.chromium.org/2321223003/ by dmurph@chromium.org.

The reason for reverting is: Causing weird MSAN and other issues:
https://bugs.chromium.org/p/chromium/issues/detail?id=645227
https://bugs.chromium.org/p/chromium/issues/detail?id=645253

Reverting..

[keishi, 2016-09-09 07:38:13]

Description was changed from

==========
Move collectGarbage* methods to ThreadState

Move collectGarbage* methods to ThreadState to indicate that these run GC for
the current thread heap.

BUG=

Committed: https://crrev.com/89d83a49a6c126d91e25e2d3520cb398e8e1f7d5
Cr-Commit-Position: refs/heads/master@{#417269}
==========

to

==========
Move collectGarbage* methods to ThreadState

Move collectGarbage* methods to ThreadState to indicate that these run GC for
the current thread heap.

BUG=

Committed: https://crrev.com/89d83a49a6c126d91e25e2d3520cb398e8e1f7d5
Cr-Commit-Position: refs/heads/master@{#417269}
==========

[keishi, 2016-09-09 07:38:24]

The CQ bit was checked by keishi@chromium.org

[commit-bot: I haz the power, 2016-09-09 07:38: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, 2016-09-09 09:05:03]

Description was changed from

==========
Move collectGarbage* methods to ThreadState

Move collectGarbage* methods to ThreadState to indicate that these run GC for
the current thread heap.

BUG=

Committed: https://crrev.com/89d83a49a6c126d91e25e2d3520cb398e8e1f7d5
Cr-Commit-Position: refs/heads/master@{#417269}
==========

to

==========
Move collectGarbage* methods to ThreadState

Move collectGarbage* methods to ThreadState to indicate that these run GC for
the current thread heap.

BUG=

Committed: https://crrev.com/89d83a49a6c126d91e25e2d3520cb398e8e1f7d5
Cr-Commit-Position: refs/heads/master@{#417269}
==========

[commit-bot: I haz the power, 2016-09-09 09:05:05]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2016-09-09 09:07:19]

Description was changed from

==========
Move collectGarbage* methods to ThreadState

Move collectGarbage* methods to ThreadState to indicate that these run GC for
the current thread heap.

BUG=

Committed: https://crrev.com/89d83a49a6c126d91e25e2d3520cb398e8e1f7d5
Cr-Commit-Position: refs/heads/master@{#417269}
==========

to

==========
Move collectGarbage* methods to ThreadState

Move collectGarbage* methods to ThreadState to indicate that these run GC for
the current thread heap.

BUG=

Committed: https://crrev.com/89d83a49a6c126d91e25e2d3520cb398e8e1f7d5
Committed: https://crrev.com/f7a7635e3c139b0230d4ac74bdb517ded2e26385
Cr-Original-Commit-Position: refs/heads/master@{#417269}
Cr-Commit-Position: refs/heads/master@{#417538}
==========

[commit-bot: I haz the power, 2016-09-09 09:07:21]

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