Leak Detector: Add support for ScriptPromise on Blink side

Leak Detector: Add support for ScriptPromise on Blink side

This CL adds support for ScriptPromise on Blink side. This CL also
adds a test that leaks ScriptPromise to LeakExpectations.

BUG=506466
Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=198273

[hajimehoshi, 2015-07-03 09:09:48]

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

[hajimehoshi, 2015-07-03 09:09:49]

PTAL

[tkent, 2015-07-03 09:47:44]

lgtm

[hajimehoshi, 2015-07-03 09:48:31]

On 2015/07/03 09:47:44, tkent wrote:
> lgtm

Thank you! Just in case: I've fixed LeakExpections just now.

[haraken, 2015-07-03 13:10:09]

LGTM

[hajimehoshi, 2015-07-03 13:20:04]

The CQ bit was checked by hajimehoshi@chromium.org

[hajimehoshi, 2015-07-03 13:20:04]

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

[commit-bot: I haz the power, 2015-07-03 13:20:09]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1223713003/40001

[commit-bot: I haz the power, 2015-07-03 14:36:04]

Committed patchset #3 (id:40001) as
https://src.chromium.org/viewvc/blink?view=rev&revision=198273

[yhirano, 2015-07-06 03:36:49]

Don't you need to protect s_instanceCount with mutex?
Don't you need to define the assignment operator as well?

[yhirano, 2015-07-06 03:53:20]

On 2015/07/06 03:36:49, yhirano wrote:
> Don't you need to protect s_instanceCount with mutex?
I think this leads to the test failure at
https://code.google.com/p/chromium/issues/detail?id=506799. The test creates
workers.

> Don't you need to define the assignment operator as well?
Sorry, I misunderstood.

[hajimehoshi, 2015-07-06 05:05:00]

On 2015/07/06 03:53:20, yhirano wrote:
> On 2015/07/06 03:36:49, yhirano wrote:
> > Don't you need to protect s_instanceCount with mutex?
> I think this leads to the test failure at
> https://code.google.com/p/chromium/issues/detail?id=506799. The test creates
> workers.

Right. I'll add a mutex or something.