Before this change we were sending sourceReference for sources we couldn't find the source for. This PR addresses that issue.
As an experiment, we implemented this change by inserting a function instead of a class like we normally do. We added some support to be able to do that.
src/chrome/cdtpDebuggee/cdtpDIContainer.ts:
Exported getSourceTextRetrievability so it can be customized by chrome-debug
src/chrome/client/sourceToClientConverter.ts:
Only include sourceReferences for sources we can actually get the source for...
src/chrome/dependencyInjection.ts/bind.ts:
Don't modify injectables that are numbers
src/chrome/dependencyInjection.ts/di.ts:
Add support for injecting functions instead of classes
Pass _componentCustomizationCallback to find places where we should've been passing it already
src/chrome/dependencyInjection.ts/types.ts:
Add isValueComponent to identify if an identifier is a class or a value
Before this change we were sending sourceReference for sources we couldn't find the source for. This PR addresses that issue. As an experiment, we implemented this change by inserting a function instead of a class like we normally do. We added some support to be able to do that.
src/chrome/cdtpDebuggee/cdtpDIContainer.ts:
src/chrome/client/sourceToClientConverter.ts:
src/chrome/dependencyInjection.ts/bind.ts:
src/chrome/dependencyInjection.ts/di.ts:
src/chrome/dependencyInjection.ts/types.ts:
src/chrome/internal/sources/sourceTextRetriever.ts:
src/chrome/logging/methodsCalledLogger.ts:
This change won't compile with a required change in chrome-debug: https://github.com/microsoft/vscode-chrome-debug/pull/930