Generated bindings for IDL callback functions

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

+// Copyright 2014 The Chromium Authors. All rights reserved.

[bashi, 2016/09/13 00:35:06]

s/2014/2016/

[lkawai, 2016/09/16 05:05:51]

Done.

+// 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"
+
+namespace blink {
+
+DEFINE_TRACE(CallbackFunctionTest)
+{
+}
+
+String CallbackFunctionTest::testDOMString(ScriptState* scriptState, V8TestCallback* callback, ExceptionState& exceptionState)
+{
+    String msg1 = "hello";
+    String msg2 = "world";
+    ScriptWrappable* scriptWrappable = nullptr;
+    String returnValue;
+
+    if (callback->call(scriptState, scriptWrappable, msg1, msg2, returnValue)) {
+        return String("SUCCESS: ") + returnValue; }

[bashi, 2016/09/13 00:35:07]

Place '{' and '}' in a new line.

[peria, 2016/09/15 01:14:33]

[style]
Not correct. Keep '{' as-is.

if (...) {
    ....;
} else {
    ....;
}

https://www.chromium.org/blink/coding-style

[lkawai, 2016/09/16 05:05:51]

Done.

[lkawai, 2016/09/16 05:05:51]

Done.

+    else { return String("Error!"); }

[bashi, 2016/09/13 00:35:06]

Place '{' and '}' in a new line.

[peria, 2016/09/15 01:14:33]

[style] In this case, you can remove 'else {' and '}'

if (...) {
    return ...;
}
return ...;

[lkawai, 2016/09/16 05:05:51]

Done.

[lkawai, 2016/09/16 05:05:51]

Done.

+}
+
+} // namespace blink