lib: coerce -0 into 0 for process APIs

[HarshithaKP]

Fixes: #32750

Checklist

• make -j4 test (UNIX), or vcbuild test (Windows) passes
• tests and/or benchmarks are included
• documentation is changed or added
• commit message follows commit guidelines

[HarshithaKP]

/test/known_issues/test-crypto-authenticated-stream.js is failing with timeout. Running it a 100 times to see if it is related to my changes.

[Trott]

Needs a test, I would think.

[targos]

It's true that this would fix #32750 but as I noted there, this is a general issue and I would like that we fix it in a general way.

[HarshithaKP]

@Trott, added a scenario in test-process-uid-gid.js, PTAL

[HarshithaKP]

@targos, I believe it is generic already (covering more APIs that setuid), do you have any better suggestions?

[addaleax]

@HarshithaKP I think the most reasonable thing is to go through the C++ source code, and check it for usages of IsUint32. For those cases where it’s possible that -0 gets passed, there is probably some ….As<Uint32>()->Value() usage, which could be replaced with ->Uint32Value(context).To(…) (like here).

test added

[BridgeAR]

Ping @HarshithaKP

[HarshithaKP]

@BridgeAR , I tried to implement the suggested approach, but got into issues.

[github-actions]

This issue/PR was marked as stalled, it will be automatically closed in 30 days. If it should remain open, please leave a comment explaining why it should remain open.

[HarshithaKP]

Intention fulfilled in #36786