fix: do not reset process_id in URLLoaderFactoryParams

[zcbenz]

Description of Change

Fix #24991.

In OverrideURLLoaderFactoryParams when disabling CORB, the process_id parameter was also changed.

This would cause a DCHECK assertion under certain cases (see #24991 (comment)), and under Release build while the DHCECK assertion would be disabled, it would cause weird permission errors due to wrong process id being set (see #24991 (comment)).

This behavior was introduced by 6d3e6ab from #19488, according to its description, the purpose was to disable CORB when web security is disabled. The original change did need to set the process_id because it was constructing a new Param struct, but after later Chromium upgrades there was no longer need to set process_id because the method was changed to override the parameters instead of creating a new one.

Checklist

• PR description included and stakeholders cc'd
• npm test passes
• tests are changed or added
• PR title follows semantic commit guidelines
• PR release notes describe the change in a way relevant to app developers, and are capitalized, punctuated, and past tense.
• This is NOT A BREAKING CHANGE. Breaking changes may not be merged to master until 11-x-y is branched.

Release Notes

Notes: Fix network permission error when there are multiple WebContents sharing same session are created with web security disabled.

[deepak1556]

Nice find!

[deepak1556]

Fix network permission error when there are multiple WebContents created with web security disabled.

maybe reword to highlight same session since the comparison was done based on browser_context.

Fix network permission error when there are multiple WebContents sharing same session are created with web security disabled.

[release-clerk]

Release Notes Persisted

Fix network permission error when there are multiple WebContents sharing same session are created with web security disabled.

[trop]

I have automatically backported this PR to "10-x-y", please check out #25178

[trop]

I was unable to backport this PR to "8-x-y" cleanly;
you will need to perform this backport manually.

[trop]

I have automatically backported this PR to "9-x-y", please check out #25179

[trop]

I have automatically backported this PR to "11-x-y", please check out #25180

[trop]

@zcbenz has manually backported this PR to "8-x-y", please check out #25181