corysanin/tubestation
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Bug 1589904 - Remove LazyScript::functionNonDelazifying() r=jandem

Browse Source 
Replace with LazyScript::function() for the same reasons as removing
JSScript::functionNonDelazifying().

Depends on D50505

Differential Revision: https://phabricator.services.mozilla.com/D50506
This commit is contained in:
Ted Campbell
2019-10-25 13:07:36 +00:00
parent 1f6b2a542d
commit 204dac71fa
8 changed files with 18 additions and 23 deletions
Download Patch File Download Diff File Expand all files Collapse all files

7
js/src/frontend/BinASTParserPerTokenizer.cpp
View File

@@ -165,7 +165,7 @@ JS::Result<FunctionNode*> BinASTParserPerTokenizer<Tok>::parseLazyFunction(
tokenizer_->seek(firstOffset);
// For now, only function declarations and function expression are supported.
RootedFunction func(cx_, lazyScript_->functionNonDelazifying());
RootedFunction func(cx_, lazyScript_->function());
bool isExpr = func->isLambda();
MOZ_ASSERT(func->kind() == FunctionFlags::FunctionKind::NormalFunction);
@@ -266,7 +266,7 @@ JS::Result<FunctionBox*> BinASTParserPerTokenizer<Tok>::buildFunctionBox(
BINJS_TRY_VAR(fun, AllocNewFunction(cx_, fcd));
MOZ_ASSERT(fun->explicitName() == atom);
} else {
BINJS_TRY_VAR(fun, lazyScript_->functionNonDelazifying());
BINJS_TRY_VAR(fun, lazyScript_->function());
}
mozilla::Maybe<Directives> directives;
@@ -321,8 +321,7 @@ JS::Result<Ok> BinASTParserPerTokenizer<Tok>::finishEagerFunction(
FunctionBox* funbox, uint32_t nargs) {
// If this is delazification of a canonical function, the JSFunction object
// already has correct `nargs_`.
if (!lazyScript_ ||
lazyScript_->functionNonDelazifying() != funbox->function()) {
if (!lazyScript_ || lazyScript_->function() != funbox->function()) {
funbox->setArgCount(nargs);
funbox->synchronizeArgCount();
} else {