[release/7.0] [wasm][debugger] fixing setting a breakpoint in an invalid IL offset after hotreload #75564
Conversation
![github-actions[bot]](https://avatars.githubusercontent.com/in/15368?s=60&v=4){kind=small}
Co-authored-by: Ankit Jain <radical@gmail.com>
Co-authored-by: Ankit Jain <radical@gmail.com>
github-actions
bot
requested review from
thaystg and
radical
as code owners
|
Tagging subscribers to this area: @thaystg Issue DetailsBackport of #75561 to release/7.0 /cc @thaystg Customer ImpactTestingRiskIMPORTANT: Is this backport for a servicing release? If so and this change touches code that ships in a NuGet package, please make certain that you have added any necessary package authoring and gotten it explicitly reviewed.
|
|
Tagging subscribers to 'arch-wasm': @lewing Issue DetailsBackport of #75561 to release/7.0 /cc @thaystg Customer ImpactTestingRiskIMPORTANT: Is this backport for a servicing release? If so and this change touches code that ships in a NuGet package, please make certain that you have added any necessary package authoring and gotten it explicitly reviewed.
|
|
Double signed off by @lewing, so that's how I know we really want this merged 😁. CI is green. Ready. |
ghost
locked as resolved and limited conversation to collaborators
Backport of #75561 to release/7.0
/cc @thaystg
Customer Impact
When applying hot reload changes the browser could close because we had a crash on BrowserDebugProxy
Testing
Manually tested
Risk
Low risk, only protecting the return of setting the breakpoint to avoid a throw.
IMPORTANT: Is this backport for a servicing release? If so and this change touches code that ships in a NuGet package, please make certain that you have added any necessary package authoring and gotten it explicitly reviewed.