Use non-deprecated versions of v8::Persistent::New/Dispose.

Use non-deprecated versions of v8::Persistent::New/Dispose.

BUG=v8::2487


Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=181970

[Sven Panne, 2013-02-06 14:51:51]

Hopefully picked the right reviewer. :-) PTAL...

[jochen (gone - plz use gerrit), 2013-02-07 12:22:16]

lgtm

[commit-bot: I haz the power, 2013-02-07 12:40:06]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/svenpanne@chromium.org/12210035/1

[commit-bot: I haz the power, 2013-02-07 12:40:08]

Presubmit check for 12210035-1 failed and returned exit status 1.


Running presubmit commit checks ...

** Presubmit ERRORS **
Missing LGTM from an OWNER for files in these directories:
    content/renderer/gpu
    content/renderer

[Sven Panne, 2013-02-08 08:08:24]

Adding jam@ as a reviewer, apatrick@'s ping TTL expired... ;-)

This is a purely mechanical change, replacing a handful of deprecated v8 API
calls with their new equivalents.

[jam, 2013-02-08 17:50:35]

On 2013/02/08 08:08:24, Sven Panne wrote:
> Adding jam@ as a reviewer, apatrick@'s ping TTL expired... ;-)
> 
> This is a purely mechanical change, replacing a handful of deprecated v8 API
> calls with their new equivalents.

i'm not familiar with this code. please get someone from
content/renderer/gpu/owners to look at this. once they do, you have my
rubberstamp for the test.

[Sven Panne, 2013-02-11 07:40:30]

Adding Ken. PTAL...

[Ken Russell (switch to Gerrit), 2013-02-11 22:00:31]

lgtm

[commit-bot: I haz the power, 2013-02-12 07:14:51]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/svenpanne@chromium.org/12210035/1

[commit-bot: I haz the power, 2013-02-12 07:14:54]

Presubmit check for 12210035-1 failed and returned exit status 1.

INFO:root:Found 2 file(s).

Running presubmit commit checks ...
Running /b/commit-queue/workdir/chromium/PRESUBMIT.py

** Presubmit ERRORS **
Missing LGTM from an OWNER for files in these directories:
    content/renderer

[Sven Panne, 2013-02-12 07:23:11]

Adding James to review the tiny content/renderer part... :-/

[jamesr, 2013-02-12 07:28:47]

lgtm

[commit-bot: I haz the power, 2013-02-12 07:43:18]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/svenpanne@chromium.org/12210035/1

[commit-bot: I haz the power, 2013-02-12 08:38:25]

Retried try job too often on mac_rel for step(s) browser_tests
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=mac_rel&nu...

[commit-bot: I haz the power, 2013-02-12 13:31:09]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/svenpanne@chromium.org/12210035/1

[commit-bot: I haz the power, 2013-02-12 14:21:47]

Retried try job too often on mac_rel for step(s) browser_tests
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=mac_rel&nu...

[commit-bot: I haz the power, 2013-02-12 14:39:57]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/svenpanne@chromium.org/12210035/1

[commit-bot: I haz the power, 2013-02-12 16:17:57]

Retried try job too often on mac_rel for step(s) browser_tests
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=mac_rel&nu...

[commit-bot: I haz the power, 2013-02-12 16:21:19]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/svenpanne@chromium.org/12210035/1

[commit-bot: I haz the power, 2013-02-12 19:17:24]

Change committed as 181970