[bindings] Let NativeValueTraits<T>::nativeValue be variadic function and merge various convers…

Source/bindings/modules/v8/V8BindingForModulesTest.cpp

 /*
  * Copyright (C) 2011 Google Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions
  * are met:
  *
  * 1.  Redistributions of source code must retain the above copyright
  *     notice, this list of conditions and the following disclaimer.
  * 2.  Redistributions in binary form must reproduce the above copyright
@@ skipping 24 matching lines @@
 #include <gtest/gtest.h>
 
 using namespace blink;
 
 namespace {
 
 IDBKey* checkKeyFromValueAndKeyPathInternal(v8::Isolate* isolate, const ScriptValue& value, const String& keyPath)
 {
     IDBKeyPath idbKeyPath(keyPath);
     EXPECT_TRUE(idbKeyPath.isValid());
-
-    return createIDBKeyFromScriptValueAndKeyPath(isolate, value, idbKeyPath);
+    NonThrowableExceptionState exceptionState;

[bashi, 2015/03/19 23:53:41]

Can we pass exceptionState from the caller of this function? We want to minimize
use of NonThrowableExceptionState. If it's difficult, please add a TODO comment.

I understand some nativeValue() methods never throw an exception via
exceptionState, but it could be changed in the future and if that happens we
will swallow exceptions unexpectedly.

+    return ScriptValue::to<IDBKey*>(isolate, value, exceptionState, idbKeyPath);
 }
 
 void checkKeyPathNullValue(v8::Isolate* isolate, const ScriptValue& value, const String& keyPath)
 {
     ASSERT_FALSE(checkKeyFromValueAndKeyPathInternal(isolate, value, keyPath));
 }
 
 bool injectKey(ScriptState* scriptState, IDBKey* key, ScriptValue& value, const String& keyPath)
 {
     IDBKeyPath idbKeyPath(keyPath);
@@ skipping 112 matching lines @@
     checkInjection(scriptState(), IDBKey::createDate(4567), scriptObject, "foo.baz");
     checkInjection(scriptState(), IDBKey::createDate(4567), scriptObject, "bar");
     checkInjection(scriptState(), IDBKey::createArray(IDBKey::KeyArray()), scriptObject, "foo.baz");
     checkInjection(scriptState(), IDBKey::createArray(IDBKey::KeyArray()), scriptObject, "bar");
 
     checkInjectionFails(scriptState(), IDBKey::createString("zoo"), scriptObject, "foo.bar.baz");
     checkInjection(scriptState(), IDBKey::createString("zoo"), scriptObject, "foo.xyz.foo");
 }
 
 } // namespace