Get rid of multiple bool parameters to takeHeapSnapshot()
This resolves a new downstream lint.
This is not marked [nfc] because there are two bugfixes in this commit:
- The reportProgress flag is now being respected. Previously we were
alwyas reporting progress.
- The exposeInternals flag is now being properly interpreted. Previously
we were passing it, un-negated, to a parameter named hide_internals.
I switched to the options struct version of the V8 TakeHeapSnapshot()
function for greater clarity.
[o11y] Assorted Streaming Tail Worker bug fixes
This may address two bugs seen in testing (which have not been reproduced yet).
- Do not provide outcome event for pipelineLogLevel == PipelineLogLevel::NONE,
we already don't provide the other event types there (matching the legacy tail
worker).
- Do not assert that a handler context is available
- Do not try to accept more events if we failed to get a JS-level handler. This
may fix "Expected only a single onset event" errors under a race condition
where the client sends an event right after the onset, before the server can
indicate that no additional events should be sent based on the invalid
handler.