[promises] Move PromiseReactionJob to c++

src/objects-debug.cc

 // Copyright 2012 the V8 project 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 "src/objects.h"
 
 #include "src/bootstrapper.h"
 #include "src/disasm.h"
 #include "src/disassembler.h"
 #include "src/field-type.h"
@@ skipping 904 matching lines @@
 void PromiseContainer::PromiseContainerVerify() {
   CHECK(IsPromiseContainer());
   thenable()->ObjectVerify();
   then()->ObjectVerify();
   resolve()->ObjectVerify();
   reject()->ObjectVerify();
   before_debug_event()->ObjectVerify();
   after_debug_event()->ObjectVerify();
 }
 
+void PromiseReactionJobInfo::PromiseReactionJobInfoVerify() {
+  CHECK(IsPromiseReactionJobInfo());
+  CHECK(tasks()->IsJSArray() || tasks()->IsCallable());

[adamk, 2016/10/12 16:18:11]

Aren't there more things you could check here? Basically I'd expect to see
CHECKs about all the expected states of the fields.

[gsathya, 2016/10/12 20:08:53]

Yeah I was curious about this too. I guess you missed my question in the
previous reply.

Done.

+}
+
 void JSModuleNamespace::JSModuleNamespaceVerify() {
   CHECK(IsJSModuleNamespace());
   module()->ObjectVerify();
 }
 
 void Module::ModuleVerify() {
   Isolate* isolate = GetIsolate();
   CHECK(IsModule());
   CHECK(code()->IsSharedFunctionInfo() || code()->IsJSFunction());
   code()->ObjectVerify();
@@ skipping 438 matching lines @@
 
   // Both are done at the same time.
   CHECK_EQ(new_it.done(), old_it.done());
 }
 
 
 #endif  // DEBUG
 
 }  // namespace internal
 }  // namespace v8