[runtime] set AsyncFunctionNext/Throw to adapt arguments

src/bootstrapper.cc

 // Copyright 2014 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/bootstrapper.h"
 
 #include "src/accessors.h"
 #include "src/api-natives.h"
 #include "src/code-stubs.h"
 #include "src/extensions/externalize-string-extension.h"
@@ skipping 2448 matching lines @@
       JSObject::AddProperty(
           async_function_prototype, factory->constructor_string(),
           async_function_constructor,
           static_cast<PropertyAttributes>(DONT_ENUM | READ_ONLY));
 
       JSFunction::SetPrototype(async_function_constructor,
                                async_function_prototype);
 
       Handle<JSFunction> async_function_next =
           SimpleInstallFunction(container, "AsyncFunctionNext",
-                                Builtins::kGeneratorPrototypeNext, 2, false);
+                                Builtins::kGeneratorPrototypeNext, 1, false);

[Dan Ehrenberg, 2016/05/18 18:58:02]

1 looks like the right value here, matching the other registration of this
builtin function; ditto Throw. Why was it 2 before?

[caitp (gmail), 2016/05/18 19:03:43]

The original implementation used a parameter for the generator object rather
than the receiver, I guess I never updated the formals count.

       Handle<JSFunction> async_function_throw =
           SimpleInstallFunction(container, "AsyncFunctionThrow",
-                                Builtins::kGeneratorPrototypeThrow, 2, false);
+                                Builtins::kGeneratorPrototypeThrow, 1, false);
       async_function_next->shared()->set_native(true);
       async_function_throw->shared()->set_native(true);
     }
   }
 }
 
 
 void Bootstrapper::ExportExperimentalFromRuntime(Isolate* isolate,
                                                  Handle<JSObject> container) {
   HandleScope scope(isolate);
@@ skipping 1293 matching lines @@
 }
 
 
 // Called when the top-level V8 mutex is destroyed.
 void Bootstrapper::FreeThreadResources() {
   DCHECK(!IsActive());
 }
 
 }  // namespace internal
 }  // namespace v8