Replaced ToV8Value::toV8Value with V8ValueTraits::toV8Value.

Replaced ToV8Value::toV8Value with V8ValueTraits::toV8Value.

BUG=369451
TEST=no new tests because of just refactoring.

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

[tasak, 2014-06-24 08:26:00]

Would you review this CL?

[yhirano, 2014-06-24 09:24:09]

https://codereview.chromium.org/350863002/diff/20001/Source/bindings/v8/V8Bin...
File Source/bindings/v8/V8Binding.h (right):

https://codereview.chromium.org/350863002/diff/20001/Source/bindings/v8/V8Bin...
Source/bindings/v8/V8Binding.h:240: typedef typename WTF::RemovePointer<T>::Type
TypeWithoutPointer;
[optional] You can use WTF::getPtr if you prefer.
See https://codereview.chromium.org/351813004/ for example.

https://codereview.chromium.org/350863002/diff/20001/Source/bindings/v8/V8Bin...
Source/bindings/v8/V8Binding.h:271: typedef typename WTF::RemoveVector<T,
Vector, 0, WTF::DefaultAllocator>::Type TypeWithoutDefaultVector;
You can use the partial template specialization. See
https://codereview.chromium.org/351813004/ for example.

[tasak, 2014-06-26 05:04:02]

Thank you for reviewing.

https://codereview.chromium.org/350863002/diff/20001/Source/bindings/v8/V8Bin...
File Source/bindings/v8/V8Binding.h (right):

https://codereview.chromium.org/350863002/diff/20001/Source/bindings/v8/V8Bin...
Source/bindings/v8/V8Binding.h:240: typedef typename WTF::RemovePointer<T>::Type
TypeWithoutPointer;
On 2014/06/24 09:24:09, yhirano wrote:
> [optional] You can use WTF::getPtr if you prefer.
> See https://codereview.chromium.org/351813004/ for example.

I see.
Since I'm using RemoveTemplate below, I will use RemovePointer here.

https://codereview.chromium.org/350863002/diff/20001/Source/bindings/v8/V8Bin...
Source/bindings/v8/V8Binding.h:271: typedef typename WTF::RemoveVector<T,
Vector, 0, WTF::DefaultAllocator>::Type TypeWithoutDefaultVector;
On 2014/06/24 09:24:09, yhirano wrote:
> You can use the partial template specialization. See
> https://codereview.chromium.org/351813004/ for example.

I see. Thanks.
Done.

[tasak, 2014-06-26 05:06:09]

jochen, would you review this CL?

I need Source/wtf owner's LGTM.

[yhirano, 2014-06-26 05:12:28]

lgtm and I think you don't need wtf changes any more.

[haraken, 2014-06-26 05:58:00]

LGTM

[jochen (gone - plz use gerrit), 2014-06-26 14:52:00]

given that you don't need RemoveVector<> anymore, you shouldn't need my
approval?

[tasak, 2014-06-27 02:01:17]

On 2014/06/26 05:12:28, yhirano wrote:
> lgtm and I think you don't need wtf changes any more.

Yeah, you are right. I reverted Source/wtf change.

[tasak, 2014-06-27 02:02:04]

jochen, thank you for comments.

On 2014/06/26 14:52:00, jochen wrote:
> given that you don't need RemoveVector<> anymore, you shouldn't need my
> approval?

Yeah, now I don't need RemoveVector<>.

[tasak, 2014-06-27 03:43:11]

The CQ bit was checked by tasak@google.com

[commit-bot: I haz the power, 2014-06-27 03:43:49]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/tasak@google.com/350863002/80001

[commit-bot: I haz the power, 2014-06-27 04:14:52]

Change committed as 177060