fix: Introduce LocationProxy for BrowserWindowProxy

[Anrock]

Fixes #15017

Description of Change

Implements proxy Location object for BrowserWindowProxy

Checklist

• PR description included and stakeholders cc'd
• npm test passes
• tests are changed or added
• relevant documentation is changed or added
• PR title follows semantic commit guidelines

Release Notes

Notes: Implement proper Location object for BrowserWindowProxy

[codebytere]

@Anrock did you test this locally? i'm seeing 10 test failures for most builds that all look related

[Anrock]

@codebytere no, i didn't - once again git pull rendered master unbuildable for me. Therefore marked as WIP.

However i'm interested in top-level review if this approach is viable and if it's a welcomed change in regards of electron architecture..

[Anrock]

Removing WIP: works for me, usual 4-5 test fails as on master.

[Anrock]

Is there relevant unit tests for location object i should change?

[Anrock]

@codebytere ping

[alexeykuzmin]

@Anrock

https://circleci.com/gh/electron/electron/82004
Among all failed tests only these two look like real failures:

• "chromium feature window.open defines a window.location getter"
• "BrowserWindow module contextIsolation option with and without sandbox option doesn't break ipc serialization"

Can you please fix them?

[Anrock]

@alexeykuzmin i'll take a look at it as soon as i can. Don't have much free time for it, unfortunately.

[zcbenz]

This looks good to me, I have fixed the failing tests by adding a setter for window.location.

[release-clerk]

Release Notes Persisted

Implement proper Location object for BrowserWindowProxy