[Bug]: BrowserView is not resized correctly on BrowserWindow resize

[staniel359]

Preflight Checklist

• I have read the Contributing Guidelines for this project.
• I agree to follow the Code of Conduct that this project adheres to.
• I have searched the issue tracker for a bug report that matches the one I want to file, without success.

Electron Version

31.0.1

What operating system are you using?

Ubuntu

Operating System Version

22.04

What arch are you using?

x64

Last Known Working Electron version

29.3.0

Expected Behavior

BrowserWindow's BrowserView should resize according to its parent.

Actual Behavior

Doesn't:
Screencast from 14-06-24 11:23:05.webm

Testcase Gist URL

No response

Additional Information

No response

[electron-issue-triage]

Hello @staniel359. Thanks for reporting this and helping to make Electron better!

Would it be possible for you to make a standalone testcase with only the code necessary to reproduce the issue? For example, Electron Fiddle is a great tool for making small test cases and makes it easy to publish your test case to a gist that Electron maintainers can use.

Stand-alone test cases make fixing issues go more smoothly: it ensure everyone's looking at the same issue, it removes all unnecessary variables from the equation, and it can also provide the basis for automated regression tests.

Now adding the blocked/need-repro Needs a test case to reproduce the bug label for this reason. After you make a test case, please link to it in a followup comment. This issue will be closed in 10 days if the above is not addressed.

[staniel359]

Found out this issue was in BrowserView.setAutoResize() method.

Removed it and used resizing on BrowserWindow's resize event.

[codebytere]

@staniel359 to clarify - you think there's a bug in setAutoresize? or you realized you were incorrectly using the API? We still need a repro if you'd like us to proceed.

[staniel359]

@codebytere setAutoResize had been working correctly until 29.3.0.
So I removed it and used the resize logic explicitly but as a workaround.

https://gist.github.com/staniel359/ef9af020415662b8069e265976910c30