Add template parameter to ReturnValue::Set.

Add template parameter to ReturnValue::Set.

E.g., v8-i18n wants to set the return value with a different type of a Persistent.

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

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

[dcarney, 2013-05-27 12:34:36]

https://codereview.chromium.org/16102002/diff/1/include/v8.h
File include/v8.h (right):

https://codereview.chromium.org/16102002/diff/1/include/v8.h#newcode5688
include/v8.h:5688: void ReturnValue<T>::Set(const Persistent<S>& handle) {
you need the correct type check here
and a test to make sure the type_cast is called correctly

[marja, 2013-05-27 13:09:28]

https://codereview.chromium.org/16102002/diff/1/include/v8.h
File include/v8.h (right):

https://codereview.chromium.org/16102002/diff/1/include/v8.h#newcode5688
include/v8.h:5688: void ReturnValue<T>::Set(const Persistent<S>& handle) {
On 2013/05/27 12:34:36, dcarney wrote:
> you need the correct type check here
> and a test to make sure the type_cast is called correctly

Done.

[dcarney, 2013-05-27 13:13:46]

lgtm

[marja, 2013-05-27 13:30:43]

(as discussed, simplified the test)

svenpanne, could you review this?

[Sven Panne, 2013-05-27 13:56:01]

lgtm

[dcarney, 2013-05-27 14:34:22]

Committed patchset #5 manually as r14826 (presubmit successful).