Skip to content

[wasm] Update javascript API usage in samples and tests #73367

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

Merged

Conversation

maraf
Copy link
Member

@maraf maraf commented Aug 4, 2022

  • Use createDotnetRuntime with object instead of callback where possible.
  • Replace bind_static_method with getAssemblyExports.

@maraf maraf added this to the 7.0.0 milestone Aug 4, 2022
@maraf maraf self-assigned this Aug 4, 2022
@maraf
Copy link
Member Author

maraf commented Aug 4, 2022

/azp run runtime-wasm

@ghost
Copy link

ghost commented Aug 4, 2022

Tagging subscribers to 'arch-wasm': @lewing
See info in area-owners.md if you want to be subscribed.

Issue Details
  • Use createDotnetRuntime with object instead of callback where possible.
  • Replace bind_static_method with getAssemblyExports.
Author: maraf
Assignees: maraf
Labels:

arch-wasm, area-System.Runtime.InteropServices.JavaScript

Milestone: 7.0.0

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@maraf
Copy link
Member Author

maraf commented Aug 4, 2022

/azp run runtime-wasm

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@maraf
Copy link
Member Author

maraf commented Aug 4, 2022

/azp run runtime-wasm

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@maraf
Copy link
Member Author

maraf commented Aug 4, 2022

/azp run runtime-wasm

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@maraf
Copy link
Member Author

maraf commented Aug 8, 2022

/azp run runtime-wasm

@azure-pipelines
Copy link

Pull request contains merge conflicts.

…dOfFunctionInCreateDotnetRuntime

# Conflicts:
#	src/mono/sample/wasm/browser-eventpipe/main.js
#	src/mono/sample/wasm/browser-profile/main.js
@maraf
Copy link
Member Author

maraf commented Aug 8, 2022

/azp run runtime-wasm

@azure-pipelines
Copy link

Comment was made before the most recent commit for PR 73367 in repo dotnet/runtime

@maraf maraf marked this pull request as ready for review August 8, 2022 10:36
@maraf
Copy link
Member Author

maraf commented Aug 8, 2022

The debugger-driver.html still has a bind_static_method for calls that pass method from C# as part of the tests. I'll address these in a separate PR.

@maraf
Copy link
Member Author

maraf commented Aug 8, 2022

/azp run runtime-wasm

@maraf maraf requested a review from pavelsavara August 8, 2022 10:37
@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

Copy link
Member

@pavelsavara pavelsavara left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Co-authored-by: Ilona Tomkowicz <[email protected]>
@maraf
Copy link
Member Author

maraf commented Aug 9, 2022

/azp run runtime-wasm

@azure-pipelines
Copy link

Pull request contains merge conflicts.

…dOfFunctionInCreateDotnetRuntime

# Conflicts:
#	src/mono/sample/wasm/browser-mt-eventpipe/Program.cs
#	src/mono/sample/wasm/browser-mt-eventpipe/main.js
@maraf
Copy link
Member Author

maraf commented Aug 9, 2022

/azp run runtime-wasm

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@maraf
Copy link
Member Author

maraf commented Aug 10, 2022

One debugger test timed out on runtime-staging, but passed on runtime-wasm.

@maraf maraf merged commit 30c1c2c into dotnet:main Aug 10, 2022
@maraf maraf deleted the WasmUseObjectInsteadOfFunctionInCreateDotnetRuntime branch August 10, 2022 09:45
@ghost ghost locked as resolved and limited conversation to collaborators Sep 9, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants