Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Assertion when using remote object as argument in method on other remote object #148

Closed
pontusn opened this issue Mar 28, 2020 · 3 comments
Labels
bug Something isn't working

Comments

@pontusn
Copy link

pontusn commented Mar 28, 2020

Our legacy API returns complex objects. To reduce latency overhead we added new helper objects with methods that convert legacy objects to JSON.

When calling this helper from JS with any other remote object as argument we consistently see this assertion in the developer console:

                    _updateSequenceNumberForProxyOptions(options, sequenceId) {
                        console.assert(options && options.remoteSequenceId <= sequenceId);
                        options.remoteSequenceId = sequenceId;
                    }
@david-risney david-risney added the bug Something isn't working label Apr 14, 2020
@david-risney
Copy link
Contributor

Another great AddRemoteObject bug. Thank you for finding these! Please let us know what other AddRemoteObject (or otherwise) issues you find.

@pontusn
Copy link
Author

pontusn commented Apr 15, 2020

You are welcome. I work as a freelance developer. My current client has a huge legacy MFC/ATL/Win32-application with some 60k users that currently embeds IE11. My current job is to prepare for migration to WebView2 ASAP. This application is great platform for finding problems. It has components from all eras since mid nineties and uses embedded webbrowser for all customized interactions. And once we flip the switch there will be many end-users.

@champnic champnic added this to the Edge 84.0.508.0 milestone Jun 22, 2020
@champnic
Copy link
Member

This should be fixed in Edge 84.0.508.0+. Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

3 participants