SIGN IN SIGN UP

fix: use HolderV2() for PropertyCallbackInfo on V8 >= 12.5 (#1459)

fix: use HolderV2() for PropertyCallbackInfo on V8 >= 13

PR #1456 replaced This() with Holder() in NODE_GETTER contexts, but
Electron 41 (V8 14.6) removed both This() and Holder() from
PropertyCallbackInfo. Only HolderV2() remains (to be renamed back to
Holder() in a future V8 version per crbug.com/333672197).

Add a PROPERTY_HOLDER compat macro that uses HolderV2() on V8 >= 13
and falls back to This() on older versions. Same pattern as the
existing GET_PROTOTYPE macro.

Verified: compiles and runs on Electron 41.0.1 (V8 14.6.202).

Fixes #1458
T
tstone-1 committed
8617ed645e7717710f83faad90b3eb6a4e5a6e95
Parent: 959a018
Committed by GitHub <noreply@github.com> on 3/13/2026, 11:08:07 PM