Oilpan: provide eager 'operator new' overload for ServiceWorker.

Oilpan: provide eager 'operator new' overload for ServiceWorker.

ServiceWorker needs to overload 'operator new' in order to have its
instances be allocated on the eagerly swept heap.

R=haraken
BUG=491488
Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=196496

[sof, 2015-06-04 11:27:38]

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

[sof, 2015-06-04 11:27:39]

please take a look.

[haraken, 2015-06-04 12:45:39]

LGTM

Just to confirm: Without having the OPERATOR_NEW, we'll hit the isEagerHeapIndex
assert (and thus we can notice the error), right?

[sof, 2015-06-04 12:49:59]

On 2015/06/04 12:45:39, haraken wrote:
> LGTM
> 
> Just to confirm: Without having the OPERATOR_NEW, we'll hit the
isEagerHeapIndex
> assert (and thus we can notice the error), right?

Indeed it will,


http://build.chromium.org/p/chromium.webkit/builders/WebKit%20Linux%20Oilpan%...

each & every "eager" CL wasn't tested on top of each other, as you can see
there.

[haraken, 2015-06-04 12:51:58]

On 2015/06/04 12:49:59, sof wrote:
> On 2015/06/04 12:45:39, haraken wrote:
> > LGTM
> > 
> > Just to confirm: Without having the OPERATOR_NEW, we'll hit the
> isEagerHeapIndex
> > assert (and thus we can notice the error), right?
> 
> Indeed it will,
> 
> 
>
http://build.chromium.org/p/chromium.webkit/builders/WebKit%20Linux%20Oilpan%...
> 
> each & every "eager" CL wasn't tested on top of each other, as you can see
> there.

oh...

[sof, 2015-06-04 12:52:40]

The CQ bit was checked by sigbjornf@opera.com

[commit-bot: I haz the power, 2015-06-04 12:53:32]

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

[commit-bot: I haz the power, 2015-06-04 12:58:02]

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