jlfwong / speedscope / 623

47%

push

travis-ci

Support importing unterminated JSON in simple cases (#208)

This PR introduces support for importing JSON based profiles that are missing a terminating `]` (and possibly have an extraneous `,`).

This is similar to #202, but takes a much more targeted and simple approach.

I'm confident that this approach is sufficient because this is exactly what `chrome://tracing` does: https://github.com/catapult-project/catapult/blob/27e047e04/tracing/tracing/extras/importer/trace_event_importer.html#L197-L208

Fixes #204

862 of 1975 branches covered (43.65%)

Branch coverage included in aggregate %.

2196 of 4546 relevant lines covered (48.31%)

6789.63 hits per line