Extended implementation to use interface as arguments

third_party/WebKit/Source/core/testing/CallbackFunctionTest.cpp

 // Copyright 2016 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.
 
 #include "core/testing/CallbackFunctionTest.h"
 
 #include "bindings/core/v8/ScriptState.h"
 #include "bindings/core/v8/V8Binding.h"
 #include "bindings/core/v8/V8TestCallback.h"
+#include "bindings/core/v8/V8TestInterfaceCallback.h"
+#include "core/html/HTMLDivElement.h"
 
 namespace blink {
 
 DEFINE_TRACE(CallbackFunctionTest)
 {
 }
 
 String CallbackFunctionTest::testCallback(ScriptState* scriptState, V8TestCallback* callback, const String& message1, const String& message2, ExceptionState& exceptionState)
 {
     ScriptWrappable* scriptWrappable;
     String returnValue;
 
     if (callback->call(scriptState, scriptWrappable = nullptr, message1, message2, returnValue)) {
         return String("SUCCESS: ") + returnValue;
     }
     return String("Error!");
 }
 
+void CallbackFunctionTest::testInterfaceCallback(ScriptState* scriptState, V8TestInterfaceCallback* callback, HeapVector<Member<HTMLDivElement>>& div, ExceptionState& exceptionState)

[bashi, 2016/09/23 02:16:05]

div -> divElements ?

[lkawai, 2016/09/23 02:43:42]

Done.

+{
+    ScriptWrappable* scriptWrappable;

[peria, 2016/09/23 02:50:05]

initialize |scriptWrappable|

[lkawai, 2016/09/23 05:35:45]

In the past CL, I got a comment that initialization should be written in the
following function. Which is better?

[peria, 2016/09/23 15:34:49]

All variables must be initialized at their declarations.  Otherwise, when
someone updates the code later, they may use uninitialized values.

+
+    callback->call(scriptState, scriptWrappable = nullptr, div);
+    return;
+}
+
 } // namespace blink