Skip to content
This repository has been archived by the owner on Oct 2, 2021. It is now read-only.

Fix breakpoint not hitting in refresh because it's being set on a negative line #282

Merged
merged 1 commit into from
Feb 10, 2018

Conversation

digeff
Copy link
Contributor

@digeff digeff commented Feb 9, 2018

Breakpoints weren't being hit after refresh because BreakOnLoadHelper.resolvePendingBreakpoints was passing a value from inside this._chromeDebugAdapter.pendingBreakpointsByUrl to chromeDebugAdapter.setBreakpoints, and setBreakpoints was calling LineColTransformer and substracting 1 to the line number again.

The deep copy on args prevents setBreakpoints from modifying the original args...

@roblourens
Copy link
Member

What's the problem exactly and why does this fix it? It sounds like you're describing the problem that this was originally added to fix, but I can't repro any issue now.

Copy link
Member

@roblourens roblourens left a comment

Choose a reason for hiding this comment

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

Discussed offline

@roblourens roblourens merged commit 92fc699 into microsoft:master Feb 10, 2018
@digeff digeff deleted the fix_lines_going_negative branch February 10, 2018 01:04
@roblourens roblourens added this to the February 2018 milestone Mar 5, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants