Skip to content

Commit d7ade26

Browse files
committed
Add comment from @pavelfeldman.
1 parent 04f408e commit d7ade26

File tree

1 file changed

+4
-0
lines changed

1 file changed

+4
-0
lines changed

wg-meetings/2016-06-01.md

+4
Original file line numberDiff line numberDiff line change
@@ -45,6 +45,10 @@ chrome://tracing is intended for platform developers while “Timeline” view i
4545

4646
Currently structural tracing is used, but work is in progress to make sampling tracing more efficient/performant and shift to sampling for most uses. See [this video](https://www.youtube.com/watch?v=nxXkquTPng8) for background on structural vs. sampling tracing.
4747

48+
* @pavelfeldman: Not exactly. We should use trace points (TRACE_EVENT) for C++ instrumentation of the engine and modules, but we should run sampler at all times for complete JS coverage (both engine and user land).
49+
50+
We can already collect JS samples in the tracing format so that the whole profile was a single payload, but we are not satisfied with the memory footprint it generates. So we are working on a revised format that would scale. In the meanwhile, we simply start both Tracing and JS profiler (sampler) and merge reports on the client side.
51+
4852
CDP allows configuration of tracing through the Tracing domain:
4953
https://chromedevtools.github.io/debugger-protocol-viewer/tot/Tracing/
5054

0 commit comments

Comments
 (0)