diff options
author | Joyee Cheung <joyeec9h3@gmail.com> | 2019-03-08 11:34:46 +0300 |
---|---|---|
committer | Joyee Cheung <joyeec9h3@gmail.com> | 2019-03-12 02:03:53 +0300 |
commit | 0a3bcdd26108bb29045a557bc555c3a53c244a55 (patch) | |
tree | b510257ef03b0ad2d04ed9c597677b39bce0405c /src/inspector | |
parent | 963ee0bc736008e01d80fbad973f80fd021735fd (diff) |
src: refactor coverage connection
- Refactor the C++ class to be resuable for other types of profiles
- Move the try-catch block around coverage collection callback
to be inside the callback to silence potential JSON or write
errors.
- Use Function::Call instead of MakeCallback to call the coverage
message callback since it does not actually need async hook
handling. This way we no longer needs to disable the async
hooks when writing the coverage results.
- Renames `lib/internal/coverage-gen/with_profiler.js` to
`lib/internal/profiler.js` because it is now the only way
to generate coverage.
PR-URL: https://github.com/nodejs/node/pull/26513
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: Ben Coe <bencoe@gmail.com>
Diffstat (limited to 'src/inspector')
-rw-r--r-- | src/inspector/node_inspector.gypi | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/inspector/node_inspector.gypi b/src/inspector/node_inspector.gypi index a59727f191e..5191543ab73 100644 --- a/src/inspector/node_inspector.gypi +++ b/src/inspector/node_inspector.gypi @@ -45,7 +45,7 @@ '../../src/inspector_io.cc', '../../src/inspector_agent.h', '../../src/inspector_io.h', - '../../src/inspector_coverage.cc', + '../../src/inspector_profiler.cc', '../../src/inspector_js_api.cc', '../../src/inspector_socket.cc', '../../src/inspector_socket.h', |