Crash reported due to Node.js #34839

[mhdawson]

nodejs/node#35620
Reported as being due to: nodejs/node@a6b6556

[mhdawson]

@gabrielschulhof in case you have a chance to take a look before the meeting on Friday.

[mhdawson]

@gabrielschulhof my first thought is that even if there is no user provided finalizer, we've still set it up for the gc to call a Finalize() in js_native_api_v8.cc and therefore there is a race condition if we delete when reference->_finalize_callback is nullptr.

[mhdawson]

Closing since we discussed, and the revert is in place.