[bindings] Remove custom binding usage in SQLResultSetRowList.

Source/bindings/modules/v8/ToV8.h

+// Copyright (c) 2015 The Chromium Authors. All rights reserved.
+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+#ifndef BINDINGS_MODULES_V8_ToV8_h

[vivekg, 2015/03/04 13:34:04]

Presubmit throws an error here:

Source/bindings/modules/v8/ToV8.h:5:  #ifndef header guard has wrong style,
please use: ToV8_h  [build/header_guard] [5]

But this would be a problem considering the existence of
Source/bindings/core/v8/ToV8.h using the same header guard. 
We should probably use a different file name instead of ToV8.h. WDYT?

[Yuki, 2015/03/04 14:11:04]

I personally think the style guide is wrong.  As we're going to merge the
Chromium repository and Blink repository, we'll have many instances of this
issue.  So the style guide (and lint) should be fixed.

But this is just my personal opinion.  What do other people think?  I'm okay to
change the file name for the time being.  (Looks inconsistent, though.)

+#define BINDINGS_MODULES_V8_ToV8_h
+
+#include "bindings/core/v8/V8Binding.h"
+#include "modules/webdatabase/sqlite/SQLValue.h"
+
+#include <v8.h>
+
+namespace blink {
+
+inline v8::Handle<v8::Value> toV8(const SQLValue& sqlValue, v8::Isolate* isolate)

[Yuki, 2015/03/04 14:11:04]

The signature of toV8() functions must be
  v8::Handle<v8::Value> toV8(T value, v8::Handle<v8::Object> creationContext,
v8::Isolate* isolate)

+{
+    switch (sqlValue.type()) {
+    case SQLValue::NullValue:
+        return v8::Null(isolate);
+    case SQLValue::NumberValue:
+        return v8::Number::New(isolate, sqlValue.number());
+    case SQLValue::StringValue:
+        return v8String(isolate, sqlValue.string());
+    }
+    ASSERT_NOT_REACHED();
+    return v8::Local<v8::Value>();
+}
+
+} // namespace blink
+
+#endif // BINDINGS_MODULES_V8_ToV8_h
+