Transforming PersistentHandleVisitor to not need to copy Persistent handles.

Transforming PersistentHandleVisitor to not need to copy Persistent handles.

This gets rid of more places where Persistent handles are copied
(see crbug.com/236290 ).

Transition plan: after this CL, Blink will be modified to work both with and
without the #define, then the #define will be removed from V8.

The corresponding Blink side changes are in https://codereview.chromium.org/15670010/ .

BUG=
R=dcarney@chromium.org, svenpanne@chromium.org

Committed: https://code.google.com/p/v8/source/detail?r=14871

[dcarney, 2013-05-28 12:58:11]

lgtm

https://codereview.chromium.org/15974006/diff/1/src/api.cc
File src/api.cc (right):

https://codereview.chromium.org/15974006/diff/1/src/api.cc#newcode5111
src/api.cc:5111: Persistent<Value> handle =
ToApi<Value>(i::Handle<i::Object>(p));
can you make this a Value* and do some casting magic here?

[dcarney, 2013-05-28 12:58:11]

lgtm

[marja, 2013-05-28 14:06:32]

svenpanne, ptal

[marja, 2013-05-28 14:07:18]

(comment done)

https://codereview.chromium.org/15974006/diff/1/src/api.cc
File src/api.cc (right):

https://codereview.chromium.org/15974006/diff/1/src/api.cc#newcode5111
src/api.cc:5111: Persistent<Value> handle =
ToApi<Value>(i::Handle<i::Object>(p));
On 2013/05/28 12:58:11, dcarney wrote:
> can you make this a Value* and do some casting magic here?

Done.

[Sven Panne, 2013-05-29 07:34:37]

lgtm

[dcarney, 2013-05-29 08:48:44]

Committed patchset #2 manually as r14871 (presubmit successful).