Skip to content
This repository has been archived by the owner on Jan 11, 2023. It is now read-only.

Handle original source maps used on code running inside an eval() #5749

Merged

Conversation

loganfsmyth
Copy link
Contributor

Refs Issue: #5561

Summary of Changes

  • Rework the processing of generated code so that it handles being run inside eval()
  • Unrelated small fix: Include the final line in a file as part of the empty line list

@@ -45,6 +45,6 @@ export function findEmptyLines(
return [];
}
const lineCount = selectedSource.text.split("\n").length;
const sourceLines = range(1, lineCount);
const sourceLines = range(1, lineCount + 1);
Copy link
Contributor Author

Choose a reason for hiding this comment

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

@jasonLaster This was the issue in the screenshot I sent you with the last line looking debuggable:
screen shot 2018-03-19 at 4 37 22 pm

Copy link
Contributor

Choose a reason for hiding this comment

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

Awesome catch!

@jasonLaster jasonLaster merged commit 777e2ee into firefox-devtools:master Mar 22, 2018
@loganfsmyth loganfsmyth deleted the eval-sourcemap-handling branch March 22, 2018 18:53
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.

None yet

3 participants