jlfwong / speedscope / 762 / 1

47%

master: 47%

push

travis-ci

Fix performance issues for the caller/callee flamegraphs in the sandwich view (#296)

This fixes two unrelated problems which together caused performance issues in the sandwich view & made hover tooltips appear to be broken.

The first issue was caused by continuously priming the `requestAnimationFrame` loop when it should be a no-op, and the second issue was caused by using different cache keys when trying to access a memoized value in the caller & callee flamegraph components. This resulted in thrash, and especially bad performance because the cache miss was resulting in us re-allocating the WebGL framebuffer on every frame, which is unsurprisingly quite slow.

Fixes #212 
Fixes #155 
Fixes #74 (though this was maybe already fixed)

934 of 2163 branches covered (43.18%)

Branch coverage included in aggregate %.

2350 of 4940 relevant lines covered (47.57%)

6348.54 hits per line