No longer clean out main thread's heap for LSan's benefit.

No longer clean out main thread's heap for LSan's benefit.

The extra GCing that cleanupMainThread() appears to be redundant, as
LSan will have performed leak detection prior to the main thread shutting
down.

R=
BUG=677684
Committed: https://crrev.com/c53d9f4af3d1f42cbf785ceab1139b61050f0e43
Cr-Commit-Position: refs/heads/master@{#441068}

[sof, 2017-01-02 14:41:02]

The CQ bit was checked by sigbjornf@opera.com to run a CQ dry run

[commit-bot: I haz the power, 2017-01-02 14:41:11]

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

[sof, 2017-01-02 14:44:34]

sigbjornf@opera.com changed reviewers:
+ oilpan-reviews@chromium.org

[sof, 2017-01-02 14:44:34]

please take a look.

I didn't detect any extra LSan leaks when trying this on
https://codereview.chromium.org/2602263002/ (ps#2)

[haraken, 2017-01-02 14:58:33]

LGTM

[commit-bot: I haz the power, 2017-01-02 15:50:31]

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

[commit-bot: I haz the power, 2017-01-02 15:50:32]

Dry run: This issue passed the CQ dry run.

[sof, 2017-01-02 15:53:52]

The CQ bit was checked by sigbjornf@opera.com

[commit-bot: I haz the power, 2017-01-02 15:54:02]

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-01-02 15:57:24]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1483372432035540, "parent_rev":
"6a797ed2cc42d8cf172b3fcab7972aff375aa995", "commit_rev":
"24fb0d6a4e52b3457559e2f8485cabe3802dc9da"}

[commit-bot: I haz the power, 2017-01-02 15:57:59]

Description was changed from

==========
No longer clean out main thread's heap for LSan's benefit.

The extra GCing that cleanupMainThread() appears to be redundant, as
LSan will have performed leak detection prior to the main thread shutting
down.

R=
BUG=677684
==========

to

==========
No longer clean out main thread's heap for LSan's benefit.

The extra GCing that cleanupMainThread() appears to be redundant, as
LSan will have performed leak detection prior to the main thread shutting
down.

R=
BUG=677684

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

[commit-bot: I haz the power, 2017-01-02 15:58:00]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2017-01-02 15:59:59]

Description was changed from

==========
No longer clean out main thread's heap for LSan's benefit.

The extra GCing that cleanupMainThread() appears to be redundant, as
LSan will have performed leak detection prior to the main thread shutting
down.

R=
BUG=677684

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

to

==========
No longer clean out main thread's heap for LSan's benefit.

The extra GCing that cleanupMainThread() appears to be redundant, as
LSan will have performed leak detection prior to the main thread shutting
down.

R=
BUG=677684

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

[commit-bot: I haz the power, 2017-01-02 16:00:00]

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