bindings: Make the sequence conversion code more complaint with WebIDL.

third_party/WebKit/Source/modules/webdatabase/SQLTransaction.cpp

 /*
  * Copyright (C) 2007, 2008, 2013 Apple 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 299 matching lines @@
              exception_state);
 }
 
 void SQLTransaction::executeSql(ScriptState* script_state,
                                 const String& sql_statement,
                                 const Nullable<Vector<ScriptValue>>& arguments,
                                 SQLStatementCallback* callback,
                                 SQLStatementErrorCallback* callback_error,
                                 ExceptionState& exception_state) {
   Vector<SQLValue> sql_values;
-  if (!arguments.IsNull())
-    sql_values = ToImplArray<Vector<SQLValue>>(
-        arguments.Get(), script_state->GetIsolate(), exception_state);
-  ExecuteSQL(sql_statement, sql_values, callback, callback_error,

[jsbell, 2017/04/10 21:09:24]

What happened to this ExecuteSQL() call?

-             exception_state);
+  sql_values.ReserveInitialCapacity(arguments.Get().size());
+  for (const auto& script_value : arguments.Get()) {
+    sql_values.emplace_back(ScriptValue::To<SQLValue>(
+        script_state->GetIsolate(), script_value, exception_state));
+  }
 }
 
 bool SQLTransaction::ComputeNextStateAndCleanupIfNeeded() {
   // Only honor the requested state transition if we're not supposed to be
   // cleaning up and shutting down:
   if (database_->Opened()) {
     SetStateToRequestedState();
     ASSERT(next_state_ == SQLTransactionState::kEnd ||
            next_state_ == SQLTransactionState::kDeliverTransactionCallback ||
            next_state_ ==
@@ skipping 17 matching lines @@
   callback_.Clear();
   success_callback_.Clear();
   error_callback_.Clear();
 }
 
 SQLTransactionErrorCallback* SQLTransaction::ReleaseErrorCallback() {
   return error_callback_.Release();
 }
 
 }  // namespace blink