lib: unify ICU and no-ICU TextDecoder

[ChALkeR]

Better reviewed with whitespace ignored

1. Fix TextDecoder defaults to stream: true instead of false with null options #61411
2. Add support for fatal UTF-8 decoder in no-ICU version unless streaming is used
3. add utf-8 fast path to no-ICU version
4. reduce duplication of TextDecoder source
5. normalize how this[kHandle] was not set in no-ICU version
6. The refactor changes the encoding reported in the no-ICU version when encoding is recognized but unsupported: now the actual resolved encoding name is reported as unsupported when previously the original input string was reported.
   I don't think this is major, and reporting the actual encoding name is better in that case.

This will allow further fixes and common fast path for UTF-16 / fixed impl for UTF-16 as string_decoder doesn't implement UTF-16 per spec.

UTF-16 is still broken in no-ICU version here, this PR does not address that.
Tracking: #61041

[nodejs-github-bot]

Review requested:

• @nodejs/startup

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 89.84%. Comparing base (77e8d44) to head (e74c6fe).
⚠️ Report is 30 commits behind head on main.

Additional details and impacted files

@@            Coverage Diff             @@
##             main   #61409      +/-   ##
==========================================
- Coverage   89.87%   89.84%   -0.03%     
==========================================
  Files         671      671              
  Lines      203178   203124      -54     
  Branches    39062    39040      -22     
==========================================
- Hits       182599   182503      -96     
- Misses      12926    12972      +46     
+ Partials     7653     7649       -4     

Files with missing lines	Coverage Δ
lib/internal/encoding.js	100.00% <100.00%> (ø)

... and 29 files with indirect coverage changes

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
• 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

[avivkeller]

There's a lazy utility in the internal utils.

[ChALkeR]

@avivkeller this is not new code, it's just moved to an outer scope
Ideally all that should go away with follow-up fixes, string_decoder path is invalid anyway
I don't think it's worth refactoring it further

[anonrig]

I still think we should use the new lazy module. Regardless of this being new code or not, git sees it as a new code.

[ChALkeR]

@anonrig Next two prs will remove this anyway. This is going in iterative steps, the intention of this PR is to make rewrite easier, not to be that rewrite. I don't think it should be blocked on rewriting a code block that wasn't introduced here and is gonna be removed.

[ChALkeR]

Rebased, no actual changes

[gurgunday]

lgtm

[nodejs-github-bot]

CI: https://ci.nodejs.org/job/node-test-pull-request/71011/