Blazor WebAssembly & WebView JS to .NET Streaming Interop Support - #33986
Conversation
SteveSandersonMS
left a comment
There was a problem hiding this comment.
Awesome work @TanayParikh! This looks great to me.
|
fyi/ Had to change the WebView base64 serialization slightly due to stack overflow concerns on large byte arrays. |
|
/azp run |
|
Azure Pipelines successfully started running 2 pipeline(s). |
|
CI passed but status isn't being reported for some reason. Could someone with admin rights please merge? @dotnet/aspnet-build https://dev.azure.com/dnceng/public/_build/results?buildId=1239168&view=results |
Pull request was closed
|
@TanayParikh all closing and reopening did was cancel the auto-merge and start another build that likely won't report back to GitHub either 😀 |
Thanks for merging. Wasn't sure why the CI status wasn't being reported to GitHub and figured closing/re-opening may re-trigger the reporting (of the new CI run). |
RemoteJSDataStreaminto a sharedBaseJSDataStreamwhich is shared across the 3 platforms. Did this instead of putting it intoJSInteropto avoid creating additional public APIs per Javier's recommendation.ReceiveJSDataChunkneeds to be refined (potential synchronization issue)BaseJSDataStream(will do this after everything else is approved)Fixes: #33638
Fixes: #27862
Part of: #30289