Skip to content

Commit

Permalink
Bug 1518812 - Redo the fix for bug 1516567 atop the moving of the rel…
Browse files Browse the repository at this point in the history
…evant code to js/public/PropertySpec.h in bug 1517624. r=jandem
  • Loading branch information
jswalden committed Jan 9, 2019
1 parent 532d683 commit d64076a
Showing 1 changed file with 3 additions and 5 deletions.
8 changes: 3 additions & 5 deletions js/public/PropertySpec.h
Original file line number Diff line number Diff line change
Expand Up @@ -106,11 +106,9 @@ struct JSPropertySpec {

private:
void checkAccessorsAreNative() const {
MOZ_ASSERT(accessors.getter.native.op);
// We may not have a setter at all. So all we can assert here, for the
// native case is that if we have a jitinfo for the setter then we have
// a setter op too. This is good enough to make sure we don't have a
// SelfHostedWrapper for the setter.
// We may have a getter or a setter or both. And whichever ones we have
// should not have a SelfHostedWrapper for the accessor.
MOZ_ASSERT_IF(accessors.getter.native.info, accessors.getter.native.op);
MOZ_ASSERT_IF(accessors.setter.native.info, accessors.setter.native.op);
}

Expand Down

0 comments on commit d64076a

Please sign in to comment.