util: inspect: avoid a crash on a DataView with a detached buffer

[ljharb]

No description provided.

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 89.21%. Comparing base (da5f7ac) to head (3b2895d).
Report is 2 commits behind head on main.

Additional details and impacted files

@@            Coverage Diff             @@
##             main   #56669      +/-   ##
==========================================
- Coverage   89.21%   89.21%   -0.01%     
==========================================
  Files         662      662              
  Lines      191893   191914      +21     
  Branches    36936    36941       +5     
==========================================
+ Hits       171189   171207      +18     
- Misses      13541    13549       +8     
+ Partials     7163     7158       -5     

Files with missing lines	Coverage Δ
lib/internal/util/inspect.js	99.91% <100.00%> (+<0.01%)	⬆️

... and 41 files with indirect coverage changes

[anonrig]

LGTM

[BridgeAR]

The current approach would impact the common case, while it's possible to separate that to only being treated special in case a DataView is handled.
@ljharb please have a look at the detached ArrayBuffer implementation handling. That works without impacting the common case :)