electron/electron

[Bug]: setBrowserView broken in v30 on macOS, works on Windows

Stanzilla opened this issue · 4 comments

Preflight Checklist

Electron Version

30.0.6

What operating system are you using?

macOS

Operating System Version

14.5

What arch are you using?

arm64 (including Apple Silicon)

Last Known Working Electron version

29

Expected Behavior

Same functionality as in v29

Actual Behavior

When this line of code if present, the app will only show a blank window:

mainWindow.setBrowserView(ads);

ads is defined as follows:

ads = new BrowserView({
    webPreferences: {
      partition: 'persist:adpartition',
    },
  });

On Windows everything works as before in v29.

Testcase Gist URL

No response

Additional Information

No response

Hello @Stanzilla. 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.

Hello @Stanzilla. 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.

Here is the Fiddle https://gist.github.com/Stanzilla/be0c9e14572178e6b3a9b2237a486074 - it is possible that this is an issue with stateful-electron-window if yes, I will create an issue over there.

Looks like it's related to #41256 being in 30 and not #41326