Leak Detector: Count ScriptPromise and ActiveDOMObject on worker threads

Leak Detector: Count ScriptPromise and ActiveDOMObject on worker threads

This CL makes the leak detector count ScriptPromise and ActiveDOMObject
not only on the main thread but also on worker threads.

There was a bug that the worker threads don't wait for the termination
(crbug.com/507224) and we couldn't count those objects on worker
threads correctly, but now the bug has been fixed.

BUG=506466, 506799, 507574
Committed: https://crrev.com/f3d1a58111d4763fb364d39498e2053cada1c50d
Cr-Commit-Position: refs/heads/master@{#413474}

[hajimehoshi, 2016-08-19 08:22:01]

Description was changed from

==========
Leak Detector: Count ScriptPromise and ActiveDOMObject on worker threads

This CL makes the leak detector count ScriptPromise and ActiveDOMObject
not only on the main thread but also on worker threads.

There was a bug that the worker threads don't wait for the termination
(crbug.com/507224) and we couldn't count those objects on worker
threads correctly, but now the bug has been fixed.

BUG=506466, 506799, 507574
==========

to

==========
Leak Detector: Count ScriptPromise and ActiveDOMObject on worker threads

This CL makes the leak detector count ScriptPromise and ActiveDOMObject
not only on the main thread but also on worker threads.

There was a bug that the worker threads don't wait for the termination
(crbug.com/507224) and we couldn't count those objects on worker
threads correctly, but now the bug has been fixed.

BUG=506466, 506799, 507574
==========

[hajimehoshi, 2016-08-19 08:22:01]

hajimehoshi@chromium.org changed reviewers:
+ haraken@chromium.org, kouhei@chromium.org

[hajimehoshi, 2016-08-19 08:23:09]

hajimehoshi@chromium.org changed reviewers:
+ yurys@chromium.org

[hajimehoshi, 2016-08-19 08:23:17]

PTAL

[hajimehoshi, 2016-08-19 08:23:55]

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

[commit-bot: I haz the power, 2016-08-19 08:24:10]

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

[hajimehoshi, 2016-08-19 08:33:16]

I tested the leak detector of 'fast' tests and confirmed no ScriptPromise or
ActiveDOMObject leaks.:

% ./third_party/WebKit/Tools/Scripts/run-webkit-tests --enable-leak-detection
--child-processes=16 -t Default third_party/WebKit/LayoutTests/fast/ 

I'll test all tests later.

[commit-bot: I haz the power, 2016-08-19 08:38:41]

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

[commit-bot: I haz the power, 2016-08-19 08:38:42]

Dry run: Try jobs failed on following builders:
  chromeos_amd64-generic_chromium_compile_only_ng on
master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromeos_amd64-...)

[haraken, 2016-08-19 09:19:23]

LGTM

[kouhei (in TOK), 2016-08-21 23:42:36]

lgtm

[hajimehoshi, 2016-08-22 05:37:24]

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

[commit-bot: I haz the power, 2016-08-22 05:37: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-08-22 09:04:48]

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

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

Dry run: This issue passed the CQ dry run.

[hajimehoshi, 2016-08-22 09:22:21]

Confirmed there are no new leaks in all layout tests.

yurys: PTAL at core/inspector

[kozy, 2016-08-22 17:42:49]

kozyatinskiy@chromium.org changed reviewers:
+ kozyatinskiy@chromium.org

[kozy, 2016-08-22 17:42:49]

@yurys transferred to another team.
lgtm.

[hajimehoshi, 2016-08-22 17:55:26]

The CQ bit was checked by hajimehoshi@chromium.org

[hajimehoshi, 2016-08-22 17:55:27]

The patchset sent to the CQ was uploaded after l-g-t-m from
haraken@chromium.org, kouhei@chromium.org
Link to the patchset: https://codereview.chromium.org/2258873004/#ps20001
(title: "Bug fix: Build fail on ChromeOS")

[commit-bot: I haz the power, 2016-08-22 17:56:15]

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-08-22 18:04:20]

Description was changed from

==========
Leak Detector: Count ScriptPromise and ActiveDOMObject on worker threads

This CL makes the leak detector count ScriptPromise and ActiveDOMObject
not only on the main thread but also on worker threads.

There was a bug that the worker threads don't wait for the termination
(crbug.com/507224) and we couldn't count those objects on worker
threads correctly, but now the bug has been fixed.

BUG=506466, 506799, 507574
==========

to

==========
Leak Detector: Count ScriptPromise and ActiveDOMObject on worker threads

This CL makes the leak detector count ScriptPromise and ActiveDOMObject
not only on the main thread but also on worker threads.

There was a bug that the worker threads don't wait for the termination
(crbug.com/507224) and we couldn't count those objects on worker
threads correctly, but now the bug has been fixed.

BUG=506466, 506799, 507574
==========

[commit-bot: I haz the power, 2016-08-22 18:04:22]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-08-22 18:06:21]

Description was changed from

==========
Leak Detector: Count ScriptPromise and ActiveDOMObject on worker threads

This CL makes the leak detector count ScriptPromise and ActiveDOMObject
not only on the main thread but also on worker threads.

There was a bug that the worker threads don't wait for the termination
(crbug.com/507224) and we couldn't count those objects on worker
threads correctly, but now the bug has been fixed.

BUG=506466, 506799, 507574
==========

to

==========
Leak Detector: Count ScriptPromise and ActiveDOMObject on worker threads

This CL makes the leak detector count ScriptPromise and ActiveDOMObject
not only on the main thread but also on worker threads.

There was a bug that the worker threads don't wait for the termination
(crbug.com/507224) and we couldn't count those objects on worker
threads correctly, but now the bug has been fixed.

BUG=506466, 506799, 507574

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

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

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