Bug 1567579 - Store inner FunctionBoxes instead of inner functions r=tcampbell

This is a good preparation fo deferred function allocation (Bug 1569315),
but also removes some awkward rooting / type issues in deferred lazyscript
allocation

Differential Revision: https://phabricator.services.mozilla.com/D39745

js/src/frontend/BinASTParserPerTokenizer.cpp

@@ -391,9 +391,10 @@ JS::Result<Ok> BinASTParserPerTokenizer<Tok>::finishLazyFunction(
 
   BINJS_TRY_DECL(lazy, LazyScript::Create(cx_, fun, sourceObject_,
                                           pc_->closedOverBindingsForLazy(),
-                                          pc_->innerFunctionsForLazy, start,
-                                          end, start, end, /* lineno = */ 0,
-                                          start, ParseGoal::Script));
+                                          pc_->innerFunctionBoxesForLazy,
+                                          start, end, start, end,
+                                          /* lineno = */ 0, start,
+                                          ParseGoal::Script));
 
   if (funbox->strict()) {
     lazy->setStrict();